fix: bug in where condition when persisting entity

williarin · Dec 5, 2023 · a7466e9 · a7466e9
1 parent 1b8e4c6
commit a7466e9
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/Bridge/Repository/AbstractEntityRepository.php b/src/Bridge/Repository/AbstractEntityRepository.php
@@ -206,7 +206,7 @@ public function persist(mixed $entity): void
             $queryBuilder = $this->entityManager->getConnection()
                 ->createQueryBuilder()
                 ->update($this->entityManager->getTablesPrefix() . static::TABLE_NAME)
-                ->where(static::TABLE_IDENTIFIER, ':' . static::TABLE_IDENTIFIER)
+                ->where(static::TABLE_IDENTIFIER . ' = :' . static::TABLE_IDENTIFIER)
                 ->setParameter(static::TABLE_IDENTIFIER, $entity->{static::TABLE_IDENTIFIER})
             ;
 

diff --git a/test/Test/Bridge/Repository/PostRepositoryTest.php b/test/Test/Bridge/Repository/PostRepositoryTest.php
@@ -267,6 +267,10 @@ public function testPersistExistingPost(): void
 
         self::assertSame('Another post with a new title', $post->postTitle);
         self::assertSame('publish', $post->postStatus);
+
+        // Assert no collateral damages
+        $anotherPost = $this->repository->find(10);
+        self::assertSame('A post', $anotherPost->postTitle);
     }
 
     public function testDynamicSetter(): void