LoginCidadao\OAuthBundle\Model\AccessTokenManager

Complexity

Total Complexity

9

Size/Duplication

Total Lines	70
Duplicated Lines	0 %

Test Coverage

Coverage

15.38%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getTokenScope()	0	12	3
A	getToken()	0	11	3
A	__construct()	0	5	1
A	getTokenClient()	0	5	1
A	hasToken()	0	4	1

<?php
/*
 * This file is part of the login-cidadao project or it's bundles.
 *
 * (c) Guilherme Donato <guilhermednt on github>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace LoginCidadao\OAuthBundle\Model;

use Doctrine\ORM\EntityManager;
use LoginCidadao\OAuthBundle\Entity\AccessToken;
use FOS\OAuthServerBundle\Security\Authentication\Token\OAuthToken;
use Symfony\Component\Security\Core\Exception\AccessDeniedException;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorageInterface;

class AccessTokenManager
{
    /** @var EntityManager */
    private $em;

    /** @var TokenStorageInterface */
    private $tokenStorage;

    public function __construct(EntityManager $em,

The first parameter of a multi-line function declaration must be on the line after the opening bracket

                                TokenStorageInterface $tokenStorage)

Multi-line function declaration not indented correctly; expected 8 spaces but found 32

The closing parenthesis of a multi-line function declaration must be on a new line

    {

The closing parenthesis and the opening brace of a multi-line function declaration must be on the same line

        $this->em           = $em;
        $this->tokenStorage = $tokenStorage;
    }

    /**
     * @return boolean
     */
    public function hasToken()
    {
        $securityToken = $this->tokenStorage->getToken();
        return ($securityToken instanceof OAuthToken);
    }

    /**
     * @param string $token
     * @throws AccessDeniedException when no token is found in the current session
     * @return AccessToken
     */
    public function getToken($token = null)
    {
        if ($token === null) {
            $securityToken = $this->tokenStorage->getToken();
            if (!($securityToken instanceof OAuthToken)) {
                throw new AccessDeniedException("Couldn't find an AccessToken in the current session.");
            }
            $token = $securityToken->getToken();
        }
        $repo = $this->em->getRepository('LoginCidadaoOAuthBundle:AccessToken');
        return $repo->findOneByToken($token);

The method findOneByToken() does not exist on LoginCidadao\OAuthBundle\Entity\AccessTokenRepository. Since you implemented __call, consider adding a @method annotation.

The expression return $repo->findOneByToken($token) also could return the type array which is incompatible with the documented return type LoginCidadao\OAuthBundle\Entity\AccessToken.

    }

    /**
     * @param string $token defaults to the current AccessToken
     * @return array
     */
    public function getTokenScope($token = null)
    {
        $accessToken = $this->getToken($token);

        $scope = $accessToken->getScope();

        if (is_string($scope)) {

The condition is_string($scope) is always true.

            return explode(' ', $scope);
        } elseif (is_array($scope)) {
            return $scope;
        } else {
            return null;
        }
    }

    /**
     * @param string $token
     * @return ClientInterface
     */
    public function getTokenClient($token = null)
    {
        $accessToken = $this->getToken($token);

        return $accessToken->getClient();
    }
}