aboutsummaryrefslogtreecommitdiffstats
path: root/activemodel
diff options
context:
space:
mode:
authorBradley Priest <bradleypriest@gmail.com>2017-05-27 21:47:07 +0800
committerBradley Priest <bradleypriest@gmail.com>2017-05-27 21:58:35 +0800
commitd83b8e65102d625c9024cd9a2727a10b0ef83b79 (patch)
tree7dbd0ce4b165494aeee9454bc4392f38aa251d7a /activemodel
parentfcc47bcfccc7578aa0414710eecdad006085a911 (diff)
downloadrails-d83b8e65102d625c9024cd9a2727a10b0ef83b79.tar.gz
rails-d83b8e65102d625c9024cd9a2727a10b0ef83b79.tar.bz2
rails-d83b8e65102d625c9024cd9a2727a10b0ef83b79.zip
Fix regression in Numericality validator where extra decimal places on
a user input for a decimal column were ignored by numerically validations
Diffstat (limited to 'activemodel')
-rw-r--r--activemodel/CHANGELOG.md5
-rw-r--r--activemodel/lib/active_model/validations/numericality.rb4
2 files changed, 8 insertions, 1 deletions
diff --git a/activemodel/CHANGELOG.md b/activemodel/CHANGELOG.md
index e707a65147..7483704212 100644
--- a/activemodel/CHANGELOG.md
+++ b/activemodel/CHANGELOG.md
@@ -1,3 +1,8 @@
+* Fix regression in numericality validator when comparing Decimal and Float input
+ values with more scale than the schema.
+
+ *Bradley Priest*
+
* Fix methods `#keys`, `#values` in `ActiveModel::Errors`.
Change `#keys` to only return the keys that don't have empty messages.
diff --git a/activemodel/lib/active_model/validations/numericality.rb b/activemodel/lib/active_model/validations/numericality.rb
index b82c85ddf4..fb053a4c4e 100644
--- a/activemodel/lib/active_model/validations/numericality.rb
+++ b/activemodel/lib/active_model/validations/numericality.rb
@@ -36,7 +36,9 @@ module ActiveModel
return
end
- unless raw_value.is_a?(Numeric)
+ if raw_value.is_a?(Numeric)
+ value = raw_value
+ else
value = parse_raw_value_as_a_number(raw_value)
end