From a5822d179cf1d1e69ea382ed6d5f603be7ff7eab Mon Sep 17 00:00:00 2001
From: Thomas Citharel <tcit@tcit.fr>
Date: Fri, 30 Jul 2021 17:31:10 +0200
Subject: [PATCH] Fix editing event from original instance on a group event

Signed-off-by: Thomas Citharel <tcit@tcit.fr>
---
 lib/federation/activity_pub/types/events.ex | 60 ++++++++++++++-------
 1 file changed, 40 insertions(+), 20 deletions(-)

diff --git a/lib/federation/activity_pub/types/events.ex b/lib/federation/activity_pub/types/events.ex
index df556d89b..11ca1f6fc 100644
--- a/lib/federation/activity_pub/types/events.ex
+++ b/lib/federation/activity_pub/types/events.ex
@@ -147,28 +147,48 @@ defmodule Mobilizon.Federation.ActivityPub.Types.Events do
 
   # Set the participant to approved if the default role for new participants is :participant
   defp approve_if_default_role_is_participant(event, activity_data, participant, role) do
-    if event.local do
-      cond do
-        Mobilizon.Events.get_default_participant_role(event) === :participant &&
-            role == :participant ->
-          {:accept,
-           ActivityPub.accept(
-             :join,
-             participant,
-             true,
-             %{"actor" => event.organizer_actor.url}
-           )}
+    case event do
+      %Event{attributed_to: %Actor{id: group_id, url: group_url}} ->
+        case Actors.get_single_group_moderator_actor(group_id) do
+          %Actor{} = actor ->
+            do_approve(event, activity_data, participant, role, %{
+              "actor" => actor.url,
+              "attributedTo" => group_url
+            })
 
-        Mobilizon.Events.get_default_participant_role(event) === :not_approved &&
-            role == :not_approved ->
-          Scheduler.pending_participation_notification(event)
-          {:ok, activity_data, participant}
+          _ ->
+            {:ok, activity_data, participant}
+        end
 
-        true ->
-          {:ok, activity_data, participant}
-      end
-    else
-      {:ok, activity_data, participant}
+      %Event{local: true} ->
+        do_approve(event, activity_data, participant, role, %{
+          "actor" => event.organizer_actor.url
+        })
+
+      _ ->
+        {:ok, activity_data, participant}
+    end
+  end
+
+  defp do_approve(event, activity_data, participant, role, additionnal) do
+    cond do
+      Mobilizon.Events.get_default_participant_role(event) === :participant &&
+          role == :participant ->
+        {:accept,
+         ActivityPub.accept(
+           :join,
+           participant,
+           true,
+           additionnal
+         )}
+
+      Mobilizon.Events.get_default_participant_role(event) === :not_approved &&
+          role == :not_approved ->
+        Scheduler.pending_participation_notification(event)
+        {:ok, activity_data, participant}
+
+      true ->
+        {:ok, activity_data, participant}
     end
   end