aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authormarcin mikołajczak <git@mkljczk.pl>2022-03-16 18:38:28 +0100
committermarcin mikołajczak <git@mkljczk.pl>2022-03-16 19:18:15 +0100
commitffe081bf4417ae7efbf24e4eaf0ee65fa2c2d8cf (patch)
tree9af33165fdfeabe30e25810d4f8c0e58ffd212da
parent6754d1f27239d3d529a3f667a6a93b267041daf0 (diff)
downloadpleroma-ffe081bf4417ae7efbf24e4eaf0ee65fa2c2d8cf.tar.gz
Use reject_follow_request
Signed-off-by: marcin mikołajczak <git@mkljczk.pl>
-rw-r--r--lib/pleroma/web/mastodon_api/controllers/account_controller.ex7
-rw-r--r--test/pleroma/web/mastodon_api/controllers/account_controller_test.exs2
2 files changed, 6 insertions, 3 deletions
diff --git a/lib/pleroma/web/mastodon_api/controllers/account_controller.ex b/lib/pleroma/web/mastodon_api/controllers/account_controller.ex
index 31d75ba85..50dd0e4c2 100644
--- a/lib/pleroma/web/mastodon_api/controllers/account_controller.ex
+++ b/lib/pleroma/web/mastodon_api/controllers/account_controller.ex
@@ -479,9 +479,12 @@ defmodule Pleroma.Web.MastodonAPI.AccountController do
{:error, "Can not unfollow yourself"}
end
- def remove_from_followers(%{assigns: %{user: follower, account: followed}} = conn, _params) do
- with {:ok, follower} <- CommonAPI.unfollow(followed, follower) do
+ def remove_from_followers(%{assigns: %{user: followed, account: follower}} = conn, _params) do
+ with {:ok, follower} <- CommonAPI.reject_follow_request(follower, followed) do
render(conn, "relationship.json", user: follower, target: followed)
+ else
+ nil ->
+ render_error(conn, :not_found, "Record not found")
end
end
diff --git a/test/pleroma/web/mastodon_api/controllers/account_controller_test.exs b/test/pleroma/web/mastodon_api/controllers/account_controller_test.exs
index b9ee173d6..f38ebdd75 100644
--- a/test/pleroma/web/mastodon_api/controllers/account_controller_test.exs
+++ b/test/pleroma/web/mastodon_api/controllers/account_controller_test.exs
@@ -1985,7 +1985,7 @@ defmodule Pleroma.Web.MastodonAPI.AccountControllerTest do
CommonAPI.follow(other_user, user)
- assert %{"id" => _id, "followed_by" => false} =
+ assert %{"id" => other_user_id, "followed_by" => false} =
conn
|> post("/api/v1/accounts/#{other_user_id}/remove_from_followers")
|> json_response_and_validate_schema(200)