treehouse
/
mastodon
21
8
Fork 5
Code Issues 32 Pull Requests Projects 1 Releases Wiki Activity

Fix nil error when removing status caused by race condition (#16099)

lolsob-rspec
Eugen Rochko 2021-04-24 13:35:39 +02:00 committed by GitHub
parent 3b3907ec9d
commit bf5d831fe7
3 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/lib/status_reach_finder.rb
View File

@ -62,7 +62,7 @@ class StatusReachFinder
end
def followers_inboxes
if @status.reply? && @status.thread.account.local? && @status.distributable?
if @status.in_reply_to_local_account? && @status.distributable?
@status.account.followers.or(@status.thread.account.followers).inboxes
else
@status.account.followers.inboxes

4
app/models/status.rb
View File

@ -161,6 +161,10 @@ class Status < ApplicationRecord
attributes['local'] || uri.nil?
end
def in_reply_to_local_account?
reply? && thread&.account&.local?
end
def reblog?
!reblog_of_id.nil?
end

2
app/workers/activitypub/distribution_worker.rb
View File

@ -35,7 +35,7 @@ class ActivityPub::DistributionWorker
# Deliver the status to all followers.
# If the status is a reply to another local status, also forward it to that
# status' authors' followers.
@inboxes ||= if @status.reply? && @status.thread.account.local? && @status.distributable?
@inboxes ||= if @status.in_reply_to_local_account? && @status.distributable?
@account.followers.or(@status.thread.account.followers).inboxes
else
@account.followers.inboxes