feat: can trash arXiv PDF without annotation

See #24
AllanChain · Nov 30, 2024 · 7deedaf · 7deedaf
1 parent b3bdea7
commit 7deedaf
Show file tree

Hide file tree

Showing 5 changed files with 18 additions and 0 deletions.
diff --git 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
@@ -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
@@ -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
@@ -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
@@ -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();