minor changes to mass assignment security patch to bring it in line with rails standards

Signed-off-by: José Valim <jose.valim@gmail.com>

1 files changed, 5 insertions, 3 deletions

diff --git a/activerecord/lib/active_record/mass_assignment_security/sanitizer.rb b/activerecord/lib/active_record/mass_assignment_security/sanitizer.rb
index 4a099a147c..11de35f9d6 100644
--- a/activerecord/lib/active_record/mass_assignment_security/sanitizer.rb
+++ b/activerecord/lib/active_record/mass_assignment_security/sanitizer.rb
@@ -13,15 +13,17 @@ module ActiveRecord
 
         def debug_protected_attribute_removal(attributes, sanitized_attributes)
           removed_keys = attributes.keys - sanitized_attributes.keys
-          if removed_keys.any?
-            logger.debug "WARNING: Can't mass-assign protected attributes: #{removed_keys.join(', ')}"
-          end
+          warn!(removed_keys) if removed_keys.any?
         end
 
         def debug?
           logger.present?
         end
 
+        def warn!(attrs)
+          logger.debug "WARNING: Can't mass-assign protected attributes: #{attrs.join(', ')}"
+        end
+
     end
   end
 end