e-suite/Backend
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Upgraded audit engine to support EFCore's has conversion implemenation

Browse Source
This commit is contained in:
Colin Dawson 2026-03-02 22:41:53 +00:00
parent e9616b18ee
commit a3531e9a3e
1 changed files with 16 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

19
e-suite.Database.Audit/e-suite.Database.Audit/AuditEngine/AuditEngineCore.cs
View File

@ -69,8 +69,17 @@ public class AuditEngineCore
continue;
}
var currentValue = property.CurrentValue == null ? null : Convert.ChangeType(property.CurrentValue, property.CurrentValue!.GetType());
var oldValue = property.OriginalValue == null ? null : Convert.ChangeType(property.OriginalValue, property.OriginalValue!.GetType());
// Value converter + comparer
var converter = property.Metadata.GetValueConverter(); //todo need a unit test for when a value converter is present on the table.
var comparer = property.Metadata.GetValueComparer();
// Raw CLR values
var currentClr = property.CurrentValue;
var oldClr = property.OriginalValue;
// Convert to provider (JSON for Tasks)
var currentValue = converter?.ConvertToProvider(currentClr) ?? currentClr;
var oldValue = converter?.ConvertToProvider(oldClr) ?? oldClr;
var isSoftDelete =
propertyInfo.CustomAttributes.SingleOrDefault(x =>
@ -96,8 +105,10 @@ public class AuditEngineCore
var currentDisplayName = GetNewName(property) ?? GetEnumAsString(currentValue);
var oldDisplayName = GetOldName(property) ?? GetEnumAsString(oldValue);
var valuesMatch = (currentValue == null && oldValue == null) ||
(currentValue != null && currentValue.Equals(oldValue));
// Compare using EF’s own comparer if available
var valuesMatch = comparer != null
? comparer.Equals(currentClr, oldClr)
: Equals(currentValue, oldValue);
if (isRedacted)
{