diff options
| author | Mike Virata-Stone <mjstone@on-site.com> | 2016-08-11 21:57:12 -0700 |
|---|---|---|
| committer | Mike Virata-Stone <mjstone@on-site.com> | 2016-08-11 22:05:48 -0700 |
| commit | efd59ab38231eca1084e85aa990321599308757f (patch) | |
| tree | 46040e745fbe79972609294418d92ce0649701e2 /actionpack/lib | |
| parent | fecc67d5152576eb9d918f6a2b90851e8fd6a23c (diff) | |
| download | rails-efd59ab38231eca1084e85aa990321599308757f.tar.gz rails-efd59ab38231eca1084e85aa990321599308757f.tar.bz2 rails-efd59ab38231eca1084e85aa990321599308757f.zip | |
Remove dead code and ensure values are strings before calling gsub
Diffstat (limited to 'actionpack/lib')
| -rw-r--r-- | actionpack/lib/action_view/helpers/tag_helper.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/actionpack/lib/action_view/helpers/tag_helper.rb b/actionpack/lib/action_view/helpers/tag_helper.rb index 6b659a7c00..e39c58ca32 100644 --- a/actionpack/lib/action_view/helpers/tag_helper.rb +++ b/actionpack/lib/action_view/helpers/tag_helper.rb @@ -147,7 +147,6 @@ module ActionView elsif BOOLEAN_ATTRIBUTES.include?(key) attrs << %(#{key}="#{key}") if value elsif !value.nil? - final_value = value.is_a?(Array) ? value.join(" ") : value attrs << tag_option(key, value, escape) end end @@ -159,7 +158,7 @@ module ActionView if value.is_a?(Array) value = escape ? safe_join(value, " ") : value.join(" ") else - value = escape ? ERB::Util.html_escape(value) : value + value = escape ? ERB::Util.html_escape(value) : value.to_s end %(#{key}="#{value.gsub(/"/, '"'.freeze)}") end |