[#1110] Use TagsSubstitutionConcern as engine for mail template tags

app/models/concerns/mail_template_concern.rb

@@ -3,28 +3,7 @@ module MailTemplateConcern
 
   include Rails.application.routes.url_helpers
   include ActionView::Helpers::UrlHelper
-
+  include TagsSubstitutionConcern
-  TAGS = []
-  TAGS << TAG_NUMERO_DOSSIER = {
-    libelle:     "numero_dossier",
-    description: "Permet d'afficher le numéro de dossier de l'utilisateur."
-  }
-  TAGS << TAG_LIEN_DOSSIER = {
-    libelle:     "lien_dossier",
-    description: "Permet d'afficher un lien vers le dossier de l'utilisateur."
-  }
-  TAGS << TAG_LIBELLE_PROCEDURE = {
-    libelle:     "libelle_procedure",
-    description: "Permet d'afficher le libellé de la procédure."
-  }
-  TAGS << TAG_DATE_DE_DECISION = {
-    libelle:     "date_de_decision",
-    description: "Permet d'afficher la date à laquelle la décision finale (acceptation, refus, classement sans suite) sur le dossier a été prise."
-  }
-  TAGS << TAG_MOTIVATION = {
-    libelle:     "motivation",
-    description: "Permet d'afficher la motivation associée à la décision finale (acceptation, refus, classement sans suite) sur le dossier. Attention, elle est facultative."
-  }
 
   def object_for_dossier(dossier)
     replace_tags(object, dossier)
@@ -34,14 +13,10 @@ module MailTemplateConcern
     replace_tags(body, dossier)
   end
 
-  def tags
+  # TODO: remove legacy argument when removing legacy tags
-    self.class.const_get(:ALLOWED_TAGS)
+  def tags(reject_legacy: true, is_dossier_termine: self.class.const_get(:IS_DOSSIER_TERMINE))
-  end
+    super(is_dossier_termine: is_dossier_termine)
-
+      .reject { |tag| reject_legacy && tag[:is_legacy] }
-  def replace_tags(string, dossier)
-    TAGS.inject(string) do |acc, tag|
-      acc.gsub("--#{tag[:libelle]}--", replace_tag(tag, dossier)) || acc
-    end
   end
 
   module ClassMethods
@@ -53,20 +28,20 @@ module MailTemplateConcern
 
   private
 
-  def replace_tag(tag, dossier)
+  def dossier_tags
-    case tag
+    super +
-    when TAG_NUMERO_DOSSIER
+      [{ libelle: 'lien dossier', description: '', lambda: -> (d) { users_dossier_recapitulatif_link(d) } },
-      dossier.id.to_s
+       # TODO: remove legacy tags
-    when TAG_LIEN_DOSSIER
+       { libelle: 'numero_dossier', description: '', target: :id, is_legacy: true },
-      link_to users_dossier_recapitulatif_url(dossier), users_dossier_recapitulatif_url(dossier), target: '_blank'
+       { libelle: 'lien_dossier', description: '', lambda: -> (d) { users_dossier_recapitulatif_link(d) }, is_legacy: true },
-    when TAG_LIBELLE_PROCEDURE
+       { libelle: 'libelle_procedure', description: '', lambda: -> (d) { d.procedure.libelle }, is_legacy: true },
-      dossier.procedure.libelle
+       { libelle: 'date_de_decision', description: '',
-    when TAG_DATE_DE_DECISION
+         lambda: -> (d) { d.processed_at.present? ? d.processed_at.localtime.strftime('%d/%m/%Y') : '' },
-      dossier.processed_at.present? ? dossier.processed_at.localtime.strftime("%d/%m/%Y") : ""
+         dossier_termine_only: true, is_legacy: true }]
-    when TAG_MOTIVATION
+  end
-      dossier.motivation || ""
+
-    else
+  def users_dossier_recapitulatif_link(dossier)
-      '--BALISE_NON_RECONNUE--'
+    url = users_dossier_recapitulatif_url(dossier)
-    end
+    link_to(url, url, target: '_blank')
   end
 end

app/models/mails/closed_mail.rb

@@ -8,6 +8,6 @@ module Mails
     TEMPLATE_NAME = "mails/closed_mail"
     DISPLAYED_NAME = "Accusé d'acceptation"
     DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été accepté'
-    ALLOWED_TAGS = [TAG_NUMERO_DOSSIER, TAG_LIEN_DOSSIER, TAG_LIBELLE_PROCEDURE, TAG_DATE_DE_DECISION, TAG_MOTIVATION]
+    IS_DOSSIER_TERMINE = true
   end
 end

app/models/mails/initiated_mail.rb

@@ -8,6 +8,6 @@ module Mails
     TEMPLATE_NAME = "mails/initiated_mail"
     DISPLAYED_NAME = 'Accusé de réception'
     DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a bien été reçu'
-    ALLOWED_TAGS = [TAG_NUMERO_DOSSIER, TAG_LIEN_DOSSIER, TAG_LIBELLE_PROCEDURE]
+    IS_DOSSIER_TERMINE = false
   end
 end

app/models/mails/received_mail.rb

@@ -8,6 +8,6 @@ module Mails
     TEMPLATE_NAME = "mails/received_mail"
     DISPLAYED_NAME = 'Accusé de passage en instruction'
     DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- va être instruit'
-    ALLOWED_TAGS = [TAG_NUMERO_DOSSIER, TAG_LIEN_DOSSIER, TAG_LIBELLE_PROCEDURE]
+    IS_DOSSIER_TERMINE = false
   end
 end

app/models/mails/refused_mail.rb

@@ -8,6 +8,6 @@ module Mails
     TEMPLATE_NAME = "mails/refused_mail"
     DISPLAYED_NAME = 'Accusé de rejet du dossier'
     DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été refusé'
-    ALLOWED_TAGS = [TAG_NUMERO_DOSSIER, TAG_LIEN_DOSSIER, TAG_LIBELLE_PROCEDURE, TAG_DATE_DE_DECISION, TAG_MOTIVATION]
+    IS_DOSSIER_TERMINE = true
   end
 end

app/models/mails/without_continuation_mail.rb

@@ -8,6 +8,6 @@ module Mails
     TEMPLATE_NAME = "mails/without_continuation_mail"
     DISPLAYED_NAME = 'Accusé de classement sans suite'
     DEFAULT_OBJECT = 'Votre dossier TPS nº --numero_dossier-- a été classé sans suite'
-    ALLOWED_TAGS = [TAG_NUMERO_DOSSIER, TAG_LIEN_DOSSIER, TAG_LIBELLE_PROCEDURE, TAG_DATE_DE_DECISION, TAG_MOTIVATION]
+    IS_DOSSIER_TERMINE = true
   end
 end

spec/factories/mail_templates.rb

@@ -1,21 +1,15 @@
 FactoryGirl.define do
-  factory :mail_template do
+  factory :mail_template, class: Mails::ClosedMail do
     object "Object, voila voila"
     body "Blabla ceci est mon body"
-    type 'MailValidated'
 
-    trait :dossier_submitted do
+    factory :dossier_submitted_mail_template, class: Mails::ReceivedMail
-      type 'MailSubmitted'
-    end
 
-    trait :dossier_refused do
+    factory :dossier_refused_mail_template, class: Mails::RefusedMail
-      type 'MailRefused'
-    end
 
-    trait :dossier_en_instruction do
+    factory :dossier_en_instruction_mail_template, class: Mails::InitiatedMail do
       object "[TPS] Accusé de réception pour votre dossier nº --numero_dossier--"
       body "Votre administration vous confirme la bonne réception de votre dossier nº --numero_dossier--"
-      type 'MailReceived'
     end
   end
 end

spec/views/admin/mail_templates/edit.html.haml_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe 'admin/mail_templates/edit.html.haml', type: :view do
+  let(:procedure) { create(:procedure) }
+  let(:mail_template) { create(:mail_template, procedure: procedure) }
+  let(:all_tags) { mail_template.tags(reject_legacy: false) }
+
+  before do
+    allow(view).to receive(:admin_procedure_mail_template_path).and_return("/toto")
+    allow(view).to receive(:admin_procedure_mail_templates_path).and_return("/toto")
+
+    assign(:mail_template, mail_template)
+  end
+
+  subject { render }
+
+  context "Legacy champs are not listed in the page" do
+    it { expect(all_tags).to include(include({ libelle: 'numero_dossier', is_legacy: true })) }
+    it { is_expected.not_to include("numero_dossier") }
+  end
+
+  context "Non-legacy champs are listed in the page" do
+    it { expect(all_tags).to include(include({ libelle: 'numéro du dossier' })) }
+    it { is_expected.to include("numéro du dossier") }
+  end
+end