kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Revert "Merge pull request #9784 from vipulnsward/change_from_blank_to_empty_on_string"

Browse source 
This reverts commit 9c4c05fc82, reversing
changes made to 4620bdcefd.

Reason: They're not completely interchangeable, since blank? will also
check for strings containing spaces.
This commit is contained in:
Carlos Antonio da Silva 2013-03-18 18:46:48 -03:00
parent 9c4c05fc82
commit 111611b511
1 changed files with 4 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
activerecord/lib/active_record/connection_adapters/column.rb
View file

@ -161,7 +161,7 @@ module ActiveRecord
def value_to_date(value)
if value.is_a?(String)
return nil if value.empty?
return nil if value.blank?
fast_string_to_date(value) || fallback_string_to_date(value)
elsif value.respond_to?(:to_date)
value.to_date
@ -172,14 +172,14 @@ module ActiveRecord
def string_to_time(string)
return string unless string.is_a?(String)
return nil if string.empty?
return nil if string.blank?
fast_string_to_time(string) || fallback_string_to_time(string)
end
def string_to_dummy_time(string)
return string unless string.is_a?(String)
return nil if string.empty?
return nil if string.blank?
dummy_time_string = "2000-01-01 #{string}"
@ -192,7 +192,7 @@ module ActiveRecord
# convert something to a boolean
def value_to_boolean(value)
if value.is_a?(String) && value.empty?
if value.is_a?(String) && value.blank?
nil
else
TRUE_VALUES.include?(value)