FSi\Bundle\AdminBundle\Menu\KnpMenu\ElementVoter

Complexity

Total Complexity

22

Size/Duplication

Total Lines	134
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	setRequest()	0	4	1
C	matchItem()	0	28	7
A	validateRequestElement()	0	17	4
A	getRequestElement()	0	4	1
A	isRouteMatchingElement()	0	5	2
A	isRouteMatchingElementDirectly()	0	8	2
A	isRouteMatchingElementAfterSuccess()	0	17	4

<?php

/**
 * (c) FSi sp. z o.o. <info@fsi.pl>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace FSi\Bundle\AdminBundle\Menu\KnpMenu;

use FSi\Bundle\AdminBundle\Admin\DependentElement;
use FSi\Bundle\AdminBundle\Admin\Element;
use FSi\Bundle\AdminBundle\Admin\ManagerInterface;
use FSi\Bundle\AdminBundle\Admin\RedirectableElement;
use Knp\Menu\ItemInterface;
use Knp\Menu\Matcher\Voter\VoterInterface;
use Symfony\Component\HttpFoundation\Request;

class ElementVoter implements VoterInterface
{
    /**
     * @var ManagerInterface
     */
    private $manager;

    /**
     * @var Request
     */
    private $request;

    public function __construct(ManagerInterface $manager)
    {
        $this->manager = $manager;
    }

    /**
     * @param Request $request
     */
    public function setRequest(Request $request)

You have injected the Request via parameter $request. This is generally not recommended as there might be multiple instances during a request cycle (f.e. when using sub-requests). Instead, it is recommended to inject the RequestStack and retrieve the current request each time you need it via getCurrentRequest().

    {
        $this->request = $request;
    }

    /**
     * @inheritdoc
     */
    public function matchItem(ItemInterface $item)
    {
        if (!$this->validateRequestElement()) {
            return null;
        }

        $element = $this->getRequestElement();

        while (true) {
            foreach ($item->getExtra('routes', []) as $testedRoute) {
                if ($this->isRouteMatchingElement($element, $testedRoute['parameters'])) {
                    return true;
                }
            }

            if (!($element instanceof DependentElement)) {
                break;
            }

            if (!$this->manager->hasElement($element->getParentId())) {
                break;
            }

            $element = $this->manager->getElement($element->getParentId());
        }

        return false;
    }

    /**
     * @return bool
     */
    private function validateRequestElement()
    {
        if (empty($this->request->attributes)) {
            return false;
        }

        if (!$this->request->attributes->has('element')) {
            return false;
        }

        $element = $this->request->attributes->get('element');
        if (!($element instanceof Element)) {
            return false;
        }

        return true;
    }

    /**
     * @return Element
     */
    private function getRequestElement()
    {
        return $this->request->attributes->get('element');
    }

    /**
     * @param Element $element
     * @param array $testedRouteParameters
     * @return bool
     */
    private function isRouteMatchingElement(Element $element, array $testedRouteParameters)
    {
        return $this->isRouteMatchingElementDirectly($element, $testedRouteParameters) ||
            $this->isRouteMatchingElementAfterSuccess($element, $testedRouteParameters);
    }

    /**
     * @param Element $element
     * @param array $testedRouteParameters
     * @return bool
     */
    private function isRouteMatchingElementDirectly(Element $element, array $testedRouteParameters)
    {
        if (!isset($testedRouteParameters['element'])) {
            return false;
        }

        return $testedRouteParameters['element'] === $element->getId();
    }

    /**
     * @param Element $element
     * @param array $testedRouteParameters
     * @return bool
     */
    private function isRouteMatchingElementAfterSuccess(Element $element, array $testedRouteParameters)
    {
        if (!($element instanceof RedirectableElement)) {
            return false;
        }

        if (!isset($testedRouteParameters['element'])) {
            return false;
        }

        $successParameters = $element->getSuccessRouteParameters();
        if (!isset($successParameters['element'])) {
            return false;
        }

        return $successParameters['element'] === $testedRouteParameters['element'];
    }
}