diff options
author | Pratik Naik <pratiknaik@gmail.com> | 2008-07-31 15:56:46 +0100 |
---|---|---|
committer | Pratik Naik <pratiknaik@gmail.com> | 2008-07-31 15:56:46 +0100 |
commit | 108db00aa90fe266564483ab301cf0669cae600f (patch) | |
tree | cbf457189f968bd61d28124c675ff20337094ef3 /activerecord/lib | |
parent | f64bd2ca85595f94cbbe809f51a52cdb9b68af19 (diff) | |
download | rails-108db00aa90fe266564483ab301cf0669cae600f.tar.gz rails-108db00aa90fe266564483ab301cf0669cae600f.tar.bz2 rails-108db00aa90fe266564483ab301cf0669cae600f.zip |
Raise UnknownAttributeError when unknown attributes are supplied via mass assignment
Diffstat (limited to 'activerecord/lib')
-rwxr-xr-x | activerecord/lib/active_record/base.rb | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/base.rb b/activerecord/lib/active_record/base.rb index 9cb64223e2..29c2995334 100755 --- a/activerecord/lib/active_record/base.rb +++ b/activerecord/lib/active_record/base.rb @@ -122,6 +122,10 @@ module ActiveRecord #:nodoc: class MissingAttributeError < NoMethodError end + # Raised when unknown attributes are supplied via mass assignment. + class UnknownAttributeError < NoMethodError + end + # Raised when an error occurred while doing a mass assignment to an attribute through the # <tt>attributes=</tt> method. The exception has an +attribute+ property that is the name of the # offending attribute. @@ -2400,7 +2404,11 @@ module ActiveRecord #:nodoc: attributes = remove_attributes_protected_from_mass_assignment(attributes) if guard_protected_attributes attributes.each do |k, v| - k.include?("(") ? multi_parameter_attributes << [ k, v ] : send(k + "=", v) + if k.include?("(") + multi_parameter_attributes << [ k, v ] + else + respond_to?(:"#{k}=") ? send(:"#{k}=", v) : raise(UnknownAttributeError, "unknown attribute: #{k}") + end end assign_multiparameter_attributes(multi_parameter_attributes) |