mirrors/catstodon
1
0
Fork 0
mirror of https://git.kescher.at/CatCatNya/catstodon.git synced 2024-11-30 17:59:03 +01:00
Code Issues Projects Releases Packages Wiki Activity Actions

serialize custom emoji reactions properly for AP

Browse source 
Akkoma and possibly others expect the `tag` field
in an EmojiReact activity to be an array, not just
a single object, so it's being wrapped into one
now.  I'm not entirely sure whether this is the
idiomatic way of doing it tbh, but it works fine.
This commit is contained in:
fef 2022-12-04 10:52:02 +00:00
parent a556d81cde
commit 14d1f7d9b3
No known key found for this signature in database
GPG key ID: EC22E476DC2D3D84
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
app/serializers/activitypub/emoji_reaction_serializer.rb
View file

@ -3,8 +3,7 @@
class ActivityPub::EmojiReactionSerializer < ActivityPub::Serializer class ActivityPub::EmojiReactionSerializer < ActivityPub::Serializer
attributes :id, :type, :actor, :content attributes :id, :type, :actor, :content
attribute :virtual_object, key: :object attribute :virtual_object, key: :object
attribute :custom_emoji, key: :tag, unless: -> { object.custom_emoji.nil? }
has_one :custom_emoji, key: :tag, serializer: ActivityPub::EmojiSerializer, unless: -> { object.custom_emoji.nil? }
def id def id
[ActivityPub::TagManager.instance.uri_for(object.account), '#emoji_reactions/', object.id].join [ActivityPub::TagManager.instance.uri_for(object.account), '#emoji_reactions/', object.id].join
@ -31,4 +30,10 @@ class ActivityPub::EmojiReactionSerializer < ActivityPub::Serializer
end end
alias reaction content alias reaction content
# Akkoma (and possibly others) expect `tag` to be an array, so we can't just
# use the has_one shorthand because we need to wrap it into an array manually
def custom_emoji
[ActivityPub::EmojiSerializer.new(object.custom_emoji).serializable_hash]
end
end end