Merge pull request #2019 from betagouv/cleanup-mailers

Cleanup mailers
2018-06-01 10:40:24 +02:00 · 2018-06-01 10:40:24 +02:00 · dc88fb46b3
commit dc88fb46b3
parent 740a83f665 23931a7591
30 changed files with 83 additions and 49 deletions
--- a/app/controllers/admin/procedures_controller.rb
+++ b/app/controllers/admin/procedures_controller.rb
@ -46,7 +46,7 @@ class Admin::ProceduresController < AdminController
    procedure = current_administrateur.procedures.find(params[:id])
    procedure.hide!

-    flash.notice = "Procédure supprimée, en cas d'erreur contactez nous : contact@demarches-simplifiees.fr"
+    flash.notice = "Procédure supprimée, en cas d'erreur contactez nous : #{CONTACT_EMAIL}"
    redirect_to admin_procedures_draft_path
  end

--- a/app/controllers/administrateurs/activate_controller.rb
+++ b/app/controllers/administrateurs/activate_controller.rb
@ -5,7 +5,7 @@ class Administrateurs::ActivateController < ApplicationController
    @administrateur = Administrateur.find_inactive_by_token(params[:token])

    if !@administrateur
-      flash.alert = "Le lien de validation d'administrateur a expiré, contactez-nous à contact@demarches-simplifiees.fr pour obtenir un nouveau lien."
+      flash.alert = "Le lien de validation d'administrateur a expiré, contactez-nous à #{CONTACT_EMAIL} pour obtenir un nouveau lien."
      redirect_to root_path
    end
  end
--- a/app/controllers/gestionnaires/activate_controller.rb
+++ b/app/controllers/gestionnaires/activate_controller.rb
@ -5,7 +5,7 @@ class Gestionnaires::ActivateController < ApplicationController
    @gestionnaire = Gestionnaire.with_reset_password_token(params[:token])

    if !@gestionnaire
-      flash.alert = "Le lien de validation du compte accompagnateur a expiré, contactez-nous à contact@demarches-simplifiees.fr pour obtenir un nouveau lien."
+      flash.alert = "Le lien de validation du compte accompagnateur a expiré, contactez-nous à #{CONTACT_EMAIL} pour obtenir un nouveau lien."
      redirect_to root_path
    end
  end
--- a/app/mailers/administrateur_mailer.rb
+++ b/app/mailers/administrateur_mailer.rb
@ -5,8 +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",
-         reply_to: "contact@demarches-simplifiees.fr")
+         subject: subject,
+         reply_to: CONTACT_EMAIL)
  end
 end
--- a/app/mailers/administration_mailer.rb
+++ b/app/mailers/administration_mailer.rb
@ -4,28 +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")
+    mail(to: TECH_EMAIL,
+         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",
-         reply_to: "contact@demarches-simplifiees.fr")
+         subject: subject,
+         reply_to: CONTACT_EMAIL)
  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",
-         reply_to: "contact@demarches-simplifiees.fr")
+         subject: subject,
+         reply_to: CONTACT_EMAIL)
  end

  def dubious_procedures(procedures_and_type_de_champs)
    @procedures_and_type_de_champs = procedures_and_type_de_champs
-    mail(to: 'equipe@demarches-simplifiees.fr',
-         subject: "[RGS] De nouvelles procédures comportent des champs interdits")
+    subject = "[RGS] De nouvelles procédures comportent des champs interdits"
+
+    mail(to: EQUIPE_EMAIL,
+         subject: subject)
  end
 end
--- a/app/mailers/application_mailer.rb
+++ b/app/mailers/application_mailer.rb
@ -1,4 +1,4 @@
 class ApplicationMailer < ActionMailer::Base
-  default from: "demarches-simplifiees.fr <#{I18n.t('dynamics.contact_email')}>"
+  default from: "demarches-simplifiees.fr <#{CONTACT_EMAIL}>"
  layout 'mailer'
 end
--- a/app/mailers/avis_mailer.rb
+++ b/app/mailers/avis_mailer.rb
@ -2,6 +2,8 @@ class AvisMailer < ApplicationMailer
  def avis_invitation(avis)
    @avis = avis
    email = @avis.gestionnaire&.email || @avis.email
-    mail(to: email, subject: "Donnez votre avis sur le dossier nº #{@avis.dossier.id} (#{@avis.dossier.procedure.libelle})")
+    subject = "Donnez votre avis sur le dossier nº #{@avis.dossier.id} (#{@avis.dossier.procedure.libelle})"
+
+    mail(to: email, subject: subject)
  end
 end
