From 4db432a4ca6018ec70df686c1e2f80a656acca4e Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Thu, 7 Jun 2018 17:23:12 +0200 Subject: [PATCH 1/4] Remove Mailers module for app mailers --- app/mailers/new_attestation_mailer.rb | 44 +++++++++---------- app/mailers/resend_attestation_mailer.rb | 36 +++++++-------- lib/tasks/2018_06_05_resend_attestations.rb | 2 +- .../2018_06_05_send_new_attestations.rake | 2 +- 4 files changed, 40 insertions(+), 44 deletions(-) diff --git a/app/mailers/new_attestation_mailer.rb b/app/mailers/new_attestation_mailer.rb index 8a8e5edea..70706316c 100644 --- a/app/mailers/new_attestation_mailer.rb +++ b/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 diff --git a/app/mailers/resend_attestation_mailer.rb b/app/mailers/resend_attestation_mailer.rb index 5997c12b9..f8a99855c 100644 --- a/app/mailers/resend_attestation_mailer.rb +++ b/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 diff --git a/lib/tasks/2018_06_05_resend_attestations.rb b/lib/tasks/2018_06_05_resend_attestations.rb index 5c6527727..bc7231f2b 100644 --- a/lib/tasks/2018_06_05_resend_attestations.rb +++ b/lib/tasks/2018_06_05_resend_attestations.rb @@ -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 diff --git a/lib/tasks/2018_06_05_send_new_attestations.rake b/lib/tasks/2018_06_05_send_new_attestations.rake index b9af48a26..672e8a0fc 100644 --- a/lib/tasks/2018_06_05_send_new_attestations.rake +++ b/lib/tasks/2018_06_05_send_new_attestations.rake @@ -37,7 +37,7 @@ namespace :'2018_06_05_send_new_attestation' do dossier.attestation = dossier.build_attestation - Mailers::NewAttestationMailer.new_attestation(dossier).deliver_later + NewAttestationMailer.new_attestation(dossier).deliver_later puts "Email envoyé à #{email} pour le dossier #{dossier.id}" end end From 96a4ebd860bdf8fdc689360a07fab169636b9871 Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Thu, 7 Jun 2018 17:25:39 +0200 Subject: [PATCH 2/4] Improve a task --- lib/tasks/2018_06_05_send_new_attestations.rake | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/lib/tasks/2018_06_05_send_new_attestations.rake b/lib/tasks/2018_06_05_send_new_attestations.rake index 672e8a0fc..8ce87337a 100644 --- a/lib/tasks/2018_06_05_send_new_attestations.rake +++ b/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 NewAttestationMailer.new_attestation(dossier).deliver_later puts "Email envoyé à #{email} pour le dossier #{dossier.id}" + puts end end end From d3e96b77dd41ebfa9c267f301941364a2017fbaa Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Thu, 7 Jun 2018 17:26:15 +0200 Subject: [PATCH 3/4] Fix a bug in a task --- lib/tasks/2018_06_05_send_new_attestations.rake | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/tasks/2018_06_05_send_new_attestations.rake b/lib/tasks/2018_06_05_send_new_attestations.rake index 8ce87337a..0f8fa6a3b 100644 --- a/lib/tasks/2018_06_05_send_new_attestations.rake +++ b/lib/tasks/2018_06_05_send_new_attestations.rake @@ -43,7 +43,7 @@ namespace :'2018_06_05_send_new_attestation' do dossier.attestation = dossier.build_attestation NewAttestationMailer.new_attestation(dossier).deliver_later - puts "Email envoyé à #{email} pour le dossier #{dossier.id}" + puts "Email envoyé à #{dossier.user.email} pour le dossier #{dossier.id}" puts end end From 8bd322f7b087b6d65b5408ee3d554865a289c71f Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Thu, 7 Jun 2018 17:28:56 +0200 Subject: [PATCH 4/4] Fix a filename --- ...resend_attestations.rb => 2018_06_05_resend_attestations.rake} | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename lib/tasks/{2018_06_05_resend_attestations.rb => 2018_06_05_resend_attestations.rake} (100%) diff --git a/lib/tasks/2018_06_05_resend_attestations.rb b/lib/tasks/2018_06_05_resend_attestations.rake similarity index 100% rename from lib/tasks/2018_06_05_resend_attestations.rb rename to lib/tasks/2018_06_05_resend_attestations.rake