Mado\QueryBundle\Services\ConfigProvider

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	getRequest()	0	3	1
A	__construct()	0	7	1
A	getUserRoles()	0	3	1
A	getUser()	0	3	1

<?php

namespace Mado\QueryBundle\Services;

use Mado\QueryBundle\Component\ConfigProvider as ConfigProviderInterface;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage;

class ConfigProvider implements ConfigProviderInterface
{
    private $user;

    private $request;

    public function __construct(
        RequestStack $requestStack,
        TokenStorage $tokenStorage
    ) {
        $this->token   = $tokenStorage->getToken();

The property token does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->user    = $this->token->getUser();

The method getUser() does not exist on null.

        $this->request = $requestStack->getCurrentRequest();
    }

    public function getUser()
    {
        return $this->user;
    }

    public function getRequest()
    {
        return $this->request;
    }

    public function getUserRoles()
    {
        return $this->token->getRoles();
    }
}