diff options
| author | Eugen Rochko <eugen@zeonfederated.com> | 2016-12-31 14:35:08 +0100 |
|---|---|---|
| committer | Eugen Rochko <eugen@zeonfederated.com> | 2016-12-31 14:36:25 +0100 |
| commit | 8d442816774e2f7974a02bf262ff9e6fdfeab4f4 (patch) | |
| tree | aafddfde01fe2d015400cf9c9ec663693a3d82b4 /app/lib | |
| parent | 777bcfc7016d4f4354af2d337dee113c6e2ad8a9 (diff) | |
Set in_reply_to_account on statuses to non-self value when possible, thus
resolving the confusion from self-chain replies ultimately linking to a non-self status. Adjust filters
Diffstat (limited to 'app/lib')
| -rw-r--r-- | app/lib/feed_manager.rb | 22 |
1 files changed, 11 insertions, 11 deletions
diff --git a/app/lib/feed_manager.rb b/app/lib/feed_manager.rb index c6262f211..0056321fa 100644 --- a/app/lib/feed_manager.rb +++ b/app/lib/feed_manager.rb @@ -78,15 +78,15 @@ class FeedManager def filter_from_home?(status, receiver) should_filter = false - if status.reply? && !status.thread.account.nil? # Filter out if it's a reply - should_filter = !receiver.following?(status.thread.account) # and I'm not following the person it's a reply to - should_filter &&= !(receiver.id == status.thread.account_id) # and it's not a reply to me - should_filter &&= !(status.account_id == status.thread.account_id) # and it's not a self-reply - elsif status.reblog? # Filter out a reblog - should_filter = receiver.blocking?(status.reblog.account) # if I'm blocking the reblogged person + if status.reply? && !status.in_reply_to_account_id.nil? # Filter out if it's a reply + should_filter = !receiver.following?(status.in_reply_to_account) # and I'm not following the person it's a reply to + should_filter &&= !(receiver.id == status.in_reply_to_account_id) # and it's not a reply to me + should_filter &&= !(status.account_id == status.in_reply_to_account_id) # and it's not a self-reply + elsif status.reblog? # Filter out a reblog + should_filter = receiver.blocking?(status.reblog.account) # if I'm blocking the reblogged person end - should_filter ||= receiver.blocking?(status.mentions.map(&:account_id)) # or if it mentions someone I blocked + should_filter ||= receiver.blocking?(status.mentions.map(&:account_id)) # or if it mentions someone I blocked should_filter end @@ -98,8 +98,8 @@ class FeedManager should_filter ||= (status.account.silenced? && !receiver.following?(status.account)) # of if the account is silenced and I'm not following them should_filter ||= (status.private_visibility? && !receiver.following?(status.account)) # or if the mentioned account is not permitted to see the private status - if status.reply? && !status.thread.account.nil? # or it's a reply - should_filter ||= receiver.blocking?(status.thread.account) # to a user I blocked + if status.reply? && !status.in_reply_to_account_id.nil? # or it's a reply + should_filter ||= receiver.blocking?(status.in_reply_to_account) # to a user I blocked end should_filter @@ -109,8 +109,8 @@ class FeedManager should_filter = receiver.blocking?(status.account) should_filter ||= receiver.blocking?(status.mentions.includes(:account).map(&:account)) - if status.reply? && !status.thread.account.nil? - should_filter ||= receiver.blocking?(status.thread.account) + if status.reply? && !status.in_reply_to_account_id.nil? + should_filter ||= receiver.blocking?(status.in_reply_to_account) elsif status.reblog? should_filter ||= receiver.blocking?(status.reblog.account) end |