Deleting a user from AdminAPI should use User.delete/1

This commit is contained in:
Mark Felder 2021-02-23 09:49:27 -06:00
parent f775a1931b
commit 14e4e2d340

View file

@ -10,8 +10,6 @@ defmodule Pleroma.Web.AdminAPI.UserController do
alias Pleroma.ModerationLog
alias Pleroma.User
alias Pleroma.Web.ActivityPub.Builder
alias Pleroma.Web.ActivityPub.Pipeline
alias Pleroma.Web.AdminAPI
alias Pleroma.Web.AdminAPI.Search
alias Pleroma.Web.Plugs.OAuthScopesPlug
@ -69,10 +67,7 @@ defmodule Pleroma.Web.AdminAPI.UserController do
defp do_deletes(%{assigns: %{user: admin}} = conn, nicknames) when is_list(nicknames) do
users = Enum.map(nicknames, &User.get_cached_by_nickname/1)
Enum.each(users, fn user ->
{:ok, delete_data, _} = Builder.delete(admin, user.ap_id)
Pipeline.common_pipeline(delete_data, local: true)
end)
Enum.each(users, &User.delete(&1))
ModerationLog.insert_log(%{
actor: admin,