Rails 3: abandono el envío de correo dentro de la acción de ActionMailer

Question

Me pregunto cómo podría abandonar de manera condicional el envío de correo dentro de la acción ActionMailer en sí misma.

 

class SomeMailer < ActionMailer::Base 
    ... 

    def some_emails 
    some_models = Model.where(:a => 1) 
    if !some_models.blank? 
     mail(...) 
    else 
     # What to add here? 
     # render :nothing => true doesn't work 
    end 
    end 

end 
 

Ahora invocando esto a través de SomeMailer.some_emails.deliver! devuelve un ArgumentError: A sender (Return-Path, Sender or From) required to send a message

Answer 1

su lugar poner sus condiciones en el lugar donde se está realizando la llamada a SomeMailer.some_emails.deliver!

Answer 2

Tuve el mismo problema. No hay forma real de hacerlo dentro de la acción ActionMailer así que hice lo siguiente en mi tarea cron:

users.each do |user| 
    begin 
    UserMailer.event_second_reminder_group_user_email(user).deliver 
    puts " - sending reminder email to user #{user.email}" 
    rescue 
    end 
end 
puts "Complete!" 

Ahora bien, si se produce un error, no se rompe la aplicación!

Answer 3

Lo extraño es que, con Rails 3.1.rc4 y WEBrick, funciona bien en mi servidor web WEBrick local. Pero tan pronto como me empuje a Heroku pila de cedro, su WEBrick lanza el

ArgumentError: A sender (Return-Path, Sender or From)

Tienes que eliminar las sentencias condicionales como se indica en la respuesta anterior. Se soluciona el problema de modo que también trabaja en Heroku, no sólo su máquina local

Answer 4

Set perform_deliveries a falso, así:

emails = get_email_list_somehow 
if emails.present? 
    mail options.merge(:bcc => emails) 
else 
    self.message.perform_deliveries = false 
end 

Este silencio no intentará enviar y debe parar el error suceda.

Answer 5

En Rails 3.2.9 puede llamar condicionalmente al mail(). Aquí está el relacionado GitHub thread. Ahora el código del OP se puede volver a trabajar así:

class SomeMailer < ActionMailer::Base 
    ... 

    def some_emails 
    some_models = Model.where(:a => 1) 
    unless some_models.blank? 
     mail(...) 
    end 
    end 

end