From 7deedaf8d94df4b1b40567fd30c895df227f9a06 Mon Sep 17 00:00:00 2001
From: Allan Chain <36528777+AllanChain@users.noreply.github.com>
Date: Sat, 30 Nov 2024 17:43:35 +0800
Subject: [PATCH] feat: can trash arXiv PDF without annotation

See #24
---
 addon/chrome/content/preferences.xhtml | 5 +++++
 addon/locale/en-US/preferences.ftl     | 2 ++
 addon/locale/zh-CN/preferences.ftl     | 2 ++
 addon/prefs.js                         | 1 +
 src/modules/arxiv-merge.ts             | 8 ++++++++
 5 files changed, 18 insertions(+)

diff --git a/addon/chrome/content/preferences.xhtml b/addon/chrome/content/preferences.xhtml
index d9cacc7..2eb9eb8 100644
--- a/addon/chrome/content/preferences.xhtml
+++ b/addon/chrome/content/preferences.xhtml
@@ -17,6 +17,11 @@
       preference="extensions.zotero.__addonRef__.downloadJournalPDF"
       data-l10n-id="pref-downloadJournalPDF"
     />
+    <checkbox
+      id="zotero-prefpane-__addonRef__-merge.trashUnannotatedPDF"
+      preference="extensions.zotero.__addonRef__.merge.trashUnannotatedPDF"
+      data-l10n-id="pref-merge-trashUnannotatedPDF"
+    />
     <html:p data-l10n-id="pref-merge-arXiv-warn"></html:p>
     <checkbox
       id="zotero-prefpane-__addonRef__-merge.arXivExtra"
diff --git a/addon/locale/en-US/preferences.ftl b/addon/locale/en-US/preferences.ftl
index 91be0f5..eb0001b 100644
--- a/addon/locale/en-US/preferences.ftl
+++ b/addon/locale/en-US/preferences.ftl
@@ -7,6 +7,8 @@ pref-merge-arXivExtra =
     .label = Keep arXiv information in the Extra field when merging
 pref-merge-arXivURL =
     .label = Use arXiv URL in the URL field when merging
+pref-merge-trashUnannotatedPDF =
+    .label = Trash arXiv PDF without annotation when merging
 pref-merge-arXiv-warn = The following two settings could add arXiv information in the exported citation:
 pref-source = arXiv Update sources
 pref-source-doi =
diff --git a/addon/locale/zh-CN/preferences.ftl b/addon/locale/zh-CN/preferences.ftl
index af61c2e..0e592b7 100644
--- a/addon/locale/zh-CN/preferences.ftl
+++ b/addon/locale/zh-CN/preferences.ftl
@@ -7,6 +7,8 @@ pref-merge-arXivExtra =
     .label = 合并条目时，保留 Extra 字段中的 arXiv 信息
 pref-merge-arXivURL =
     .label = 合并条目时，在 URL 字段中保留 arXiv 的 URL
+pref-merge-trashUnannotatedPDF =
+    .label = 合并条目时，删除无注释的 arXiv PDF
 pref-merge-arXiv-warn = 以下两条设置可能导致导出的引用信息包含 arXiv 信息：
 pref-source = arXiv 更新源
 pref-source-doi =
diff --git a/addon/prefs.js b/addon/prefs.js
index c64c6cb..acc837d 100644
--- a/addon/prefs.js
+++ b/addon/prefs.js
@@ -12,3 +12,4 @@ pref("__prefsPrefix__.updateSource.arXiv", true);
 pref("__prefsPrefix__.mergePreferJournalPDF", true);
 pref("__prefsPrefix__.merge.arXivExtra", false);
 pref("__prefsPrefix__.merge.arXivURL", false);
+pref("__prefsPrefix__.merge.trashUnannotatedPDF", false);
diff --git a/src/modules/arxiv-merge.ts b/src/modules/arxiv-merge.ts
index 6043d98..b89bf79 100644
--- a/src/modules/arxiv-merge.ts
+++ b/src/modules/arxiv-merge.ts
@@ -143,6 +143,14 @@ export class arXivMerge {
         title: preprintJSON.archiveID ?? "Preprint URL",
       });
     }
+    if (getPref("merge.trashUnannotatedPDF")) {
+      for (const attachmentID of preprintItem.getAttachments()) {
+        const attachment = await Zotero.Items.getAsync(attachmentID);
+        if (!attachment.isPDFAttachment()) continue;
+        if (attachment.getAnnotations().length) continue;
+        await Zotero.Items.trashTx(attachmentID);
+      }
+    }
     // Set prefered PDF
     if (getPref("mergePreferJournalPDF")) {
       let oldestPDFDate = new Date();