erin/akkoma
1
0
Fork 0
forked from mirrors/akkoma
Code Activity

Don't delete new delete activity...

Browse source
This commit is contained in:
Roger Braun 2017-09-04 20:47:43 +02:00
parent fcf3128a56
commit 3438a120cc
2 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
lib/pleroma/web/activity_pub/activity_pub.ex
View file

@ -83,11 +83,11 @@ def delete(%Object{data: %{"id" => id, "actor" => actor}} = object, local \\ tru
"object" => id,
"to" => [user.follower_address, "https://www.w3.org/ns/activitystreams#Public"]
}
with {:ok, activity} <- insert(data, local),
with Repo.delete(object),
Repo.delete_all(Activity.all_non_create_by_object_ap_id_q(id)),
Repo.delete_all(Activity.all_by_object_ap_id_q(id)),
{:ok, activity} <- insert(data, local),
:ok <- maybe_federate(activity) do
Repo.delete(object)
Repo.delete_all(Activity.all_non_create_by_object_ap_id_q(id))
Repo.delete_all(Activity.all_by_object_ap_id_q(id))
{:ok, activity}
end
end

2
test/web/activity_pub/activity_pub_test.exs
View file

@ -256,6 +256,8 @@ test "it creates a delete activity and deletes the original object" do
assert delete.data["actor"] == note.data["actor"]
assert delete.data["object"] == note.data["object"]["id"]
assert Repo.get(Activity, delete.id) != nil
assert Repo.get(Object, object.id) == nil
end
end