From aa2094a2ccaf8a657d620fe91c0cf4fbc84d3322 Mon Sep 17 00:00:00 2001 From: Alan Evans Date: Thu, 14 Jan 2021 10:18:56 -0400 Subject: [PATCH] Fix group recipient showing in verify safety number change "learn more". --- .../securesms/conversation/ConversationFragment.java | 5 ++++- .../securesms/conversation/ConversationUpdateItem.java | 2 +- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationFragment.java b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationFragment.java index 01ea93e92d..4a4cc3b781 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationFragment.java +++ b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationFragment.java @@ -22,7 +22,6 @@ import android.app.Activity; import android.content.ClipData; import android.content.ClipboardManager; import android.content.Context; -import android.content.DialogInterface; import android.content.Intent; import android.net.Uri; import android.os.AsyncTask; @@ -1438,6 +1437,10 @@ public class ConversationFragment extends LoggingFragment { @Override public void onSafetyNumberLearnMoreClicked(@NonNull Recipient recipient) { + if (recipient.isGroup()) { + throw new AssertionError("Must be individual"); + } + AlertDialog dialog = new AlertDialog.Builder(requireContext()) .setView(R.layout.safety_number_changed_learn_more_dialog) .setPositiveButton(R.string.ConversationFragment_verify, (d, w) -> { diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationUpdateItem.java b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationUpdateItem.java index e65e89c043..7ce12ebf1e 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationUpdateItem.java +++ b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationUpdateItem.java @@ -221,7 +221,7 @@ public final class ConversationUpdateItem extends LinearLayout actionButton.setVisibility(VISIBLE); actionButton.setOnClickListener(v -> { if (batchSelected.isEmpty() && eventListener != null) { - eventListener.onSafetyNumberLearnMoreClicked(conversationRecipient); + eventListener.onSafetyNumberLearnMoreClicked(conversationMessage.getMessageRecord().getIndividualRecipient()); } }); } else if (conversationMessage.getMessageRecord().isGroupCall()) {