diff --git a/app/src/main/java/org/thoughtcrime/securesms/components/LinkPreviewView.java b/app/src/main/java/org/thoughtcrime/securesms/components/LinkPreviewView.java index cbec416a70..971e840dbe 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/components/LinkPreviewView.java +++ b/app/src/main/java/org/thoughtcrime/securesms/components/LinkPreviewView.java @@ -172,11 +172,11 @@ public class LinkPreviewView extends FrameLayout { spinner.setVisibility(GONE); noPreview.setVisibility(GONE); - CallLinks.CallLinkParseResult linkParseResult = CallLinks.parseUrl(linkPreview.getUrl()); + CallLinks.CallLinkParseResult callLinkParseResult = CallLinks.isCallLink(linkPreview.getUrl()) ? CallLinks.parseUrl(linkPreview.getUrl()) : null; if (!Util.isEmpty(linkPreview.getTitle())) { title.setText(linkPreview.getTitle()); title.setVisibility(VISIBLE); - } else if (linkParseResult != null) { + } else if (callLinkParseResult != null) { title.setText(R.string.Recipient_signal_call); title.setVisibility(VISIBLE); } else { @@ -186,7 +186,7 @@ public class LinkPreviewView extends FrameLayout { if (showDescription && !Util.isEmpty(linkPreview.getDescription())) { description.setText(linkPreview.getDescription()); description.setVisibility(VISIBLE); - } else if (linkParseResult != null) { + } else if (callLinkParseResult != null) { description.setText(R.string.LinkPreviewView__use_this_link_to_join_a_signal_call); description.setVisibility(VISIBLE); } else { @@ -221,14 +221,14 @@ public class LinkPreviewView extends FrameLayout { thumbnail.get().setImageResource(requestManager, new ImageSlide(linkPreview.getThumbnail().get()), type == TYPE_CONVERSATION && !scheduleMessageMode, false); thumbnail.get().showSecondaryText(false); thumbnail.get().setOutlineEnabled(true); - } else if (linkParseResult != null) { + } else if (callLinkParseResult != null) { thumbnail.setVisibility(VISIBLE); thumbnailState.applyState(thumbnail); thumbnail.get().setImageDrawable( requestManager, new FallbackAvatarDrawable( getContext(), - new FallbackAvatar.Resource.CallLink(AvatarColorHash.forCallLink(linkParseResult.getRootKey().getKeyBytes())) + new FallbackAvatar.Resource.CallLink(AvatarColorHash.forCallLink(callLinkParseResult.getRootKey().getKeyBytes())) ).circleCrop() ); thumbnail.get().showSecondaryText(false); diff --git a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationItem.java b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationItem.java index 5cd14a6426..a960bbd089 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationItem.java +++ b/app/src/main/java/org/thoughtcrime/securesms/conversation/ConversationItem.java @@ -1231,14 +1231,14 @@ public final class ConversationItem extends RelativeLayout implements BindableCo //noinspection ConstantConditions LinkPreview linkPreview = ((MmsMessageRecord) messageRecord).getLinkPreviews().get(0); - CallLinks.CallLinkParseResult linkParseResult = CallLinks.parseUrl(linkPreview.getUrl()); - if (linkParseResult != null) { + CallLinks.CallLinkParseResult callLinkParseResult = CallLinks.isCallLink(linkPreview.getUrl()) ? CallLinks.parseUrl(linkPreview.getUrl()) : null; + if (callLinkParseResult != null) { joinCallLinkStub.setVisibility(View.VISIBLE); joinCallLinkStub.get().setTextColor(ContextCompat.getColor(context, messageRecord.isOutgoing() ? R.color.signal_light_colorOnPrimary : R.color.signal_colorOnPrimaryContainer)); joinCallLinkStub.get().setBackgroundColor(ContextCompat.getColor(context, messageRecord.isOutgoing() ? R.color.signal_light_colorTransparent2 : R.color.signal_colorOnPrimary)); joinCallLinkStub.get().setOnClickListener(v -> { if (eventListener != null) { - eventListener.onJoinCallLink(linkParseResult.getRootKey(), linkParseResult.getEpoch()); + eventListener.onJoinCallLink(callLinkParseResult.getRootKey(), callLinkParseResult.getEpoch()); } }); }