AdminLte::buildMenu() - Code Metrics - Inspection of "Merge pull request #307 from jeroennoten/update-ad..." - jeroennoten/Laravel-AdminLTE - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( bbce29...ca7080 )

unknown

created 2019-07-01 14:07 UTC

AdminLte::buildMenu() A

↳ Parent: AdminLte

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	5
CRAP Score	2.0185

Importance

Changes

Metric	Value
dl	0
loc	12
ccs	5
cts	6
cp	0.8333
rs	9.8666
c	0
b	0
f	0
cc	2
nc	2
nop	0
crap	2.0185

<?php

namespace JeroenNoten\LaravelAdminLte;

use Illuminate\Contracts\Events\Dispatcher;
use Illuminate\Contracts\Container\Container;
use JeroenNoten\LaravelAdminLte\Menu\Builder;
use JeroenNoten\LaravelAdminLte\Events\BuildingMenu;

class AdminLte
{
    protected $menu;

    protected $filters;

    protected $events;

    protected $container;

    public function __construct(
        array $filters,
        Dispatcher $events,
        Container $container
    ) {
        $this->filters = $filters;
        $this->events = $events;
        $this->container = $container;
    }

    public function menu()
    {
        if (! $this->menu) {
            $this->menu = $this->buildMenu();
        }

        return $this->menu;
    }

    protected function buildMenu()
    {
        $builder = new Builder($this->buildFilters());

        if (method_exists($this->events, 'dispatch')) {
            $this->events->dispatch(new BuildingMenu($builder));
        } else {
            $this->events->fire(new BuildingMenu($builder));

        }

        return $builder->menu;
    }

    protected function buildFilters()
    {
        return array_map([$this->container, 'make'], $this->filters);
    }
}


1		<?php
2
3		namespace JeroenNoten\LaravelAdminLte;
4
5		use Illuminate\Contracts\Events\Dispatcher;
6		use Illuminate\Contracts\Container\Container;
7		use JeroenNoten\LaravelAdminLte\Menu\Builder;
8		use JeroenNoten\LaravelAdminLte\Events\BuildingMenu;
9
10		class AdminLte
11		{
12		protected $menu;
13
14		protected $filters;
15
16		protected $events;
17
18		protected $container;
19
20	1	public function __construct(
21		array $filters,
22		Dispatcher $events,
23		Container $container
24		) {
25	1	$this->filters = $filters;
26	1	$this->events = $events;
27	1	$this->container = $container;
28	1	}
29
30	1	public function menu()
31		{
32	1	if (! $this->menu) {
33	1	$this->menu = $this->buildMenu();
34		}
35
36	1	return $this->menu;
37		}
38
39	1	protected function buildMenu()
40		{
41	1	$builder = new Builder($this->buildFilters());
42
43	1	if (method_exists($this->events, 'dispatch')) {
44		$this->events->dispatch(new BuildingMenu($builder));
45		} else {
46	1	$this->events->fire(new BuildingMenu($builder));
		0 ignored issues – show Bug introduced 2019-02-28 08:27 UTC by Report Bug Copy Issue Report The method `fire()` does not seem to exist on `object<Illuminate\Contracts\Events\Dispatcher>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
47		}
48
49	1	return $builder->menu;
50		}
51
52	1	protected function buildFilters()
53		{
54	1	return array_map([$this->container, 'make'], $this->filters);
55		}
56		}
57

jeroennoten / Laravel-AdminLTE

Push — master ( bbce29...ca7080 )

AdminLte::buildMenu() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like