Create deletes activities for incoming deletes.
authorRoger Braun <roger@rogerbraun.net>
Mon, 4 Sep 2017 16:47:46 +0000 (18:47 +0200)
committerRoger Braun <roger@rogerbraun.net>
Mon, 4 Sep 2017 16:47:46 +0000 (18:47 +0200)
lib/pleroma/web/ostatus/handlers/delete_handler.ex
test/web/ostatus/incoming_documents/delete_handling_test.exs

index f54a037a0ef3cdf56e5ff9b70b7f8c40b76806fc..29fe4052c915e10648965090ddb0bff6d3dbca37 100644 (file)
@@ -2,14 +2,13 @@ defmodule Pleroma.Web.OStatus.DeleteHandler do
   require Logger
   alias Pleroma.Web.{XML, OStatus}
   alias Pleroma.{Activity, Object, Repo}
+  alias Pleroma.Web.ActivityPub.ActivityPub
 
   def handle_delete(entry, doc \\ nil) do
     with id <- XML.string_from_xpath("//id", entry),
-         object when not is_nil(object) <- Object.get_by_ap_id(id) 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))
-      nil
+         object when not is_nil(object) <- Object.get_by_ap_id(id),
+         {:ok, delete} <- ActivityPub.delete(object, false) do
+      delete
     end
   end
 end
index 989c87afa14fd4a6258b198b725243541de19bdf..74c6a5994711763cd7cbe86d218ff38450f8acaa 100644 (file)
@@ -16,13 +16,15 @@ defmodule Pleroma.Web.OStatus.DeleteHandlingTest do
 
       incoming = File.read!("test/fixtures/delete.xml")
       |> String.replace("tag:mastodon.sdf.org,2017-06-10:objectId=310513:objectType=Status", note.data["object"]["id"])
-      {:ok, []} = OStatus.handle_incoming(incoming)
+      {:ok, [delete]} = OStatus.handle_incoming(incoming)
 
       refute Repo.get(Activity, note.id)
       refute Repo.get(Activity, like.id)
       refute Object.get_by_ap_id(note.data["object"]["id"])
       assert Repo.get(Activity, second_note.id)
       assert Object.get_by_ap_id(second_note.data["object"]["id"])
+
+      assert delete.data["type"] == "Delete"
     end
   end
 end