This project is archived and is in readonly mode.

#6623 new
Jacob Tjørnholm

render_to_string in mailer causes subsequent render to fail

Reported by Jacob Tjørnholm | March 25th, 2011 @ 02:31 PM

I'm using render_to_string in a mailer, and this prevents the rendering of the actual mail from finding the template.

This seems to be caused by @lookup_context being initialized on the mailer by render_to_string, then reused when rendering the mail. And this breaks because the mail extension (*.text.erb) is not found by the lookup context.

Here's a minimal example:

class NoteMailer < ActionMailer::Base
  default :from => ""

  def foo
    @notes = []
    render_to_string(:template => "notes/index.html.erb") # <-- Removing this line makes the test pass
    # self.instance_variable_set(:@lookup_context, nil) # <-- So does enabling this
    mail(:to => "")

I'm attaching a minimal project containing a functional test that demonstrates this.

Comments and changes to this ticket

  • bhu

    bhu April 6th, 2011 @ 04:31 PM

    • Tag changed from actioncontroller, actionmailer to actioncontroller, actionmailer, rendering

    think i got the same error

     def cancellation(contract, user)
        @contract = contract
        kit ='pdf/cancellation', :layout => false))
        kit.stylesheets << 'private/stylesheets/cancellation.css'
        attachments['Cancellation.pdf'] = kit.to_pdf
        options = {}
        options[:to] =
        options[:from] =
        options[:bcc] = if Variable.send_employee_email_bcc
        options[:subject] = "Cancellation"
        mail options

    it renders nothing except the attachment which breaks the mail completely if there is no body.

    the format block helps getting around this but i dont think that it's intended for this kind of situation

    mail options do |format|

    with this block the default behavior is some kind of restored. you have to explicitly specify every format. per real default rails is auto detecting which formats are available.

    Sorry for not really having a solution, just a workaround :/

Create your profile

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href=""></a>

People watching this ticket