PermissionsCount::run() - Code Metrics - Inspection of "Widgets system" - devnile/microboard - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1)

by Mohamed

created 2020-07-03 02:05 UTC

PermissionsCount::run() A

↳ Parent: PermissionsCount

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

<?php

namespace Microboard\Widgets;

use Microboard\Foundations\Widget;
use Microboard\Models\Permission;
use Microboard\Models\Role;

class PermissionsCount extends Widget
{
    /**
     * The configuration array.
     *
     * @var array
     */
    protected $config = [
        'background' => 'bg-white',
        'icon-background' => 'red',
        'text' => 'dark'
    ];

    /**
     * Treat this method as a controller action.
     * Return view() or other content to display.
     */
    public function run()
    {
        return view('microboard::state', [
            'config' => $this->config,
            'title' => trans('microboard::pages.widgets.permissions-count.title'),
            'info' => trans('microboard::pages.widgets.permissions-count.info'),
            'icon' => 'ni ni-key-25',
            'count' => Permission::count()
        ]);
    }

    /**
     * Determine if the widget should be displayed.
     *
     * @return bool
     */
    public function shouldBeDisplayed()
    {
        return auth()->user()->can('viewAny', new Role);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
    }
}


1			<?php
2
3			namespace Microboard\Widgets;
4
5			use Microboard\Foundations\Widget;
6			use Microboard\Models\Permission;
7			use Microboard\Models\Role;
8
9			class PermissionsCount extends Widget
10			{
11			/**
12			* The configuration array.
13			*
14			* @var array
15			*/
16			protected $config = [
17			'background' => 'bg-white',
18			'icon-background' => 'red',
19			'text' => 'dark'
20			];
21
22			/**
23			* Treat this method as a controller action.
24			* Return view() or other content to display.
25			*/
26			public function run()
27			{
28			return view('microboard::state', [
29			'config' => $this->config,
30			'title' => trans('microboard::pages.widgets.permissions-count.title'),
31			'info' => trans('microboard::pages.widgets.permissions-count.info'),
32			'icon' => 'ni ni-key-25',
33			'count' => Permission::count()
34			]);
35			}
36
37			/**
38			* Determine if the widget should be displayed.
39			*
40			* @return bool
41			*/
42			public function shouldBeDisplayed()
43			{
44			return auth()->user()->can('viewAny', new Role);
			0 ignored issues – show Bug introduced 2020-07-03 02:14 UTC by Report Bug Copy Issue Report The method `user` does only exist in `Illuminate\Contracts\Auth\Guard`, but not in `Illuminate\Contracts\Auth\Factory`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
45			}
46			}
47

devnile / microboard

Pull Request — master (#1)

PermissionsCount::run() A

Complexity

Size

Duplication

Importance

Available Fixes

Duplication Side-by-Side

Filter issues like