Permalink
Browse files

attribute_present? should return false for empty strings

  • Loading branch information...
1 parent d87ec9d commit 04aca254cc927663c9f53560bbe0d0b680194c5e @Jacobkg Jacobkg committed Mar 7, 2012
@@ -189,7 +189,7 @@ def attribute_for_inspect(attr_name)
# nil nor empty? (the latter only applies to objects that respond to empty?, most notably Strings).
def attribute_present?(attribute)
value = read_attribute(attribute)
- !value.nil? || (value.respond_to?(:empty?) && !value.empty?)
+ !value.nil? && !(value.respond_to?(:empty?) && value.empty?)
end
# Returns the column object for the named attribute.
@@ -30,9 +30,12 @@ def test_attribute_present
t = Topic.new
t.title = "hello there!"
t.written_on = Time.now
+ t.author_name = ""
assert t.attribute_present?("title")
assert t.attribute_present?("written_on")
assert !t.attribute_present?("content")
+ assert !t.attribute_present?("author_name")
+
end
def test_attribute_present_with_booleans

0 comments on commit 04aca25

Please sign in to comment.