diff options
Diffstat (limited to 'activemodel')
| -rw-r--r-- | activemodel/lib/active_model/validations/length.rb | 3 | ||||
| -rw-r--r-- | activemodel/test/cases/validations/length_validation_test.rb | 11 |
2 files changed, 13 insertions, 1 deletions
diff --git a/activemodel/lib/active_model/validations/length.rb b/activemodel/lib/active_model/validations/length.rb index 7af6c83460..3e6f6e62e6 100644 --- a/activemodel/lib/active_model/validations/length.rb +++ b/activemodel/lib/active_model/validations/length.rb @@ -42,7 +42,8 @@ module ActiveModel next unless check_value = options[key] value ||= [] if key == :maximum - + + next if value.kind_of?(Fixnum) && value.to_s.size.send(validity_check, check_value) next if value && value.size.send(validity_check, check_value) errors_options = options.except(*RESERVED_OPTIONS) diff --git a/activemodel/test/cases/validations/length_validation_test.rb b/activemodel/test/cases/validations/length_validation_test.rb index 1e6180a938..5e4f4f5cae 100644 --- a/activemodel/test/cases/validations/length_validation_test.rb +++ b/activemodel/test/cases/validations/length_validation_test.rb @@ -341,6 +341,17 @@ class LengthValidationTest < ActiveModel::TestCase assert t.errors[:content].any? assert_equal ["Your essay must be at least 5 words."], t.errors[:content] end + + def test_validates_length_of_for_fixnum + Topic.validates_length_of(:approved, :is => 4) + + t = Topic.new("title" => "uhohuhoh", "content" => "whatever", :approved => 1) + assert t.invalid? + assert t.errors[:approved].any? + + t = Topic.new("title" => "uhohuhoh", "content" => "whatever", :approved => 1234) + assert t.valid? + end def test_validates_length_of_for_ruby_class Person.validates_length_of :karma, :minimum => 5 |