Handle the empty array correctly.

cyu · Jul 27, 2011 · efe4cbe · efe4cbe
1 parent 02691d3
commit efe4cbe
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/activemodel/lib/active_model/errors.rb b/activemodel/lib/active_model/errors.rb
@@ -86,7 +86,7 @@ def clear
 
     # Do the error messages include an error with key +error+?
     def include?(error)
-      messages.include? error
+      (v = messages[error]) && v.any?
     end
 
     # Get messages for +key+

diff --git a/activemodel/test/cases/errors_test.rb b/activemodel/test/cases/errors_test.rb
@@ -38,6 +38,7 @@ def test_include?
     person.errors[:foo]
     assert person.errors.empty?
     assert person.errors.blank?
+    assert !person.errors.include?(:foo)
   end
 
   test "method validate! should work" do