aboutsummaryrefslogtreecommitdiffstats
path: root/actionview
diff options
context:
space:
mode:
authorRafael Mendonça França <rafaelmfranca@gmail.com>2014-06-30 17:41:50 -0300
committerRafael Mendonça França <rafaelmfranca@gmail.com>2014-06-30 17:41:50 -0300
commit25466109200919efcbbb8dc4c72f947b891de770 (patch)
tree5373caee612574732981baf62e7f334ebeac3926 /actionview
parentf39595a6307147170781050cb4f1ba4d4a9b358b (diff)
downloadrails-25466109200919efcbbb8dc4c72f947b891de770.tar.gz
rails-25466109200919efcbbb8dc4c72f947b891de770.tar.bz2
rails-25466109200919efcbbb8dc4c72f947b891de770.zip
Use if/else
Since we are using both branches of the code is preferable to use if/else over the early return.
Diffstat (limited to 'actionview')
-rw-r--r--actionview/lib/action_view/helpers/tags/datetime_field.rb7
1 files changed, 5 insertions, 2 deletions
diff --git a/actionview/lib/action_view/helpers/tags/datetime_field.rb b/actionview/lib/action_view/helpers/tags/datetime_field.rb
index 4296e52f21..b2cee9d198 100644
--- a/actionview/lib/action_view/helpers/tags/datetime_field.rb
+++ b/actionview/lib/action_view/helpers/tags/datetime_field.rb
@@ -18,8 +18,11 @@ module ActionView
end
def datetime_value(value)
- return value unless value.is_a? String
- DateTime.parse(value) rescue nil
+ if value.is_a? String
+ DateTime.parse(value) rescue nil
+ else
+ value
+ end
end
end
end