DGNum/demarches-normaliennes
14
1
Fork 0
Code Issues 1 Pull requests Projects Releases Packages Wiki Activity

Merge pull request #3780 from betagouv/fix-missing-logo

Browse source 
Fix when missing logo on remote storage crash email sending
This commit is contained in:
Mathieu Magnin 2019-04-11 15:49:49 +02:00 committed by GitHub
commit 62082f585a
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
app/mailers/notification_mailer.rb
View file

@ -39,9 +39,14 @@ class NotificationMailer < ApplicationMailer
create_commentaire_for_notification(dossier, subject, body) create_commentaire_for_notification(dossier, subject, body)
if dossier.procedure.logo? if dossier.procedure.logo?
logo_filename = dossier.procedure.logo.filename begin
attachments.inline[logo_filename] = dossier.procedure.logo.read logo_filename = dossier.procedure.logo.filename
@logo_url = attachments[logo_filename].url attachments.inline[logo_filename] = dossier.procedure.logo.read
@logo_url = attachments[logo_filename].url
rescue StandardError => e
# A problem occured when reading logo, maybe the logo is missing and we should clean the procedure to remove logo reference ?
Raven.capture_exception(e)
end
end end
@dossier = dossier @dossier = dossier