aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/lib
diff options
context:
space:
mode:
authorJon Leighton <j@jonathanleighton.com>2012-05-05 13:40:42 -0700
committerJon Leighton <j@jonathanleighton.com>2012-05-05 13:40:42 -0700
commit3af9e721b46d04512653ec356385bcd9e90c3212 (patch)
treea652d8b3e980b0a29385030c848318e27ad4a6dd /activerecord/lib
parent06c787d2359660529ff5b927d3d34f22379eef00 (diff)
parent7f160b06a24547a41a59994a736d6b11beb0c30e (diff)
downloadrails-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.rb6
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