Browse Source

Fix regression from #3842 (#3892)

* Fix regression from #3842

Simplify the query by omitting all direct statuses. Private statuses
are allowed because they are from accounts we are following (so
by definition)

Resolves #3887 (alternative)

* Adjust test
closed-social-glitch-2
Eugen Rochko 6 years ago
committed by GitHub
parent
commit
0190aac240
2 changed files with 5 additions and 13 deletions
  1. +1
    -9
      app/models/status.rb
  2. +4
    -4
      spec/models/status_spec.rb

+ 1
- 9
app/models/status.rb View File

@ -131,15 +131,7 @@ class Status < ApplicationRecord
end end
def as_home_timeline(account) def as_home_timeline(account)
# 'references' is a workaround for the following issue:
# Inconsistent results with #or in ActiveRecord::Relation with respect to documentation Issue #24055 rails/rails
# https://github.com/rails/rails/issues/24055
references(:mentions)
.where.not(visibility: :direct)
.or(where(mentions: { account: account }))
.where(follows: { account_id: account })
.or(references(:mentions, :follows).where(account: account))
.left_outer_joins(account: :followers).left_outer_joins(:mentions).group(:id)
where(account: [account] + account.following).where(visibility: [:public, :unlisted, :private])
end end
def as_public_timeline(account = nil, local_only = false) def as_public_timeline(account = nil, local_only = false)

+ 4
- 4
spec/models/status_spec.rb View File

@ -201,17 +201,17 @@ RSpec.describe Status, type: :model do
expect(@results).to include(@self_status) expect(@results).to include(@self_status)
end end
it 'includes direct statuses from self' do
expect(@results).to include(@self_direct_status)
it 'does not include direct statuses from self' do
expect(@results).to_not include(@self_direct_status)
end end
it 'includes statuses from followed' do it 'includes statuses from followed' do
expect(@results).to include(@followed_status) expect(@results).to include(@followed_status)
end end
it 'includes direct statuses mentioning recipient from followed' do
it 'does not include direct statuses mentioning recipient from followed' do
Fabricate(:mention, account: account, status: @followed_direct_status) Fabricate(:mention, account: account, status: @followed_direct_status)
expect(@results).to include(@followed_direct_status)
expect(@results).to_not include(@followed_direct_status)
end end
it 'does not include direct statuses not mentioning recipient from followed' do it 'does not include direct statuses not mentioning recipient from followed' do

Loading…
Cancel
Save