diff --git a/actionpack/CHANGELOG b/actionpack/CHANGELOG index 939e9f5b8f36ea665d5e18d0dd18b3f82a29b712..d8a898f8ef3a03946186e59eea5cc02d2fb74dcc 100644 --- a/actionpack/CHANGELOG +++ b/actionpack/CHANGELOG @@ -1,5 +1,7 @@ *SVN* +* Don't include a layout when rendering an rjs template using render's :template option. [Marcel Molina Jr.] + * Update to script.aculo.us 1.5.0 final (equals 1.5.0_rc6) [Thomas Fuchs] * Update to Prototype 1.4.0 final [Sam Stephenson] diff --git a/actionpack/lib/action_controller/base.rb b/actionpack/lib/action_controller/base.rb index 9aab2c50286082b6f13fd63096cde5a1cea92de9..c1adbf3b8adc90f4abbd271a4b1035ccc550c1f4 100755 --- a/actionpack/lib/action_controller/base.rb +++ b/actionpack/lib/action_controller/base.rb @@ -937,7 +937,17 @@ def assert_existance_of_template_file(template_name) end def default_template_name(default_action_name = action_name) + default_action_name = default_action_name.dup + strip_out_controller!(default_action_name) if template_path_includes_controller?(default_action_name) "#{self.class.controller_path}/#{default_action_name}" end + + def strip_out_controller!(path) + path.replace path.split('/', 2).last + end + + def template_path_includes_controller?(path) + path.to_s['/'] && self.class.controller_path.split('/')[-1] == path.split('/')[0] + end end end diff --git a/actionpack/lib/action_controller/layout.rb b/actionpack/lib/action_controller/layout.rb index d6a4789befbc2cf87ef95bcf943efa151b5c7f89..3ec125f08817df0a726a1967f4315188e235510b 100644 --- a/actionpack/lib/action_controller/layout.rb +++ b/actionpack/lib/action_controller/layout.rb @@ -237,7 +237,7 @@ def apply_layout?(template_with_options, options) def candidate_for_layout?(options) (options.has_key?(:layout) && options[:layout] != false) || options.values_at(:text, :file, :inline, :partial, :nothing).compact.empty? && - !template_exempt_from_layout?(default_template_name(options[:action])) + !template_exempt_from_layout?(default_template_name(options[:action] || options[:template])) end def pick_layout(template_with_options, options, deprecated_layout) diff --git a/actionpack/test/controller/new_render_test.rb b/actionpack/test/controller/new_render_test.rb index a1ed9f71d06f9f3db2e180b355ccd6465b40550a..5652803e1d90e01a50c773d801c27efe15a97c6b 100644 --- a/actionpack/test/controller/new_render_test.rb +++ b/actionpack/test/controller/new_render_test.rb @@ -209,7 +209,10 @@ def determine_layout "accessing_params_in_template", "accessing_params_in_template_with_layout", "render_with_explicit_template", - "test_rendering_rjs_action_explicitly" + "render_js_with_explicit_template", + "render_js_with_explicit_action_template", + "delete_with_js" + "layouts/standard" when "builder_layout_test" "layouts/builder"