From cd91ed6f451af4aec236c65c8bb45cb3d6c2299b Mon Sep 17 00:00:00 2001
From: Josh Perez <60019601+josh-signal@users.noreply.github.com>
Date: Wed, 1 Dec 2021 15:19:54 -0500
Subject: [PATCH] Do not allow attachments if there are link previews

---
 ts/components/CompositionArea.tsx |  2 +-
 ts/models/conversations.ts        | 14 +++++++++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/ts/components/CompositionArea.tsx b/ts/components/CompositionArea.tsx
index f7c7d62ac5..9b50d8c87d 100644
--- a/ts/components/CompositionArea.tsx
+++ b/ts/components/CompositionArea.tsx
@@ -405,7 +405,7 @@ export const CompositionArea = ({
     />
   ) : null;
 
-  const attButton = (
+  const attButton = linkPreviewResult ? undefined : (
     <div className="CompositionArea__button-cell">
       <button
         type="button"
diff --git a/ts/models/conversations.ts b/ts/models/conversations.ts
index da117a8d2d..53d2bcd85b 100644
--- a/ts/models/conversations.ts
+++ b/ts/models/conversations.ts
@@ -3895,6 +3895,18 @@ export class ConversationModel extends window.Backbone
       [window.ConversationController.getOurConversationIdOrThrow()]
     );
 
+    // If there are link previews present in the message we shouldn't include
+    // any attachments as well.
+    const attachmentsToSend = preview && preview.length ? [] : attachments;
+
+    if (preview && preview.length) {
+      attachments.forEach(attachment => {
+        if (attachment.path) {
+          deleteAttachmentData(attachment.path);
+        }
+      });
+    }
+
     // Here we move attachments to disk
     const messageWithSchema = await upgradeMessageSchema({
       timestamp: now,
@@ -3903,7 +3915,7 @@ export class ConversationModel extends window.Backbone
       conversationId: this.id,
       quote,
       preview,
-      attachments,
+      attachments: attachmentsToSend,
       sent_at: now,
       received_at: window.Signal.Util.incrementMessageCounter(),
       received_at_ms: now,