tommy/mobilizon
1
0
Fork 0
forked from potsda.mn/mobilizon
Code Pull requests Activity

fix(backend): only send suspension notification emails when actor's suspended and not just deleted

Browse source 
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2023-12-04 09:02:38 +01:00
parent 51e1ed642c
commit 9e41bc1ad6
No known key found for this signature in database
GPG key ID: A061B9DDE0CA0773
2 changed files with 13 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
lib/service/actor_suspension.ex
View file

@ -35,6 +35,8 @@ defmodule Mobilizon.Service.ActorSuspension do
delete_actor_options = Keyword.merge(@delete_actor_default_options, options) delete_actor_options = Keyword.merge(@delete_actor_default_options, options)
Logger.debug(inspect(delete_actor_options)) Logger.debug(inspect(delete_actor_options))
# Only send suspension notifications if we actually are suspending the actor
if Keyword.get(delete_actor_options, :suspension, false) do
send_suspension_notification(actor) send_suspension_notification(actor)
Logger.debug( Logger.debug(
@ -42,6 +44,8 @@ defmodule Mobilizon.Service.ActorSuspension do
) )
notify_event_participants_from_suspension(actor) notify_event_participants_from_suspension(actor)
end
delete_participations(actor) delete_participations(actor)
multi = multi =

5
lib/service/workers/background.ex
View file

@ -20,7 +20,10 @@ defmodule Mobilizon.Service.Workers.Background do
reserve_username when is_boolean(reserve_username) -> reserve_username when is_boolean(reserve_username) ->
case Actors.get_actor(actor_id) do case Actors.get_actor(actor_id) do
%Actor{} = actor -> %Actor{} = actor ->
ActorSuspension.suspend_actor(actor, reserve_username: reserve_username) ActorSuspension.suspend_actor(actor,
reserve_username: reserve_username,
suspension: Map.get(args, "suspension", false)
)
nil -> nil ->
{:error, :actor_not_found} {:error, :actor_not_found}