]> nmode's Git Repositories - signal-cli/commitdiff
Don’t remove self from group when sending group messages
authorAsamK <asamk@gmx.de>
Tue, 28 Jun 2016 10:35:18 +0000 (12:35 +0200)
committerAsamK <asamk@gmx.de>
Tue, 28 Jun 2016 10:35:18 +0000 (12:35 +0200)
src/main/java/org/asamk/signal/Manager.java

index d96537feaf90e283744b2f17f2133364e13cc871..907858247892d41d6c8792887fe45c87f70683b4 100644 (file)
@@ -478,9 +478,11 @@ class Manager implements Signal {
         if (g == null) {
             throw new GroupNotFoundException(groupId);
         }
         if (g == null) {
             throw new GroupNotFoundException(groupId);
         }
-        Set<String> members = g.members;
-        members.remove(this.username);
-        sendMessage(message, members);
+
+        // Don't send group message to ourself
+        final List<String> membersSend = new ArrayList<>(g.members);
+        membersSend.remove(this.username);
+        sendMessage(message, membersSend);
     }
 
     public void sendQuitGroupMessage(byte[] groupId) throws GroupNotFoundException, IOException, EncapsulatedExceptions, UntrustedIdentityException {
     }
 
     public void sendQuitGroupMessage(byte[] groupId) throws GroupNotFoundException, IOException, EncapsulatedExceptions, UntrustedIdentityException {
@@ -555,6 +557,7 @@ class Manager implements Signal {
                 .asGroupMessage(group.build())
                 .build();
 
                 .asGroupMessage(group.build())
                 .build();
 
+        // Don't send group message to ourself
         final List<String> membersSend = new ArrayList<>(g.members);
         membersSend.remove(this.username);
         sendMessage(message, membersSend);
         final List<String> membersSend = new ArrayList<>(g.members);
         membersSend.remove(this.username);
         sendMessage(message, membersSend);