diff options
author | Claire <claire.github-309c@sitedethib.com> | 2022-05-03 12:05:24 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-05-03 12:05:24 +0200 |
commit | 58ac5ae643dc57ffd20017c54e7be523deaef156 (patch) | |
tree | 491e4afab8ab269096f6f68a3dfab08433954cda /app/services/activitypub/fetch_featured_collection_service.rb | |
parent | cb9a919ff28294a2f1cf3642a165ad433cbf856b (diff) | |
parent | fa22b8ed229f93de1236a74d4df4a0b5fa57dddd (diff) |
Merge pull request #1761 from ClearlyClaire/glitch-soc/merge-upstream
Merge upstream changes
Diffstat (limited to 'app/services/activitypub/fetch_featured_collection_service.rb')
-rw-r--r-- | app/services/activitypub/fetch_featured_collection_service.rb | 34 |
1 files changed, 23 insertions, 11 deletions
diff --git a/app/services/activitypub/fetch_featured_collection_service.rb b/app/services/activitypub/fetch_featured_collection_service.rb index 07a9fe039..37d05e055 100644 --- a/app/services/activitypub/fetch_featured_collection_service.rb +++ b/app/services/activitypub/fetch_featured_collection_service.rb @@ -7,19 +7,33 @@ class ActivityPub::FetchFeaturedCollectionService < BaseService return if account.featured_collection_url.blank? || account.suspended? || account.local? @account = account - @json = fetch_resource(@account.featured_collection_url, true) + @json = fetch_resource(@account.featured_collection_url, true, local_follower) - return unless supported_context? + return unless supported_context?(@json) - case @json['type'] + process_items(collection_items(@json)) + end + + private + + def collection_items(collection) + collection = fetch_collection(collection['first']) if collection['first'].present? + return unless collection.is_a?(Hash) + + case collection['type'] when 'Collection', 'CollectionPage' - process_items @json['items'] + collection['items'] when 'OrderedCollection', 'OrderedCollectionPage' - process_items @json['orderedItems'] + collection['orderedItems'] end end - private + def fetch_collection(collection_or_uri) + return collection_or_uri if collection_or_uri.is_a?(Hash) + return if invalid_origin?(collection_or_uri) + + fetch_resource_without_id_validation(collection_or_uri, local_follower, true) + end def process_items(items) status_ids = items.filter_map do |item| @@ -53,11 +67,9 @@ class ActivityPub::FetchFeaturedCollectionService < BaseService end end - def supported_context? - super(@json) - end - def local_follower - @local_follower ||= @account.followers.local.without_suspended.first + return @local_follower if defined?(@local_follower) + + @local_follower = @account.followers.local.without_suspended.first end end |