diff options
author | lain <lain@soykaf.club> | 2020-05-01 14:22:39 +0200 |
---|---|---|
committer | lain <lain@soykaf.club> | 2020-05-01 14:22:39 +0200 |
commit | ebbd9c7f369f986b7a66f66eddab91537c490c79 (patch) | |
tree | e39ad2ecbcf798b5277b8a3d745c0a5efd5ba4d5 | |
parent | 51f1dbf0a2bf6b61fdef0be56fd8f20a40827100 (diff) | |
download | pleroma-ebbd9c7f369f986b7a66f66eddab91537c490c79.tar.gz |
AdminAPIController: Refactor.
-rw-r--r-- | lib/pleroma/web/admin_api/admin_api_controller.ex | 14 | ||||
-rw-r--r-- | test/web/admin_api/admin_api_controller_test.exs | 2 |
2 files changed, 3 insertions, 13 deletions
diff --git a/lib/pleroma/web/admin_api/admin_api_controller.ex b/lib/pleroma/web/admin_api/admin_api_controller.ex index 816c11e01..c09584fd1 100644 --- a/lib/pleroma/web/admin_api/admin_api_controller.ex +++ b/lib/pleroma/web/admin_api/admin_api_controller.ex @@ -133,18 +133,8 @@ defmodule Pleroma.Web.AdminAPI.AdminAPIController do action_fallback(:errors) - def user_delete(%{assigns: %{user: admin}} = conn, %{"nickname" => nickname}) do - user = User.get_cached_by_nickname(nickname) - User.delete(user) - - ModerationLog.insert_log(%{ - actor: admin, - subject: [user], - action: "delete" - }) - - conn - |> json(nickname) + def user_delete(conn, %{"nickname" => nickname}) do + user_delete(conn, %{"nicknames" => [nickname]}) end def user_delete(%{assigns: %{user: admin}} = conn, %{"nicknames" => nicknames}) do diff --git a/test/web/admin_api/admin_api_controller_test.exs b/test/web/admin_api/admin_api_controller_test.exs index f80dbf8dd..c92715fab 100644 --- a/test/web/admin_api/admin_api_controller_test.exs +++ b/test/web/admin_api/admin_api_controller_test.exs @@ -156,7 +156,7 @@ defmodule Pleroma.Web.AdminAPI.AdminAPIControllerTest do assert ModerationLog.get_log_entry_message(log_entry) == "@#{admin.nickname} deleted users: @#{user.nickname}" - assert json_response(conn, 200) == user.nickname + assert json_response(conn, 200) == [user.nickname] end test "multiple users", %{admin: admin, conn: conn} do |