Merge pull request #13795 from bobbus/errors-has-key

Fix ActiveModel::Errors#has_key? returning nil

activemodel/lib/active_model/errors.rb

@@ -94,7 +94,7 @@ def clear
     #   person.errors.include?(:name) # => true
     #   person.errors.include?(:age)  # => false
     def include?(attribute)
-      (v = messages[attribute]) && v.any?
+      messages[attribute].present?
     end
     # aliases include?
     alias :has_key? :include?

activemodel/test/cases/errors_test.rb

@@ -51,7 +51,12 @@ def test_dup
   def test_has_key?
     errors = ActiveModel::Errors.new(self)
     errors[:foo] = 'omg'
-    assert errors.has_key?(:foo), 'errors should have key :foo'
+    assert_equal true, errors.has_key?(:foo), 'errors should have key :foo'
+  end
+
+  def test_has_no_key
+    errors = ActiveModel::Errors.new(self)
+    assert_equal false, errors.has_key?(:name), 'errors should not have key :name'
   end
 
   test "clear errors" do