diff --git a/app/mailers/notification_mailer.rb b/app/mailers/notification_mailer.rb index b493f8407..7d803b9cf 100644 --- a/app/mailers/notification_mailer.rb +++ b/app/mailers/notification_mailer.rb @@ -1,8 +1,6 @@ class NotificationMailer < ApplicationMailer default to: Proc.new { @user.email } - after_action :create_commentaire_for_notification, only: [:send_notification, :send_dossier_received] - def send_dossier_received(dossier_id) dossier = Dossier.find(dossier_id) send_notification(dossier, dossier.procedure.received_mail_template) @@ -20,10 +18,6 @@ class NotificationMailer < ApplicationMailer send_notification(dossier, dossier.procedure.initiated_mail_template) end - def send_received_notification(dossier) - send_notification(dossier, dossier.procedure.received_mail_template) - end - def send_closed_notification(dossier) send_notification(dossier, dossier.procedure.closed_mail_template) end @@ -48,6 +42,8 @@ class NotificationMailer < ApplicationMailer @subject = mail_template.subject_for_dossier dossier @body = mail_template.body_for_dossier dossier + create_commentaire_for_notification + mail(subject: @subject) { |format| format.html { @body } } end diff --git a/lib/tasks/dev.rake b/lib/tasks/dev.rake index b363774b1..1a7b21198 100644 --- a/lib/tasks/dev.rake +++ b/lib/tasks/dev.rake @@ -71,7 +71,7 @@ namespace :dev do task :import_db do filename = "tps_prod_#{1.day.ago.strftime("%d-%m-%Y")}.sql" local_file = "/tmp/#{filename}" - run_and_stop_if_error "scp deploy@sgmap_backup:/var/backup/production1/db/#{filename} #{local_file}" + run_and_stop_if_error "scp -C deploy@sgmap_backup:/var/backup/production1/db/#{filename} #{local_file}" dev_env_param = "RAILS_ENV=development" diff --git a/spec/controllers/new_user/dossiers_controller_spec.rb b/spec/controllers/new_user/dossiers_controller_spec.rb index 0f2b958eb..2717a6883 100644 --- a/spec/controllers/new_user/dossiers_controller_spec.rb +++ b/spec/controllers/new_user/dossiers_controller_spec.rb @@ -246,7 +246,7 @@ describe NewUser::DossiersController, type: :controller do it { expect(flash.alert).to eq(['nop']) } it 'does not send an email' do - expect(NotificationMailer).not_to receive(:send_received_notification) + expect(NotificationMailer).not_to receive(:send_initiated_notification) subject end