diff options
author | Sean Griffin <sean@seantheprogrammer.com> | 2015-10-29 11:15:19 -0600 |
---|---|---|
committer | Sean Griffin <sean@seantheprogrammer.com> | 2015-10-29 11:15:19 -0600 |
commit | f2ae4af9e122fd1daa95fd7249bdcf1068ebeecb (patch) | |
tree | 8073c86c3f1e025e0e279db2ed1492081dada441 | |
parent | 462698b2c704eff962060ed5851af80f4215958b (diff) | |
download | rails-f2ae4af9e122fd1daa95fd7249bdcf1068ebeecb.tar.gz rails-f2ae4af9e122fd1daa95fd7249bdcf1068ebeecb.tar.bz2 rails-f2ae4af9e122fd1daa95fd7249bdcf1068ebeecb.zip |
Fix style issues with #16252
-rw-r--r-- | actionview/CHANGELOG.md | 8 | ||||
-rw-r--r-- | actionview/lib/action_view/helpers/date_helper.rb | 13 |
2 files changed, 9 insertions, 12 deletions
diff --git a/actionview/CHANGELOG.md b/actionview/CHANGELOG.md index b37a275fe0..65314184c8 100644 --- a/actionview/CHANGELOG.md +++ b/actionview/CHANGELOG.md @@ -1,3 +1,7 @@ +* Allow `date_select` helper selected option to accept hash like the default options. + + *Lecky Lao* + * Collection input propagates input's `id` to the label's `for` attribute when using html options as the last element of collection. @@ -208,8 +212,4 @@ *Daniel Gomez de Souza* -* Allow date_select helper selected option to accept hash like the default options. - - *Lecky Lao* - Please check [4-2-stable](https://github.com/rails/rails/blob/4-2-stable/actionview/CHANGELOG.md) for previous changes. diff --git a/actionview/lib/action_view/helpers/date_helper.rb b/actionview/lib/action_view/helpers/date_helper.rb index c69e4cc77e..233e613e97 100644 --- a/actionview/lib/action_view/helpers/date_helper.rb +++ b/actionview/lib/action_view/helpers/date_helper.rb @@ -845,14 +845,11 @@ module ActionView private %w( sec min hour day month year ).each do |method| define_method(method) do - if @datetime - if @datetime.kind_of?(Hash) - @datetime[method.to_sym] - elsif @datetime.kind_of?(Numeric) - @datetime - else - @datetime.send(method) - end + case @datetime + when Hash then @datetime[method.to_sym] + when Numeric then @datetime + when nil then nil + else @datetime.send(method) end end end |