[PATCH] Doc RXML rendering shouldn't use RHTML layouts

Slight fix, the curl command should be: curl -H "Accept: application/xml" http://localhost:4000/test/my_action

Workaround, add the following to your application controller:

alias true_render render
    
def render(options = nil, deprecated_status = nil, &block)
  options[:layout] = false if options[:action] and options[:action].match /.*\.rxml$/
  true_render options, deprecated_status, &block
end

There's a bug in my previous comment, if options is nil it'll die:

workaround v2

alias true_render render
    
def render(options = nil, deprecated_status = nil, &block)
  options[:layout] = false if options and options[:action] and options[:action].match /.*\.rxml$/
  true_render options, deprecated_status, &block
end

Version 3: this handles if you do render action => :action_name

    alias true_render render
    
    def render(options = nil, deprecated_status = nil, &block)
        options[:layout] = false if options and options[:action] and options[:action].to_s.match /.*\.rxml$/
        true_render options, deprecated_status, &block
    end

Without the workaround not even overrideing the :layout option works.

format.xml {render :layout => false }

This does still render the layout.

It might be related to what was done in #307.

If I remove the rhtml layout and tries to access with xml format using Firefox I get a error. In my example the url is "http://localhost:3000/users.xml" ...

 Errno::ENOENT in Users#index

 Showing app/views/layouts/users.rhtml where line # raised:

 No such file or directory - script/../config/../app/views/layouts/users.rhtml


 vendor/rails/actionpack/lib/action_view/base.rb:440:in `mtime'
 vendor/rails/actionpack/lib/action_view/base.rb:440:in `compile_template?'
 vendor/rails/actionpack/lib/action_view/base.rb:317:in `compile_and_render_template'
 vendor/rails/actionpack/lib/action_view/base.rb:301:in `render_template'
 vendor/rails/actionpack/lib/action_view/base.rb:260:in `render_file'
 vendor/rails/actionpack/lib/action_controller/layout.rb:254:in `render_without_benchmark'
 vendor/rails/actionpack/lib/action_controller/benchmarking.rb:50:in `render'
 /usr/lib/ruby/1.8/benchmark.rb:293:in `measure'
 vendor/rails/actionpack/lib/action_controller/benchmarking.rb:50:in `render'
 app/controllers/users_controller.rb:7:in `index'
 app/controllers/users_controller.rb:7:in `index'

 Request

 Parameters: {"format"=>"xml"}

 --- 
 flash: !map:ActionController::Flash::FlashHash {}


 Response
 Headers: {"Status"=>"200 OK", "cookie"=>[], "Content-Type"=>"application/xml", "Cache-Control"=>"no-cache"}

I haven't looked at the code for #307, but looking at the description that definitely seems like it would be the root of the problem.

crippledcanary: did you restart your server before you tried accessing that url (after you deleted the .rhtml file)? The server (mongrel at least) I don't think picks up those sorts of changes.

I have the same issue. While the workarounds might be a temporary solution. I think a more permanent fix is applicable.

Replying to jwieland:

I have the same issue. While the workarounds might be a temporary solution. I think a more permanent fix is applicable.

Pardon me if I'm wrong but, after digging through ActionController and ActionView in 1.2.3, this is actually intended behavior. You're allowed to mix and match ERB and Builder templates to your heart's content.

If you don't want Builder templates rendered with layouts in your app, you can specify that with the following snippet in your ApplicationController:

exempt_from_layout :rxml

I certainly agree that method could be documented better, especially with all of the blogger tutorials on respond_to ignoring it completely.

Hope this helps.

Attached is a doc-patch that better explains the use of exempt_from_layout in the context of respond_to.