From 456d79b853aaec14f14c4c4deaa04cfb66c5b357 Mon Sep 17 00:00:00 2001
From: David Heinemeier Hansson <david@loudthinking.com>
Date: Fri, 28 Sep 2018 11:01:49 -0700
Subject: Extract MessageId concern

---
 app/models/action_mailroom/inbound_email.rb        | 26 +++-------------------
 .../action_mailroom/inbound_email/message_id.rb    | 25 +++++++++++++++++++++
 2 files changed, 28 insertions(+), 23 deletions(-)
 create mode 100644 app/models/action_mailroom/inbound_email/message_id.rb

(limited to 'app/models/action_mailroom')

diff --git a/app/models/action_mailroom/inbound_email.rb b/app/models/action_mailroom/inbound_email.rb
index 1f086b24bf..cf7370d543 100644
--- a/app/models/action_mailroom/inbound_email.rb
+++ b/app/models/action_mailroom/inbound_email.rb
@@ -3,28 +3,13 @@ require "mail"
 class ActionMailroom::InboundEmail < ActiveRecord::Base
   self.table_name = "action_mailroom_inbound_emails"
 
-  include Incineratable, Routable
+  include Incineratable, MessageId, Routable
 
   has_one_attached :raw_email
   enum status: %i[ pending processing delivered failed bounced ]
 
-  before_save :generate_missing_message_id
-
-  class << self
-    def create_from_raw_email!(raw_email, **options)
-      create! raw_email: raw_email, message_id: extract_message_id(raw_email), **options
-    end
-    
-    def mail_from_source(source)
-      Mail.new(Mail::Utilities.binary_unsafe_to_crlf(source.to_s))
-    end
-
-    private
-      def extract_message_id(raw_email)
-        mail_from_source(raw_email.read).message_id
-      rescue => e
-        # FIXME: Add logging with "Couldn't extract Message ID, so will generating a new random ID instead"
-      end
+  def self.mail_from_source(source)
+    Mail.new Mail::Utilities.binary_unsafe_to_crlf(source.to_s)
   end
 
   def mail
@@ -34,9 +19,4 @@ class ActionMailroom::InboundEmail < ActiveRecord::Base
   def source
     @source ||= raw_email.download
   end
-
-  private
-    def generate_missing_message_id
-      self.message_id ||= Mail::MessageIdField.new.message_id
-    end
 end
diff --git a/app/models/action_mailroom/inbound_email/message_id.rb b/app/models/action_mailroom/inbound_email/message_id.rb
new file mode 100644
index 0000000000..6bdda8c1c7
--- /dev/null
+++ b/app/models/action_mailroom/inbound_email/message_id.rb
@@ -0,0 +1,25 @@
+module ActionMailroom::InboundEmail::MessageId
+  extend ActiveSupport::Concern
+
+  included do
+    before_save :generate_missing_message_id
+  end
+
+  module ClassMethods
+    def create_and_extract_message_id!(raw_email, **options)
+      create! raw_email: raw_email, message_id: extract_message_id(raw_email), **options
+    end
+
+    private
+      def extract_message_id(raw_email)
+        mail_from_source(raw_email.read).message_id
+      rescue => e
+        # FIXME: Add logging with "Couldn't extract Message ID, so will generating a new random ID instead"
+      end
+  end
+
+  private
+    def generate_missing_message_id
+      self.message_id ||= Mail::MessageIdField.new.message_id
+    end
+end
-- 
cgit v1.2.3