Add In-Reply-To, References headers to email

[rails.git] / app / mailers / notifier.rb

diff --git a/app/mailers/notifier.rb b/app/mailers/notifier.rb
index 4b53c66f6825f1b38d5fdbde69653002762e9896..676ae6f3b5cf56f19bc305718d3995161963e340 100644 (file)
--- a/app/mailers/notifier.rb
+++ b/app/mailers/notifier.rb
@@ -1,6 +1,10 @@
-class Notifier < ActionMailer::Base
-  default :from => EMAIL_FROM,
-          :return_path => EMAIL_RETURN_PATH,
+class Notifier < ApplicationMailer
+  include ActionView::Helpers::AssetUrlHelper
+
+  self.delivery_job = ActionMailer::MailDeliveryJob
+
+  default :from => Settings.email_from,
+          :return_path => Settings.email_return_path,

           :auto_submitted => "auto-generated"
   helper :application
   before_action :set_shared_template_vars
           :auto_submitted => "auto-generated"
   helper :application
   before_action :set_shared_template_vars


@@ -90,11 +94,14 @@ class Notifier < ActionMailer::Base
       @readurl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "comment#{comment.id}")
       @commenturl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "newcomment")
       @replyurl = new_message_url(comment.user, :message => { :title => "Re: #{comment.diary_entry.title}" })
       @readurl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "comment#{comment.id}")
       @commenturl = diary_entry_url(comment.diary_entry.user, comment.diary_entry, :anchor => "newcomment")
       @replyurl = new_message_url(comment.user, :message => { :title => "Re: #{comment.diary_entry.title}" })

-
+      @ref = "osm-diary-#{comment.diary_entry.id}@#{Settings.server_url}"

       @author = @from_user
 
       attach_user_avatar(comment.user)
 
       @author = @from_user
 
       attach_user_avatar(comment.user)
 

+      headers["In-Reply-To"] = @ref
+      headers["References"]  = @ref
+

       mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest, recipient.id),
            :to => recipient.email,
            :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name)
       mail :from => from_address(comment.user.display_name, "c", comment.id, comment.digest, recipient.id),
            :to => recipient.email,
            :subject => I18n.t("notifier.diary_comment_notification.subject", :user => comment.user.display_name)


@@ -122,6 +129,7 @@ class Notifier < ActionMailer::Base
       @comment = comment.body
       @owner = recipient == comment.note.author
       @event = comment.event
       @comment = comment.body
       @owner = recipient == comment.note.author
       @event = comment.event

+      @ref = "osm-note-#{comment.note.id}@#{Settings.server_url}"

 
       @commenter = if comment.author
                      comment.author.display_name
 
       @commenter = if comment.author
                      comment.author.display_name


@@ -132,6 +140,9 @@ class Notifier < ActionMailer::Base
       @author = @commenter
       attach_user_avatar(comment.author)
 
       @author = @commenter
       attach_user_avatar(comment.author)
 

+      headers["In-Reply-To"] = @ref
+      headers["References"]  = @ref
+

       subject = if @owner
                   I18n.t("notifier.note_comment_notification.#{@event}.subject_own", :commenter => @commenter)
                 else
       subject = if @owner
                   I18n.t("notifier.note_comment_notification.#{@event}.subject_own", :commenter => @commenter)
                 else


@@ -153,6 +164,7 @@ class Notifier < ActionMailer::Base
       @time = comment.created_at
       @changeset_author = comment.changeset.user.display_name
       @author = @commenter
       @time = comment.created_at
       @changeset_author = comment.changeset.user.display_name
       @author = @commenter

+      @ref = "osm-changeset-#{comment.changeset.id}@#{Settings.server_url}"

 
       subject = if @owner
                   I18n.t("notifier.changeset_comment_notification.commented.subject_own", :commenter => @commenter)
 
       subject = if @owner
                   I18n.t("notifier.changeset_comment_notification.commented.subject_own", :commenter => @commenter)


@@ -162,6 +174,9 @@ class Notifier < ActionMailer::Base
 
       attach_user_avatar(comment.author)
 
 
       attach_user_avatar(comment.author)
 

+      headers["In-Reply-To"] = @ref
+      headers["References"]  = @ref
+

       mail :to => recipient.email, :subject => subject
     end
   end
       mail :to => recipient.email, :subject => subject
     end
   end


@@ -173,19 +188,19 @@ class Notifier < ActionMailer::Base
   end
 
   def attach_project_logo
   end
 
   def attach_project_logo

-    attachments.inline["logo.png"] = File.read(Rails.root.join("app", "assets", "images", "osm_logo_30.png"))
+    attachments.inline["logo.png"] = File.read(Rails.root.join("app/assets/images/osm_logo_30.png"))

   end
 
   def attach_user_avatar(user)
   end
 
   def attach_user_avatar(user)

-    attachments.inline["avatar.png"] = File.read(user_avatar_file_path(user))
+    attachments.inline["avatar.png"] = user_avatar_file(user)

   end
 
   end
 

-  def user_avatar_file_path(user)
-    image = user&.image
-    if image&.file?
-      return image.path(:small)
+  def user_avatar_file(user)
+    avatar = user&.avatar
+    if avatar&.attached?
+      avatar.variant(:resize => "50x50>").blob.download

     else
     else

-      return Rails.root.join("app", "assets", "images", "users", "images", "small.png")
+      File.read(Rails.root.join("app/assets/images/avatar_small.png"))

     end
   end
 
     end
   end
 


@@ -196,14 +211,14 @@ class Notifier < ActionMailer::Base
   end
 
   def from_address(name, type, id, digest, user_id = nil)
   end
 
   def from_address(name, type, id, digest, user_id = nil)

-    if Object.const_defined?(:MESSAGES_DOMAIN) && domain = MESSAGES_DOMAIN
+    if Settings.key?(:messages_domain) && domain = Settings.messages_domain

       if user_id
         "#{name} <#{type}-#{id}-#{user_id}-#{digest[0, 6]}@#{domain}>"
       else
         "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>"
       end
     else
       if user_id
         "#{name} <#{type}-#{id}-#{user_id}-#{digest[0, 6]}@#{domain}>"
       else
         "#{name} <#{type}-#{id}-#{digest[0, 6]}@#{domain}>"
       end
     else

-      EMAIL_FROM
+      Settings.email_from

     end
   end
 end
     end
   end
 end