From c79cfbd30d3e82a71004a1f948b70c849f6f7203 Mon Sep 17 00:00:00 2001
From: Tom Moor <tom.moor@gmail.com>
Date: Sat, 20 Feb 2021 13:22:02 -0800
Subject: [PATCH] fix: Documents in trash should still load their attachments
 closes #1896

---
 server/api/attachments.js      |  1 +
 server/api/attachments.test.js | 25 +++++++++++++++++++++++++
 2 files changed, 26 insertions(+)

diff --git a/server/api/attachments.js b/server/api/attachments.js
index 0cc3b377..792c1a42 100644
--- a/server/api/attachments.js
+++ b/server/api/attachments.js
@@ -139,6 +139,7 @@ router.post("attachments.redirect", auth(), async (ctx) => {
     if (attachment.documentId) {
       const document = await Document.findByPk(attachment.documentId, {
         userId: user.id,
+        paranoid: false,
       });
       authorize(user, "read", document);
     }
diff --git a/server/api/attachments.test.js b/server/api/attachments.test.js
index 3a6e47b4..a61105bc 100644
--- a/server/api/attachments.test.js
+++ b/server/api/attachments.test.js
@@ -153,6 +153,31 @@ describe("#attachments.redirect", () => {
     expect(res.status).toEqual(302);
   });
 
+  it("should return a redirect for an attachment belonging to a trashed document user has access to", async () => {
+    const user = await buildUser();
+    const collection = await buildCollection({
+      teamId: user.teamId,
+      userId: user.id,
+    });
+    const document = await buildDocument({
+      teamId: user.teamId,
+      userId: user.id,
+      collectionId: collection.id,
+      deletedAt: new Date(),
+    });
+    const attachment = await buildAttachment({
+      documentId: document.id,
+      teamId: user.teamId,
+      userId: user.id,
+    });
+    const res = await server.post("/api/attachments.redirect", {
+      body: { token: user.getJwtToken(), id: attachment.id },
+      redirect: "manual",
+    });
+
+    expect(res.status).toEqual(302);
+  });
+
   it("should always return a redirect for a public attachment", async () => {
     const user = await buildUser();
     const collection = await buildCollection({