aboutsummaryrefslogtreecommitdiffstats
path: root/app/models/action_mailbox/inbound_email/incineratable.rb
diff options
context:
space:
mode:
authorGeorge Claghorn <george@basecamp.com>2018-10-06 22:18:09 -0400
committerGeorge Claghorn <george@basecamp.com>2018-10-06 22:18:09 -0400
commit40fcc1ea95795b0a69fe4ee690c031b5ea974042 (patch)
treed9d2a5e4008f81d4c4d889462810c741577bf222 /app/models/action_mailbox/inbound_email/incineratable.rb
parent7b95ebc9e44b7dba1e4aa32b773a124f75de9474 (diff)
downloadrails-40fcc1ea95795b0a69fe4ee690c031b5ea974042.tar.gz
rails-40fcc1ea95795b0a69fe4ee690c031b5ea974042.tar.bz2
rails-40fcc1ea95795b0a69fe4ee690c031b5ea974042.zip
Shush interpreter warnings regarding uninitialized instance variable
Diffstat (limited to 'app/models/action_mailbox/inbound_email/incineratable.rb')
-rw-r--r--app/models/action_mailbox/inbound_email/incineratable.rb8
1 files changed, 4 insertions, 4 deletions
diff --git a/app/models/action_mailbox/inbound_email/incineratable.rb b/app/models/action_mailbox/inbound_email/incineratable.rb
index 6ba73c0c6d..e85f3e264d 100644
--- a/app/models/action_mailbox/inbound_email/incineratable.rb
+++ b/app/models/action_mailbox/inbound_email/incineratable.rb
@@ -6,7 +6,7 @@ module ActionMailbox::InboundEmail::Incineratable
included do
before_update :remember_to_incinerate_later
- after_update_commit :incinerate_later, if: :need_to_incinerate_later?
+ after_update_commit :incinerate_later, if: :incinerating_later?
end
def incinerate
@@ -17,12 +17,12 @@ module ActionMailbox::InboundEmail::Incineratable
# TODO: Use enum change tracking once merged into Active Support
def remember_to_incinerate_later
if status_changed? && (delivered? || failed?)
- @incinerate_later = true
+ @incinerating_later = true
end
end
- def need_to_incinerate_later?
- @incinerate_later
+ def incinerating_later?
+ @incinerating_later ||= false
end
def incinerate_later