MenuSubscriber::__construct() - Code Metrics - Inspection of "Merge pull request #798 from OpencachingDeutschlan..." - OpencachingDeutschland/oc-server3 - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — development ( 9736ae...1c21f9 )

by Mirco

created 2020-11-10 20:18 UTC

MenuSubscriber::__construct() A

↳ Parent: MenuSubscriber

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
nc	1
nop	1
dl	0
loc	4
rs	10
c	0
b	0
f	0

<?php

declare(strict_types=1);

namespace Oc\Components\Subscriber;

use KevinPapst\AdminLTEBundle\Event\KnpMenuEvent;
use Oc\Entity\UserEntity;
use Symfony\Component\EventDispatcher\EventSubscriberInterface;
use Symfony\Component\Security\Core\Security;

class MenuSubscriber implements EventSubscriberInterface
{
    /**
     * @var Security
     */
    private $security;

    public function __construct(Security $security)
    {
        $this->security = $security;
    }

    public static function getSubscribedEvents(): array
    {
        return [
            KnpMenuEvent::class => ['onSetupMenu', 100],
        ];
    }

    public function onSetupMenu(KnpMenuEvent $event)
    {
        $menu = $event->getMenu();

        $menu->addChild('MainNavigationMenuItem', [
            'label' => 'MAIN NAVIGATION',
            'childOptions' => $event->getChildOptions()
        ])->setAttribute('class', 'header');



        if ($this->security->isGranted("CAN_VIEW", UserEntity::class)) {
            $cacheMenu = $menu->addChild('cache', [
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
                'label' => 'Caches',
                'route' => 'backend_user_index',
                'childOptions' => $event->getChildOptions(),
            ])->setLabelAttribute('icon', 'fas fa-map-marker-alt');
        }

        if ($this->security->isGranted("CAN_VIEW", UserEntity::class)) {
            $userMenu = $menu->addChild('user', [
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
                'label' => 'Benutzer',
                'route' => 'backend_user_index',
                'childOptions' => $event->getChildOptions(),
            ])->setLabelAttribute('icon', 'fas fa-users');
        }
    }
}


1		<?php
2
3		declare(strict_types=1);
4
5		namespace Oc\Components\Subscriber;
6
7		use KevinPapst\AdminLTEBundle\Event\KnpMenuEvent;
8		use Oc\Entity\UserEntity;
9		use Symfony\Component\EventDispatcher\EventSubscriberInterface;
10		use Symfony\Component\Security\Core\Security;
11
12		class MenuSubscriber implements EventSubscriberInterface
13		{
14		/**
15		* @var Security
16		*/
17		private $security;
18
19		public function __construct(Security $security)
20		{
21		$this->security = $security;
22		}
23
24		public static function getSubscribedEvents(): array
25		{
26		return [
27		KnpMenuEvent::class => ['onSetupMenu', 100],
28		];
29		}
30
31		public function onSetupMenu(KnpMenuEvent $event)
32		{
33		$menu = $event->getMenu();
34
35		$menu->addChild('MainNavigationMenuItem', [
36		'label' => 'MAIN NAVIGATION',
37		'childOptions' => $event->getChildOptions()
38		])->setAttribute('class', 'header');
39
40
41
42	View Code Duplication	if ($this->security->isGranted("CAN_VIEW", UserEntity::class)) {
43		$cacheMenu = $menu->addChild('cache', [
		0 ignored issues – show Unused Code introduced 2020-11-10 19:18 UTC by Report Bug Copy Issue Report `$cacheMenu` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
44		'label' => 'Caches',
45		'route' => 'backend_user_index',
46		'childOptions' => $event->getChildOptions(),
47		])->setLabelAttribute('icon', 'fas fa-map-marker-alt');
48		}
49
50	View Code Duplication	if ($this->security->isGranted("CAN_VIEW", UserEntity::class)) {
51		$userMenu = $menu->addChild('user', [
		0 ignored issues – show Unused Code introduced 2020-11-10 19:18 UTC by Report Bug Copy Issue Report `$userMenu` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
52		'label' => 'Benutzer',
53		'route' => 'backend_user_index',
54		'childOptions' => $event->getChildOptions(),
55		])->setLabelAttribute('icon', 'fas fa-users');
56		}
57		}
58		}
59

OpencachingDeutschland / oc-server3

Push — development ( 9736ae...1c21f9 )

MenuSubscriber::__construct() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like