fix: Revert change to Tags

Let's keep this for a later refactor

Signed-off-by: Carl Schwan <carl@carlschwan.eu>

Author: CarlSchwan

lib/private/Tagging/TagMapper.php

@@ -8,6 +8,7 @@
 
 namespace OC\Tagging;
 
+use OCP\AppFramework\Db\DoesNotExistException;
 use OCP\AppFramework\ORM\Repository;
 
 /**
@@ -33,4 +34,17 @@ public function loadTags(array $owners, string $type): array {
 			'name' => 'ASC',
 		]));
 	}
+
+	public function tagExists(Tag $tag): bool {
+		try {
+			$this->findOneBy([
+				'uid' => $tag->owner,
+				'type' => $tag->type,
+				'category' => $tag->name,
+			]);
+			return true;
+		} catch (DoesNotExistException) {
+			return false;
+		}
+	}
 }

lib/private/Tags.php

@@ -320,15 +320,8 @@ public function addMultiple(string|array $names, bool $sync = false, ?int $id =
 	 */
 	protected function save(): void {
 		foreach ($this->tags as $tag) {
-			try {
+			if (!$this->mapper->tagExists($tag)) {
 				$this->mapper->insert($tag);
-			} catch (Exception $e) {
-				if ($e->getReason() !== Exception::REASON_UNIQUE_CONSTRAINT_VIOLATION) {
-					$this->logger->error($e->getMessage(), [
-						'exception' => $e,
-						'app' => 'core',
-					]);
-				}
 			}
 		}