Skip to content

Commit 9ff33fb

Browse files
committed
change from blank? to empty? on obvious string values to save extra method calls
1 parent 4e7292c commit 9ff33fb

File tree

1 file changed

+4
-4
lines changed
  • activerecord/lib/active_record/connection_adapters

1 file changed

+4
-4
lines changed

activerecord/lib/active_record/connection_adapters/column.rb

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -161,7 +161,7 @@ def binary_to_string(value)
161161

162162
def value_to_date(value)
163163
if value.is_a?(String)
164-
return nil if value.blank?
164+
return nil if value.empty?
165165
fast_string_to_date(value) || fallback_string_to_date(value)
166166
elsif value.respond_to?(:to_date)
167167
value.to_date
@@ -172,14 +172,14 @@ def value_to_date(value)
172172

173173
def string_to_time(string)
174174
return string unless string.is_a?(String)
175-
return nil if string.blank?
175+
return nil if string.empty?
176176

177177
fast_string_to_time(string) || fallback_string_to_time(string)
178178
end
179179

180180
def string_to_dummy_time(string)
181181
return string unless string.is_a?(String)
182-
return nil if string.blank?
182+
return nil if string.empty?
183183

184184
dummy_time_string = "2000-01-01 #{string}"
185185

@@ -192,7 +192,7 @@ def string_to_dummy_time(string)
192192

193193
# convert something to a boolean
194194
def value_to_boolean(value)
195-
if value.is_a?(String) && value.blank?
195+
if value.is_a?(String) && value.empty?
196196
nil
197197
else
198198
TRUE_VALUES.include?(value)

0 commit comments

Comments
 (0)