fix: add term to an entity which already have it won't throw exception anymore

Author: williarin

src/Bridge/Repository/TermRepository.php

@@ -4,6 +4,7 @@
 
 namespace Williarin\WordpressInterop\Bridge\Repository;
 
+use Doctrine\DBAL\Exception\UniqueConstraintViolationException;
 use Doctrine\DBAL\Query\QueryBuilder;
 use Williarin\WordpressInterop\Bridge\Entity\BaseEntity;
 use Williarin\WordpressInterop\Bridge\Entity\Term;
@@ -62,17 +63,20 @@ public function addTermsToEntity(BaseEntity $entity, array $terms): void
                 continue;
             }
 
-            $this->entityManager->getConnection()
-                ->createQueryBuilder()
-                ->insert($this->entityManager->getTablesPrefix() . 'term_relationships')
-                ->values([
-                    'object_id' => '?',
-                    'term_taxonomy_id' => '?',
-                    'term_order' => '0',
-                ])
-                ->setParameters([$entity->id, (int) $term->termTaxonomyId])
-                ->executeStatement()
-            ;
+            try {
+                $this->entityManager->getConnection()
+                    ->createQueryBuilder()
+                    ->insert($this->entityManager->getTablesPrefix() . 'term_relationships')
+                    ->values([
+                        'object_id' => '?',
+                        'term_taxonomy_id' => '?',
+                        'term_order' => '0',
+                    ])
+                    ->setParameters([$entity->id, (int) $term->termTaxonomyId])
+                    ->executeStatement()
+                ;
+            } catch (UniqueConstraintViolationException) {
+            }
         }
 
         $this->recountTerms();

test/Test/Bridge/Repository/TermRepositoryTest.php

@@ -133,6 +133,39 @@ public function testAddTermsToEntity(): void
         self::assertEquals([1, 6, 1, 2], array_column($termsProduct, 'count'));
     }
 
+    public function testAddDuplicateTermsToEntityDoesNotDuplicateThem(): void
+    {
+        $hoodieTerms = $this->repository->findBy([
+            new PostRelationshipCondition(Product::class, [
+                'post_status' => new Operand(['publish', 'private'], Operand::OPERATOR_IN),
+                'sku' => 'super-forces-hoodie',
+            ]),
+        ]);
+
+        $product = $this->manager->getRepository(Product::class)
+            ->find(16)
+        ;
+
+        $termsProduct = $this->repository->findBy([
+            new PostRelationshipCondition(Product::class, [
+                'id' => $product->id,
+            ]),
+        ]);
+
+        self::assertEquals(['simple', 'Hoodies'], array_column($termsProduct, 'name'));
+
+        $this->repository->addTermsToEntity($product, $hoodieTerms);
+
+        $termsProduct = $this->repository->findBy([
+            new PostRelationshipCondition(Product::class, [
+                'id' => $product->id,
+            ]),
+        ]);
+
+        self::assertEquals(['simple', 'Hoodies', 'MegaBrand'], array_column($termsProduct, 'name'));
+        self::assertEquals([15, 5, 2], array_column($termsProduct, 'count'));
+    }
+
     public function testAddTermsToEntityWithoutTermTaxonomyIdAreIgnored(): void
     {
         $hoodieTerms = array_map(