Ensure tags are unique when renaming

shaarli · May 14, 2024 · 7088857 · 7088857
1 parent 2321bcf
commit 7088857
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 5 deletions.
diff --git a/application/bookmark/Bookmark.php b/application/bookmark/Bookmark.php
@@ -505,15 +505,19 @@ public function getAdditionalContentEntry(string $key, $default = null)
     }
 
     /**
-     * Rename a tag in tags list.
+     * Rename a tag in tags list. If the new tag already exists, merge them
      *
      * @param string $fromTag
      * @param string $toTag
      */
     public function renameTag(string $fromTag, string $toTag): void
     {
         if (($pos = array_search($fromTag, $this->tags ?? [])) !== false) {
-            $this->tags[$pos] = trim($toTag);
+            if (in_array($toTag, $this->tags ?? []) !== false) {
+                $this->deleteTag($fromTag);
+            } else {
+                $this->tags[$pos] = trim($toTag);
+            }
         }
     }
 

diff --git a/tests/bookmark/BookmarkTest.php b/tests/bookmark/BookmarkTest.php
@@ -374,6 +374,9 @@ public function testSetTags()
     public function testRenameTag()
     {
         $bookmark = new Bookmark();
+        $bookmark->renameTag('chair', 'table');
+        $this->assertEquals([], $bookmark->getTags());
+
         $bookmark->setTags(['tag1', 'tag2', 'chair']);
         $bookmark->renameTag('chair', 'table');
         $this->assertEquals(['tag1', 'tag2', 'table'], $bookmark->getTags());

diff --git a/tests/front/controller/admin/ManageTagControllerTest.php b/tests/front/controller/admin/ManageTagControllerTest.php
@@ -13,7 +13,6 @@
 use Shaarli\TestCase;
 use Slim\Http\Request;
 use Slim\Http\Response;
-
 // These are declared for the bookmark service
 use malkusch\lock\mutex\NoMutex;
 use Shaarli\History;
@@ -56,8 +55,18 @@ public function setUp(): void
         self::$refDB = new ReferenceLinkDB();
         self::$refDB->write(self::$testDatastore);
         $history = new History('sandbox/history.php');
-        $this->container->bookmarkService = new FakeBookmarkService($conf, static::$pluginManager, $history, $mutex, true);
-        $this->linkFilter = new BookmarkFilter($this->container->bookmarkService->getBookmarks(), $conf, static::$pluginManager);
+        $this->container->bookmarkService = new FakeBookmarkService(
+            $conf,
+            static::$pluginManager,
+            $history,
+            $mutex,
+            true
+        );
+        $this->linkFilter = new BookmarkFilter(
+            $this->container->bookmarkService->getBookmarks(),
+            $conf,
+            static::$pluginManager
+        );
     }
 
     /**