diff options
author | Ilja <ilja@ilja.space> | 2022-03-07 14:00:42 +0100 |
---|---|---|
committer | Ilja <ilja@ilja.space> | 2022-03-07 14:00:42 +0100 |
commit | cdc5bbe8369d4fc66d642bb3e845a237d11e34d7 (patch) | |
tree | cb634431aced993833db185ac66c744bfd43f99e | |
parent | 89667189b840fc79d85336739e6b2512684d7be0 (diff) | |
download | pleroma-cdc5bbe8369d4fc66d642bb3e845a237d11e34d7.tar.gz |
After code review
Use patern matching to see if someone was superuser before
-rw-r--r-- | lib/pleroma/user.ex | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/lib/pleroma/user.ex b/lib/pleroma/user.ex index 7ecd37337..2f3c6eb84 100644 --- a/lib/pleroma/user.ex +++ b/lib/pleroma/user.ex @@ -1136,11 +1136,8 @@ defmodule Pleroma.User do |> maybe_remove_report_notifications(was_superuser_before_update) end - defp maybe_remove_report_notifications( - {:ok, %Pleroma.User{} = user} = result, - was_superuser_before_update - ) do - if was_superuser_before_update and not User.superuser?(user), + defp maybe_remove_report_notifications({:ok, %Pleroma.User{} = user} = result, true) do + if not User.superuser?(user), do: user |> Notification.destroy_multiple_from_types(["pleroma:report"]) result |