length validation for fixnums

Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>

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