Don't create duplicate tags; Tagging is idempotent

diff --git a/vopamoi.ts b/vopamoi.ts
index 7c51935a2c14535444199ddb68da35ec2eca4e17..3b2f1638beb180e3e6b294cfa59b23ee2c73fe3d 100644 (file)
--- a/vopamoi.ts
+++ b/vopamoi.ts
@@ -44,6 +44,8 @@ const Model = {
   addTag: function (createTimestamp: string, tagName: string): Element | null {
     const task = this.getTask(createTimestamp);
     if (!task) return null;
   addTag: function (createTimestamp: string, tagName: string): Element | null {
     const task = this.getTask(createTimestamp);
     if (!task) return null;

+    const existingTag = this.hasTag(task, tagName);
+    if (existingTag) return existingTag;

     const tag = document.createElement("span");
     tag.appendChild(document.createTextNode(tagName));
     tag.classList.add("tag");
     const tag = document.createElement("span");
     tag.appendChild(document.createTextNode(tagName));
     tag.classList.add("tag");


@@ -77,13 +79,13 @@ const Model = {
     return target;
   },
 
     return target;
   },
 

-  hasTag: function (task: Element, tag: string) {
+  hasTag: function (task: Element, tag: string): Element | null {

     for (const child of task.children) {
       if (child.classList.contains("tag") && child.textContent === tag) {
     for (const child of task.children) {
       if (child.classList.contains("tag") && child.textContent === tag) {

-        return true;
+        return child;

       }
     }
       }
     }

-    return false;
+    return null;

   },
 
   getPriority: function (task: Element): number {
   },
 
   getPriority: function (task: Element): number {