From 9614e7468710a2cd9786db20d0ef0a6ea28ba165 Mon Sep 17 00:00:00 2001
From: Maximilian Beck <contact@maximilianbeck.de>
Date: Mon, 14 Dec 2020 18:45:13 +0100
Subject: [PATCH] #255: Set revoked as parameter

- Fixes an issue with postgres where dbal converts the boolean values to smallint if passed directly in the `set()`-method
---
 .../DoctrineCredentialsRevoker.php             | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/Service/CredentialsRevoker/DoctrineCredentialsRevoker.php b/Service/CredentialsRevoker/DoctrineCredentialsRevoker.php
index 9158e706..acf02e5f 100644
--- a/Service/CredentialsRevoker/DoctrineCredentialsRevoker.php
+++ b/Service/CredentialsRevoker/DoctrineCredentialsRevoker.php
@@ -30,7 +30,8 @@ public function revokeCredentialsForUser(UserInterface $user): void
 
         $this->entityManager->createQueryBuilder()
             ->update(AccessToken::class, 'at')
-            ->set('at.revoked', true)
+            ->set('at.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where('at.userIdentifier = :userIdentifier')
             ->setParameter('userIdentifier', $userIdentifier)
             ->getQuery()
@@ -39,7 +40,8 @@ public function revokeCredentialsForUser(UserInterface $user): void
         $queryBuilder = $this->entityManager->createQueryBuilder();
         $queryBuilder
             ->update(RefreshToken::class, 'rt')
-            ->set('rt.revoked', true)
+            ->set('rt.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where($queryBuilder->expr()->in(
                 'rt.accessToken',
                 $this->entityManager->createQueryBuilder()
@@ -54,7 +56,8 @@ public function revokeCredentialsForUser(UserInterface $user): void
 
         $this->entityManager->createQueryBuilder()
             ->update(AuthorizationCode::class, 'ac')
-            ->set('ac.revoked', true)
+            ->set('ac.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where('ac.userIdentifier = :userIdentifier')
             ->setParameter('userIdentifier', $userIdentifier)
             ->getQuery()
@@ -69,7 +72,8 @@ public function revokeCredentialsForClient(Client $client): void
 
         $this->entityManager->createQueryBuilder()
             ->update(AccessToken::class, 'at')
-            ->set('at.revoked', true)
+            ->set('at.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where('at.client = :client')
             ->setParameter('client', $doctrineClient)
             ->getQuery()
@@ -77,7 +81,8 @@ public function revokeCredentialsForClient(Client $client): void
 
         $queryBuilder = $this->entityManager->createQueryBuilder();
         $queryBuilder->update(RefreshToken::class, 'rt')
-            ->set('rt.revoked', true)
+            ->set('rt.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where($queryBuilder->expr()->in(
                 'rt.accessToken',
                 $this->entityManager->createQueryBuilder()
@@ -92,7 +97,8 @@ public function revokeCredentialsForClient(Client $client): void
 
         $this->entityManager->createQueryBuilder()
             ->update(AuthorizationCode::class, 'ac')
-            ->set('ac.revoked', true)
+            ->set('ac.revoked', ':revoked')
+            ->setParameter('revoked', true)
             ->where('ac.client = :client')
             ->setParameter('client', $doctrineClient)
             ->getQuery()