fix: use add tags folder hierarchy & isTemplateItem (#13)

* fix: use new snjs add tag hierarchy * fix: use new snjs isTemplateItem
standardnotes · Dec 23, 2021 · 592cd88 · 592cd88
1 parent c2b6168
commit 592cd88
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 33 deletions.
diff --git a/app/assets/javascripts/ui_models/app_state/note_tags_state.ts b/app/assets/javascripts/ui_models/app_state/note_tags_state.ts
@@ -174,32 +174,11 @@ export class NoteTagsState {
     }
   }
 
-  public static addTagHierarchyToNote(
-    application: WebApplication,
-    tag: SNTag,
-    note: SNNote
-  ): Promise<unknown> {
-    const parentChainTags = application.getTagParentChain(tag);
-    const tagsToAdd = [...parentChainTags, tag];
-
-    return Promise.all(
-      tagsToAdd.map(async (tag) => {
-        await application.changeItem(tag.uuid, (mutator) => {
-          mutator.addItemAsRelationship(note);
-        });
-      })
-    );
-  }
-
   async addTagToActiveNote(tag: SNTag): Promise<void> {
     const { activeNote } = this;
 
     if (activeNote) {
-      await NoteTagsState.addTagHierarchyToNote(
-        this.application,
-        tag,
-        activeNote
-      );
+      await this.application.addTagHierarchyToNote(activeNote, tag);
       this.application.sync();
       this.reloadTags();
     }

diff --git a/app/assets/javascripts/ui_models/app_state/tags_state.ts b/app/assets/javascripts/ui_models/app_state/tags_state.ts
@@ -60,18 +60,16 @@ export class TagsState {
       return [];
     }
 
-    try {
-      const children = this.application.getTagChildren(tag);
-      const childrenUuids = children.map((childTag) => childTag.uuid);
-      const childrenTags = this.tags.filter((tag) =>
-        childrenUuids.includes(tag.uuid)
-      );
-      return childrenTags;
-    } catch {
-      // In the case of template tags, this code throw,
-      // We implement a general catch-all for now.
+    if (this.application.isTemplateItem(tag)) {
       return [];
     }
+
+    const children = this.application.getTagChildren(tag);
+    const childrenUuids = children.map((childTag) => childTag.uuid);
+    const childrenTags = this.tags.filter((tag) =>
+      childrenUuids.includes(tag.uuid)
+    );
+    return childrenTags;
   }
 
   isValidTagParent(parentUuid: UuidString, tagUuid: UuidString): boolean {

diff --git a/app/assets/javascripts/ui_models/editor.ts b/app/assets/javascripts/ui_models/editor.ts
@@ -81,7 +81,7 @@ export class Editor {
     })) as SNNote;
     if (noteTag) {
       const tag = this.application.findItem(noteTag) as SNTag;
-      await NoteTagsState.addTagHierarchyToNote(this.application, tag, note);
+      await this.application.addTagHierarchyToNote(note, tag);
     }
     if (!this.isTemplateNote || this.note.title !== note.title) {
       this.setNote(note as SNNote, true);