From 0ca9bff61b226f590ac7ad61c73636db05e335b3 Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Mon, 7 Feb 2022 22:50:28 +0100
Subject: [PATCH 1/7] fix multiple UI bugs, closes #2616

---
 src/becca/entities/etapi_token.js       |  4 ++-
 src/public/app/dialogs/options/etapi.js | 30 ++++++++++-----------
 src/public/app/dialogs/prompt.js        |  7 ++---
 src/routes/api/etapi_tokens.js          | 10 +++----
 src/services/etapi_tokens.js            | 35 ++++++++++++-------------
 src/views/dialogs/prompt.ejs            |  2 +-
 6 files changed, 44 insertions(+), 44 deletions(-)

diff --git a/src/becca/entities/etapi_token.js b/src/becca/entities/etapi_token.js
index 04c882536..170e79bc8 100644
--- a/src/becca/entities/etapi_token.js
+++ b/src/becca/entities/etapi_token.js
@@ -42,7 +42,9 @@ class EtapiToken extends AbstractEntity {
         /** @type {boolean} */
         this.isDeleted = !!row.isDeleted;
 
-        this.becca.etapiTokens[this.etapiTokenId] = this;
+        if (this.etapiTokenId) {
+            this.becca.etapiTokens[this.etapiTokenId] = this;
+        }
     }
 
     init() {
diff --git a/src/public/app/dialogs/options/etapi.js b/src/public/app/dialogs/options/etapi.js
index 2246d3ecd..aba500d44 100644
--- a/src/public/app/dialogs/options/etapi.js
+++ b/src/public/app/dialogs/options/etapi.js
@@ -48,46 +48,46 @@ const TPL = `
 export default class EtapiOptions {
     constructor() {
         $("#options-etapi").html(TPL);
-        
+
         $("#create-etapi-token").on("click", async () => {
             const promptDialog = await import('../../dialogs/prompt.js');
             const tokenName = await promptDialog.ask({
                 title: "New ETAPI token",
-                message: "Please enter new token's name", 
-                defaultValue: "new token" 
+                message: "Please enter new token's name",
+                defaultValue: "new token"
             });
-            
+
             if (!tokenName.trim()) {
                 alert("Token name can't be empty");
                 return;
             }
-            
-            const {token} = await server.post('etapi-tokens', {tokenName});
 
-            await promptDialog.ask({ 
+            const {authToken} = await server.post('etapi-tokens', {tokenName});
+
+            await promptDialog.ask({
                 title: "ETAPI token created",
-                message: 'Copy the created token into clipboard. Trilium stores the token hashed and this is the last time you see it.', 
-                defaultValue: token 
+                message: 'Copy the created token into clipboard. Trilium stores the token hashed and this is the last time you see it.',
+                defaultValue: authToken
             });
-            
+
             this.refreshTokens();
         });
 
         this.refreshTokens();
     }
-    
+
     async refreshTokens() {
         const $noTokensYet = $("#no-tokens-yet");
         const $tokensTable = $("#tokens-table");
-        
+
         const tokens = await server.get('etapi-tokens');
-        
+
         $noTokensYet.toggle(tokens.length === 0);
         $tokensTable.toggle(tokens.length > 0);
 
         const $tokensTableBody = $tokensTable.find("tbody");
         $tokensTableBody.empty();
-        
+
         for (const token of tokens) {
             $tokensTableBody.append(
                 $("<tr>")
@@ -112,7 +112,7 @@ export default class EtapiOptions {
         });
 
         await server.patch(`etapi-tokens/${etapiTokenId}`, {name: tokenName});
-        
+
         this.refreshTokens();
     }
 
diff --git a/src/public/app/dialogs/prompt.js b/src/public/app/dialogs/prompt.js
index c4e03b6b1..94a8e9c46 100644
--- a/src/public/app/dialogs/prompt.js
+++ b/src/public/app/dialogs/prompt.js
@@ -13,9 +13,9 @@ let shownCb;
 
 export function ask({ title, message, defaultValue, shown }) {
     shownCb = shown;
-    
+
     $("#prompt-title").text(title || "Prompt");
-    
+
     $question = $("<label>")
         .prop("for", "prompt-dialog-answer")
         .text(message);
@@ -51,7 +51,8 @@ $dialog.on("hidden.bs.modal", () => {
     }
 });
 
-$form.on('submit', () => {
+$form.on('submit', e => {
+    e.preventDefault();
     resolve($answer.val());
 
     $dialog.modal('hide');
diff --git a/src/routes/api/etapi_tokens.js b/src/routes/api/etapi_tokens.js
index b6a2ebfa7..565ebc53c 100644
--- a/src/routes/api/etapi_tokens.js
+++ b/src/routes/api/etapi_tokens.js
@@ -2,16 +2,14 @@ const etapiTokenService = require("../../services/etapi_tokens");
 
 function getTokens() {
     const tokens = etapiTokenService.getTokens();
-    
+
     tokens.sort((a, b) => a.utcDateCreated < b.utcDateCreated ? -1 : 1);
-    
+
     return tokens;
 }
 
 function createToken(req) {
-    return {
-        authToken: etapiTokenService.createToken(req.body.tokenName)
-    };
+    return etapiTokenService.createToken(req.body.tokenName);
 }
 
 function patchToken(req) {
@@ -27,4 +25,4 @@ module.exports = {
     createToken,
     patchToken,
     deleteToken
-};
\ No newline at end of file
+};
diff --git a/src/services/etapi_tokens.js b/src/services/etapi_tokens.js
index abc8c09c0..6b4ec6277 100644
--- a/src/services/etapi_tokens.js
+++ b/src/services/etapi_tokens.js
@@ -19,7 +19,7 @@ function createToken(tokenName) {
         name: tokenName,
         tokenHash
     }).save();
-    
+
     return {
         authToken: `${etapiToken.etapiTokenId}_${token}`
     };
@@ -29,14 +29,14 @@ function parseAuthToken(auth) {
     if (!auth) {
         return null;
     }
-    
+
     const chunks = auth.split("_");
-    
+
     if (chunks.length === 1) {
         return { token: auth }; // legacy format without etapiTokenId
     }
     else if (chunks.length === 2) {
-        return { 
+        return {
             etapiTokenId: chunks[0],
             token: chunks[1]
         }
@@ -48,20 +48,20 @@ function parseAuthToken(auth) {
 
 function isValidAuthHeader(auth) {
     const parsed = parseAuthToken(auth);
-    
+
     if (!parsed) {
         return false;
     }
-    
-    const authTokenHash = getTokenHash(parsed.token); 
-    
+
+    const authTokenHash = getTokenHash(parsed.token);
+
     if (parsed.etapiTokenId) {
         const etapiToken = becca.getEtapiToken(parsed.etapiTokenId);
-        
+
         if (!etapiToken) {
             return false;
         }
-        
+
         return etapiToken.tokenHash === authTokenHash;
     }
     else {
@@ -70,31 +70,30 @@ function isValidAuthHeader(auth) {
                 return true;
             }
         }
-        
+
         return false;
     }
 }
 
 function renameToken(etapiTokenId, newName) {
     const etapiToken = becca.getEtapiToken(etapiTokenId);
-    
+
     if (!etapiToken) {
         throw new Error(`Token ${etapiTokenId} does not exist`);
     }
-    
+
     etapiToken.name = newName;
     etapiToken.save();
 }
 
 function deleteToken(etapiTokenId) {
     const etapiToken = becca.getEtapiToken(etapiTokenId);
-    
+
     if (!etapiToken) {
         return; // ok, already deleted
     }
-    
-    etapiToken.isDeleted = true;
-    etapiToken.save();
+
+    etapiToken.markAsDeletedSimple();
 }
 
 module.exports = {
@@ -104,4 +103,4 @@ module.exports = {
     deleteToken,
     parseAuthToken,
     isValidAuthHeader
-};
\ No newline at end of file
+};
diff --git a/src/views/dialogs/prompt.ejs b/src/views/dialogs/prompt.ejs
index e89b30c88..2f135ad1f 100644
--- a/src/views/dialogs/prompt.ejs
+++ b/src/views/dialogs/prompt.ejs
@@ -1,5 +1,5 @@
 <div id="prompt-dialog" class="modal mx-auto" tabindex="-1" role="dialog">
-    <div class="modal-dialog" role="document">
+    <div class="modal-dialog modal-lg" role="document">
         <div class="modal-content">
             <form id="prompt-dialog-form">
                 <div class="modal-header">

From 83f8fac088f4bacbe9bed15637895f4b594fac14 Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Mon, 7 Feb 2022 23:01:14 +0100
Subject: [PATCH 2/7] fixed background color of input label in CKEditor search
 & replace, #2254

---
 src/public/stylesheets/ckeditor-theme.css | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/public/stylesheets/ckeditor-theme.css b/src/public/stylesheets/ckeditor-theme.css
index f1af43c98..8ef29a790 100644
--- a/src/public/stylesheets/ckeditor-theme.css
+++ b/src/public/stylesheets/ckeditor-theme.css
@@ -91,6 +91,8 @@ body {
 
     --ck-color-table-focused-cell-background: var(--more-accented-background-color);
 
+    --ck-color-labeled-field-label-background: var(--accented-background-color);
+
     /* todo lists */
 
     --ck-color-todo-list-checkmark-border: var(--main-border-color);

From 1dfde76b95e20e6f2902907d223dab65cde9fbb6 Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Tue, 8 Feb 2022 23:38:54 +0100
Subject: [PATCH 3/7] fix updating becca after undeleting notes

---
 src/public/app/services/tree.js | 2 +-
 src/services/notes.js           | 5 ++++-
 2 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/public/app/services/tree.js b/src/public/app/services/tree.js
index 4bbdafdb4..30eb85688 100644
--- a/src/public/app/services/tree.js
+++ b/src/public/app/services/tree.js
@@ -65,7 +65,7 @@ async function resolveNotePathToSegments(notePath, hoistedNoteId = 'root', logEr
 
             if (!parents.length) {
                 if (logErrors) {
-                    ws.logError(`No parents found for ${childNoteId} (${child.title}) for path ${notePath}`);
+                    ws.logError(`No parents found for note ${childNoteId} (${child.title}) for path ${notePath}`);
                 }
 
                 return;
diff --git a/src/services/notes.js b/src/services/notes.js
index 5b330efd6..df227291d 100644
--- a/src/services/notes.js
+++ b/src/services/notes.js
@@ -634,7 +634,10 @@ function undeleteBranch(branchId, deleteId, taskContext) {
     taskContext.increaseProgressCount();
 
     if (note.isDeleted && note.deleteId === deleteId) {
-        new Note(note).save();
+        // becca entity was already created as skeleton in "new Branch()" above
+        const noteEntity = becca.getNote(note.noteId);
+        noteEntity.updateFromRow(note);
+        noteEntity.save();
 
         const attributes = sql.getRows(`
                 SELECT * FROM attributes 

From d6016f9b81ac47ac65b4ef5f396fa83c7313125a Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Wed, 9 Feb 2022 21:21:17 +0100
Subject: [PATCH 4/7] introduced mutex to avoid conflicts during tab closing

---
 src/public/app/services/tab_manager.js | 71 ++++++++++++++------------
 src/public/app/utils/mutex.js          | 28 ++++++++++
 2 files changed, 67 insertions(+), 32 deletions(-)
 create mode 100644 src/public/app/utils/mutex.js

diff --git a/src/public/app/services/tab_manager.js b/src/public/app/services/tab_manager.js
index 2947886f6..15f939bfe 100644
--- a/src/public/app/services/tab_manager.js
+++ b/src/public/app/services/tab_manager.js
@@ -7,11 +7,14 @@ import treeService from "./tree.js";
 import utils from "./utils.js";
 import NoteContext from "./note_context.js";
 import appContext from "./app_context.js";
+import Mutex from "../utils/mutex.js";
 
 export default class TabManager extends Component {
     constructor() {
         super();
 
+        this.mutex = new Mutex();
+
         this.activeNtxId = null;
 
         // elements are arrays of note contexts for each tab (one main context + subcontexts [splits])
@@ -292,51 +295,55 @@ export default class TabManager extends Component {
     }
 
     async removeNoteContext(ntxId) {
-        const noteContextToRemove = this.getNoteContextById(ntxId);
+        // removing note context is async process which can take some time, if users presses CTRL-W quickly, two
+        // close events could interleave which would then lead to attempting to activate already removed context.
+        await this.mutex.runExclusively(async () => {
+            const noteContextToRemove = this.getNoteContextById(ntxId);
 
-        if (noteContextToRemove.isMainContext()) {
-            // forbid removing last main note context
-            // this was previously allowed (was replaced with empty tab) but this proved to be prone to race conditions
-            const mainNoteContexts = this.getNoteContexts().filter(nc => nc.isMainContext());
+            if (noteContextToRemove.isMainContext()) {
+                // forbid removing last main note context
+                // this was previously allowed (was replaced with empty tab) but this proved to be prone to race conditions
+                const mainNoteContexts = this.getNoteContexts().filter(nc => nc.isMainContext());
 
-            if (mainNoteContexts.length === 1) {
-                mainNoteContexts[0].setEmpty();
-                return;
+                if (mainNoteContexts.length === 1) {
+                    mainNoteContexts[0].setEmpty();
+                    return;
+                }
             }
-        }
 
-        // close dangling autocompletes after closing the tab
-        $(".aa-input").autocomplete("close");
+            // close dangling autocompletes after closing the tab
+            $(".aa-input").autocomplete("close");
 
-        const noteContextsToRemove = noteContextToRemove.getSubContexts();
-        const ntxIdsToRemove = noteContextsToRemove.map(nc => nc.ntxId);
+            const noteContextsToRemove = noteContextToRemove.getSubContexts();
+            const ntxIdsToRemove = noteContextsToRemove.map(nc => nc.ntxId);
 
-        await this.triggerEvent('beforeTabRemove', { ntxIds: ntxIdsToRemove });
+            await this.triggerEvent('beforeTabRemove', { ntxIds: ntxIdsToRemove });
 
-        if (!noteContextToRemove.isMainContext()) {
-            await this.activateNoteContext(noteContextToRemove.getMainContext().ntxId);
-        }
-        else if (this.mainNoteContexts.length <= 1) {
-            await this.openAndActivateEmptyTab();
-        }
-        else if (ntxIdsToRemove.includes(this.activeNtxId)) {
-            const idx = this.mainNoteContexts.findIndex(nc => nc.ntxId === noteContextToRemove.ntxId);
-
-            if (idx === this.mainNoteContexts.length - 1) {
-                await this.activatePreviousTabCommand();
+            if (!noteContextToRemove.isMainContext()) {
+                await this.activateNoteContext(noteContextToRemove.getMainContext().ntxId);
             }
-            else {
-                await this.activateNextTabCommand();
+            else if (this.mainNoteContexts.length <= 1) {
+                await this.openAndActivateEmptyTab();
             }
-        }
+            else if (ntxIdsToRemove.includes(this.activeNtxId)) {
+                const idx = this.mainNoteContexts.findIndex(nc => nc.ntxId === noteContextToRemove.ntxId);
 
-        this.children = this.children.filter(nc => !ntxIdsToRemove.includes(nc.ntxId));
+                if (idx === this.mainNoteContexts.length - 1) {
+                    await this.activatePreviousTabCommand();
+                }
+                else {
+                    await this.activateNextTabCommand();
+                }
+            }
 
-        this.recentlyClosedTabs.push(noteContextsToRemove);
+            this.children = this.children.filter(nc => !ntxIdsToRemove.includes(nc.ntxId));
 
-        this.triggerEvent('noteContextRemoved', {ntxIds: ntxIdsToRemove});
+            this.recentlyClosedTabs.push(noteContextsToRemove);
 
-        this.tabsUpdate.scheduleUpdate();
+            this.triggerEvent('noteContextRemoved', {ntxIds: ntxIdsToRemove});
+
+            this.tabsUpdate.scheduleUpdate();
+        });
     }
 
     tabReorderEvent({ntxIdsInOrder}) {
diff --git a/src/public/app/utils/mutex.js b/src/public/app/utils/mutex.js
new file mode 100644
index 000000000..a48b2660e
--- /dev/null
+++ b/src/public/app/utils/mutex.js
@@ -0,0 +1,28 @@
+export default class Mutex {
+    constructor() {
+        this.current = Promise.resolve();
+    }
+
+    /** @returns {Promise} */
+    lock() {
+        let resolveFun;
+        const subPromise = new Promise(resolve => resolveFun = () => resolve());
+        // Caller gets a promise that resolves when the current outstanding lock resolves
+        const newPromise = this.current.then(() => resolveFun);
+        // Don't allow the next request until the new promise is done
+        this.current = subPromise;
+        // Return the new promise
+        return newPromise;
+    };
+
+    async runExclusively(cb) {
+        const unlock = await this.lock();
+
+        try {
+            await cb();
+        }
+        finally {
+            unlock();
+        }
+    }
+}

From 23daaa2387a0655685377f0a541d154aeec2aae8 Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Wed, 9 Feb 2022 22:39:37 +0100
Subject: [PATCH 5/7] fix exporting huge text notes (export would get stuck on
 HTML pretty printing)

---
 src/services/export/zip.js | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/src/services/export/zip.js b/src/services/export/zip.js
index 2a4ea2495..b1336e32a 100644
--- a/src/services/export/zip.js
+++ b/src/services/export/zip.js
@@ -13,6 +13,7 @@ const fs = require("fs");
 const becca = require("../../becca/becca");
 const RESOURCE_DIR = require('../../services/resource_dir').RESOURCE_DIR;
 const archiver = require('archiver');
+const log = require("../log");
 
 /**
  * @param {TaskContext} taskContext
@@ -54,7 +55,7 @@ function exportToZip(taskContext, branch, format, res) {
 
         let existingExtension = path.extname(fileName).toLowerCase();
         let newExtension;
-        
+
         if (fileName.length > 30) {
             fileName = fileName.substr(0, 30);
         }
@@ -254,7 +255,9 @@ ${content}
 </html>`;
             }
 
-            return html.prettyPrint(content, {indent_size: 2});
+            return content.length < 100000
+                ? html.prettyPrint(content, {indent_size: 2})
+                : content;
         }
         else if (noteMeta.format === 'markdown') {
             let markdownContent = mdService.toMarkdown(content);
@@ -274,6 +277,8 @@ ${content}
     const notePaths = {};
 
     function saveNote(noteMeta, filePathPrefix) {
+        log.info(`Exporting note ${noteMeta.noteId}`);
+
         if (noteMeta.isClone) {
             const targetUrl = getTargetUrl(noteMeta.noteId, noteMeta);
 

From 832cac106ebd321805febb06785f7e477e365fc5 Mon Sep 17 00:00:00 2001
From: zadam <zadam.apps@gmail.com>
Date: Wed, 9 Feb 2022 22:52:36 +0100
Subject: [PATCH 6/7] release 0.50.2

---
 package.json          | 2 +-
 src/services/build.js | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/package.json b/package.json
index 57cc60ea8..0d6dc3620 100644
--- a/package.json
+++ b/package.json
@@ -2,7 +2,7 @@
   "name": "trilium",
   "productName": "Trilium Notes",
   "description": "Trilium Notes",
-  "version": "0.50.1",
+  "version": "0.50.2",
   "license": "AGPL-3.0-only",
   "main": "electron.js",
   "bin": {
diff --git a/src/services/build.js b/src/services/build.js
index bc71e3264..7d0fb0cf6 100644
--- a/src/services/build.js
+++ b/src/services/build.js
@@ -1 +1 @@
-module.exports = { buildDate:"2022-02-02T21:38:21+01:00", buildRevision: "0917fc8be171253449219cf29c0e603ac29eb26e" };
+module.exports = { buildDate:"2022-02-09T22:52:36+01:00", buildRevision: "23daaa2387a0655685377f0a541d154aeec2aae8" };

From 40598d2663fc8c942b1484e04e3c8d734361d3c2 Mon Sep 17 00:00:00 2001
From: Nriver <6752679+Nriver@users.noreply.github.com>
Date: Fri, 11 Feb 2022 02:28:42 +0800
Subject: [PATCH 7/7] fix param validation (#2624)

(cherry picked from commit 1aeb6747336c645e9268c1d1adc01ac6845aac9d)
---
 src/etapi/notes.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/etapi/notes.js b/src/etapi/notes.js
index f5110d646..500b4d6bc 100644
--- a/src/etapi/notes.js
+++ b/src/etapi/notes.js
@@ -162,7 +162,7 @@ function parseBoolean(obj, name) {
     return obj[name] === 'true';
 }
 
-function parseInteger(obj, name) {
+function parseOrderDirection(obj, name) {
     if (!(name in obj)) {
         return undefined;
     }
@@ -176,7 +176,7 @@ function parseInteger(obj, name) {
     return integer;
 }
 
-function parseOrderDirection(obj, name) {
+function parseInteger(obj, name) {
     if (!(name in obj)) {
         return undefined;
     }