diff options
author | Jon Leighton <j@jonathanleighton.com> | 2012-05-05 13:40:42 -0700 |
---|---|---|
committer | Jon Leighton <j@jonathanleighton.com> | 2012-05-05 13:40:42 -0700 |
commit | 3af9e721b46d04512653ec356385bcd9e90c3212 (patch) | |
tree | a652d8b3e980b0a29385030c848318e27ad4a6dd /activerecord/lib | |
parent | 06c787d2359660529ff5b927d3d34f22379eef00 (diff) | |
parent | 7f160b06a24547a41a59994a736d6b11beb0c30e (diff) | |
download | rails-3af9e721b46d04512653ec356385bcd9e90c3212.tar.gz rails-3af9e721b46d04512653ec356385bcd9e90c3212.tar.bz2 rails-3af9e721b46d04512653ec356385bcd9e90c3212.zip |
Merge pull request #6054 from flexoid/column-fix
Prevent creating valid time-like objects from blank string from db
Diffstat (limited to 'activerecord/lib')
-rw-r--r-- | activerecord/lib/active_record/connection_adapters/column.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/activerecord/lib/active_record/connection_adapters/column.rb b/activerecord/lib/active_record/connection_adapters/column.rb index 9af8e46120..1933ce2b46 100644 --- a/activerecord/lib/active_record/connection_adapters/column.rb +++ b/activerecord/lib/active_record/connection_adapters/column.rb @@ -158,7 +158,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 @@ -169,14 +169,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? string_to_time "2000-01-01 #{string}" end |