TEST_CODEOWNERS/rails
3
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Ensure content type gets reset after render_to_string [#1182 state:resolved]

Browse Source
This commit is contained in:
Joshua Peek 2008-10-30 15:25:36 -05:00
parent 7857e42103
commit 2092687bcb
2 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
actionpack/lib/action_controller/base.rb
View File

@ -933,6 +933,7 @@ def render(options = nil, extra_options = {}, &block) #:doc:
def render_to_string(options = nil, &block) #:doc: def render_to_string(options = nil, &block) #:doc:
render(options, &block) render(options, &block)
ensure ensure
response.content_type = nil
erase_render_results erase_render_results
reset_variables_added_to_assigns reset_variables_added_to_assigns
end end

10
actionpack/test/controller/render_test.rb
View File

@ -154,6 +154,10 @@ def render_symbol_json
render :json => {:hello => 'world'}.to_json render :json => {:hello => 'world'}.to_json
end end
def render_json_with_render_to_string
render :json => {:hello => render_to_string(:partial => 'partial')}
end
def render_custom_code def render_custom_code
render :text => "hello world", :status => 404 render :text => "hello world", :status => 404
end end
@ -772,6 +776,12 @@ def test_render_symbol_json
assert_equal 'application/json', @response.content_type assert_equal 'application/json', @response.content_type
end end
def test_render_json_with_render_to_string
get :render_json_with_render_to_string
assert_equal '{"hello": "partial html"}', @response.body
assert_equal 'application/json', @response.content_type
end
def test_render_custom_code def test_render_custom_code
get :render_custom_code get :render_custom_code
assert_response 404 assert_response 404