--- a/app/mailers/dossier_mailer.rb
+++ b/app/mailers/dossier_mailer.rb
@ -3,11 +3,15 @@ class DossierMailer < ApplicationMailer

  def notify_deletion_to_user(deleted_dossier, to_email)
    @deleted_dossier = deleted_dossier
-    mail(to: to_email, subject: "Votre dossier n° #{@deleted_dossier.dossier_id} a bien été supprimé")
+    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
-    mail(to: to_email, subject: "Le dossier n° #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager")
+    subject = "Le dossier n° #{@deleted_dossier.dossier_id} a été supprimé à la demande de l'usager"
+
+    mail(to: to_email, subject: subject)
  end
 end
--- a/app/mailers/gestionnaire_mailer.rb
+++ b/app/mailers/gestionnaire_mailer.rb
@ -4,19 +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",
-         reply_to: "contact@demarches-simplifiees.fr")
+         subject: subject,
+         reply_to: CONTACT_EMAIL)
  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
-    send_mail gestionnaire.email, overview, 'Vos activités sur demarches-simplifiees.fr'
+    subject = 'Vos activités sur demarches-simplifiees.fr'
+
+    send_mail(gestionnaire.email, overview, subject)
  end

  def send_dossier(sender, dossier, recipient)
--- 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
+    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
+    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
--- a/app/mailers/notification_mailer.rb
+++ b/app/mailers/notification_mailer.rb
@ -54,7 +54,7 @@ class NotificationMailer < ApplicationMailer
  def create_commentaire_for_notification(dossier, subject, body)
    Commentaire.create(
      dossier: dossier,
-      email: I18n.t("dynamics.contact_email"),
+      email: CONTACT_EMAIL,
      body: ["[#{subject}]", body].join("<br><br>")
    )
  end
--- a/app/mailers/user_mailer.rb
+++ b/app/mailers/user_mailer.rb
@ -3,6 +3,8 @@ class UserMailer < ApplicationMailer

  def new_account_warning(user)
    @user = user
-    mail(to: user.email, subject: "Création de compte")
+    subject = "Création de compte"
+
+    mail(to: user.email, subject: subject)
  end
 end
--- a/app/models/commentaire.rb
+++ b/app/models/commentaire.rb
@ -36,7 +36,7 @@ class Commentaire < ApplicationRecord
    # - If a user or an invited user posted a commentaire, do nothing,
    #   the notification system will properly
    # - Otherwise, a gestionnaire posted a commentaire, we need to notify the user
-    if !email.in?([I18n.t("dynamics.contact_email"), dossier_user_email, *invited_users_emails])
+    if !email.in?([CONTACT_EMAIL, dossier_user_email, *invited_users_emails])
      notify_user
    end
  end
--- a/app/models/gestionnaire.rb
+++ b/app/models/gestionnaire.rb
@ -102,8 +102,8 @@ class Gestionnaire < ApplicationRecord

      messagerie = follow.messagerie_seen_at.present? &&
        dossier.commentaires
-          .where.not(email: 'contact@tps.apientreprise.fr')
-          .where.not(email: 'contact@demarches-simplifiees.fr')
+          .where.not(email: OLD_CONTACT_EMAIL)
+          .where.not(email: CONTACT_EMAIL)
          .updated_since?(follow.messagerie_seen_at).any?

      annotations_hash(demande, annotations_privees, avis_notif, messagerie)
@ -208,8 +208,8 @@ class Gestionnaire < ApplicationRecord
    updated_messagerie = dossiers
      .joins(:commentaires)
      .where('commentaires.updated_at > follows.messagerie_seen_at')
