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

Passed

Push — master ( 2856b4...cf0423 )

by Felipe

created 2018-04-16 16:05 UTC

SqleditController::doFind() F

↳ Parent: SqleditController

Complexity

Conditions	22
Paths	> 20000

Size

Total Lines	51
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	22
eloc	36
nc	1064960
nop	0
dl	0
loc	51
rs	2.9305
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

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

namespace PHPPgAdmin\Controller;

use PHPPgAdmin\Decorators\Decorator;

/**
 * Base controller class.
 *
 * @package PHPPgAdmin
 */
class SqleditController extends BaseController
{
    public $controller_name = 'SqleditController';
    public $query           = '';
    public $subject         = '';
    public $start_time;
    public $duration;

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


        $action = $this->action;

        if ('tree' == $action) {
            return $this->doTree();
        }

        switch ($action) {
            case 'find':
                $title     = $this->lang['strfind'];
                $body_text = $this->doFind();

                break;
            case 'sql':
            default:
                $title     = $this->lang['strsql'];
                $body_text = $this->doDefault();

                break;
        }

        $this->setWindowName('sqledit');

        $this->scripts = '<script type="text/javascript">window.inPopUp=true;</script>';

        $this->printHeader($title, $this->scripts, true, 'header_sqledit.twig');
        $this->printBody(true, 'sql_edit');
        echo $body_text;

