Reverted reject_change? check to old style

boxt · Aug 22, 2022 · bc42eb1 · bc42eb1
1 parent 1a5882a
commit bc42eb1
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/lib/logga/active_record.rb b/lib/logga/active_record.rb
@@ -116,10 +116,11 @@ def field_changes_to_message(changes)
     def reject_change?(key)
       sym_key = key.to_sym
       return allowed_fields.exclude?(sym_key) if allowed_fields.present?
-      return true if config_excluded_fields.include?(sym_key)
-      return true if config_excluded_suffixes.any? { |suffix| key.to_s.end_with?(suffix.to_s) }
 
-      fields.exclude?(sym_key) && excluded_fields.include?(sym_key)
+      config_excluded_fields.include?(sym_key) ||
+        (fields.exclude?(sym_key) &&
+         (excluded_fields.include?(sym_key) ||
+          config_excluded_suffixes.any? { |suffix| key.to_s.end_with?(suffix.to_s) }))
     end
 
     def should_log?