diff --git a/app/mailers/notification_mailer.rb b/app/mailers/notification_mailer.rb index cdd587493..185543d22 100644 --- a/app/mailers/notification_mailer.rb +++ b/app/mailers/notification_mailer.rb @@ -2,11 +2,11 @@ class NotificationMailer < ApplicationMailer default from: 'tps@apientreprise.fr', to: Proc.new { @user.email } - def send_notification dossier, email + def send_notification dossier, mail_template vars_mailer(dossier) - obj = email.object_for_dossier dossier - body = email.body_for_dossier dossier + obj = mail_template.object_for_dossier dossier + body = mail_template.body_for_dossier dossier mail(subject: obj) { |format| format.html { body } } end diff --git a/app/models/concerns/mail_template_concern.rb b/app/models/concerns/mail_template_concern.rb index f0e9863e9..0da40c322 100644 --- a/app/models/concerns/mail_template_concern.rb +++ b/app/models/concerns/mail_template_concern.rb @@ -32,7 +32,7 @@ module MailTemplateConcern module ClassMethods def slug - self.name.underscore.parameterize + self.name.demodulize.underscore.parameterize end def default @@ -48,7 +48,6 @@ module MailTemplateConcern when :numero_dossier dossier.id.to_s when :lien_dossier - # TPS::Application::URL # quickfix link_to users_dossier_recapitulatif_url(dossier), users_dossier_recapitulatif_url(dossier), target: '_blank' when :libelle_procedure dossier.procedure.libelle diff --git a/app/models/mail_received.rb b/app/models/mail_received.rb deleted file mode 100644 index 2199e0543..000000000 --- a/app/models/mail_received.rb +++ /dev/null @@ -1,23 +0,0 @@ -class MailReceived < MailTemplate - before_save :default_values - - - def name - "E-mail d'accusé de réception" - end - - def default_values - self.object ||= "[TPS] Accusé de réception pour votre dossier n°--numero_dossier--" - self.body ||= "Bonjour, -
-
- Votre administration vous confirme la bonne réception de votre dossier n°--numero_dossier-- complet. Celui-ci sera instruit dans le délai légal déclaré par votre interlocuteur.
-
- En vous souhaitant une bonne journée, -
-
- --- -
- L'équipe TPS" - end -end diff --git a/app/views/mails/closed_mail.html.haml b/app/views/mails/closed_mail.html.haml index 22321d848..636817517 100644 --- a/app/views/mails/closed_mail.html.haml +++ b/app/views/mails/closed_mail.html.haml @@ -1,17 +1,17 @@ Bonjour -
-
+%br +%br Votre dossier N°--numero_dossier-- a été accepté. -
-
+%br +%br A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier-- -
-
+%br +%br Bonne journée -
-
+%br +%br \--- -
+%br Merci de ne pas répondre à ce mail. Postez directement vos questions dans votre dossier sur la plateforme. -
+%br \--- diff --git a/app/views/mails/initiated_mail.html.haml b/app/views/mails/initiated_mail.html.haml index 5b008a02c..d1e0ab917 100644 --- a/app/views/mails/initiated_mail.html.haml +++ b/app/views/mails/initiated_mail.html.haml @@ -1,17 +1,17 @@ Bonjour -
-
+%br +%br Votre administration vous confirme la bonne réception de votre dossier n°--numero_dossier--. -
-
+%br +%br A tout moment, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier-- -
-
+%br +%br Bonne journée -
-
+%br +%br \--- -
+%br Merci de ne pas répondre à ce mail. Postez directement vos questions dans votre dossier sur la plateforme. -
+%br \--- diff --git a/app/views/mails/received_mail.html.haml b/app/views/mails/received_mail.html.haml index ea25ec013..bc0b64ffa 100644 --- a/app/views/mails/received_mail.html.haml +++ b/app/views/mails/received_mail.html.haml @@ -1,14 +1,14 @@ Bonjour -
-
+%br +%br Votre administration vous confirme la bonne réception de votre dossier n°--numero_dossier--. Celui-ci sera instruit dans le délai légal déclaré par votre interlocuteur. -
-
+%br +%br Bonne journée -
-
+%br +%br \--- -
+%br Merci de ne pas répondre à ce mail. Postez directement vos questions dans votre dossier sur la plateforme. -
+%br \--- diff --git a/app/views/mails/refused_mail.html.haml b/app/views/mails/refused_mail.html.haml index 5ea109569..d98bacab6 100644 --- a/app/views/mails/refused_mail.html.haml +++ b/app/views/mails/refused_mail.html.haml @@ -1,17 +1,17 @@ Bonjour -
-
+%br +%br Votre dossier N°--numero_dossier-- a été refusé. -
-
+%br +%br Pour en savoir plus sur le motif du refus, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier-- -
-
+%br +%br Bonne journée -
-
+%br +%br \--- -
+%br Merci de ne pas répondre à ce mail. Postez directement vos questions dans votre dossier sur la plateforme. -
+%br \--- diff --git a/app/views/mails/without_continuation_mail.html.haml b/app/views/mails/without_continuation_mail.html.haml index 9acff2424..5277c51a6 100644 --- a/app/views/mails/without_continuation_mail.html.haml +++ b/app/views/mails/without_continuation_mail.html.haml @@ -1,17 +1,17 @@ Bonjour -
-
+%br +%br Votre dossier N°--numero_dossier-- a été classé sans suite. -
-
+%br +%br Pour en savoir plus sur les raisons de ce classement sans suite, vous pouvez consulter le contenu de vos dossiers et les éventuels commentaires de l'administration à cette adresse : --lien_dossier-- -
-
+%br +%br Bonne journée -
-
+%br +%br \--- -
+%br Merci de ne pas répondre à ce mail. Postez directement vos questions dans votre dossier sur la plateforme. -
+%br \--- diff --git a/db/migrate/20170302105557_create_initiated_mails.rb b/db/migrate/20170302105557_create_initiated_mails.rb index 36c0ef332..91e005e6a 100644 --- a/db/migrate/20170302105557_create_initiated_mails.rb +++ b/db/migrate/20170302105557_create_initiated_mails.rb @@ -1,7 +1,7 @@ class CreateInitiatedMails < ActiveRecord::Migration[5.0] def change create_table :initiated_mails do |t| - t.text :object + t.string :object t.text :body t.belongs_to :procedure, index: true, unique: true, foreign_key: true diff --git a/db/migrate/20170302112312_remove_deposit_datetime_from_dossiers.rb b/db/migrate/20170302112312_remove_deposit_datetime_from_dossiers.rb index 600beb4a3..823290306 100644 --- a/db/migrate/20170302112312_remove_deposit_datetime_from_dossiers.rb +++ b/db/migrate/20170302112312_remove_deposit_datetime_from_dossiers.rb @@ -1,8 +1,17 @@ class RemoveDepositDatetimeFromDossiers < ActiveRecord::Migration[5.0] def change + remove_column :dossiers, :deposit_datetime, :datetime + end + + def up Dossier.where.not(deposit_datetime: nil).each do |dossier| dossier.update(initiated_at: dossier.deposit_datetime) end - remove_column :dossiers, :deposit_datetime, :datetime + end + + def down + Dossier.where.not(initiated_at: nil).each do |dossier| + dossier.update(deposit_datetime: dossier.initiated_at) + end end end diff --git a/db/migrate/20170306102116_create_received_mails.rb b/db/migrate/20170306102116_create_received_mails.rb index 2b1526ce8..aaaaefb42 100644 --- a/db/migrate/20170306102116_create_received_mails.rb +++ b/db/migrate/20170306102116_create_received_mails.rb @@ -2,7 +2,7 @@ class CreateReceivedMails < ActiveRecord::Migration[5.0] def change create_table :received_mails do |t| t.text :body - t.text :object + t.string :object t.references :procedure, foreign_key: true t.column :created_at, :timestamp, null: true diff --git a/db/migrate/20170306102238_create_closed_mails.rb b/db/migrate/20170306102238_create_closed_mails.rb index 11f6c07ed..49f3bf5e8 100644 --- a/db/migrate/20170306102238_create_closed_mails.rb +++ b/db/migrate/20170306102238_create_closed_mails.rb @@ -2,7 +2,7 @@ class CreateClosedMails < ActiveRecord::Migration[5.0] def change create_table :closed_mails do |t| t.text :body - t.text :object + t.string :object t.belongs_to :procedure, index: true, unique: true, foreign_key: true t.timestamps diff --git a/db/migrate/20170306102256_create_refused_mails.rb b/db/migrate/20170306102256_create_refused_mails.rb index 527866530..ea7b58846 100644 --- a/db/migrate/20170306102256_create_refused_mails.rb +++ b/db/migrate/20170306102256_create_refused_mails.rb @@ -2,7 +2,7 @@ class CreateRefusedMails < ActiveRecord::Migration[5.0] def change create_table :refused_mails do |t| t.text :body - t.text :object + t.string :object t.belongs_to :procedure, index: true, unique: true, foreign_key: true t.timestamps diff --git a/db/migrate/20170306102320_create_without_continuation_mails.rb b/db/migrate/20170306102320_create_without_continuation_mails.rb index da65dc8de..09dbe1b8e 100644 --- a/db/migrate/20170306102320_create_without_continuation_mails.rb +++ b/db/migrate/20170306102320_create_without_continuation_mails.rb @@ -2,7 +2,7 @@ class CreateWithoutContinuationMails < ActiveRecord::Migration[5.0] def change create_table :without_continuation_mails do |t| t.text :body - t.text :object + t.string :object t.belongs_to :procedure, index: true, unique: true, foreign_key: true t.timestamps diff --git a/db/schema.rb b/db/schema.rb index 770109651..db86ebfaf 100644 --- a/db/schema.rb +++ b/db/schema.rb @@ -10,7 +10,6 @@ # # It's strongly recommended that you check this file into your version control system. - ActiveRecord::Schema.define(version: 20170307092820) do # These are extensions that must be enabled in order to support this database @@ -107,7 +106,7 @@ ActiveRecord::Schema.define(version: 20170307092820) do create_table "closed_mails", force: :cascade do |t| t.text "body" - t.text "object" + t.string "object" t.integer "procedure_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -238,7 +237,7 @@ ActiveRecord::Schema.define(version: 20170307092820) do end create_table "initiated_mails", force: :cascade do |t| - t.text "object" + t.string "object" t.text "body" t.integer "procedure_id" t.datetime "created_at", null: false @@ -357,7 +356,7 @@ ActiveRecord::Schema.define(version: 20170307092820) do create_table "received_mails", force: :cascade do |t| t.text "body" - t.text "object" + t.string "object" t.integer "procedure_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -366,7 +365,7 @@ ActiveRecord::Schema.define(version: 20170307092820) do create_table "refused_mails", force: :cascade do |t| t.text "body" - t.text "object" + t.string "object" t.integer "procedure_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false @@ -426,7 +425,7 @@ ActiveRecord::Schema.define(version: 20170307092820) do create_table "without_continuation_mails", force: :cascade do |t| t.text "body" - t.text "object" + t.string "object" t.integer "procedure_id" t.datetime "created_at", null: false t.datetime "updated_at", null: false diff --git a/spec/mailers/previews/notification_mailer_preview.rb b/spec/mailers/previews/notification_mailer_preview.rb index 5eb4a0bb3..25fa735c2 100644 --- a/spec/mailers/previews/notification_mailer_preview.rb +++ b/spec/mailers/previews/notification_mailer_preview.rb @@ -1,7 +1,7 @@ class NotificationMailerPreview < ActionMailer::Preview - def dossier_received - NotificationMailer.dossier_received(Dossier.last) + def send_notification + NotificationMailer.send_notification(Dossier.last, Dossier.last.procedure.initiated_mail) end end