From 6591702c08f367926b3984c6e0ab8bb4ba7d3375 Mon Sep 17 00:00:00 2001
From: Audric Ackermann <audric@loki.network>
Date: Mon, 10 May 2021 10:29:42 +1000
Subject: [PATCH] create convo for members if they don't exist

also, removing a private convo does not remove it entirely as we need
the convo to be able to remove members
---
 .../conversation/SessionConversation.tsx      | 15 +++++++++++-
 .../conversations/ConversationController.ts   | 24 +++++++++++++++----
 2 files changed, 33 insertions(+), 6 deletions(-)

diff --git a/ts/components/session/conversation/SessionConversation.tsx b/ts/components/session/conversation/SessionConversation.tsx
index b7e2ca7d1..4df91edcc 100644
--- a/ts/components/session/conversation/SessionConversation.tsx
+++ b/ts/components/session/conversation/SessionConversation.tsx
@@ -486,7 +486,20 @@ export class SessionConversation extends React.Component<Props, State> {
       onUpdateGroupName: () => {
         window.Whisper.events.trigger('updateGroupName', conversation);
       },
-      onUpdateGroupMembers: () => {
+      onUpdateGroupMembers: async () => {
+        if (conversation.isMediumGroup()) {
+          // make sure all the members' convo exists so we can add or remove them
+          await Promise.all(
+            conversation
+              .get('members')
+              .map(m =>
+                ConversationController.getInstance().getOrCreateAndWait(
+                  m,
+                  ConversationTypeEnum.PRIVATE
+                )
+              )
+          );
+        }
         window.Whisper.events.trigger('updateGroupMembers', conversation);
       },
       onInviteContacts: () => {
diff --git a/ts/session/conversations/ConversationController.ts b/ts/session/conversations/ConversationController.ts
index ceec6b612..c23a1cfaf 100644
--- a/ts/session/conversations/ConversationController.ts
+++ b/ts/session/conversations/ConversationController.ts
@@ -34,6 +34,7 @@ export class ConversationController {
       return ConversationController.instance;
     }
     ConversationController.instance = new ConversationController();
+
     return ConversationController.instance;
   }
 
@@ -187,9 +188,10 @@ export class ConversationController {
       return;
     }
 
-    // Close group leaving
+    // Closed/Medium group leaving
     if (conversation.isClosedGroup()) {
       await conversation.leaveClosedGroup();
+      // open group v1
     } else if (conversation.isPublic() && !conversation.isOpenGroupV2()) {
       const channelAPI = await conversation.getPublicSendData();
       if (channelAPI === null) {
@@ -197,6 +199,7 @@ export class ConversationController {
       } else {
         channelAPI.serverAPI.partChannel((channelAPI as any).channelId);
       }
+      // open group v2
     } else if (conversation.isOpenGroupV2()) {
       window.log.info('leaving open group v2', conversation.id);
       const roomInfos = await getV2OpenGroupRoom(conversation.id);
@@ -220,10 +223,21 @@ export class ConversationController {
     // those are the stuff to do for all contact types
     await conversation.destroyMessages();
 
-    await removeConversation(id);
-    this.conversations.remove(conversation);
-    if (window.inboxStore) {
-      window.inboxStore?.dispatch(conversationActions.conversationRemoved(conversation.id));
+    // if this conversation is a private conversation it's in fact a `contact` for desktop.
+    // we just want to remove everything related to it, set the active_at to undefined
+    // so conversation still exists (useful for medium groups members or opengroups) but is not shown on the UI
+    if (conversation.isPrivate()) {
+      conversation.set('active_at', undefined);
+      await conversation.commit();
+    } else {
+      await removeConversation(id);
+      this.conversations.remove(conversation);
+      if (window.inboxStore) {
+        window.inboxStore?.dispatch(conversationActions.conversationRemoved(conversation.id));
+        window.inboxStore?.dispatch(
+          conversationActions.conversationChanged(conversation.id, conversation.getProps())
+        );
+      }
     }
   }