-      .where.not(commentaires: { email: 'contact@tps.apientreprise.fr' })
-      .where.not(commentaires: { email: 'contact@demarches-simplifiees.fr' })
+      .where.not(commentaires: { email: OLD_CONTACT_EMAIL })
+      .where.not(commentaires: { email: CONTACT_EMAIL })

    [
      updated_demandes,
--- a/app/views/accessibilite/index.html.haml
+++ b/app/views/accessibilite/index.html.haml
@ -9,7 +9,7 @@

  %h2.new-h2 Signaler un dysfonctionnement
  %p.new-p
-    Si, malgré notre vigilance, vous rencontriez le moindre problème d’accessibilité sur notre site, n’hésitez pas à nous écrire à <a href="mailto:contact@demarches-simplifiees.fr">contact@demarches-simplifiees.fr</a>.
+    = "Si, malgré notre vigilance, vous rencontriez le moindre problème d’accessibilité sur notre site, n’hésitez pas à nous écrire à <a href='mailto:#{CONTACT_EMAIL}'>#{CONTACT_EMAIL}</a>."

  %h2.new-h2 Défenseur des droits
  %p.new-p
--- a/app/views/layouts/_footer.html.haml
+++ b/app/views/layouts/_footer.html.haml
@ -9,4 +9,4 @@
    \-
    = link_to 'CGU / Mentions légales', CGU_URL
    \-
-    = link_to 'Contact', "mailto:"+t('dynamics.contact_email')
+    = link_to 'Contact', "mailto:#{CONTACT_EMAIL}"
--- a/app/views/layouts/_new_footer.html.haml
+++ b/app/views/layouts/_new_footer.html.haml
@ -30,7 +30,7 @@
        %ul.footer-links
          %li.footer-link
            = link_to "Contact",
-              "mailto:#{t('dynamics.contact_email')}",
+              "mailto:#{CONTACT_EMAIL}",
              :class => "footer-link"
          %li.footer-link
            = link_to "Documentation", DOC_URL, :class => "footer-link", :target => "_blank", rel: "noopener noreferrer"
--- a/app/views/layouts/_new_header.haml
+++ b/app/views/layouts/_new_header.haml
@ -29,7 +29,7 @@
                %br
                – par téléphone : 01 76 42 02 87
                %br
-                – par email : <a href="mailto:contact@demarches-simplifiees.fr">contact@demarches-simplifiees.fr</a>
+                = "– par email : <a href='mailto:#{CONTACT_EMAIL}'>#{CONTACT_EMAIL}</a>"

      - if nav_bar_profile == :user
        %ul.header-tabs
--- a/app/views/new_gestionnaire/shared/messages/_message.html.haml
+++ b/app/views/new_gestionnaire/shared/messages/_message.html.haml
@ -4,7 +4,7 @@
  %h2
    %span.mail
      = render partial: 'new_gestionnaire/shared/messages/message_issuer', locals: { commentaire: commentaire, current_gestionnaire: current_gestionnaire }
-    - if ![current_gestionnaire.email, commentaire.dossier.user.email, 'contact@tps.apientreprise.fr', 'contact@demarches-simplifiees.fr'].include?(commentaire.email)
+    - if ![current_gestionnaire.email, commentaire.dossier.user.email, OLD_CONTACT_EMAIL, CONTACT_EMAIL].include?(commentaire.email)
      %span.guest Invité
    %span.date{ class: highlight_if_unseen_class(messagerie_seen_at, commentaire.created_at) }
      = I18n.l(commentaire.created_at.localtime, format: '%d/%m/%Y à %H:%M ')
--- a/app/views/new_gestionnaire/shared/messages/_message_icon.html.haml
+++ b/app/views/new_gestionnaire/shared/messages/_message_icon.html.haml
@ -1,7 +1,8 @@
 - case commentaire.email
 - when current_gestionnaire.email
  = image_tag('icons/account-circle.svg', class: 'person-icon')
- when 'contact@demarches-simplifiees.fr'
+- when OLD_CONTACT_EMAIL
+- when CONTACT_EMAIL
  = image_tag('icons/mail.svg', class: 'person-icon')
 - else
  = image_tag('icons/blue-person.svg', class: 'person-icon')
--- a/app/views/new_gestionnaire/shared/messages/_message_issuer.html.haml
+++ b/app/views/new_gestionnaire/shared/messages/_message_issuer.html.haml
@ -1,8 +1,8 @@
 - case commentaire.email
 - when current_gestionnaire.email
  Vous
- when 'contact@tps.apientreprise.fr'
- when 'contact@demarches-simplifiees.fr'
+- when OLD_CONTACT_EMAIL
+- when CONTACT_EMAIL
  Email automatique
 - else
  = commentaire.email
--- a/app/views/root/landing.html.haml
+++ b/app/views/root/landing.html.haml
@ -204,7 +204,7 @@
          %p.cta-panel-explanation Notre équipe est disponible pour vous renseigner et vous aider
        %div
          = link_to "Contactez-nous",
-            "mailto:#{t("dynamics.contact_email")}?subject=Question%20à%20propos%20de%20demarches-simplifiees.fr",
+            "mailto:#{CONTACT_EMAIL}?subject=Question%20à%20propos%20de%20demarches-simplifiees.fr",
            class: "cta-panel-button-white",
            target: "_blank",
            rel: "noopener noreferrer"
--- a/app/views/root/patron.html.haml
+++ b/app/views/root/patron.html.haml
@ -190,7 +190,7 @@
            .icon.bubble.avis-icon
            .width-100
              %h2.gestionnaire
-                contact@demarches-simplifiees.fr
+                = CONTACT_EMAIL
                %span.date Réponse donnée le 13/01/2012
              %p Je donne un avis favorable

--- a/config/initializers/devise.rb
+++ b/config/initializers/devise.rb
@ -1,3 +1,5 @@
+require_relative "emails"
+
 # Use this hook to configure devise mailer, warden hooks and so forth.
 # Many of these configuration options can be set straight in your model.
 Devise.setup do |config|
@ -10,7 +12,7 @@ Devise.setup do |config|
  # Configure the e-mail address which will be shown in Devise::Mailer,
  # note that it will be overwritten if you use your own mailer class
  # with default "from" parameter.
-  config.mailer_sender = "demarches-simplifiees.fr <#{I18n.t('dynamics.contact_email')}>"
+  config.mailer_sender = "demarches-simplifiees.fr <#{CONTACT_EMAIL}>"

  # Configure the class responsible to send e-mails.
  config.mailer = 'DeviseUserMailer'
--- a/config/initializers/emails.rb
+++ b/config/initializers/emails.rb
@ -0,0 +1,7 @@
+if !defined?(CONTACT_EMAIL)
+  CONTACT_EMAIL = "contact@demarches-simplifiees.fr"
+  EQUIPE_EMAIL = "equipe@demarches-simplifiees.fr"
+  TECH_EMAIL = "tech@demarches-simplifiees.fr"
+
+  OLD_CONTACT_EMAIL = "contact@tps.apientreprise.fr"
+end
--- a/config/locales/dynamics/fr.yml
+++ b/config/locales/dynamics/fr.yml
@ -1,7 +1,6 @@
 fr:
  dynamics:
    page_title: demarches-simplifiees.fr
-    contact_email: contact@demarches-simplifiees.fr
    users:
      connexion_title: Connexion
      mes_dossiers: 'Mes dossiers'
--- a/lib/mailers/attestation_closed_mail_discrepancy_mailer.rb
+++ b/lib/mailers/attestation_closed_mail_discrepancy_mailer.rb
@ -35,7 +35,7 @@ module Mailers
        #{detail_procedures(procedures)}

        Pour toute question vous pouvez nous joindre par téléphone au 01 76 42 02 87
-        ou sur l’adresse email contact@demarches-simplifiees.fr.
+        ou sur l’adresse email #{CONTACT_EMAIL}.
        -- \nL’équipe demarches-simplifiees.fr
      HEREDOC
    end
--- a/lib/tasks/2018_05_30_missed_ar_messages.rake
+++ b/lib/tasks/2018_05_30_missed_ar_messages.rake
@ -21,7 +21,7 @@ namespace :'2018_05_30_missed_ar_messages' do

    Commentaire.create(
      dossier: dossier,
-      email: I18n.t("dynamics.contact_email"),
+      email: CONTACT_EMAIL,
      body: "[#{subject}]<br><br>#{body}",
      created_at: date
    )
--- a/spec/models/commentaire_spec.rb
+++ b/spec/models/commentaire_spec.rb
@ -31,7 +31,7 @@ describe Commentaire do
        expect(commentaire).not_to receive(:notify_user)
        expect(commentaire).not_to receive(:notify_gestionnaires)

-        commentaire.email = "contact@demarches-simplifiees.fr"
+        commentaire.email = CONTACT_EMAIL
        commentaire.save
      end
    end
--- a/spec/models/gestionnaire_spec.rb
+++ b/spec/models/gestionnaire_spec.rb
@ -246,7 +246,7 @@ describe Gestionnaire, type: :model do
      end

      context 'when there is a new commentaire issued by tps' do
-        before { create(:commentaire, dossier: dossier, email: 'contact@demarches-simplifiees.fr') }
+        before { create(:commentaire, dossier: dossier, email: CONTACT_EMAIL) }

        it { is_expected.to match({ demande: false, annotations_privees: false, avis: false, messagerie: false }) }
      end
@ -327,7 +327,7 @@ describe Gestionnaire, type: :model do
      end

      context 'when there is a new commentaire issued by tps' do
-        before { create(:commentaire, dossier: dossier, email: 'contact@demarches-simplifiees.fr') }
+        before { create(:commentaire, dossier: dossier, email: CONTACT_EMAIL) }

        it { is_expected.to match([]) }
      end