From a3f5ae9d0b1383d72636bbdd2d863c3ad2c76379 Mon Sep 17 00:00:00 2001 From: gregoirenovel Date: Thu, 31 May 2018 15:26:02 +0200 Subject: [PATCH] Add a variable to improve readability --- app/mailers/administrateur_mailer.rb | 3 ++- app/mailers/administration_mailer.rb | 13 +++++++++---- app/mailers/avis_mailer.rb | 3 ++- app/mailers/dossier_mailer.rb | 6 ++++-- app/mailers/gestionnaire_mailer.rb | 10 +++++++--- app/mailers/invite_mailer.rb | 6 ++++-- app/mailers/user_mailer.rb | 3 ++- 7 files changed, 30 insertions(+), 14 deletions(-) diff --git a/app/mailers/administrateur_mailer.rb b/app/mailers/administrateur_mailer.rb index 7f2c647da..91d289672 100644 --- a/app/mailers/administrateur_mailer.rb +++ b/app/mailers/administrateur_mailer.rb @@ -5,9 +5,10 @@ class AdministrateurMailer < ApplicationMailer @administrateur = administrateur @reset_password_token = reset_password_token @expiration_date = @administrateur.reset_password_sent_at + Devise.reset_password_within + subject = "demarches-simplifiees.fr - N'oubliez pas d'activer votre compte administrateur" mail(to: administrateur.email, - subject: "demarches-simplifiees.fr - N'oubliez pas d'activer votre compte administrateur", + subject: subject, reply_to: "contact@demarches-simplifiees.fr") end end diff --git a/app/mailers/administration_mailer.rb b/app/mailers/administration_mailer.rb index c9a584de5..1f2b222ea 100644 --- a/app/mailers/administration_mailer.rb +++ b/app/mailers/administration_mailer.rb @@ -4,30 +4,35 @@ class AdministrationMailer < ApplicationMailer def new_admin_email(admin, administration) @admin = admin @administration = administration + subject = "Création d'un compte Admin demarches-simplifiees.fr" mail(to: 'tech@demarches-simplifiees.fr', - subject: "Création d'un compte Admin demarches-simplifiees.fr") + subject: subject) end def invite_admin(admin, reset_password_token) @reset_password_token = reset_password_token @admin = admin + subject = "demarches-simplifiees.fr - Activez votre compte administrateur" mail(to: admin.email, - subject: "demarches-simplifiees.fr - Activez votre compte administrateur", + subject: subject, reply_to: "contact@demarches-simplifiees.fr") end def refuse_admin(admin_email) + subject = "demarches-simplifiees.fr - Votre demande de compte a été refusée" + mail(to: admin_email, - subject: "demarches-simplifiees.fr - Votre demande de compte a été refusée", + subject: subject, reply_to: "contact@demarches-simplifiees.fr") end def dubious_procedures(procedures_and_type_de_champs) @procedures_and_type_de_champs = procedures_and_type_de_champs + subject = "[RGS] De nouvelles procédures comportent des champs interdits" mail(to: 'equipe@demarches-simplifiees.fr', - subject: "[RGS] De nouvelles procédures comportent des champs interdits") + subject: subject) end end diff --git a/app/mailers/avis_mailer.rb b/app/mailers/avis_mailer.rb index 2fd8401f4..2580f4be6 100644 --- a/app/mailers/avis_mailer.rb +++ b/app/mailers/avis_mailer.rb @@ -2,7 +2,8 @@ class AvisMailer < ApplicationMailer def avis_invitation(avis) @avis = avis email = @avis.gestionnaire&.email || @avis.email + subject = "Donnez votre avis sur le dossier nº #{@avis.dossier.id} (#{@avis.dossier.procedure.libelle})" - mail(to: email, subject: "Donnez votre avis sur le dossier nº #{@avis.dossier.id} (#{@avis.dossier.procedure.libelle})") + mail(to: email, subject: subject) end end diff --git a/app/mailers/dossier_mailer.rb b/app/mailers/dossier_mailer.rb index 10fd85a14..96e2b46b0 100644 --- a/app/mailers/dossier_mailer.rb +++ b/app/mailers/dossier_mailer.rb @@ -3,13 +3,15 @@ class DossierMailer < ApplicationMailer def notify_deletion_to_user(deleted_dossier, to_email) @deleted_dossier = deleted_dossier + subject = "Votre dossier n° #{@deleted_dossier.dossier_id} a bien été supprimé" - mail(to: to_email, subject: "Votre dossier n° #{@deleted_dossier.dossier_id} a bien été supprimé") + mail(to: to_email, subject: subject) end def notify_deletion_to_administration(deleted_dossier, to_email) @deleted_dossier = deleted_dossier + subject = "Le dossier n° #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager" - mail(to: to_email, subject: "Le dossier n° #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager") + mail(to: to_email, subject: subject) end end diff --git a/app/mailers/gestionnaire_mailer.rb b/app/mailers/gestionnaire_mailer.rb index 7410192ff..c3e3edc6a 100644 --- a/app/mailers/gestionnaire_mailer.rb +++ b/app/mailers/gestionnaire_mailer.rb @@ -4,21 +4,25 @@ class GestionnaireMailer < ApplicationMailer def invite_gestionnaire(gestionnaire, reset_password_token) @reset_password_token = reset_password_token @gestionnaire = gestionnaire + subject = "demarches-simplifiees.fr - Activez votre compte accompagnateur" mail(to: gestionnaire.email, - subject: "demarches-simplifiees.fr - Activez votre compte accompagnateur", + subject: subject, reply_to: "contact@demarches-simplifiees.fr") end def user_to_gestionnaire(email) - send_mail(email, nil, "Vous avez été nommé accompagnateur sur demarches-simplifiees.fr") + subject = "Vous avez été nommé accompagnateur sur demarches-simplifiees.fr" + + send_mail(email, nil, subject) end def last_week_overview(gestionnaire) headers['X-mailjet-campaign'] = 'last_week_overview' overview = gestionnaire.last_week_overview + subject = 'Vos activités sur demarches-simplifiees.fr' - send_mail(gestionnaire.email, overview, 'Vos activités sur demarches-simplifiees.fr') + send_mail(gestionnaire.email, overview, subject) end def send_dossier(sender, dossier, recipient) diff --git a/app/mailers/invite_mailer.rb b/app/mailers/invite_mailer.rb index 220235fcd..1b07d39ff 100644 --- a/app/mailers/invite_mailer.rb +++ b/app/mailers/invite_mailer.rb @@ -1,14 +1,16 @@ class InviteMailer < ApplicationMailer def invite_user(invite) vars_mailer(invite) + subject = "demarches-simplifiees.fr - Participez à l'élaboration d'un dossier" - send_mail(invite.email, "demarches-simplifiees.fr - Participez à l'élaboration d'un dossier", invite.email_sender) if invite.user.present? + send_mail(invite.email, subject, invite.email_sender) if invite.user.present? end def invite_guest(invite) vars_mailer(invite) + subject = "Invitation - #{invite.email_sender} vous invite à consulter un dossier sur demarches-simplifiees.fr" - send_mail(invite.email, "Invitation - #{invite.email_sender} vous invite à consulter un dossier sur demarches-simplifiees.fr", invite.email_sender) + send_mail(invite.email, subject, invite.email_sender) end private diff --git a/app/mailers/user_mailer.rb b/app/mailers/user_mailer.rb index fde3369a9..2f1ae9873 100644 --- a/app/mailers/user_mailer.rb +++ b/app/mailers/user_mailer.rb @@ -3,7 +3,8 @@ class UserMailer < ApplicationMailer def new_account_warning(user) @user = user + subject = "Création de compte" - mail(to: user.email, subject: "Création de compte") + mail(to: user.email, subject: subject) end end