resolve: #135

windingwind · Oct 24, 2023 · 59fb70a · 59fb70a
1 parent 616952c
commit 59fb70a
Showing 1 changed file with 15 additions and 6 deletions.
diff --git a/src/utils/actions.ts b/src/utils/actions.ts
@@ -128,21 +128,30 @@ async function applyAction(rule: ActionData, args: ActionArgs) {
     .map((tag) => tag.trim())
     .filter((tag) => tag);
   let message: string = "";
+  let hasChanged = false;
   switch (rule.operation) {
     case ActionOperationTypes.add: {
       for (const tag of tags) {
-        item?.addTag(tag, 1);
+        if (!item?.hasTag(tag)) {
+          hasChanged || (hasChanged = true);
+          item?.addTag(tag, 1);
+        }
       }
-      message = `Add tag ${tags.join(",")} to item ${item?.getField("title")}`;
+      message = hasChanged
+        ? `Add tag ${tags.join(",")} to item ${item?.getField("title")}`
+        : "";
       break;
     }
     case ActionOperationTypes.remove: {
       for (const tag of tags) {
-        item?.removeTag(tag);
+        if (item?.hasTag(tag)) {
+          hasChanged || (hasChanged = true);
+          item?.removeTag(tag);
+        }
       }
-      message = `Remove tag ${tags.join(",")} from item ${item?.getField(
-        "title",
-      )}`;
+      message = hasChanged
+        ? `Remove tag ${tags.join(",")} from item ${item?.getField("title")}`
+        : "";
       break;
     }
     case ActionOperationTypes.toggle: {