ModuleAdmin - Code Metrics - laravelflare/flare - Measure and Improve Code Quality continuously with Scrutinizer

ModuleAdmin A
last analyzed 2019-02-11 23:01 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

Metric	Value
wmc	5
c	0
b	0
f	0
lcom	1
cbo	1
dl	0
loc	51
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	getView()	0	20	5

<?php

namespace LaravelFlare\Flare\Admin\Modules;

use LaravelFlare\Flare\Admin\Admin;

abstract class ModuleAdmin extends Admin
{
    /**
     * The Controller to be used by the Module Admin.
     *
     * This defaults to parent::getController()
     * if it has been left undefined. 
     * 
     * @var string
     */
    protected $controller = '\LaravelFlare\Flare\Admin\Modules\ModuleAdminController';

    /**
     * The Module Admin Default View.
     *
     * @var string
     */
    protected $view = 'admin.modules.index';

    /**
     * Returns the Module Admin View.
     *
     * Determines if a view exists by:
     * Looking for $this->view
     * Then looks for  'admin.modulename.index',
     * Then looks for  'admin.modulename',
     * Then defaults to 
     * 
     * @return string
     */
    public function getView()
    {
        if (view()->exists($this->view)) {
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.
}
            return $this->view;
        }

        if (view()->exists('admin.'.$this->urlPrefix().'.index')) {
            return 'admin.'.$this->urlPrefix().'.index';
        }

        if (view()->exists('admin.'.$this->urlPrefix())) {
            return 'admin.'.$this->urlPrefix();
        }

        if (view()->exists('flare::'.$this->view)) {
            return 'flare::'.$this->view;
        }

        return parent::getView();
    }
}


1			<?php
2
3			namespace LaravelFlare\Flare\Admin\Modules;
4
5			use LaravelFlare\Flare\Admin\Admin;
6
7			abstract class ModuleAdmin extends Admin
8			{
9			/**
10			* The Controller to be used by the Module Admin.
11			*
12			* This defaults to parent::getController()
13			* if it has been left undefined.
14			*
15			* @var string
16			*/
17			protected $controller = '\LaravelFlare\Flare\Admin\Modules\ModuleAdminController';
18
19			/**
20			* The Module Admin Default View.
21			*
22			* @var string
23			*/
24			protected $view = 'admin.modules.index';
25
26			/**
27			* Returns the Module Admin View.
28			*
29			* Determines if a view exists by:
30			* Looking for $this->view
31			* Then looks for 'admin.modulename.index',
32			* Then looks for 'admin.modulename',
33			* Then defaults to
34			*
35			* @return string
36			*/
37			public function getView()
38			{
39			if (view()->exists($this->view)) {
			0 ignored issues – show Bug introduced 2015-11-29 15:51 UTC by Report Bug Copy Issue Report The method `exists` does only exist in `Illuminate\Contracts\View\Factory`, but not in `Illuminate\View\View`. 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...
40			return $this->view;
41			}
42
43			if (view()->exists('admin.'.$this->urlPrefix().'.index')) {
44			return 'admin.'.$this->urlPrefix().'.index';
45			}
46
47			if (view()->exists('admin.'.$this->urlPrefix())) {
48			return 'admin.'.$this->urlPrefix();
49			}
50
51			if (view()->exists('flare::'.$this->view)) {
52			return 'flare::'.$this->view;
53			}
54
55			return parent::getView();
56			}
57			}
58

laravelflare / flare

GitHub Access Token became invalid

ModuleAdmin A last analyzed 2019-02-11 23:01 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Available Fixes

Duplication Side-by-Side

Filter issues like

ModuleAdmin A
last analyzed 2019-02-11 23:01 UTC