Avoid setting null when an string is required

thephpleague · Apr 22, 2024 · f9c0b4f · f9c0b4f
1 parent 3482a76
commit f9c0b4f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/Repository/AccessTokenRepository.php b/src/Repository/AccessTokenRepository.php
@@ -47,7 +47,9 @@ public function getNewToken(ClientEntityInterface $clientEntity, array $scopes,
         /** @var int|string|null $userIdentifier */
         $accessToken = new AccessTokenEntity();
         $accessToken->setClient($clientEntity);
-        $accessToken->setUserIdentifier($userIdentifier);
+        if (null !== $userIdentifier) {
+            $accessToken->setUserIdentifier((string) $userIdentifier);
+        }
 
         foreach ($scopes as $scope) {
             $accessToken->addScope($scope);

diff --git a/tests/TestHelper.php b/tests/TestHelper.php
@@ -82,7 +82,7 @@ public static function generateJwtToken(AccessTokenModel $accessToken): string
         $accessTokenEntity->setIdentifier($accessToken->getIdentifier());
         $accessTokenEntity->setExpiryDateTime($accessToken->getExpiry());
         $accessTokenEntity->setClient($clientEntity);
-        $accessTokenEntity->setUserIdentifier($accessToken->getUserIdentifier());
+        $accessTokenEntity->setUserIdentifier((string) $accessToken->getUserIdentifier());
 
         foreach ($accessToken->getScopes() as $scope) {
             $scopeEntity = new ScopeEntity();