TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Merge pull request #2305 from thedarkone/errors-include

Browse Source 
Improve AM::Errors#include?
This commit is contained in:
Santiago Pastorino 2011-07-27 09:44:15 -07:00
commit fa2bfd832c
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
activemodel/lib/active_model/errors.rb
View File

@ -86,7 +86,7 @@ def clear
# Do the error messages include an error with key +error+? # Do the error messages include an error with key +error+?
def include?(error) def include?(error)
messages.include? error (v = messages[error]) && v.any?
end end
# Get messages for +key+ # Get messages for +key+

1
activemodel/test/cases/errors_test.rb
View File

@ -38,6 +38,7 @@ def test_include?
person.errors[:foo] person.errors[:foo]
assert person.errors.empty? assert person.errors.empty?
assert person.errors.blank? assert person.errors.blank?
assert !person.errors.include?(:foo)
end end
test "method validate! should work" do test "method validate! should work" do