After code review

Use patern matching to see if someone was superuser before
This commit is contained in:
Ilja 2022-03-07 14:00:42 +01:00 committed by FloatingGhost
parent 1048bc1bb9
commit a20d2847e2

View file

@ -1099,11 +1099,8 @@ def update_and_set_cache(%{data: %Pleroma.User{} = user} = changeset) do
|> maybe_remove_report_notifications(was_superuser_before_update) |> maybe_remove_report_notifications(was_superuser_before_update)
end end
defp maybe_remove_report_notifications( defp maybe_remove_report_notifications({:ok, %Pleroma.User{} = user} = result, true) do
{:ok, %Pleroma.User{} = user} = result, if not User.superuser?(user),
was_superuser_before_update
) do
if was_superuser_before_update and not User.superuser?(user),
do: user |> Notification.destroy_multiple_from_types(["pleroma:report"]) do: user |> Notification.destroy_multiple_from_types(["pleroma:report"])
result result