fix: download content and file attachment only if included in the tem…

…plate
omnivore-app · May 30, 2023 · dc92a38 · dc92a38
1 parent 4c6df1e
commit dc92a38
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/src/main.ts b/src/main.ts
@@ -196,7 +196,13 @@ export default class OmnivorePlugin extends Plugin {
       manualSync && new Notice("🚀 Fetching articles ...");
 
       // pre-parse template
-      preParseTemplate(template);
+      const templateSpans = preParseTemplate(template);
+      const includeContent = templateSpans.some(
+        (templateSpan) => templateSpan[1] === "content"
+      );
+      const includeFileAttachment = templateSpans.some(
+        (templateSpan) => templateSpan[1] === "fileAttachment"
+      );
 
       const size = 50;
       for (
@@ -211,7 +217,7 @@ export default class OmnivorePlugin extends Plugin {
           size,
           parseDateTime(syncAt).toISO(),
           getQueryFromFilter(filter, customQuery),
-          true,
+          includeContent,
           "highlightedMarkdown"
         );
 
@@ -229,7 +235,7 @@ export default class OmnivorePlugin extends Plugin {
             await this.app.vault.createFolder(folderName);
           }
           const fileAttachment =
-            article.pageType === PageType.File
+            article.pageType === PageType.File && includeFileAttachment
               ? await this.downloadFileAsAttachment(article)
               : undefined;
           const content = await renderArticleContnet(

diff --git a/src/settings/template.ts b/src/settings/template.ts
@@ -298,5 +298,5 @@ export const renderFolderName = (folder: string, folderDate: string) => {
 };
 
 export const preParseTemplate = (template: string) => {
-  Mustache.parse(template);
+  return Mustache.parse(template);
 };