diff options
author | David Heinemeier Hansson <david@loudthinking.com> | 2018-09-19 16:40:56 -0700 |
---|---|---|
committer | David Heinemeier Hansson <david@loudthinking.com> | 2018-09-19 16:40:56 -0700 |
commit | 4b5e1e982740723648638ab06fbe7e38e2e70dc1 (patch) | |
tree | 1c2f342de3d862950ace59deb3f105253b6c1ad4 /app | |
parent | 3e199600ec1117b6595e402a27d1ae58c18b39e7 (diff) | |
download | rails-4b5e1e982740723648638ab06fbe7e38e2e70dc1.tar.gz rails-4b5e1e982740723648638ab06fbe7e38e2e70dc1.tar.bz2 rails-4b5e1e982740723648638ab06fbe7e38e2e70dc1.zip |
Extract routable concern
Diffstat (limited to 'app')
-rw-r--r-- | app/models/action_mailroom/inbound_email.rb | 13 | ||||
-rw-r--r-- | app/models/action_mailroom/inbound_email/routable.rb | 12 |
2 files changed, 14 insertions, 11 deletions
diff --git a/app/models/action_mailroom/inbound_email.rb b/app/models/action_mailroom/inbound_email.rb index d44007c2f1..c439988e91 100644 --- a/app/models/action_mailroom/inbound_email.rb +++ b/app/models/action_mailroom/inbound_email.rb @@ -1,23 +1,14 @@ require "mail" class ActionMailroom::InboundEmail < ActiveRecord::Base - include Incineratable - self.table_name = "action_mailroom_inbound_emails" - has_one_attached :raw_email + include Incineratable, Routable + has_one_attached :raw_email enum status: %i[ pending processing delivered failed bounced ] - after_create_commit :deliver_to_mailroom_later, if: ->(r) { r.pending? } - - def mail @mail ||= Mail.new(Mail::Utilities.binary_unsafe_to_crlf(raw_email.download)) end - - private - def deliver_to_mailroom_later - ActionMailroom::RoutingJob.perform_later self - end end diff --git a/app/models/action_mailroom/inbound_email/routable.rb b/app/models/action_mailroom/inbound_email/routable.rb new file mode 100644 index 0000000000..b888c592f5 --- /dev/null +++ b/app/models/action_mailroom/inbound_email/routable.rb @@ -0,0 +1,12 @@ +module ActionMailroom::InboundEmail::Routable + extend ActiveSupport::Concern + + included do + after_create_commit :route_later, if: ->(r) { r.pending? } + end + + private + def route_later + ActionMailroom::RoutingJob.perform_later self + end +end |