diff options
author | Claire <claire.github-309c@sitedethib.com> | 2023-01-15 16:53:30 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-01-15 16:53:30 +0100 |
commit | 13227e1dafd308dfe1a3effc3379b766274809b3 (patch) | |
tree | c06274b2eaaf6fdb6d1ffb8c2d0ff6b94befdf6b /app/models/admin/status_batch_action.rb | |
parent | afd0d424da4928b9e20a3c7a943f970252ed3a29 (diff) | |
parent | ab59743c131574076af7fa3640493a866c4528ad (diff) |
Merge pull request #2081 from ClearlyClaire/glitch-soc/merge-upstream
Merge upstream changes
Diffstat (limited to 'app/models/admin/status_batch_action.rb')
-rw-r--r-- | app/models/admin/status_batch_action.rb | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/app/models/admin/status_batch_action.rb b/app/models/admin/status_batch_action.rb index 0f019b854..39cd7d0eb 100644 --- a/app/models/admin/status_batch_action.rb +++ b/app/models/admin/status_batch_action.rb @@ -73,7 +73,7 @@ class Admin::StatusBatchAction # Can't use a transaction here because UpdateStatusService queues # Sidekiq jobs statuses.includes(:media_attachments, :preview_cards).find_each do |status| - next unless status.with_media? || status.with_preview_card? + next if status.discarded? || !(status.with_media? || status.with_preview_card?) authorize([:admin, status], :update?) @@ -89,15 +89,15 @@ class Admin::StatusBatchAction report.resolve!(current_account) log_action(:resolve, report) end - - @warning = target_account.strikes.create!( - action: :mark_statuses_as_sensitive, - account: current_account, - report: report, - status_ids: status_ids - ) end + @warning = target_account.strikes.create!( + action: :mark_statuses_as_sensitive, + account: current_account, + report: report, + status_ids: status_ids + ) + UserMailer.warning(target_account.user, @warning).deliver_later! if warnable? end |