EcommerceDashboard::AllPanels() - Code Metrics - sunnysideup/silverstripe-ecommerce_dashboard - Measure and Improve Code Quality continuously with Scrutinizer

EcommerceDashboard::AllPanels() A
last analyzed 2018-07-10 10:37 UTC

↳ Parent: EcommerceDashboard

Complexity

Conditions	5
Paths	5

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	5
nc	5
nop	0
dl	0
loc	15
rs	9.4555
c	0
b	0
f	0

<?php


class EcommerceDashboard extends Dashboard
namespace YourVendor;

class YourClass { }
{
    private static $menu_title = "Dashboard";


    private static $url_segment = "ecom-dashboard";


    private static $menu_priority = 100;


    private static $url_priority = 30;


    private static $menu_icon = "dashboard/images/dashboard.png";


    private static $tree_class = 'EcommerceDashboardPanel';


    /**
     * Gets all the available panels that can be installed on the dashboard. All subclasses of
     * {@link DashboardPanel} are included
     *
     * @return ArrayList
     */
    public function AllPanels()
    {
        $panels = parent::AllPanels();
        $al = ArrayList::create();
        foreach ($panels as $panel) {
            if ($panel instanceof EcommerceDashboardPanel) {
                if ($panel->registered()) {
                    if ($panel->ClassName !== 'EcommerceDashboardPanel') {
                        $al->push($panel);
                    }
                }
            }
        }
        return $al;
    }
}


1			<?php
2
3
4			class EcommerceDashboard extends Dashboard
			0 ignored issues – show Coding Style Compatibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions. You can fix this by adding a namespace to your class: namespace YourVendor; class YourClass { } When choosing a vendor namespace, try to pick something that is not too generic to avoid conflicts with other libraries. Loading history...
5			{
6			private static $menu_title = "Dashboard";
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$menu_title` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
7
8			private static $url_segment = "ecom-dashboard";
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$url_segment` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
9
10			private static $menu_priority = 100;
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$menu_priority` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
11
12			private static $url_priority = 30;
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$url_priority` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
13
14			private static $menu_icon = "dashboard/images/dashboard.png";
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$menu_icon` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
15
16			private static $tree_class = 'EcommerceDashboardPanel';
			0 ignored issues – show Comprehensibility introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report Consider using a different property name as you override a private property of the parent class. Loading history... Unused Code introduced 2017-01-03 00:41 UTC by Report Bug Copy Issue Report The property `$tree_class` is not used and could be removed. This check marks private properties in classes that are never used. Those properties can be removed. Loading history...
17
18			/**
19			* Gets all the available panels that can be installed on the dashboard. All subclasses of
20			* {@link DashboardPanel} are included
21			*
22			* @return ArrayList
23			*/
24			public function AllPanels()
25			{
26			$panels = parent::AllPanels();
27			$al = ArrayList::create();
28			foreach ($panels as $panel) {
29			if ($panel instanceof EcommerceDashboardPanel) {
30			if ($panel->registered()) {
31			if ($panel->ClassName !== 'EcommerceDashboardPanel') {
32			$al->push($panel);
33			}
34			}
35			}
36			}
37			return $al;
38			}
39			}
40

sunnysideup / silverstripe-ecommerce_dashboard

GitHub Access Token became invalid

EcommerceDashboard::AllPanels() A last analyzed 2018-07-10 10:37 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

EcommerceDashboard::AllPanels() A
last analyzed 2018-07-10 10:37 UTC