mirror of
https://github.com/kikobar/mastodon.git
synced 2024-12-29 09:14:15 +00:00
9d3c6f1849
* Fetch up to 5 replies when discovering a new remote status This is used for resolving threads downwards. The originating server must add a “replies” attributes with such replies for it to be useful. * Add some tests for ActivityPub::FetchRepliesWorker * Add specs for ActivityPub::FetchRepliesService * Serialize up to 5 public self-replies for ActivityPub notes * Add specs for ActivityPub::NoteSerializer * Move exponential backoff logic to a worker concern * Fetch first page of paginated collections when fetching thread replies * Add specs for paginated collections in replies * Move Note replies serialization to a first CollectionPage The collection isn't actually paginable yet as it has no id nor a `next` field. This may come in another PR. * Use pluck(:uri) instead of map(&:uri) to improve performances * Fix fetching replies when they are in a CollectionPage
137 lines
4.1 KiB
Ruby
137 lines
4.1 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
module StatusThreadingConcern
|
|
extend ActiveSupport::Concern
|
|
|
|
def ancestors(limit, account = nil)
|
|
find_statuses_from_tree_path(ancestor_ids(limit), account)
|
|
end
|
|
|
|
def descendants(limit, account = nil, max_child_id = nil, since_child_id = nil, depth = nil)
|
|
find_statuses_from_tree_path(descendant_ids(limit, max_child_id, since_child_id, depth), account, promote: true)
|
|
end
|
|
|
|
def self_replies(limit)
|
|
account.statuses.where(in_reply_to_id: id, visibility: [:public, :unlisted]).reorder(id: :asc).limit(limit)
|
|
end
|
|
|
|
private
|
|
|
|
def ancestor_ids(limit)
|
|
key = "ancestors:#{id}"
|
|
ancestors = Rails.cache.fetch(key)
|
|
|
|
if ancestors.nil? || ancestors[:limit] < limit
|
|
ids = ancestor_statuses(limit).pluck(:id).reverse!
|
|
Rails.cache.write key, limit: limit, ids: ids
|
|
ids
|
|
else
|
|
ancestors[:ids].last(limit)
|
|
end
|
|
end
|
|
|
|
def ancestor_statuses(limit)
|
|
Status.find_by_sql([<<-SQL.squish, id: in_reply_to_id, limit: limit])
|
|
WITH RECURSIVE search_tree(id, in_reply_to_id, path)
|
|
AS (
|
|
SELECT id, in_reply_to_id, ARRAY[id]
|
|
FROM statuses
|
|
WHERE id = :id
|
|
UNION ALL
|
|
SELECT statuses.id, statuses.in_reply_to_id, path || statuses.id
|
|
FROM search_tree
|
|
JOIN statuses ON statuses.id = search_tree.in_reply_to_id
|
|
WHERE NOT statuses.id = ANY(path)
|
|
)
|
|
SELECT id
|
|
FROM search_tree
|
|
ORDER BY path
|
|
LIMIT :limit
|
|
SQL
|
|
end
|
|
|
|
def descendant_ids(limit, max_child_id, since_child_id, depth)
|
|
descendant_statuses(limit, max_child_id, since_child_id, depth).pluck(:id)
|
|
end
|
|
|
|
def descendant_statuses(limit, max_child_id, since_child_id, depth)
|
|
# use limit + 1 and depth + 1 because 'self' is included
|
|
depth += 1 if depth.present?
|
|
limit += 1 if limit.present?
|
|
|
|
descendants_with_self = Status.find_by_sql([<<-SQL.squish, id: id, limit: limit, max_child_id: max_child_id, since_child_id: since_child_id, depth: depth])
|
|
WITH RECURSIVE search_tree(id, path)
|
|
AS (
|
|
SELECT id, ARRAY[id]
|
|
FROM statuses
|
|
WHERE id = :id AND COALESCE(id < :max_child_id, TRUE) AND COALESCE(id > :since_child_id, TRUE)
|
|
UNION ALL
|
|
SELECT statuses.id, path || statuses.id
|
|
FROM search_tree
|
|
JOIN statuses ON statuses.in_reply_to_id = search_tree.id
|
|
WHERE COALESCE(array_length(path, 1) < :depth, TRUE) AND NOT statuses.id = ANY(path)
|
|
)
|
|
SELECT id
|
|
FROM search_tree
|
|
ORDER BY path
|
|
LIMIT :limit
|
|
SQL
|
|
|
|
descendants_with_self - [self]
|
|
end
|
|
|
|
def find_statuses_from_tree_path(ids, account, promote: false)
|
|
statuses = statuses_with_accounts(ids).to_a
|
|
account_ids = statuses.map(&:account_id).uniq
|
|
domains = statuses.map(&:account_domain).compact.uniq
|
|
relations = relations_map_for_account(account, account_ids, domains)
|
|
|
|
statuses.reject! { |status| filter_from_context?(status, account, relations) }
|
|
|
|
# Order ancestors/descendants by tree path
|
|
statuses.sort_by! { |status| ids.index(status.id) }
|
|
|
|
# Bring self-replies to the top
|
|
if promote
|
|
promote_by!(statuses) { |status| status.in_reply_to_account_id == status.account_id }
|
|
else
|
|
statuses
|
|
end
|
|
end
|
|
|
|
def promote_by!(arr)
|
|
insert_at = arr.find_index { |item| !yield(item) }
|
|
|
|
return arr if insert_at.nil?
|
|
|
|
arr.each_with_index do |item, index|
|
|
next if index <= insert_at || !yield(item)
|
|
|
|
arr.insert(insert_at, arr.delete_at(index))
|
|
insert_at += 1
|
|
end
|
|
|
|
arr
|
|
end
|
|
|
|
def relations_map_for_account(account, account_ids, domains)
|
|
return {} if account.nil?
|
|
|
|
{
|
|
blocking: Account.blocking_map(account_ids, account.id),
|
|
blocked_by: Account.blocked_by_map(account_ids, account.id),
|
|
muting: Account.muting_map(account_ids, account.id),
|
|
following: Account.following_map(account_ids, account.id),
|
|
domain_blocking_by_domain: Account.domain_blocking_map_by_domain(domains, account.id),
|
|
}
|
|
end
|
|
|
|
def statuses_with_accounts(ids)
|
|
Status.where(id: ids).includes(:account)
|
|
end
|
|
|
|
def filter_from_context?(status, account, relations)
|
|
StatusFilter.new(status, account, relations).filtered?
|
|
end
|
|
end
|