2016-03-29 6 views
1

Ich versuche, mehrere E-Mails an mehrere Marken zu senden. Ich habe versucht, eine Schleife zu machen, die diese mehrfach durchläuft, während ich die E-Mail des Mail-Empfängers ändere, aber sie wird nur an die letzte Mail-Anfrage gesendet. Wie würde ich das Problem lösen? hier ist mein aktueller Code:Schienen senden mehrere E-Mails?

@user_products.each do |p| 
    @brands << p.brand 
end 
count = 0 
@brands = @brands.uniq 
while count < @brands.count 
    debugger 
     @brand = @brands[count] 
      mail(to: @brands[count].email, subject: 'A purchase has been made!') 
      count += 1 
    end 

Mein HTML-Code, die Produkte auf gemacht durch die bestimmte Marke zu erhalten:

<body style="margin-left: auto; margin-right: auto; background-color: #d55d5d; width: 75%;"> 
    <div align="center"> 
     <img src="http://localhost:3000/assets/motobanner-d2fde8a6e30060905a6f72b0c8128d222a7596e690a7a4c648e5e81109bf4600.jpg" style="margin-top: 0% ;width: 75%; height: 240px;" ></img> 
    </div> 
    <div align="center"> 
     <h2>Hello, <%= @brand.name %>!</h2> 
      <p style="font-size: 1.2em;">There has been a purchase of the following items:</p> 
      <% @user_products.each do |item| %> 
       <div align="center"> <%= image_tag "http://localhost:3000/#{item.picture.url}", alt: item.product_name if item.picture && item.brand_id == @brand.id %><br /></div> 
       <b><%= "Product name:" if item.brand_id == @brand.id %></b> <%= item.product_name if item.brand_id == @brand.id %> 
       <b><%= "Product description:" if item.brand_id == @brand.id %></b><br /> <p class="product-description"><%= item.product_description if item.brand_id == @brand.id %></p> 
       <b><%= "Product price:" if item.brand_id == @brand.id %></b> <%= "£" + item.product_price.to_s if item.brand_id == @brand.id %> 
      <% end %> 
</body> 

Der Debugger für meine E-Mail ist es, wie ich durch die Schleife lief und es lief die mail(to: @brands[count].email, subject: 'A purchase has been made!') Zeile jedes Mal, wenn es geloopt, weshalb ich nicht verstehe, warum es nicht funktioniert.

+0

Wenn ich mich richtig erinnere, hat das Mail-Objekt eine send-Methode Was passiert ist, ist, dass Sie effektiv all diese Mails erstellen, aber Schienen sendet nur die letzte für Sie, da es die eine ist das wird zuletzt zurückgegeben – Sidewinder94

+0

Ah ich sehe Gibt es einen besseren Weg, diese Aktion durchzuführen? –

Antwort

1

Sie sollen die Schleife außerhalb des Mailer tun, wie es in offiziellem Beispiel:

class SendWeeklySummary 
    def run 
    User.find_each do |user| 
     UserMailer.weekly_summary(user).deliver_now 
    end 
    end 
end 

wie hier erklärt: http://guides.rubyonrails.org/action_mailer_basics.html und Sie sind wahrscheinlich den deliver_now Teil fehlen (kann durch deliver_later und ausgeführt durch Arbeiter ersetzt werden process