BrowserController::doTree() - Code Metrics - Inspection of "Tag v 6.0.0-beta.35" - HuasoFoundries/phpPgAdmin6 - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( cf0423...965dbc )

by Felipe

created 2018-04-17 12:55 UTC

BrowserController::doTree() A

↳ Parent: BrowserController

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	9

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	14
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	9
nc	1
nop	0

<?php

/**
 * PHPPgAdmin v6.0.0-beta.33
 */

namespace PHPPgAdmin\Controller;

use PHPPgAdmin\Decorators\Decorator;

/**
 * Base controller class.
 *
 * @package PHPPgAdmin
 */
class BrowserController extends BaseController
{
    public $controller_name = 'BrowserController';

    /**
     * Default method to render the controller according to the action parameter.
     */
    public function render($action = null)
    {
        if ($action === null) {
            $action = $this->action;
        }

        switch ($action) {
            case 'tree':
                return $this->doTree();

                break;
switch ($x) {
    case 1:
        return 'foo';
        break; // This break is not necessary and can be left off.
}
            case 'jstree':
                return $this->jsTree();
                break;

            default:
                return $this->doDefault();
                break;
        }
    }

    /**
     * Default method to render the controller according to the action parameter.
     */
    public function doDefault()
    {
        $this->misc->setNoDBConnection(true);

        $this->setNoBottomLink(true);

        $viewVars = ['icon' => [
            'blank'          => $this->misc->icon('blank'),
            'I'              => $this->misc->icon('I'),
            'L'              => $this->misc->icon('L'),
            'Lminus'         => $this->misc->icon('Lminus'),
            'Loading'        => $this->misc->icon('Loading'),
            'Lplus'          => $this->misc->icon('Lplus'),
            'ObjectNotFound' => $this->misc->icon('ObjectNotFound'),
            'Refresh'        => $this->misc->icon('Refresh'),
            'Servers'        => $this->misc->icon('Servers'),
            'T'              => $this->misc->icon('T'),
            'Tminus'         => $this->misc->icon('Tminus'),
            'Tplus'          => $this->misc->icon('Tplus'),
        ]];

        return $this->view->fetch('browser.twig', $viewVars);
    }

    /**
     * Default method to render the controller according to the action parameter.
     */
    public function jsTree()
    {
        $this->misc->setNoDBConnection(true);

        $this->setNoBottomLink(true);

        $viewVars = ['icon' => [
            'blank'          => $this->misc->icon('blank'),
            'I'              => $this->misc->icon('I'),
            'L'              => $this->misc->icon('L'),
            'Lminus'         => $this->misc->icon('Lminus'),
            'Loading'        => $this->misc->icon('Loading'),
            'Lplus'          => $this->misc->icon('Lplus'),
            'ObjectNotFound' => $this->misc->icon('ObjectNotFound'),
            'Refresh'        => $this->misc->icon('Refresh'),
            'Servers'        => $this->misc->icon('Servers'),
            'T'              => $this->misc->icon('T'),
            'Tminus'         => $this->misc->icon('Tminus'),
            'Tplus'          => $this->misc->icon('Tplus'),
        ]];

        return $this->view->fetch('jstree.twig', $viewVars);
    }

    public function doTree()
    {
        $treedata = new \PHPPgAdmin\ArrayRecordSet([]);
        $reqvars  = [];

        $attrs = [
            'text'    => 'Servers',
            'icon'    => 'Servers',
            'is_root' => 'true',
            'action'  => Decorator::url('/src/views/servers'),
            'branch'  => Decorator::url('/src/views/servers', $reqvars, ['action' => 'tree']),
        ];

        return $this->printTree($treedata, $attrs, 'server');
    }

}


