Testing w clg

js/background.js

@@ -641,7 +641,6 @@
       const ev = new Event('message');
       ev.confirm = () => {};
 
-
       const convo = await ConversationController.getOrCreateAndWait(
         groupId,
         'group'
@@ -667,11 +666,30 @@
         // if pubkey already in devices to remove, skip
         const currentMembers = convo.attributes.members;
 
-        const membersToRemove = currentMembers.filter(member => !_.includes(members, member));
+        const membersToRemove = currentMembers.filter(
+          member => !_.includes(members, member)
+        );
+
+        const allMembersToRemove = [];
+        membersToRemove.forEach(async member => {
+          const pairedDevices = await libloki.storage.getPairedDevicesFor(member);
+          allMembersToRemove.push(member, ...pairedDevices);
+
+          console.log('[vince] ALL DEVICES FOR THIS USER:', [member, ...pairedDevices]);
+        });
+
+        console.log('[vince] members:', members);
+        console.log('[vince] currentMembers:', currentMembers);
+        console.log('[vince] membersToRemove:', membersToRemove);
+        console.log('[vince] allMembersToRemove:', allMembersToRemove);
+
+        /// OOOOOH interesting. The member is already removed by the
+        // time this function is called
 
-        // console.log('[vince] members:', members);
-        // console.log('[vince] currentMembers:', currentMembers);
-        // console.log('[vince] membersToRemove:', membersToRemove);
+
+
+        // For each of allMembersToRemove,
+        // if you exist in members,
 
 
         // allPubkeys.forEach(pubkey => {
@@ -683,10 +701,12 @@
         // console.log('[vince] providedGroupUpdate:', providedGroupUpdate);
         // console.log('[vince] groupUpdate:', groupUpdate);
 
-        
         console.log('[vince] doUpdateGroup: members:', members);
-      }
 
+        // rturn early for testing
+        alert('return early for testing');
+        return;
+      }
 
       ev.data = {
         source: ourKey,

js/conversation_controller.js

@@ -184,11 +184,6 @@
             return sessionCipher.deleteAllSessionsForDevice();
           })
         );
-
-
-
-
-
       } else if (conversation.isPublic()) {
         const channelAPI = await conversation.getPublicSendData();
         if (channelAPI === null) {

js/models/conversations.js

@@ -2235,10 +2235,8 @@
       });
       message.set({ id });
 
-      
       console.log('[vince] conversations.js --> groupUpdate:', groupUpdate);
 
-
       const options = this.getSendOptions();
       message.send(
         this.wrapSend(
@@ -2276,7 +2274,6 @@
         const groupNumbers = this.getRecipients();
         this.set({ left: true });
 
-
         // await window.Signal.Data.updateConversation(this.id, this.attributes, {
         //   Conversation: Whisper.Conversation,
         // });

js/views/create_group_dialog_view.js

@@ -1,4 +1,4 @@
-/* global Whisper, i18n, textsecure, _ */
+/* global Whisper, i18n, textsecure, libloki, _ */
 
 // eslint-disable-next-line func-names
 (function() {
@@ -153,6 +153,8 @@
         if (!Array.isArray(this.existingMembers)) {
           this.existingMembers = [];
         }
+
+        console.log('[vince] UpdateGroupMembersDialogView this AFTER', this);
       }
 
       this.$el.focus();
@@ -183,13 +185,30 @@
       const ourPK = textsecure.storage.user.getNumber();
       const allMembers = window.Lodash.concat(newMembers, [ourPK]);
 
+      
+
       // We need to NOT trigger an group update if the list of member is the same.
       const notPresentInOld = allMembers.filter(
         m => !this.existingMembers.includes(m)
       );
+
+      // Filter out all linked devices for cases in which one device
+      // exists in group, but hasn't yet synced with its other devices.
       const notPresentInNew = this.existingMembers.filter(
         m => !allMembers.includes(m)
       );
+
+      // Get all devices for notPresentInNew
+      const allDevicesOfMembersToRemove = [];
+      notPresentInNew.forEach(async member => {
+        const pairedDevices = await libloki.storage.getPairedDevicesFor(member);
+        allDevicesOfMembersToRemove.push(member, ...pairedDevices);
+      });
+
+      const allToRemove = allDevicesOfMembersToRemove.filter(
+        m => this.existingMembers.includes(m)
+      );
+
       // would be easer with _.xor but for some reason we do not have it
       const xor = notPresentInNew.concat(notPresentInOld);
       if (xor.length === 0) {
@@ -200,12 +219,21 @@
         return;
       }
 
-      window.doUpdateGroup(
-        this.groupId,
-        this.groupName,
-        allMembers,
-        this.avatarPath
-      );
+      console.log('[vince] allDevicesOfMembersToRemove:', allDevicesOfMembersToRemove);
+      console.log('[vince] allMembers:', allMembers);
+      console.log('[vince] notPresentInOld:', notPresentInOld);
+      console.log('[vince] notPresentInNew:', notPresentInNew);
+      console.log('[vince] xor:', xor);
+      console.log('[vince] allToRemove:', allToRemove);
+      
+      alert('returning earlyyyy');
+
+      // window.doUpdateGroup(
+      //   this.groupId,
+      //   this.groupName,
+      //   allMembers,
+      //   this.avatarPath
+      // );
     },
     close() {
       this.remove();

libtextsecure/message_receiver.js

@@ -1427,12 +1427,10 @@ MessageReceiver.prototype.extend({
       );
     }
 
-
     console.log('[vince] THIS IS A syncMessage:');
     console.log('[vince] envelope:', envelope);
     console.log('[vince] syncMessage:', syncMessage);
 
-
     if (syncMessage.sent) {
       const sentMessage = syncMessage.sent;
       const to = sentMessage.message.group