        $this->printFooter(true, 'footer_sqledit.twig');
    }

    /**
     * Allow execution of arbitrary SQL statements on a database.
     */
    public function doDefault()
    {
        $lang = $this->lang;
        $data = $this->misc->getDatabaseAccessor();

        if (!isset($_SESSION['sqlquery'])) {
            $_SESSION['sqlquery'] = '';
        }

        if (!isset($_REQUEST['search_path'])) {
            $_REQUEST['search_path'] = implode(',', $data->getSearchPath());
        }
        $search_path = htmlspecialchars($_REQUEST['search_path']);
        $sqlquery    = htmlspecialchars($_SESSION['sqlquery']);

        $default_html = $this->printTabs($this->misc->getNavTabs('popup'), 'sql', false);

        $default_html .= '<form action="' . \SUBFOLDER . '/src/views/sql" method="post" enctype="multipart/form-data" class="sqlform" id="sqlform" target="detail">';
        $default_html .= "\n";
        $default_html .= $this->_printConnection('sql');

        $default_html .= "\n";

        $default_html .= ' <div class="searchpath">';
        $default_html .= '<label>';
        $default_html .= $this->misc->printHelp($lang['strsearchpath'], 'pg.schema.search_path', false);

        $default_html .= ': <input type="text" name="search_path" size="45" value="' . $search_path . '" />';
        $default_html .= "</label>\n";

        $default_html .= "</div>\n";

        $default_html .= '<div id="queryedition" style="padding:1%;width:98%;float:left;">';
        $default_html .= "\n";
        $default_html .= '<textarea style="width:98%;" rows="10" cols="50" name="query" id="query" resizable="true">' . $sqlquery . '</textarea>';
        $default_html .= "\n";
        $default_html .= "</div>\n";

        $default_html .= '<div class="sqledit_bottom_inputs" >';

        if (ini_get('file_uploads')) {
            // Don't show upload option if max size of uploads is zero
            $max_size = $this->misc->inisizeToBytes(ini_get('upload_max_filesize'));
            if (is_double($max_size) && $max_size > 0) {
                $default_html .= '<p class="upload_sql_script">';
                $default_html .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . $max_size . '" />';
                $default_html .= "\n";
                $default_html .= '<label for="script">' . $lang['struploadscript'] . '</label>';
                $default_html .= '&nbsp;&nbsp; <input class="btn btn-small"  id="script" name="script" type="file" /></p>';
                $default_html .= "</p>\n";
            }
        }

        // Check that file uploads are enabled
        $checked = (isset($_REQUEST['paginate']) ? ' checked="checked"' : '');

        $default_html .= '<p><input type="submit" class="btn btn-small" name="execute" accesskey="r" value="' . $lang['strexecute'] . '" />';
        $default_html .= "\n";

        $default_html .= '<input type="reset" class="btn btn-small"  accesskey="q" value="' . $lang['strreset'] . '" /></p>';
        $default_html .= "\n";

        $default_html .= '<p>';
        $default_html .= '<label for="paginate">';
        $default_html .= '<input type="checkbox" id="paginate" name="paginate"' . $checked . ' />&nbsp;' . $lang['strpaginate'] . '&nbsp;';
        $default_html .= "</label>\n";
        $default_html .= "</p>\n";

        $default_html .= "</div>\n";
        $default_html .= '</form>';
        $default_html .= "\n";

        // Default focus
        //$this->setFocus('forms[0].query');
        return $default_html;
    }

    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');
    }

    /**
     * Private function to display server and list of databases.
     *
     * @param mixed $action
     */
    public function _printConnection($action)

    {
        $lang = $this->lang;

        $data = $this->misc->getDatabaseAccessor();


        // The javascript action on the select box reloads the
        // popup whenever the server or database is changed.
        // This ensures that the correct page encoding is used.
        $onchange = "onchange=\"location.href='" . \SUBFOLDER . '/sqledit/' .
        urlencode($action) . "?server=' + encodeURI(server.options[server.selectedIndex].value) + '&amp;database=' + encodeURI(database.options[database.selectedIndex].value) + ";

        // The exact URL to reload to is different between SQL and Find mode, however.
        if ('find' == $action) {
            $onchange .= "'&amp;term=' + encodeURI(term.value) + '&amp;filter=' + encodeURI(filter.value) + '&amp;'\"";
        } else {
            $onchange .= "'&amp;query=' + encodeURI(query.value) + '&amp;search_path=' + encodeURI(search_path.value) + (paginate.checked ? '&amp;paginate=on' : '')  + '&amp;'\"";
        }

        return $this->misc->printConnection($onchange, false);
    }

    /**
     * Searches for a named database object.
     */
    public function doFind()
    {
        $lang = $this->lang;
        $data = $this->misc->getDatabaseAccessor();

        if (!isset($_REQUEST['term'])) {
            $_REQUEST['term'] = '';
        }

        if (!isset($_REQUEST['filter'])) {
            $_REQUEST['filter'] = '';
        }

        $default_html = $this->printTabs($this->misc->getNavTabs('popup'), 'find', false);

        $default_html .= "<form action=\"database\" method=\"post\" target=\"detail\">\n";
        $default_html .= $this->_printConnection('find');
        $default_html .= '<p><input class="focusme" name="term" value="' . htmlspecialchars($_REQUEST['term']) . "\" size=\"32\" maxlength=\"{$data->_maxNameLen}\" />\n";

        // Output list of filters.  This is complex due to all the 'has' and 'conf' feature possibilities
        $default_html .= "<select name=\"filter\">\n";
        $default_html .= "\t<option value=\"\"" . ('' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strallobjects']}</option>\n";
        $default_html .= "\t<option value=\"SCHEMA\"" . ('SCHEMA' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strschemas']}</option>\n";
        $default_html .= "\t<option value=\"TABLE\"" . ('TABLE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtables']}</option>\n";
        $default_html .= "\t<option value=\"VIEW\"" . ('VIEW' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strviews']}</option>\n";
        $default_html .= "\t<option value=\"SEQUENCE\"" . ('SEQUENCE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strsequences']}</option>\n";
        $default_html .= "\t<option value=\"COLUMN\"" . ('COLUMN' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strcolumns']}</option>\n";
        $default_html .= "\t<option value=\"RULE\"" . ('RULE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strrules']}</option>\n";
        $default_html .= "\t<option value=\"INDEX\"" . ('INDEX' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strindexes']}</option>\n";
        $default_html .= "\t<option value=\"TRIGGER\"" . ('TRIGGER' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtriggers']}</option>\n";
        $default_html .= "\t<option value=\"CONSTRAINT\"" . ('CONSTRAINT' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strconstraints']}</option>\n";
        $default_html .= "\t<option value=\"FUNCTION\"" . ('FUNCTION' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strfunctions']}</option>\n";
        $default_html .= "\t<option value=\"DOMAIN\"" . ('DOMAIN' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strdomains']}</option>\n";
        if ($this->conf['show_advanced']) {
            $default_html .= "\t<option value=\"AGGREGATE\"" . ('AGGREGATE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['straggregates']}</option>\n";
            $default_html .= "\t<option value=\"TYPE\"" . ('TYPE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtypes']}</option>\n";
            $default_html .= "\t<option value=\"OPERATOR\"" . ('OPERATOR' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['stroperators']}</option>\n";
            $default_html .= "\t<option value=\"OPCLASS\"" . ('OPCLASS' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['stropclasses']}</option>\n";
            $default_html .= "\t<option value=\"CONVERSION\"" . ('CONVERSION' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strconversions']}</option>\n";
            $default_html .= "\t<option value=\"LANGUAGE\"" . ('LANGUAGE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strlanguages']}</option>\n";
        }
        $default_html .= "</select>\n";

        $default_html .= "<input type=\"submit\" value=\"{$lang['strfind']}\" />\n";
        $default_html .= "<input type=\"hidden\" name=\"action\" value=\"find\" /></p>\n";
        $default_html .= "</form>\n";

        // Default focus
        $this->setFocus('forms[0].term');

        return $default_html;
    }
}


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 SqleditController extends BaseController
17			{
18			public $controller_name = 'SqleditController';
19			public $query = '';
20			public $subject = '';
21			public $start_time;
22			public $duration;
23
24			/**
25			* Default method to render the controller according to the action parameter.
26			*/
27			public function render()
28			{
29			$lang = $this->lang;
			0 ignored issues – show Unused Code introduced 2018-02-15 19:53 UTC by Report Bug Copy Issue Report The assignment to `$lang` is dead and can be removed. Loading history...
30
31			$action = $this->action;
32
33			if ('tree' == $action) {
34			return $this->doTree();
35			}
36
37			switch ($action) {
38			case 'find':
39			$title = $this->lang['strfind'];
40			$body_text = $this->doFind();
41
42			break;
43			case 'sql':
44			default:
45			$title = $this->lang['strsql'];
46			$body_text = $this->doDefault();
47
48			break;
49			}
50
51			$this->setWindowName('sqledit');
52
53			$this->scripts = '<script type="text/javascript">window.inPopUp=true;</script>';
54
55			$this->printHeader($title, $this->scripts, true, 'header_sqledit.twig');
56			$this->printBody(true, 'sql_edit');
57			echo $body_text;
58
59			$this->printFooter(true, 'footer_sqledit.twig');
60			}
61
62			/**
63			* Allow execution of arbitrary SQL statements on a database.
64			*/
65			public function doDefault()
66			{
67			$lang = $this->lang;
68			$data = $this->misc->getDatabaseAccessor();
69
70			if (!isset($_SESSION['sqlquery'])) {
71			$_SESSION['sqlquery'] = '';
72			}
73
74			if (!isset($_REQUEST['search_path'])) {
75			$_REQUEST['search_path'] = implode(',', $data->getSearchPath());
76			}
77			$search_path = htmlspecialchars($_REQUEST['search_path']);
78			$sqlquery = htmlspecialchars($_SESSION['sqlquery']);
79
80			$default_html = $this->printTabs($this->misc->getNavTabs('popup'), 'sql', false);
81
82			$default_html .= '<form action="' . \SUBFOLDER . '/src/views/sql" method="post" enctype="multipart/form-data" class="sqlform" id="sqlform" target="detail">';
83			$default_html .= "\n";
84			$default_html .= $this->_printConnection('sql');
85
86			$default_html .= "\n";
87
88			$default_html .= ' <div class="searchpath">';
89			$default_html .= '<label>';
90			$default_html .= $this->misc->printHelp($lang['strsearchpath'], 'pg.schema.search_path', false);
91
92			$default_html .= ': <input type="text" name="search_path" size="45" value="' . $search_path . '" />';
93			$default_html .= "</label>\n";
94
95			$default_html .= "</div>\n";
96
97			$default_html .= '<div id="queryedition" style="padding:1%;width:98%;float:left;">';
98			$default_html .= "\n";
99			$default_html .= '<textarea style="width:98%;" rows="10" cols="50" name="query" id="query" resizable="true">' . $sqlquery . '</textarea>';
100			$default_html .= "\n";
101			$default_html .= "</div>\n";
102
103			$default_html .= '<div class="sqledit_bottom_inputs" >';
104
105			if (ini_get('file_uploads')) {
106			// Don't show upload option if max size of uploads is zero
107			$max_size = $this->misc->inisizeToBytes(ini_get('upload_max_filesize'));
108			if (is_double($max_size) && $max_size > 0) {
109			$default_html .= '<p class="upload_sql_script">';
110			$default_html .= '<input type="hidden" name="MAX_FILE_SIZE" value="' . $max_size . '" />';
111			$default_html .= "\n";
112			$default_html .= '<label for="script">' . $lang['struploadscript'] . '</label>';
113			$default_html .= '   <input class="btn btn-small" id="script" name="script" type="file" /></p>';
114			$default_html .= "</p>\n";
115			}
116			}
117
118			// Check that file uploads are enabled
119			$checked = (isset($_REQUEST['paginate']) ? ' checked="checked"' : '');
120
121			$default_html .= '<p><input type="submit" class="btn btn-small" name="execute" accesskey="r" value="' . $lang['strexecute'] . '" />';
122			$default_html .= "\n";
123
124			$default_html .= '<input type="reset" class="btn btn-small" accesskey="q" value="' . $lang['strreset'] . '" /></p>';
125			$default_html .= "\n";
126
127			$default_html .= '<p>';
128			$default_html .= '<label for="paginate">';
129			$default_html .= '<input type="checkbox" id="paginate" name="paginate"' . $checked . ' /> ' . $lang['strpaginate'] . ' ';
130			$default_html .= "</label>\n";
131			$default_html .= "</p>\n";
132
133			$default_html .= "</div>\n";
134			$default_html .= '</form>';
135			$default_html .= "\n";
136
137			// Default focus
138			//$this->setFocus('forms[0].query');
139			return $default_html;
140			}
141
142			public function doTree()
143			{
144			$treedata = new \PHPPgAdmin\ArrayRecordSet([]);
145			$reqvars = [];
146
147			$attrs = [
148			'text' => 'Servers',
149			'icon' => 'Servers',
150			'is_root' => 'true',
151			'action' => Decorator::url('/src/views/servers'),
152			'branch' => Decorator::url('/src/views/servers', $reqvars, ['action' => 'tree']),
153			];
154
155			return $this->printTree($treedata, $attrs, 'server');
156			}
157
158			/**
159			* Private function to display server and list of databases.
160			*
161			* @param mixed $action
162			*/
163			public function _printConnection($action)
			1 ignored issue – show Coding Style introduced 2017-10-08 18:23 UTC by Report Bug Copy Issue Report Public method name "SqleditController::_printConnection" must not be prefixed with an underscore Loading history...
164			{
165			$lang = $this->lang;
			0 ignored issues – show Unused Code introduced 2018-02-15 15:30 UTC by Report Bug Copy Issue Report The assignment to `$lang` is dead and can be removed. Loading history...
166			$data = $this->misc->getDatabaseAccessor();
			0 ignored issues – show Unused Code introduced 2017-09-30 13:55 UTC by Report Bug Copy Issue Report The assignment to `$data` is dead and can be removed. Loading history...
167
168			// The javascript action on the select box reloads the
169			// popup whenever the server or database is changed.
170			// This ensures that the correct page encoding is used.
171			$onchange = "onchange=\"location.href='" . \SUBFOLDER . '/sqledit/' .
172			urlencode($action) . "?server=' + encodeURI(server.options[server.selectedIndex].value) + '&database=' + encodeURI(database.options[database.selectedIndex].value) + ";
173
174			// The exact URL to reload to is different between SQL and Find mode, however.
175			if ('find' == $action) {
176			$onchange .= "'&term=' + encodeURI(term.value) + '&filter=' + encodeURI(filter.value) + '&'\"";
177			} else {
178			$onchange .= "'&query=' + encodeURI(query.value) + '&search_path=' + encodeURI(search_path.value) + (paginate.checked ? '&paginate=on' : '') + '&'\"";
179			}
180
181			return $this->misc->printConnection($onchange, false);
182			}
183
184			/**
185			* Searches for a named database object.
186			*/
187			public function doFind()
188			{
189			$lang = $this->lang;
190			$data = $this->misc->getDatabaseAccessor();
191
192			if (!isset($_REQUEST['term'])) {
193			$_REQUEST['term'] = '';
194			}
195
196			if (!isset($_REQUEST['filter'])) {
197			$_REQUEST['filter'] = '';
198			}
199
200			$default_html = $this->printTabs($this->misc->getNavTabs('popup'), 'find', false);
201
202			$default_html .= "<form action=\"database\" method=\"post\" target=\"detail\">\n";
203			$default_html .= $this->_printConnection('find');
204			$default_html .= '<p><input class="focusme" name="term" value="' . htmlspecialchars($_REQUEST['term']) . "\" size=\"32\" maxlength=\"{$data->_maxNameLen}\" />\n";
205
206			// Output list of filters. This is complex due to all the 'has' and 'conf' feature possibilities
207			$default_html .= "<select name=\"filter\">\n";
208			$default_html .= "\t<option value=\"\"" . ('' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strallobjects']}</option>\n";
209			$default_html .= "\t<option value=\"SCHEMA\"" . ('SCHEMA' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strschemas']}</option>\n";
210			$default_html .= "\t<option value=\"TABLE\"" . ('TABLE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtables']}</option>\n";
211			$default_html .= "\t<option value=\"VIEW\"" . ('VIEW' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strviews']}</option>\n";
212			$default_html .= "\t<option value=\"SEQUENCE\"" . ('SEQUENCE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strsequences']}</option>\n";
213			$default_html .= "\t<option value=\"COLUMN\"" . ('COLUMN' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strcolumns']}</option>\n";
214			$default_html .= "\t<option value=\"RULE\"" . ('RULE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strrules']}</option>\n";
215			$default_html .= "\t<option value=\"INDEX\"" . ('INDEX' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strindexes']}</option>\n";
216			$default_html .= "\t<option value=\"TRIGGER\"" . ('TRIGGER' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtriggers']}</option>\n";
217			$default_html .= "\t<option value=\"CONSTRAINT\"" . ('CONSTRAINT' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strconstraints']}</option>\n";
218			$default_html .= "\t<option value=\"FUNCTION\"" . ('FUNCTION' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strfunctions']}</option>\n";
219			$default_html .= "\t<option value=\"DOMAIN\"" . ('DOMAIN' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strdomains']}</option>\n";
220			if ($this->conf['show_advanced']) {
221			$default_html .= "\t<option value=\"AGGREGATE\"" . ('AGGREGATE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['straggregates']}</option>\n";
222			$default_html .= "\t<option value=\"TYPE\"" . ('TYPE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strtypes']}</option>\n";
223			$default_html .= "\t<option value=\"OPERATOR\"" . ('OPERATOR' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['stroperators']}</option>\n";
224			$default_html .= "\t<option value=\"OPCLASS\"" . ('OPCLASS' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['stropclasses']}</option>\n";
225			$default_html .= "\t<option value=\"CONVERSION\"" . ('CONVERSION' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strconversions']}</option>\n";
226			$default_html .= "\t<option value=\"LANGUAGE\"" . ('LANGUAGE' == $_REQUEST['filter'] ? ' selected="selected" ' : '') . ">{$lang['strlanguages']}</option>\n";
227			}
228			$default_html .= "</select>\n";
229
230			$default_html .= "<input type=\"submit\" value=\"{$lang['strfind']}\" />\n";
231			$default_html .= "<input type=\"hidden\" name=\"action\" value=\"find\" /></p>\n";
232			$default_html .= "</form>\n";
233
234			// Default focus
235			$this->setFocus('forms[0].term');
236
237			return $default_html;
238			}
239			}
240

HuasoFoundries / phpPgAdmin6

Push — master ( 2856b4...cf0423 )

SqleditController::doFind() F

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like