OAuth\Repository\AccessTokenRepository

Complexity

Total Complexity

7

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Test Coverage

Coverage

57.89%

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	revokeAccessToken()	0	9	2
A	getNewToken()	0	5	1
A	persistNewAccessToken()	0	5	1
A	isAccessTokenRevoked()	0	8	3

<?php

namespace OAuth\Repository;

use Doctrine\ORM\EntityRepository;
use Exception;
use League\OAuth2\Server\Entities\AccessTokenEntityInterface;
use League\OAuth2\Server\Entities\ClientEntityInterface;
use League\OAuth2\Server\Repositories\AccessTokenRepositoryInterface;
use OAuth\AccessToken;

class AccessTokenRepository extends EntityRepository implements AccessTokenRepositoryInterface
{
    /**
     * @param AccessTokenEntityInterface $accessTokenEntity
     * @return AccessTokenEntityInterface
     * @throws \Doctrine\ORM\OptimisticLockException
     */
    public function persistNewAccessToken(AccessTokenEntityInterface $accessTokenEntity): AccessTokenEntityInterface
    {
        $this->_em->persist($accessTokenEntity);
        $this->_em->flush();
        return $accessTokenEntity;
    }

    /**
     * @param string $tokenId
     * @throws Exception
     */
    public function revokeAccessToken($tokenId)
    {
        /** @var AccessToken $token */
        $token = $this->find($tokenId);
        if(!$token) {

$token is of type OAuth\AccessToken, thus it always evaluated to true.

            throw new Exception('Token not found', 404);
        }
        $token->setRevoked(true);
        $this->_em->flush($token);
    }

    /**
     * {@inheritdoc}
     */
    public function isAccessTokenRevoked($tokenId)
    {
        /** @var null|AccessToken $token */
        $token = $this->findOneBy(['identifier' => $tokenId]);
        if(!$token || $token->isRevoked()) {
            return true;
        }
        return false;
    }

    /**
     * @param ClientEntityInterface $clientEntity
     * @param array $scopes
     * @param null $userIdentifier

Are you sure the doc-type for parameter $userIdentifier is correct as it would always require null to be passed?

     * @return AccessTokenEntityInterface|AccessToken
     */
    public function getNewToken(ClientEntityInterface $clientEntity, array $scopes, $userIdentifier = null): AccessToken
    {
        $token =  new AccessToken();

        return $token;
    }
}