Rails 3 - abandonner l'envoi de courrier dans l'action ActionMailer

Question

Je me demande comment je pourrais abandonner conditionnellement l'envoi de courrier dans l'action ActionMailer elle-même.

 

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 
 

Maintenant, l'invocation de cette grâce SomeMailer.some_emails.deliver! retourne un ArgumentError: A sender (Return-Path, Sender or From) required to send a message

Answer 1

mettre vos conditions lieu à l'endroit où vous faites l'appel à SomeMailer.some_emails.deliver!

Answer 2

J'ai eu ce même problème. Il n'y a aucun moyen de le faire dans l'action ActionMailer donc je ne suit dans ma tâche 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!" 

Maintenant, si une erreur est renvoyée, il ne rompt pas l'application!

Answer 3

La chose étrange est que, avec Rails 3.1.rc4 et WEBrick, cela fonctionne très bien sur mon serveur web WEBrick local. Mais dès que je pousse à pile de cèdre Heroku, leur WEBrick jette le

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

Vous devez supprimer les instructions conditionnelles comme indiqué dans la réponse ci-dessus. Ce problème est résolu pour que cela fonctionne aussi sur Heroku, pas seulement votre machine locale

Answer 4

Set perform_deliveries false, comme suit:

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

Cela va essayer tranquillement de ne pas envoyer et devrait cesser l'erreur de se produire.

Answer 5

Dans Rails 3.2.9, vous pouvez enfin appeler le mail(). Voici les GitHub thread connexes. Maintenant, le code de l'OP peut être retravaillé comme ceci:

class SomeMailer < ActionMailer::Base 
    ... 

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

end