1			<?php
2
3			/**
4			* PHPPgAdmin v6.0.0-beta.33
5			*/
6
7			namespace PHPPgAdmin\Controller;
8
9			use PHPPgAdmin\Decorators\Decorator;
10
11			/**
12			* Base controller class.
13			*
14			* @package PHPPgAdmin
15			*/
16			class BrowserController extends BaseController
17			{
18			public $controller_name = 'BrowserController';
19
20			/**
21			* Default method to render the controller according to the action parameter.
22			*/
23			public function render($action = null)
24			{
25			if ($action === null) {
26			$action = $this->action;
27			}
28
29			switch ($action) {
30			case 'tree':
31			return $this->doTree();
32
33			break;
			1 ignored issue – show Unused Code introduced 2018-04-17 12:58 UTC by Report Bug Copy Issue Report `break` is not strictly necessary here and could be removed. The `break` statement is not necessary if it is preceded for example by a `return` statement: switch ($x) { case 1: return 'foo'; break; // This break is not necessary and can be left off. } If you would like to keep this construct to be consistent with other `case` statements, you can safely mark this issue as a false-positive. Loading history...
34			case 'jstree':
35			return $this->jsTree();
36			break;
37
38			default:
39			return $this->doDefault();
40			break;
41			}
42			}
43
44			/**
45			* Default method to render the controller according to the action parameter.
46			*/
47			public function doDefault()
48			{
49			$this->misc->setNoDBConnection(true);
50
51			$this->setNoBottomLink(true);
52
53			$viewVars = ['icon' => [
54			'blank' => $this->misc->icon('blank'),
55			'I' => $this->misc->icon('I'),
56			'L' => $this->misc->icon('L'),
57			'Lminus' => $this->misc->icon('Lminus'),
58			'Loading' => $this->misc->icon('Loading'),
59			'Lplus' => $this->misc->icon('Lplus'),
60			'ObjectNotFound' => $this->misc->icon('ObjectNotFound'),
61			'Refresh' => $this->misc->icon('Refresh'),
62			'Servers' => $this->misc->icon('Servers'),
63			'T' => $this->misc->icon('T'),
64			'Tminus' => $this->misc->icon('Tminus'),
65			'Tplus' => $this->misc->icon('Tplus'),
66			]];
67
68			return $this->view->fetch('browser.twig', $viewVars);
69			}
70
71			/**
72			* Default method to render the controller according to the action parameter.
73			*/
74			public function jsTree()
75			{
76			$this->misc->setNoDBConnection(true);
77
78			$this->setNoBottomLink(true);
79
80			$viewVars = ['icon' => [
81			'blank' => $this->misc->icon('blank'),
82			'I' => $this->misc->icon('I'),
83			'L' => $this->misc->icon('L'),
84			'Lminus' => $this->misc->icon('Lminus'),
85			'Loading' => $this->misc->icon('Loading'),
86			'Lplus' => $this->misc->icon('Lplus'),
87			'ObjectNotFound' => $this->misc->icon('ObjectNotFound'),
88			'Refresh' => $this->misc->icon('Refresh'),
89			'Servers' => $this->misc->icon('Servers'),
90			'T' => $this->misc->icon('T'),
91			'Tminus' => $this->misc->icon('Tminus'),
92			'Tplus' => $this->misc->icon('Tplus'),
93			]];
94
95			return $this->view->fetch('jstree.twig', $viewVars);
96			}
97
98			public function doTree()
99			{
100			$treedata = new \PHPPgAdmin\ArrayRecordSet([]);
101			$reqvars = [];
102
103			$attrs = [
104			'text' => 'Servers',
105			'icon' => 'Servers',
106			'is_root' => 'true',
107			'action' => Decorator::url('/src/views/servers'),
108			'branch' => Decorator::url('/src/views/servers', $reqvars, ['action' => 'tree']),
109			];
110
111			return $this->printTree($treedata, $attrs, 'server');
112			}
113
114			}
115

HuasoFoundries / phpPgAdmin6

Push — master ( cf0423...965dbc )

BrowserController::doTree() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like