Kunstmaan\CacheBundle\Helper\Menu\VarnishMenuAdaptor

Complexity

Total Complexity

4

Size/Duplication

Total Lines	28
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	adaptChildren()	17	17	4

<?php

namespace Kunstmaan\CacheBundle\Helper\Menu;

use Kunstmaan\AdminBundle\Helper\Menu\MenuAdaptorInterface;
use Kunstmaan\AdminBundle\Helper\Menu\MenuBuilder;
use Kunstmaan\AdminBundle\Helper\Menu\MenuItem;
use Symfony\Component\HttpFoundation\Request;

class VarnishMenuAdaptor implements MenuAdaptorInterface

This class seems to be duplicated in your project.

{
    /**
     * 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) && 'KunstmaanAdminBundle_settings' == $parent->getRoute()) {
            $menuItem = new MenuItem($menu);
            $menuItem
                ->setRoute('kunstmaancachebundle_varnish_settings_ban')
                ->setLabel('Varnish ban')
                ->setUniqueId('varnish_ban')
                ->setParent($parent);

            if (stripos($request->attributes->get('_route'), $menuItem->getRoute()) === 0) {
                $menuItem->setActive(true);
                $parent->setActive(true);
            }
            $children[] = $menuItem;
        }
    }
}