diff options
author | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2013-11-15 00:19:03 -0200 |
---|---|---|
committer | Carlos Antonio da Silva <carlosantoniodasilva@gmail.com> | 2013-11-15 00:19:03 -0200 |
commit | 4c7e3a3087d48ffbdf1c185ab12ef259b5f8e074 (patch) | |
tree | 60a3a1413d1d0296fac7a54d2b812aedc686f402 | |
parent | d2992818e02bd380f6a68385b4e97a301ec2f8aa (diff) | |
download | rails-4c7e3a3087d48ffbdf1c185ab12ef259b5f8e074.tar.gz rails-4c7e3a3087d48ffbdf1c185ab12ef259b5f8e074.tar.bz2 rails-4c7e3a3087d48ffbdf1c185ab12ef259b5f8e074.zip |
Use a simple conditional rather than short circuit with next
-rw-r--r-- | activemodel/lib/active_model/validations/numericality.rb | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/activemodel/lib/active_model/validations/numericality.rb b/activemodel/lib/active_model/validations/numericality.rb index 8f4ae693a1..0f131de923 100644 --- a/activemodel/lib/active_model/validations/numericality.rb +++ b/activemodel/lib/active_model/validations/numericality.rb @@ -11,8 +11,9 @@ module ActiveModel def check_validity! keys = CHECKS.keys - [:odd, :even] options.slice(*keys).each do |option, value| - next if value.is_a?(Numeric) || value.is_a?(Proc) || value.is_a?(Symbol) - raise ArgumentError, ":#{option} must be a number, a symbol or a proc" + unless value.is_a?(Numeric) || value.is_a?(Proc) || value.is_a?(Symbol) + raise ArgumentError, ":#{option} must be a number, a symbol or a proc" + end end end |