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

Fix group notification of new event being sent multiple times

Browse source 
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
This commit is contained in:
Thomas Citharel 2022-05-10 13:14:45 +02:00
parent 1eb111f52f
commit 46236dbe1d
No known key found for this signature in database
GPG key ID: A061B9DDE0CA0773
1 changed files with 15 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

24
lib/web/email/group.ex
View file

@ -17,25 +17,27 @@ defmodule Mobilizon.Web.Email.Group do
# TODO: When we have events restricted to members, don't send emails to followers
group
|> Actors.list_actors_to_notify_from_group_event()
|> Enum.reduce([], fn actor, users ->
# No emails for remote actors
if is_nil(actor.user_id) do
users
else
users ++ [Users.get_user_with_activity_settings!(actor.user_id)]
end
end)
|> Enum.each(&notify_follower(event, group, &1))
end
def notify_of_new_event(%Event{}), do: :ok
defp notify_follower(%Event{} = _event, %Actor{}, %Actor{user_id: nil}), do: :ok
defp notify_follower(%Event{} = event, %Actor{type: :Group} = group, %Actor{
id: profile_id,
user_id: user_id
}) do
%User{
defp notify_follower(%Event{} = event, %Actor{type: :Group} = group, %User{
email: email,
locale: locale,
settings: %Setting{timezone: timezone},
activity_settings: activity_settings
} = Users.get_user_with_activity_settings!(user_id)
if profile_id != event.organizer_actor_id &&
activity_settings: activity_settings,
default_actor_id: default_actor_id
}) do
if default_actor_id != event.organizer_actor_id &&
accepts_new_events_notifications(activity_settings) do
Gettext.put_locale(locale)