Kunstmaan\AdminBundle\Helper\Menu\SettingsMenuAdaptor

Complexity

Total Complexity

10

Size/Duplication

Total Lines	76
Duplicated Lines	17.11 %

Coupling/Cohesion

Components	1
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
B	adaptChildren()	13	43	9

<?php

namespace Kunstmaan\AdminBundle\Helper\Menu;

use Symfony\Component\Security\Core\Authorization\AuthorizationCheckerInterface;
use Symfony\Component\HttpFoundation\Request;

/**
 * SettingsMenuAdaptor to add the Settings MenuItem to the top menu and build the Settings tree
 */
class SettingsMenuAdaptor implements MenuAdaptorInterface
{
    /**
     * @var AuthorizationCheckerInterface
     */
    private $authorizationChecker;

    /**
     * @var bool
     */
    private $isEnabledVersionChecker;

    /**
     * Constructor
     *
     * @param AuthorizationCheckerInterface $authorizationChecker
     * @param bool                          $isEnabledVersionChecker
     */
    public function __construct(AuthorizationCheckerInterface $authorizationChecker, $isEnabledVersionChecker)
    {
        $this->authorizationChecker = $authorizationChecker;
        $this->isEnabledVersionChecker = (bool) $isEnabledVersionChecker;
    }

    /**
     * In this method you can add children for a specific parent, but also remove and change the already created children
     *
     * @param MenuBuilder $menu      The MenuBuilder
     * @param MenuItem[]  &$children The current children
     * @param MenuItem    $parent    The parent Menu item

Should the type for parameter $parent not be null|MenuItem?

     * @param Request     $request   The Request

Should the type for parameter $request not be null|Request?

     */
    public function adaptChildren(MenuBuilder $menu, array &$children, MenuItem $parent = null, Request $request = null)
    {
        if (is_null($parent)) {

This code seems to be duplicated across your project.

            $menuItem = new TopMenuItem($menu);
            $menuItem
                ->setRoute('KunstmaanAdminBundle_settings')
                ->setLabel('settings.title')
                ->setUniqueId('settings')
                ->setParent($parent)
                ->setRole('settings');
            if (stripos($request->attributes->get('_route'), $menuItem->getRoute()) === 0) {
                $menuItem->setActive(true);
            }
            $children[] = $menuItem;
        }

        if (!is_null($parent) && 'KunstmaanAdminBundle_settings' == $parent->getRoute()) {
            if ($this->authorizationChecker->isGranted('ROLE_SUPER_ADMIN') && $this->isEnabledVersionChecker) {
                $menuItem = new MenuItem($menu);
                $menuItem
                    ->setRoute('KunstmaanAdminBundle_settings_bundle_version')
                    ->setLabel('settings.version.bundle')
                    ->setUniqueId('bundle_versions')
                    ->setParent($parent);
                if (stripos($request->attributes->get('_route'), $menuItem->getRoute()) === 0) {
                    $menuItem->setActive(true);
                }
                $children[] = $menuItem;
            }

            $menuItem = new MenuItem($menu);
            $menuItem
                ->setRoute('kunstmaanadminbundle_admin_exception')
                ->setLabel('settings.exceptions.title')
                ->setUniqueId('exceptions')
                ->setParent($parent);
            if (stripos($request->attributes->get('_route'), $menuItem->getRoute()) === 0) {
                $menuItem->setActive(true);
                $parent->setActive(true);
            }
            $children[] = $menuItem;
        }
    }
}