diff options
| author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-02 14:01:44 -0300 |
|---|---|---|
| committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2012-08-02 14:11:51 -0300 |
| commit | 99f622de230e582c372e20f4b72792c0fce0fd4a (patch) | |
| tree | 11e9b53483059998d333f9ed5534022c7a21bcdc /activerecord/lib/active_record/attribute_methods | |
| parent | 59c4b22c4528e9f97d3eb394f603dc50c3cf41a9 (diff) | |
| download | rails-99f622de230e582c372e20f4b72792c0fce0fd4a.tar.gz rails-99f622de230e582c372e20f4b72792c0fce0fd4a.tar.bz2 rails-99f622de230e582c372e20f4b72792c0fce0fd4a.zip | |
Do not consider the numeric attribute as changed if the old value is
zero and the new value is not a string.
Before this commit this was the behavior
r = Review.find_by_issue(0)
r.issue
=> 0
r.changes
=> {}
r.issue = 0
=> 0
r.changed?
=> true
r.changes
=> {"issue"=>[0,0]}
Fixes #7237
Diffstat (limited to 'activerecord/lib/active_record/attribute_methods')
| -rw-r--r-- | activerecord/lib/active_record/attribute_methods/dirty.rb | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/activerecord/lib/active_record/attribute_methods/dirty.rb b/activerecord/lib/active_record/attribute_methods/dirty.rb index 586a189011..ed863a9696 100644 --- a/activerecord/lib/active_record/attribute_methods/dirty.rb +++ b/activerecord/lib/active_record/attribute_methods/dirty.rb @@ -104,7 +104,7 @@ module ActiveRecord def changes_from_zero_to_string?(old, value) # For columns with old 0 and value non-empty string - old == 0 && value.present? && value != '0' + old == 0 && value.is_a?(String) && value.present? && value != '0' end end end |