Merge pull request #2064 from betagouv/more-fixes

More fixes

app/mailers/new_attestation_mailer.rb

@@ -1,33 +1,31 @@
-module Mailers
-  class NewAttestationMailer < ApplicationMailer
-    include Rails.application.routes.url_helpers
+class NewAttestationMailer < ApplicationMailer
+  include Rails.application.routes.url_helpers
 
-    def new_attestation(dossier)
-      to = dossier.user.email
-      subject = "Nouvelle attestation pour votre dossier nº #{dossier.id}"
+  def new_attestation(dossier)
+    to = dossier.user.email
+    subject = "Nouvelle attestation pour votre dossier nº #{dossier.id}"
 
-      mail(to: to, subject: subject, body: body(dossier))
-    end
+    mail(to: to, subject: subject, body: body(dossier))
+  end
 
-    private
+  private
 
-    def body(dossier)
-      <<~HEREDOC
-        Bonjour,
+  def body(dossier)
+    <<~HEREDOC
+      Bonjour,
 
-        Votre dossier nº #{dossier.id} (procédure "#{dossier.procedure.libelle}") a subi, à un moment, un "aller-retour" :
-        - Acceptation de votre dossier
-        - Passage en instruction du dossier car besoin de le modifier
-        - Seconde acceptation de votre dossier
+      Votre dossier nº #{dossier.id} (procédure "#{dossier.procedure.libelle}") a subi, à un moment, un "aller-retour" :
+      - Acceptation de votre dossier
+      - Passage en instruction du dossier car besoin de le modifier
+      - Seconde acceptation de votre dossier
 
-        Suite à cette opération, l'attestation liée à votre dossier n'a pas été regénérée.
-        Ce problème est désormais reglé, votre nouvelle attestation est disponible à l'adresse suivante :
-        #{dossier_attestation_url(dossier)}
+      Suite à cette opération, l'attestation liée à votre dossier n'a pas été regénérée.
+      Ce problème est désormais reglé, votre nouvelle attestation est disponible à l'adresse suivante :
+      #{dossier_attestation_url(dossier)}
 
-        Cordialement,
+      Cordialement,
 
-        L’équipe demarches-simplifiees.fr
-      HEREDOC
-    end
+      L’équipe demarches-simplifiees.fr
+    HEREDOC
   end
 end

app/mailers/resend_attestation_mailer.rb

@@ -1,29 +1,27 @@
-module Mailers
-  class ResendAttestationMailer < ApplicationMailer
-    include Rails.application.routes.url_helpers
+class ResendAttestationMailer < ApplicationMailer
+  include Rails.application.routes.url_helpers
 
-    def resend_attestation(dossier)
-      to = dossier.user.email
-      subject = "Nouvelle attestation pour votre dossier nº #{dossier.id}"
+  def resend_attestation(dossier)
+    to = dossier.user.email
+    subject = "Nouvelle attestation pour votre dossier nº #{dossier.id}"
 
-      mail(to: to, subject: subject, body: body(dossier))
-    end
+    mail(to: to, subject: subject, body: body(dossier))
+  end
 
-    private
+  private
 
-    def body(dossier)
-      <<~HEREDOC
-        Bonjour,
+  def body(dossier)
+    <<~HEREDOC
+      Bonjour,
 
-        L'attestation de votre dossier nº #{dossier.id} (procédure "#{dossier.procedure.libelle}") a été modifiée.
+      L'attestation de votre dossier nº #{dossier.id} (procédure "#{dossier.procedure.libelle}") a été modifiée.
 
-        Votre nouvelle attestation est disponible à l'adresse suivante :
-        #{dossier_attestation_url(dossier)}
+      Votre nouvelle attestation est disponible à l'adresse suivante :
+      #{dossier_attestation_url(dossier)}
 
-        Cordialement,
+      Cordialement,
 
-        L’équipe demarches-simplifiees.fr
-      HEREDOC
-    end
+      L’équipe demarches-simplifiees.fr
+    HEREDOC
   end
 end

lib/tasks/2018_06_05_resend_attestations.rake

@@ -11,7 +11,7 @@ namespace :'2018_06_05_resend_attestations' do
 
       dossier.attestation = dossier.build_attestation
 
-      Mailers::ResendAttestationMailer.resend_attestation(dossier).deliver_later
+      ResendAttestationMailer.resend_attestation(dossier).deliver_later
       puts "Email envoyé à #{email} pour le dossier #{dossier.id}"
     end
   end

lib/tasks/2018_06_05_send_new_attestations.rake

@@ -33,12 +33,18 @@ namespace :'2018_06_05_send_new_attestation' do
 
     dossiers.each do |dossier|
       attestation = dossier.attestation
-      attestation.destroy
+
+      if attestation
+        id = attestation.id
+        attestation.destroy
+        puts "Attestation #{id} détruite"
+      end
 
       dossier.attestation = dossier.build_attestation
 
-      Mailers::NewAttestationMailer.new_attestation(dossier).deliver_later
-      puts "Email envoyé à #{email} pour le dossier #{dossier.id}"
+      NewAttestationMailer.new_attestation(dossier).deliver_later
+      puts "Email envoyé à #{dossier.user.email} pour le dossier #{dossier.id}"
+      puts
     end
   end
 end