fix(audit): key props now respect configured property exclusions

IntelliTect · Sep 25, 2024 · 63f8d86 · 63f8d86
1 parent cf4f3ba
commit 63f8d86
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,7 @@
 
 - feat: better "not found" messages from data sources when the ID is null or empty string. (#447)
 - fix(template): adjust manual chunking configuration to avoid circular deps. (#455)
+- fix(audit): key props now respect configured property exclusions
 
 # 5.0.1
 

diff --git a/src/IntelliTect.Coalesce.AuditLogging/Internal/CoalesceAudit.cs b/src/IntelliTect.Coalesce.AuditLogging/Internal/CoalesceAudit.cs
@@ -59,7 +59,7 @@ internal void PreSaveChanges(DbContext context)
                     audit.Entries.Add(auditEntry);
 
                     auditEntry.Properties = item.Properties
-                        .Where(p => p.Metadata.IsKey() || audit.Configuration.IsAuditedProperty(p))
+                        .Where(p => audit.Configuration.IsAuditedProperty(p))
                         .Select(p => new AuditEntryProperty
                         {
                             Parent = auditEntry,
@@ -80,10 +80,10 @@ internal void PreSaveChanges(DbContext context)
                     audit.Entries.Add(auditEntry);
 
                     auditEntry.Properties = item.Properties
-                        .Where(p => p.Metadata.IsKey() || (
+                        .Where(p => 
                             audit.Configuration.IsAuditedProperty(p) && 
                             !object.Equals(p.CurrentValue, p.OriginalValue)
-                        ))
+                        )
                         .Select(p => new AuditEntryProperty
                         {
                             Parent = auditEntry,
@@ -106,7 +106,7 @@ internal void PostSaveChanges()
                 case AuditEntryState.EntityAdded:
                     foreach (var propertyEntry in auditEntry.Entry.Properties)
                     {
-                        if (!propertyEntry.Metadata.IsKey() && !auditEntry.Parent.Configuration.IsAuditedProperty(propertyEntry))
+                        if (!auditEntry.Parent.Configuration.IsAuditedProperty(propertyEntry))
                         {
                             continue;
                         }
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,7 @@ @@
     - feat: better "not found" messages from data sources when the ID is null or empty string. (#447)
     - fix(template): adjust manual chunking configuration to avoid circular deps. (#455)
+    - fix(audit): key props now respect configured property exclusions
     # 5.0.1
@@ Expand Down @@