DamienHarper / auditor

auditor, the missing audit log library
MIT License
164 stars 53 forks source link

improved the TransactionProcessor to convert encoding of diff properly (#150) #151

Closed joerndyherrn closed 1 year ago

joerndyherrn commented 1 year ago

improved the TransactionProcessor to convert encoding of diff properly (https://github.com/DamienHarper/auditor/issues/150)

codecov[bot] commented 1 year ago

Codecov Report

Merging #151 (5e0a6ac) into master (0f03f17) will increase coverage by 0.71%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #151      +/-   ##
==========================================
+ Coverage   95.69%   96.40%   +0.71%     
==========================================
  Files          43       42       -1     
  Lines        2391     1752     -639     
==========================================
- Hits         2288     1689     -599     
+ Misses        103       63      -40     
Impacted Files Coverage Δ
...rine/Auditing/Transaction/TransactionProcessor.php 100.00% <100.00%> (ø)
...der/Doctrine/Persistence/Helper/PlatformHelper.php 74.46% <0.00%> (-7.08%) :arrow_down:
...der/Doctrine/Persistence/Helper/DoctrineHelper.php 76.92% <0.00%> (-5.32%) :arrow_down:
...Doctrine/Persistence/Reader/Filter/RangeFilter.php 80.00% <0.00%> (-4.62%) :arrow_down:
...rine/Persistence/Reader/Filter/DateRangeFilter.php 84.00% <0.00%> (-3.88%) :arrow_down:
...vider/Doctrine/Auditing/Transaction/AuditTrait.php 95.79% <0.00%> (-1.91%) :arrow_down:
...octrine/Persistence/Event/CreateSchemaListener.php 96.87% <0.00%> (-0.50%) :arrow_down:
...rc/Provider/Doctrine/Persistence/Reader/Reader.php 96.07% <0.00%> (-0.43%) :arrow_down:
...rine/Persistence/Command/CleanAuditLogsCommand.php 98.79% <0.00%> (-0.24%) :arrow_down:
...ider/Doctrine/Persistence/Schema/SchemaManager.php 99.32% <0.00%> (-0.22%) :arrow_down:
... and 19 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

DamienHarper commented 1 year ago

@joerndyherrn Thanks for the contribution!

joerndyherrn commented 1 year ago

@DamienHarper this is done :)