@@ -2594,7 +2594,7 @@ module ActiveRecord #:nodoc: removed_attributes = attributes.keys - safe_attributes.keys if removed_attributes.any? - logger.debug "WARNING: Can't mass-assign these protected attributes: #{removed_attributes.join(', ')}" + log_protected_attribute_removal(removed_attributes) end safe_attributes @@ -2609,6 +2609,10 @@ module ActiveRecord #:nodoc: end end + def log_protected_attribute_removal(*attributes) + logger.debug "WARNING: Can't mass-assign these protected attributes: #{attributes.join(', ')}" + end + # The primary key and inheritance column can never be set by mass-assignment for security reasons. def attributes_protected_by_default default = [ self.class.primary_key, self.class.inheritance_column ] activerecord/lib/active_record/base.rb c7375d74d9fff3219d4ea389ba9e36a90afe9d33 Trevor Turk trevorturk@yahoo.com http://github.com/rails/rails/commit/81c12d1f6359eb5e52b376f1f3552097a144cc8b 81c12d1f6359eb5e52b376f1f3552097a144cc8b 2008-08-12T00:52:06-07:00 2008-08-11T19:17:14-07:00 move logging of protected attribute removal into log_protected_attribute_removal method Signed-off-by: Michael Koziarski <michael@koziarski.com> [#804 status:committed] fb59cc6d80e944b443a9564ab9fccc62ce018113 Michael Koziarski michael@koziarski.com