diff options
| author | Claire <claire.github-309c@sitedethib.com> | 2023-01-05 13:47:21 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-01-05 13:47:21 +0100 |
| commit | 18fb01ef7ca3829b07bb8ea101bdd073da72cfbc (patch) | |
| tree | 5e493ef3997176e4a98be57ee68068d427176906 /app/services/unsuspend_account_service.rb | |
| parent | 8683a082ddc04b2028b73a039325fd70fe3c55a9 (diff) | |
Fix possible race conditions when suspending/unsuspending accounts (#22363)
* Fix possible race conditions when suspending/unsuspending accounts * Fix tests Tests were assuming SuspensionWorker and UnsuspensionWorker would do the suspending/unsuspending themselves, but this has changed.
Diffstat (limited to 'app/services/unsuspend_account_service.rb')
| -rw-r--r-- | app/services/unsuspend_account_service.rb | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/app/services/unsuspend_account_service.rb b/app/services/unsuspend_account_service.rb index 534203dce..70667308e 100644 --- a/app/services/unsuspend_account_service.rb +++ b/app/services/unsuspend_account_service.rb @@ -2,10 +2,12 @@ class UnsuspendAccountService < BaseService include Payloadable + + # Restores a recently-unsuspended account + # @param [Account] account Account to restore def call(account) @account = account - unsuspend! refresh_remote_account! return if @account.nil? || @account.suspended? @@ -18,10 +20,6 @@ class UnsuspendAccountService < BaseService private - def unsuspend! - @account.unsuspend! if @account.suspended? - end - def refresh_remote_account! return if @account.local? |