Embed announce activity data instead of linking to it

This commit is contained in:
Francis Dinh 2018-04-22 21:28:51 -04:00
parent b1742eca42
commit 8c0806539c
3 changed files with 15 additions and 7 deletions

View file

@ -140,9 +140,14 @@ defmodule Pleroma.Web.ActivityPub.ActivityPub do
end
end
def unannounce(%User{} = actor, %Object{} = object, local \\ true) do
def unannounce(
%User{} = actor,
%Object{} = object,
local \\ true,
activity_id \\ nil
) do
with %Activity{} = activity <- get_existing_announce(actor.ap_id, object),
unannounce_data <- make_unannounce_data(actor, activity),
unannounce_data <- make_unannounce_data(actor, activity, activity_id),
{:ok, unannounce_activity} <- insert(unannounce_data, local),
:ok <- maybe_federate(activity),
{:ok, _activity} <- Repo.delete(activity),

View file

@ -283,16 +283,19 @@ defmodule Pleroma.Web.ActivityPub.Utils do
"""
def make_unannounce_data(
%User{ap_id: ap_id} = user,
%Activity{data: %{"id" => id, "context" => context}} = activity
%Activity{data: %{"context" => context}} = activity,
activity_id
) do
%{
data = %{
"type" => "Undo",
"actor" => ap_id,
"object" => id,
"object" => activity.data,
"to" => [user.follower_address, activity.data["actor"]],
"cc" => ["https://www.w3.org/ns/activitystreams#Public"],
"context" => context
}
if activity_id, do: Map.put(data, "id", activity_id), else: data
end
def add_announce_to_object(%Activity{data: %{"actor" => actor}}, object) do

View file

@ -294,7 +294,7 @@ defmodule Pleroma.Web.ActivityPub.ActivityPubTest do
announce_activity.data["actor"]
]
assert unannounce_activity.data["type"] == "Undo"
assert unannounce_activity.data["object"] == announce_activity.data["id"]
assert unannounce_activity.data["object"] == announce_activity.data
assert unannounce_activity.data["actor"] == user.ap_id
assert unannounce_activity.data["context"] == announce_activity.data["context"]