Merge pull request #28 from drekinov/fix-orm-2-3-setparameters

doctrine/orm v3 compatibility fix
klapaudius · Sep 2, 2024 · 260d2bd · 260d2bd
2 parents 8ad6cb4 + 916a4f3
commit 260d2bd
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 11 deletions.
diff --git a/Entity/AuthCodeManager.php b/Entity/AuthCodeManager.php
@@ -17,6 +17,8 @@
 use FOS\OAuthServerBundle\Model\AuthCodeInterface;
 use FOS\OAuthServerBundle\Model\AuthCodeManager as BaseAuthCodeManager;
 
+use function time;
+
 class AuthCodeManager extends BaseAuthCodeManager
 {
     /**
@@ -80,8 +82,8 @@ public function deleteExpired(): int
         $qb = $repository->createQueryBuilder('a');
         $qb
             ->delete()
-            ->where('a.expiresAt < ?1')
-            ->setParameters([1 => time()])
+            ->where('a.expiresAt < :time')
+            ->setParameter('time', time())
         ;
 
         return $qb->getQuery()->execute();

diff --git a/Entity/TokenManager.php b/Entity/TokenManager.php
@@ -18,6 +18,8 @@
 use FOS\OAuthServerBundle\Model\TokenInterface;
 use FOS\OAuthServerBundle\Model\TokenManager as BaseTokenManager;
 
+use function time;
+
 class TokenManager extends BaseTokenManager
 {
     /**
@@ -88,8 +90,8 @@ public function deleteExpired(): int
         $qb = $this->repository->createQueryBuilder('t');
         $qb
             ->delete()
-            ->where('t.expiresAt < ?1')
-            ->setParameters([1 => time()])
+            ->where('t.expiresAt < :time')
+            ->setParameter('time', time())
         ;
 
         return $qb->getQuery()->execute();

diff --git a/Tests/Entity/AuthCodeManagerTest.php b/Tests/Entity/AuthCodeManagerTest.php
@@ -170,14 +170,14 @@ public function testDeleteExpired(): void
         $queryBuilder
             ->expects($this->once())
             ->method('where')
-            ->with('a.expiresAt < ?1')
+            ->with('a.expiresAt < :time')
             ->willReturn($queryBuilder)
         ;
 
         $queryBuilder
             ->expects($this->once())
-            ->method('setParameters')
-            ->with([1 => time()])
+            ->method('setParameter')
+            ->with('time', time())
             ->willReturn($queryBuilder)
         ;
 
@@ -202,4 +202,4 @@ public function testDeleteExpired(): void
 
         $this->assertSame($randomResult, $this->instance->deleteExpired());
     }
-}
+}
diff --git a/Tests/Entity/TokenManagerTest.php b/Tests/Entity/TokenManagerTest.php
@@ -177,14 +177,14 @@ public function testDeleteExpired(): void
         $queryBuilder
             ->expects($this->once())
             ->method('where')
-            ->with('t.expiresAt < ?1')
+            ->with('t.expiresAt < :time')
             ->willReturn($queryBuilder)
         ;
 
         $queryBuilder
             ->expects($this->once())
-            ->method('setParameters')
-            ->with([1 => time()])
+            ->method('setParameter')
+            ->with('time', time())
             ->willReturn($queryBuilder)
         ;