From efd59ab38231eca1084e85aa990321599308757f Mon Sep 17 00:00:00 2001
From: Mike Virata-Stone <mjstone@on-site.com>
Date: Thu, 11 Aug 2016 21:57:12 -0700
Subject: [PATCH] Remove dead code and ensure values are strings before calling
 gsub

---
 actionpack/lib/action_view/helpers/tag_helper.rb | 3 +--
 actionpack/test/template/tag_helper_test.rb      | 8 ++++++++
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/actionpack/lib/action_view/helpers/tag_helper.rb b/actionpack/lib/action_view/helpers/tag_helper.rb
index 6b659a7c00b0f..e39c58ca32563 100644
--- a/actionpack/lib/action_view/helpers/tag_helper.rb
+++ b/actionpack/lib/action_view/helpers/tag_helper.rb
@@ -147,7 +147,6 @@ def tag_options(options, escape = true)
               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 @@ def tag_option(key, value, escape)
           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(/"/, '&quot;'.freeze)}")
         end
diff --git a/actionpack/test/template/tag_helper_test.rb b/actionpack/test/template/tag_helper_test.rb
index 9c3d636765b8f..e38c9beb8c858 100644
--- a/actionpack/test/template/tag_helper_test.rb
+++ b/actionpack/test/template/tag_helper_test.rb
@@ -29,6 +29,14 @@ def test_tag_options_accepts_blank_option
     assert_equal "<p included=\"\" />", tag("p", :included => '')
   end
 
+  def test_tag_options_accepts_symbol_option_when_not_escaping
+    assert_equal "<p value=\"symbol\" />", tag("p", { :value => :symbol }, false, false)
+  end
+
+  def test_tag_options_accepts_integer_option_when_not_escaping
+    assert_equal "<p value=\"42\" />", tag("p", { :value => 42 }, false, false)
+  end
+
   def test_tag_options_converts_boolean_option
     assert_equal '<p disabled="disabled" multiple="multiple" readonly="readonly" />',
       tag("p", :disabled => true, :multiple => true, :readonly => true)