[sorting query builder] Adjust filtering

This commit is contained in:
Fosco Marotto 2021-01-21 17:37:12 -05:00
parent 5b4b633702
commit 852d2f78e9
4 changed files with 13 additions and 8 deletions

View File

@ -44,9 +44,10 @@ class Api::V1::Timelines::ExploreController < Api::BaseController
def explore_statuses def explore_statuses
if current_account if current_account
SortingQueryBuilder.new.call(@sort_type, nil, params[:page]).reject {|status| SortingQueryBuilder.new.call(@sort_type, nil, params[:page], current_account)
FeedManager.instance.filter?(:home, status, current_account.id) .reject {|status|
} FeedManager.instance.filter?(:home, status, current_account.id)
}
else else
page = [params[:page].to_i.abs, MIN_UNAUTHENTICATED_PAGES].min page = [params[:page].to_i.abs, MIN_UNAUTHENTICATED_PAGES].min
SortingQueryBuilder.new.call(@sort_type, nil, page) SortingQueryBuilder.new.call(@sort_type, nil, page)

View File

@ -68,7 +68,7 @@ class Api::V1::Timelines::GroupCollectionController < Api::BaseController
end end
if current_account if current_account
SortingQueryBuilder.new.call(@sort_type, @groupIds, params[:page]).reject {|status| SortingQueryBuilder.new.call(@sort_type, @groupIds, params[:page], current_account).reject {|status|
FeedManager.instance.filter?(:home, status, current_account.id) FeedManager.instance.filter?(:home, status, current_account.id)
} }
else else

View File

@ -51,7 +51,7 @@ class Api::V1::Timelines::GroupController < Api::BaseController
def group_statuses def group_statuses
if current_account if current_account
SortingQueryBuilder.new.call(@sort_type, @group, params[:page]).reject {|status| SortingQueryBuilder.new.call(@sort_type, @group, params[:page], current_account).reject {|status|
FeedManager.instance.filter?(:home, status, current_account.id) FeedManager.instance.filter?(:home, status, current_account.id)
} }
else else

View File

@ -1,7 +1,7 @@
# frozen_string_literal: true # frozen_string_literal: true
class SortingQueryBuilder < BaseService class SortingQueryBuilder < BaseService
def call(sort_type, group = nil, page = 1) def call(sort_type, group = nil, page = 1, account = nil)
limit = 20 limit = 20
min_likes = 5 min_likes = 5
@ -44,6 +44,8 @@ class SortingQueryBuilder < BaseService
query = query.with_public_visibility if group.nil? query = query.with_public_visibility if group.nil?
query = query.where('statuses.created_at > ?', date_limit) query = query.where('statuses.created_at > ?', date_limit)
query = query.where(group: group) unless group.nil? query = query.where(group: group) unless group.nil?
query = query.excluding_blocked_reblogs(account) unless account.nil?
query = query.not_excluded_by_account(current_account) unless account.nil?
query = query.page(page.to_i) query = query.page(page.to_i)
query = query.per(limit) query = query.per(limit)
return query return query
@ -55,6 +57,8 @@ class SortingQueryBuilder < BaseService
query = query.where('status_stats.reblogs_count > ?', min_reblogs) unless sort_type == 'recent' query = query.where('status_stats.reblogs_count > ?', min_reblogs) unless sort_type == 'recent'
query = query.where('status_stats.favourites_count > ?', min_likes) unless sort_type == 'recent' query = query.where('status_stats.favourites_count > ?', min_likes) unless sort_type == 'recent'
query = query.joins(:status) query = query.joins(:status)
query = query.excluding_blocked_reblogs(account) unless account.nil?
query = query.not_excluded_by_account(current_account) unless account.nil?
query = query.where('statuses.reblog_of_id IS NULL') query = query.where('statuses.reblog_of_id IS NULL')
query = query.where('statuses.in_reply_to_id IS NULL') query = query.where('statuses.in_reply_to_id IS NULL')
query = query.where('statuses.group_id': group) unless group.nil? query = query.where('statuses.group_id': group) unless group.nil?