Merge pull request #5792 from lest/patch-4

no need in temporary array in number_to_phone helper
rails · Apr 11, 2012 · 6da5002 · 6da5002
2 parents b79b784 + 332c3e1
commit 6da5002
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/actionpack/lib/action_view/helpers/number_helper.rb b/actionpack/lib/action_view/helpers/number_helper.rb
@@ -74,11 +74,11 @@ def number_to_phone(number, options = {})
           number.slice!(0, 1) if number.start_with?(delimiter) && !delimiter.blank?
         end
 
-        str = []
+        str = ''
         str << "+#{country_code}#{delimiter}" unless country_code.blank?
         str << number
         str << " x #{extension}" unless extension.blank?
-        ERB::Util.html_escape(str.join)
+        ERB::Util.html_escape(str)
       end
 
       # Formats a +number+ into a currency string (e.g., $13.65). You can customize the format