Fix delete group from message request state bug.

Fixes #12193
signalapp · Jul 11, 2022 · 60e366e · 60e366e
1 parent 1a80cb7
commit 60e366e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/MultiDeviceMessageRequestResponseJob.java
@@ -101,8 +101,8 @@ public void onRun() throws IOException, UntrustedIdentityException {
     SignalServiceMessageSender messageSender = ApplicationDependencies.getSignalServiceMessageSender();
     Recipient                  recipient     = Recipient.resolved(threadRecipient);
 
-    if (!recipient.hasServiceId()) {
-      Log.i(TAG, "Queued for recipient without ServiceId");
+    if (!recipient.isGroup() && !recipient.hasServiceId()) {
+      Log.i(TAG, "Queued for non-group recipient without ServiceId");
       return;
     }
 

diff --git a/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java b/app/src/main/java/org/thoughtcrime/securesms/messages/MessageContentProcessor.java
@@ -1102,7 +1102,7 @@ private void handleSynchronizeMessageRequestResponse(@NonNull MessageRequestResp
     if (response.getPerson().isPresent()) {
       recipient = Recipient.externalPush(response.getPerson().get());
     } else if (response.getGroupId().isPresent()) {
-      GroupId groupId = GroupId.v1(response.getGroupId().get());
+      GroupId groupId = GroupId.push(response.getGroupId().get());
       recipient = Recipient.externalPossiblyMigratedGroup(groupId);
     } else {
       warn("Message request response was missing a thread recipient! Skipping.");