Add deletion to masto api.
This commit is contained in:
parent
be04f725e9
commit
4dc517a0bb
|
@ -0,0 +1,13 @@
|
||||||
|
defmodule Pleroma.Web.CommonAPI do
|
||||||
|
alias Pleroma.{Repo, Activity, Object}
|
||||||
|
alias Pleroma.Web.ActivityPub.ActivityPub
|
||||||
|
|
||||||
|
def delete(activity_id, user) do
|
||||||
|
with %Activity{data: %{"object" => %{"id" => object_id}}} <- Repo.get(Activity, activity_id),
|
||||||
|
%Object{} = object <- Object.get_by_ap_id(object_id),
|
||||||
|
true <- user.ap_id == object.data["actor"],
|
||||||
|
{:ok, delete} <- ActivityPub.delete(object) do
|
||||||
|
{:ok, delete}
|
||||||
|
end
|
||||||
|
end
|
||||||
|
end
|
|
@ -6,6 +6,7 @@ defmodule Pleroma.Web.MastodonAPI.MastodonAPIController do
|
||||||
alias Pleroma.Web.MastodonAPI.{StatusView, AccountView}
|
alias Pleroma.Web.MastodonAPI.{StatusView, AccountView}
|
||||||
alias Pleroma.Web.ActivityPub.ActivityPub
|
alias Pleroma.Web.ActivityPub.ActivityPub
|
||||||
alias Pleroma.Web.TwitterAPI.TwitterAPI
|
alias Pleroma.Web.TwitterAPI.TwitterAPI
|
||||||
|
alias Pleroma.Web.CommonAPI
|
||||||
|
|
||||||
def create_app(conn, params) do
|
def create_app(conn, params) do
|
||||||
with cs <- App.register_changeset(%App{}, params) |> IO.inspect,
|
with cs <- App.register_changeset(%App{}, params) |> IO.inspect,
|
||||||
|
@ -68,4 +69,15 @@ def post_status(%{assigns: %{user: user}} = conn, %{"status" => status} = params
|
||||||
render conn, StatusView, "status.json", %{activity: activity, for: user, as: :activity}
|
render conn, StatusView, "status.json", %{activity: activity, for: user, as: :activity}
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
def delete_status(%{assigns: %{user: user}} = conn, %{"id" => id}) do
|
||||||
|
with {:ok, %Activity{}} <- CommonAPI.delete(id, user) do
|
||||||
|
json(conn, %{})
|
||||||
|
else
|
||||||
|
_e ->
|
||||||
|
conn
|
||||||
|
|> put_status(403)
|
||||||
|
|> json(%{error: "Can't delete this post"})
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
|
@ -56,6 +56,7 @@ def user_fetcher(username) do
|
||||||
get "/timelines/home", MastodonAPIController, :home_timeline
|
get "/timelines/home", MastodonAPIController, :home_timeline
|
||||||
|
|
||||||
post "/statuses", MastodonAPIController, :post_status
|
post "/statuses", MastodonAPIController, :post_status
|
||||||
|
delete "/statuses/:id", MastodonAPIController, :delete_status
|
||||||
end
|
end
|
||||||
|
|
||||||
scope "/api", Pleroma.Web do
|
scope "/api", Pleroma.Web do
|
||||||
|
|
|
@ -2,6 +2,7 @@ defmodule Pleroma.Web.TwitterAPI.Controller do
|
||||||
use Pleroma.Web, :controller
|
use Pleroma.Web, :controller
|
||||||
alias Pleroma.Web.TwitterAPI.{TwitterAPI, UserView}
|
alias Pleroma.Web.TwitterAPI.{TwitterAPI, UserView}
|
||||||
alias Pleroma.Web.TwitterAPI.Representers.ActivityRepresenter
|
alias Pleroma.Web.TwitterAPI.Representers.ActivityRepresenter
|
||||||
|
alias Pleroma.Web.CommonAPI
|
||||||
alias Pleroma.{Repo, Activity, User, Object}
|
alias Pleroma.{Repo, Activity, User, Object}
|
||||||
alias Pleroma.Web.ActivityPub.ActivityPub
|
alias Pleroma.Web.ActivityPub.ActivityPub
|
||||||
alias Ecto.Changeset
|
alias Ecto.Changeset
|
||||||
|
@ -95,10 +96,7 @@ def follow(%{assigns: %{user: user}} = conn, params) do
|
||||||
end
|
end
|
||||||
|
|
||||||
def delete_post(%{assigns: %{user: user}} = conn, %{"id" => id}) do
|
def delete_post(%{assigns: %{user: user}} = conn, %{"id" => id}) do
|
||||||
with %Activity{data: %{"object" => %{"id" => object_id}}} <- Repo.get(Activity, id),
|
with {:ok, delete} <- CommonAPI.delete(id, user) do
|
||||||
%Object{} = object <- Object.get_by_ap_id(object_id),
|
|
||||||
true <- user.ap_id == object.data["actor"],
|
|
||||||
{:ok, delete} <- ActivityPub.delete(object) |> IO.inspect do
|
|
||||||
json = ActivityRepresenter.to_json(delete, %{user: user, for: user})
|
json = ActivityRepresenter.to_json(delete, %{user: user, for: user})
|
||||||
conn
|
conn
|
||||||
|> json_reply(200, json)
|
|> json_reply(200, json)
|
||||||
|
|
|
@ -93,4 +93,32 @@ test "get a status", %{conn: conn} do
|
||||||
assert %{"id" => id} = json_response(conn, 200)
|
assert %{"id" => id} = json_response(conn, 200)
|
||||||
assert id == activity.id
|
assert id == activity.id
|
||||||
end
|
end
|
||||||
|
|
||||||
|
describe "deleting a status" do
|
||||||
|
test "when you created it", %{conn: conn} do
|
||||||
|
activity = insert(:note_activity)
|
||||||
|
author = User.get_by_ap_id(activity.data["actor"])
|
||||||
|
|
||||||
|
conn = conn
|
||||||
|
|> assign(:user, author)
|
||||||
|
|> delete("/api/v1/statuses/#{activity.id}")
|
||||||
|
|
||||||
|
assert %{} = json_response(conn, 200)
|
||||||
|
|
||||||
|
assert Repo.get(Activity, activity.id) == nil
|
||||||
|
end
|
||||||
|
|
||||||
|
test "when you didn't create it", %{conn: conn} do
|
||||||
|
activity = insert(:note_activity)
|
||||||
|
user = insert(:user)
|
||||||
|
|
||||||
|
conn = conn
|
||||||
|
|> assign(:user, user)
|
||||||
|
|> delete("/api/v1/statuses/#{activity.id}")
|
||||||
|
|
||||||
|
assert %{"error" => _} = json_response(conn, 403)
|
||||||
|
|
||||||
|
assert Repo.get(Activity, activity.id) == activity
|
||||||
|
end
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
Loading…
Reference in New Issue