note tree refactorings

src/public/javascripts/dialogs/edit_tree_prefix.js

@@ -19,8 +19,9 @@ const editTreePrefix = (function() {
         const currentNode = noteTree.getCurrentNode();
 
         noteTreeId = currentNode.data.noteTreeId;
+        const nt = noteTree.getNoteTree(noteTreeId);
 
-        $treePrefixInput.val(currentNode.data.prefix).focus();
+        $treePrefixInput.val(nt.prefix).focus();
 
         const noteTitle = noteTree.getNoteTitle(currentNode.data.noteId);
 

src/public/javascripts/note_tree.js

@@ -89,11 +89,18 @@ const noteTree = (function() {
 
         notesTreeMap[noteTreeId].prefix = prefix;
 
-        getNodesByNoteTreeId(noteTreeId).map(node => {
+        getNodesByNoteTreeId(noteTreeId).map(node => setNodeTitleWithPrefix(node));
-            node.data.prefix = prefix;
+    }
 
-            treeUtils.setNodeTitleWithPrefix(node);
+    function setNodeTitleWithPrefix(node) {
-        });
+        const noteTitle = getNoteTitle(node.data.noteId);
+        const noteTree = notesTreeMap[node.data.noteTreeId];
+
+        const prefix = noteTree.prefix;
+
+        const title = (prefix ? (prefix + " - ") : "") + noteTitle;
+
+        node.setTitle(escapeHtml(title));
     }
 
     function removeParentChildRelation(parentNoteId, childNoteId) {
@@ -138,6 +145,7 @@ const noteTree = (function() {
             notesTreeMap[note.noteTreeId] = note;
 
             noteIdToNote[note.noteId] = {
+                noteId: note.noteId,
                 title: note.title,
                 isProtected: note.isProtected,
                 type: note.type,
@@ -182,24 +190,26 @@ const noteTree = (function() {
         const noteList = [];
 
         for (const noteId of childNoteIds) {
+            const note = getNote(noteId);
             const noteTreeId = getNoteTreeId(parentNoteId, noteId);
             const noteTree = notesTreeMap[noteTreeId];
 
-            const title = (noteTree.prefix ? (noteTree.prefix + " - ") : "") + getNote(noteTree.noteId).title;
+            const title = (noteTree.prefix ? (noteTree.prefix + " - ") : "") + note.title;
 
             const node = {
-                noteId: noteTree.noteId,
+                noteId: noteId,
                 parentNoteId: noteTree.parentNoteId,
                 noteTreeId: noteTree.noteTreeId,
-                isProtected: noteTree.isProtected,
+                isProtected: note.isProtected,
-                prefix: noteTree.prefix,
                 title: escapeHtml(title),
-                extraClasses: getExtraClasses(noteTree),
+                extraClasses: getExtraClasses(note),
-                refKey: noteTree.noteId,
+                refKey: noteId,
                 expanded: noteTree.isExpanded
             };
 
-            if (parentToChildren[noteId] && parentToChildren[noteId].length > 0) {
+            const hasChildren = parentToChildren[noteId] && parentToChildren[noteId].length > 0;
+
+            if (hasChildren || note.type === 'search') {
                 node.folder = true;
 
                 if (node.expanded) {
@@ -626,9 +636,15 @@ const noteTree = (function() {
             },
             dnd: dragAndDropSetup,
             lazyLoad: function(event, data){
-                const node = data.node.data;
+                const noteId = data.node.data.noteId;
+                const note = getNote(noteId);
 
-                data.result = prepareNoteTreeInner(node.noteId);
+                if (note.type === 'search') {
+                    data.result = loadSearchNote(noteId);
+                }
+                else {
+                    data.result = prepareNoteTreeInner(noteId);
+                }
             },
             clones: {
                 highlightActiveClones: true
@@ -638,6 +654,16 @@ const noteTree = (function() {
         $tree.contextmenu(contextMenu.contextMenuSettings);
     }
 
+    async function loadSearchNote(noteId) {
+        const note = await server.get('notes/' + noteId);
+
+        const json = JSON.parse(note.detail.content);
+
+        const noteIds = await server.get('notes?search=' + encodeURIComponent(json.searchString));
+
+        console.log("Found: ", noteIds);
+    }
+
     function getTree() {
         return $tree.fancytree('getTree');
     }
@@ -755,7 +781,7 @@ const noteTree = (function() {
 
         getNote(noteId).title = title;
 
-        getNodesByNoteId(noteId).map(clone => treeUtils.setNodeTitleWithPrefix(clone));
+        getNodesByNoteId(noteId).map(clone => setNodeTitleWithPrefix(clone));
     }
 
     async function createNewTopLevelNote() {
@@ -787,6 +813,7 @@ const noteTree = (function() {
         notesTreeMap[result.noteTreeId] = result;
 
         noteIdToNote[result.noteId] = {
+            noteId: result.noteId,
             title: result.title,
             isProtected: result.isProtected,
             type: result.type,
@@ -844,6 +871,10 @@ const noteTree = (function() {
         return instanceName;
     }
 
+    function getNoteTree(noteTreeId) {
+        return notesTreeMap[noteTreeId];
+    }
+
     $(document).bind('keydown', 'ctrl+o', e => {
         const node = getCurrentNode();
         const parentNoteId = node.data.parentNoteId;
@@ -919,6 +950,7 @@ const noteTree = (function() {
         getSelectedNodes,
         sortAlphabetically,
         noteExists,
-        getInstanceName
+        getInstanceName,
+        getNoteTree
     };
 })();

src/public/javascripts/tree_utils.js

@@ -31,20 +31,10 @@ const treeUtils = (function() {
         return path.reverse().join("/");
     }
 
-    function setNodeTitleWithPrefix(node) {
-        const noteTitle = noteTree.getNoteTitle(node.data.noteId);
-        const prefix = node.data.prefix;
-
-        const title = (prefix ? (prefix + " - ") : "") + noteTitle;
-
-        node.setTitle(escapeHtml(title));
-    }
-
     return {
         getParentProtectedStatus,
         getNodeByKey,
         getNotePath,
         getNoteIdFromNotePath,
-        setNodeTitleWithPrefix
     };
 })();