about summary refs log tree commit diff
path: root/app/models/admin
diff options
context:
space:
mode:
authorEugen Rochko <eugen@zeonfederated.com>2022-03-15 04:11:13 +0100
committerGitHub <noreply@github.com>2022-03-15 04:11:13 +0100
commiteebafe24a8000ac77bb54660170f5b47bfc5b3a3 (patch)
tree01d74338fdbb82fe3e3501f6738992f971ed2ddc /app/models/admin
parenteb9a7e36260c99aec980d097ee819c17ebb93631 (diff)
Fix statuses not being referenced in strike when category is spam (#17786)
Diffstat (limited to 'app/models/admin')
-rw-r--r--app/models/admin/account_action.rb14
1 files changed, 12 insertions, 2 deletions
diff --git a/app/models/admin/account_action.rb b/app/models/admin/account_action.rb
index d3be4be3f..850ea6d82 100644
--- a/app/models/admin/account_action.rb
+++ b/app/models/admin/account_action.rb
@@ -22,6 +22,16 @@ class Admin::AccountAction
 
   attr_reader :warning, :send_email_notification, :include_statuses
 
+  alias send_email_notification? send_email_notification
+  alias include_statuses? include_statuses
+
+  def initialize(attributes = {})
+    @send_email_notification = true
+    @include_statuses        = true
+
+    super
+  end
+
   def send_email_notification=(value)
     @send_email_notification = ActiveModel::Type::Boolean.new.cast(value)
   end
@@ -141,11 +151,11 @@ class Admin::AccountAction
   end
 
   def warnable?
-    send_email_notification && target_account.local?
+    send_email_notification? && target_account.local?
   end
 
   def status_ids
-    report.status_ids if with_report? && include_statuses
+    report.status_ids if with_report? && include_statuses?
   end
 
   def reports