Mado\QueryBundle\Services\QueryBuilder

Complexity

Total Complexity

2

Size/Duplication

Total Lines	44
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	15	1
A	getRepository()	0	13	1

<?php

namespace Mado\QueryBundle\Services;

use Doctrine\ORM\EntityManager;
use Mado\QueryBundle\Interfaces\CustomFilter;
use Psr\Log\LoggerInterface;
use Symfony\Component\HttpFoundation\RequestStack;
use Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage;

The type Symfony\Component\Security\Core\Authentication\Token\Storage\TokenStorage was not found. Maybe you did not declare it correctly or list all dependencies?

final class QueryBuilder
{
    private $customFilter;

    private $storage;

    private $token;

    private $user;

    private $request;

    private $logger;

    public function __construct(
        CustomFilter $customFilter,
        TokenStorage $storage,
        EntityManager $manager,
        RequestStack $requestStack,
        LoggerInterface $logger
    ) {
        $this->customFilter = $customFilter;
        $this->storage      = $storage;
        $this->manager      = $manager;

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

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

        $this->token = $this->storage->getToken();
        $this->user  = $this->token->getUser();
    }

    public function getRepository(string $rootEntity)
    {
        $this->rootEntity = $rootEntity;

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

        $additionalFilters = $this->customFilter
            ->setUser($this->user)
            ->allItemsTo($rootEntity);

        return $this->manager
            ->getRepository($this->rootEntity)
            ->setRequestWithFilter(
                $this->request,
                $additionalFilters
            );
    }
}