diff options
author | Sean Griffin <sean@thoughtbot.com> | 2014-11-30 18:24:53 -0700 |
---|---|---|
committer | Sean Griffin <sean@thoughtbot.com> | 2014-12-01 05:31:44 -0700 |
commit | 704c658531ae202715cba29d6b2ba64651f220fd (patch) | |
tree | 23d985b99313d0b41df014b8e29ea82f0f9a5799 /activerecord/test/cases/primary_keys_test.rb | |
parent | 8226bad7101fd35e774ca6d233c605c4cee2abd8 (diff) | |
download | rails-704c658531ae202715cba29d6b2ba64651f220fd.tar.gz rails-704c658531ae202715cba29d6b2ba64651f220fd.tar.bz2 rails-704c658531ae202715cba29d6b2ba64651f220fd.zip |
Ensure numericality validations work with mutation
The detection of in-place changes caused a weird unexpected issue with
numericality validations. That validator (out of necessity) works on the
`_before_type_cast` version of the attribute, since on an `:integer`
type column, a non-numeric string would type cast to 0.
However, strings are mutable, and we changed strings to ensure that the
post type cast version of the attribute was a different instance than
the before type cast version (so the mutation detection can work
properly).
Even though strings are the only mutable type for which a numericality
validation makes sense, special casing strings would feel like a strange
change to make here. Instead, we can make the assumption that for all
mutable types, we should work on the post-type-cast version of the
attribute, since all cases which would return 0 for non-numeric strings
are immutable.
Fixes #17852
Diffstat (limited to 'activerecord/test/cases/primary_keys_test.rb')
0 files changed, 0 insertions, 0 deletions