BackMenus::register() - Code Metrics - spatie-custom/blender - Measure and Improve Code Quality continuously with Scrutinizer

BackMenus::register() A
last analyzed 2017-01-20 14:45 UTC

↳ Parent: BackMenus

Complexity

Conditions	2
Paths	1

Size

Total Lines	63
Code Lines	45

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	45
nc	1
nop	0
dl	0
loc	63
rs	9.4347
c	0
b	0
f	0

How to fix Long Method

<?php

namespace App\Services\Navigation\Menu;

use Html;
use Spatie\Menu\Laravel\Link;
use Spatie\Menu\Laravel\Menu;

class BackMenus
{
    public function register()
    {
        Menu::macro('back', function () {
            return Menu::new()
                ->setActiveClass('-active')
                ->setActiveFromRequest('/blender');
        });

        Menu::macro('moduleGroup', function ($title) {
            return Menu::back()
                ->addParentClass('menu__group')
                ->setParentAttribute('data-menu-group', fragment("back.nav.{$title}"))
                ->registerFilter(function (Link $link) {
                    $link->addParentClass('menu__group__item');
                });
        });

        Menu::macro('module', function (string $action, string $name) {
            return $this->action("Back\\{$action}", fragment("back.{$name}"));

        });

        Menu::macro('backMain', function () {
            return Menu::back()
                ->addClass('menu__groups')
                ->setAttribute('data-menu-groups')
                ->add(Menu::moduleGroup('content')
                    ->module('ArticlesController@index', 'articles.title')
                    ->module('NewsController@index', 'news.title')
                    ->module('PeopleController@index', 'people.title'))
                ->add(Menu::moduleGroup('modules')
                    ->module('FragmentsController@index', 'fragments.title')
                    ->module('FormResponsesController@showDownloadButton', 'formResponses.title')
                    ->module('TagsController@index', 'tags.title'))
                ->add(Menu::moduleGroup('users')
                    ->module('MembersController@index', 'members.title')
                    ->module('AdministratorsController@index', 'administrators.title'))
                ->add(Menu::moduleGroup('system')
                    ->module('ActivitylogController@index', 'log.title')
                    ->module('RedirectsController@index', 'redirects.title')
                    ->module('StatisticsController@index', 'statistics.menuTitle'));
        });

        Menu::macro('backUser', function () {
            $avatar = Html::avatar(current_user(), '-small').
                el('span.:response-desktop-only', current_user()->email);

            return Menu::new()
                ->action('Back\AdministratorsController@edit', $avatar, [current_user()->id])
                ->html(view('back.auth._partials.logoutForm'));
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        });

        Menu::macro('breadcrumbs', function (array $breadcrumbs) {
            return Menu::build($breadcrumbs, function (Menu $menu, $actionWithParameters, $label) {
                if (! is_array($actionWithParameters)) {
                    $actionWithParameters = [$actionWithParameters];
                }

                $action = array_shift($actionWithParameters);

                return $menu->action($action, $label, $actionWithParameters);
            })->addClass('breadcrumb')->setActiveFromRequest('/blender');
        });
    }
}


1			<?php
2
3			namespace App\Services\Navigation\Menu;
4
5			use Html;
6			use Spatie\Menu\Laravel\Link;
7			use Spatie\Menu\Laravel\Menu;
8
9			class BackMenus
10			{
11			public function register()
12			{
13			Menu::macro('back', function () {
14			return Menu::new()
15			->setActiveClass('-active')
16			->setActiveFromRequest('/blender');
17			});
18
19			Menu::macro('moduleGroup', function ($title) {
20			return Menu::back()
21			->addParentClass('menu__group')
22			->setParentAttribute('data-menu-group', fragment("back.nav.{$title}"))
23			->registerFilter(function (Link $link) {
24			$link->addParentClass('menu__group__item');
25			});
26			});
27
28			Menu::macro('module', function (string $action, string $name) {
29			return $this->action("Back\\{$action}", fragment("back.{$name}"));
			0 ignored issues – show Bug introduced 2016-04-11 15:15 UTC by Report Bug Copy Issue Report The method `action()` does not seem to exist on `object<App\Services\Navigation\Menu\BackMenus>`. 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...
30			});
31
32			Menu::macro('backMain', function () {
33			return Menu::back()
34			->addClass('menu__groups')
35			->setAttribute('data-menu-groups')
36			->add(Menu::moduleGroup('content')
37			->module('ArticlesController@index', 'articles.title')
38			->module('NewsController@index', 'news.title')
39			->module('PeopleController@index', 'people.title'))
40			->add(Menu::moduleGroup('modules')
41			->module('FragmentsController@index', 'fragments.title')
42			->module('FormResponsesController@showDownloadButton', 'formResponses.title')
43			->module('TagsController@index', 'tags.title'))
44			->add(Menu::moduleGroup('users')
45			->module('MembersController@index', 'members.title')
46			->module('AdministratorsController@index', 'administrators.title'))
47			->add(Menu::moduleGroup('system')
48			->module('ActivitylogController@index', 'log.title')
49			->module('RedirectsController@index', 'redirects.title')
50			->module('StatisticsController@index', 'statistics.menuTitle'));
51			});
52
53			Menu::macro('backUser', function () {
54			$avatar = Html::avatar(current_user(), '-small').
55			el('span.:response-desktop-only', current_user()->email);
56
57			return Menu::new()
58			->action('Back\AdministratorsController@edit', $avatar, [current_user()->id])
59			->html(view('back.auth._partials.logoutForm'));
			0 ignored issues – show Documentation introduced 2016-08-27 00:12 UTC by Report Bug Copy Issue Report `view('back.auth._partials.logoutForm')` is of type `object<Illuminate\View\V...Contracts\View\Factory>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
60			});
61
62			Menu::macro('breadcrumbs', function (array $breadcrumbs) {
63			return Menu::build($breadcrumbs, function (Menu $menu, $actionWithParameters, $label) {
64			if (! is_array($actionWithParameters)) {
65			$actionWithParameters = [$actionWithParameters];
66			}
67
68			$action = array_shift($actionWithParameters);
69
70			return $menu->action($action, $label, $actionWithParameters);
71			})->addClass('breadcrumb')->setActiveFromRequest('/blender');
72			});
73			}
74			}
75

spatie-custom / blender

BackMenus::register() A last analyzed 2017-01-20 14:45 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like

BackMenus::register() A
last analyzed 2017-01-20 14:45 UTC