summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorClaire <claire.github-309c@sitedethib.com>2021-04-23 22:51:21 +0200
committerGitHub <noreply@github.com>2021-04-23 22:51:21 +0200
commita6564d56d6af0467eb8cfa96444b4503f437e0a6 (patch)
tree72c47386ec1dc5b1a823acd2db574dbe687bceac
parent23601914345d201b8f06ceea637c70e0a23674db (diff)
Fix edge case where accepted follow cannot be processed because of follow limit (#16098)
-rw-r--r--app/models/follow_request.rb2
-rw-r--r--spec/models/follow_request_spec.rb2
2 files changed, 2 insertions, 2 deletions
diff --git a/app/models/follow_request.rb b/app/models/follow_request.rb
index 59fefcdf64e..0b6f7629af9 100644
--- a/app/models/follow_request.rb
+++ b/app/models/follow_request.rb
@@ -29,7 +29,7 @@ class FollowRequest < ApplicationRecord
validates :account_id, uniqueness: { scope: :target_account_id }
def authorize!
- account.follow!(target_account, reblogs: show_reblogs, notify: notify, uri: uri)
+ account.follow!(target_account, reblogs: show_reblogs, notify: notify, uri: uri, bypass_limit: true)
MergeWorker.perform_async(target_account.id, account.id) if account.local?
destroy!
end
diff --git a/spec/models/follow_request_spec.rb b/spec/models/follow_request_spec.rb
index cc484a5b921..36ce8ee60b1 100644
--- a/spec/models/follow_request_spec.rb
+++ b/spec/models/follow_request_spec.rb
@@ -7,7 +7,7 @@ RSpec.describe FollowRequest, type: :model do
let(:target_account) { Fabricate(:account) }
it 'calls Account#follow!, MergeWorker.perform_async, and #destroy!' do
- expect(account).to receive(:follow!).with(target_account, reblogs: true, notify: false, uri: follow_request.uri)
+ expect(account).to receive(:follow!).with(target_account, reblogs: true, notify: false, uri: follow_request.uri, bypass_limit: true)
expect(MergeWorker).to receive(:perform_async).with(target_account.id, account.id)
expect(follow_request).to receive(:destroy!)
follow_request.authorize!