kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

doesn't modify params in SafeBuffer#%

Browse source
This commit is contained in:
Vasiliy Ermolovich 2012-05-18 18:17:43 +03:00
parent 62c6ac2f8d
commit 3b1c30c99f
2 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
activesupport/lib/active_support/core_ext/string/output_safety.rb
View file

@ -151,9 +151,7 @@ module ActiveSupport #:nodoc:
end
def %(args)
args = Array(args)
args.map! do |arg|
args = Array(args).map do |arg|
if !html_safe? || arg.html_safe?
arg
else

7
activesupport/test/core_ext/string_ext_test.rb
View file

@ -463,6 +463,13 @@ class OutputSafetyTest < ActiveSupport::TestCase
assert @other_string.html_safe?
end
test "Concatting with % doesn't modify a string" do
@other_string = ["<p>", "<b>", "<h1>"]
"%s %s %s".html_safe % @other_string
assert_equal ["<p>", "<b>", "<h1>"], @other_string
end
test "Concatting a fixnum to safe always yields safe" do
string = @string.html_safe
string = string.concat(13)