diff options
author | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-06-30 17:41:50 -0300 |
---|---|---|
committer | Rafael Mendonça França <rafaelmfranca@gmail.com> | 2014-06-30 17:41:50 -0300 |
commit | 25466109200919efcbbb8dc4c72f947b891de770 (patch) | |
tree | 5373caee612574732981baf62e7f334ebeac3926 /actionview/lib/action_view/helpers | |
parent | f39595a6307147170781050cb4f1ba4d4a9b358b (diff) | |
download | rails-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/lib/action_view/helpers')
-rw-r--r-- | actionview/lib/action_view/helpers/tags/datetime_field.rb | 7 |
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 |