Completed
Push — master ( 0c98dc...038288 )
by Richard
15s queued 11s
created
htdocs/modules/system/class/extension.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@  discard block
 block discarded – undo
67 67
         $i = 0;
68 68
         foreach ($this->modulesList as $file) {
69 69
             $file = trim($file);
70
-            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $file . '/xoops_version.php')) {
70
+            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$file.'/xoops_version.php')) {
71 71
                 clearstatcache();
72 72
                 /* @var $module XoopsModule */
73 73
                 $module = $module_handler->create();
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
                         ) {
98 98
                             $module->setInfo(
99 99
                                 'link_pref',
100
-                                \XoopsBaseConfig::get('url') . '/modules/system/admin.php?fct=preferences&op=showmod&mod='
100
+                                \XoopsBaseConfig::get('url').'/modules/system/admin.php?fct=preferences&op=showmod&mod='
101 101
                                 . $module->getInfo('mid')
102 102
                             );
103 103
                         }
@@ -106,28 +106,28 @@  discard block
 block discarded – undo
106 106
                     }
107 107
                     $module->setInfo('version', round($module->getInfo('version'), 2));
108 108
                     if (XoopsLoad::fileExists(
109
-                        \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_small.png'
109
+                        \XoopsBaseConfig::get('root-path').'/modules/'.$module->getInfo('dirname').'/icons/logo_small.png'
110 110
                     )) {
111 111
                         $module->setInfo(
112 112
                             'logo_small',
113
-                            \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_small.png'
113
+                            \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('dirname').'/icons/logo_small.png'
114 114
                         );
115 115
                     } else {
116
-                        $module->setInfo('logo_small', \XoopsBaseConfig::get('url') . '/media/xoops/images/icons/16/default.png');
116
+                        $module->setInfo('logo_small', \XoopsBaseConfig::get('url').'/media/xoops/images/icons/16/default.png');
117 117
                     }
118 118
                     if (XoopsLoad::fileExists(
119
-                        \XoopsBaseConfig::get('root-path') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_large.png'
119
+                        \XoopsBaseConfig::get('root-path').'/modules/'.$module->getInfo('dirname').'/icons/logo_large.png'
120 120
                     )) {
121 121
                         $module->setInfo(
122 122
                             'logo_large',
123
-                            \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/icons/logo_large.png'
123
+                            \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('dirname').'/icons/logo_large.png'
124 124
                         );
125 125
                     } else {
126
-                        $module->setInfo('logo_large', \XoopsBaseConfig::get('url') . '/media/xoops/images/icons/32/default.png');
126
+                        $module->setInfo('logo_large', \XoopsBaseConfig::get('url').'/media/xoops/images/icons/32/default.png');
127 127
                     }
128 128
                     $module->setInfo(
129 129
                         'link_admin',
130
-                        \XoopsBaseConfig::get('url') . '/modules/' . $module->getInfo('dirname') . '/' . $module->getInfo('adminindex')
130
+                        \XoopsBaseConfig::get('url').'/modules/'.$module->getInfo('dirname').'/'.$module->getInfo('adminindex')
131 131
                     );
132 132
                     $module->setInfo('options', $module->getAdminMenu());
133 133
                     $ret[] = $module;
@@ -153,7 +153,7 @@  discard block
 block discarded – undo
153 153
         $ret = array();
154 154
         $i = 0;
155 155
         foreach ($this->modulesList as $file) {
156
-            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path') . '/modules/' . $file . '/xoops_version.php')) {
156
+            if (XoopsLoad::fileExists(\XoopsBaseConfig::get('root-path').'/modules/'.$file.'/xoops_version.php')) {
157 157
                 clearstatcache();
158 158
                 $file = trim($file);
159 159
                 if (!in_array($file, $this->modulesDirnames)) {
Please login to merge, or discard this patch.
htdocs/modules/system/class/form/group.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -70,12 +70,12 @@  discard block
 block discarded – undo
70 70
 
71 71
         $s_cat_checkbox = new Xoops\Form\Checkbox('', "system_catids", $s_cat_value);
72 72
         //$s_cat_checkbox->columns = 6;
73
-        $admin_dir = \XoopsBaseConfig::get('root-path') . '/modules/system/admin/';
73
+        $admin_dir = \XoopsBaseConfig::get('root-path').'/modules/system/admin/';
74 74
         $dirlist = XoopsLists::getDirListAsArray($admin_dir);
75 75
         foreach ($dirlist as $file) {
76
-            include \XoopsBaseConfig::get('root-path') . '/modules/system/admin/' . $file . '/xoops_version.php';
76
+            include \XoopsBaseConfig::get('root-path').'/modules/system/admin/'.$file.'/xoops_version.php';
77 77
             if (!empty($modversion['category'])) {
78
-                if ($xoops->getModuleConfig('active_' . $file, 'system') == 1) {
78
+                if ($xoops->getModuleConfig('active_'.$file, 'system') == 1) {
79 79
                     $s_cat_checkbox->addOption($modversion['category'], $modversion['name']);
80 80
                 }
81 81
             }
@@ -124,7 +124,7 @@  discard block
 block discarded – undo
124 124
 
125 125
         $block_handler = $xoops->getHandlerBlock();
126 126
         $blocks_obj = $block_handler->getDistinctObjects(
127
-            new Criteria("mid", "('" . implode("', '", array_keys($module_list)) . "')", "IN"),
127
+            new Criteria("mid", "('".implode("', '", array_keys($module_list))."')", "IN"),
128 128
             true
129 129
         );
130 130
 
@@ -146,12 +146,12 @@  discard block
 block discarded – undo
146 146
 
147 147
             $new_blocks_array = array();
148 148
             foreach ($blocks_module[$mid] as $key => $value) {
149
-                $new_blocks_array[$key] = "<a href='" . \XoopsBaseConfig::get('url')
149
+                $new_blocks_array[$key] = "<a href='".\XoopsBaseConfig::get('url')
150 150
                     . "/modules/system/admin.php?fct=blocksadmin&amp;op=edit&amp;bid={$key}' "
151 151
                     . "title='ID: {$key}' rel='external'>{$value}</a>";
152 152
             }
153 153
             $r_block_checkbox = new Xoops\Form\Checkbox(
154
-                '<strong>' . $module_list[$mid] . '</strong><br />',
154
+                '<strong>'.$module_list[$mid].'</strong><br />',
155 155
                 "read_bids[]",
156 156
                 $r_block_value
157 157
             );
Please login to merge, or discard this patch.
htdocs/modules/system/class/form/preferences.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         $configs = $mod->getInfo('config');
58 58
         $configNames = array();
59 59
         foreach (array_keys($configs) as $i) {
60
-            $configNames[$configs[$i]['name']] =& $configs[$i];
60
+            $configNames[$configs[$i]['name']] = & $configs[$i];
61 61
         }
62 62
         $configCats = $mod->getInfo('configcat');
63 63
         if (!$configCats) {
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
         $tabTray = new Xoops\Form\TabTray('', 'pref_tabtay');
86 86
         $tabs = array();
87 87
         foreach ($configCats as $name => $info) {
88
-            $tabs[$name] = new Xoops\Form\Tab($info['name'], 'pref_tab_' . $name);
88
+            $tabs[$name] = new Xoops\Form\Tab($info['name'], 'pref_tab_'.$name);
89 89
             if (isset($info['description']) && $info['description'] != '') {
90 90
                 $tabs[$name]->addElement(new Xoops\Form\Label('', $info['description']));
91 91
             }
@@ -99,7 +99,7 @@  discard block
 block discarded – undo
99 99
         } elseif ($mod->getInfo('adminindex')) {
100 100
             $this->addElement(new Xoops\Form\Hidden(
101 101
                 'redirect',
102
-                \XoopsBaseConfig::get('url') . '/modules/' . $mod->getVar('dirname') . '/' . $mod->getInfo('adminindex')
102
+                \XoopsBaseConfig::get('url').'/modules/'.$mod->getVar('dirname').'/'.$mod->getInfo('adminindex')
103 103
             ));
104 104
         }
105 105
         $count = count($obj);
@@ -260,7 +260,7 @@  discard block
 block discarded – undo
260 260
                         $ele = new Xoops\Form\ElementTray($title, '<br />');
261 261
                         foreach (array_keys($modules) as $mid) {
262 262
                             $c_val = isset($currrent_val[$mid]) ? (int)($currrent_val[$mid]) : null;
263
-                            $selform = new Xoops\Form\Select($modules[$mid]->getVar('name'), $obj[$i]->getVar('conf_name') . "[$mid]", $c_val);
263
+                            $selform = new Xoops\Form\Select($modules[$mid]->getVar('name'), $obj[$i]->getVar('conf_name')."[$mid]", $c_val);
264 264
                             $selform->addOptionArray($cache_options);
265 265
                             $ele->addElement($selform);
266 266
                             unset($selform);
Please login to merge, or discard this patch.
htdocs/modules/system/class/form/block.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -152,17 +152,17 @@
 block discarded – undo
152 152
                 if (count($btemplate) > 0) {
153 153
                     $this->addElement(new Xoops\Form\Label(
154 154
                         XoopsLocale::CONTENT,
155
-                        '<a href="' . \XoopsBaseConfig::get('url') . '/modules/system/admin.php?fct=tplsets&amp;op=edittpl&amp;id='
156
-                        . $btemplate[0]->getVar('tpl_id') . '">' . SystemLocale::EDIT_TEMPLATE . '</a>'
155
+                        '<a href="'.\XoopsBaseConfig::get('url').'/modules/system/admin.php?fct=tplsets&amp;op=edittpl&amp;id='
156
+                        . $btemplate[0]->getVar('tpl_id').'">'.SystemLocale::EDIT_TEMPLATE.'</a>'
157 157
                     ));
158 158
                 } else {
159 159
                     $btemplate2 = $tplfile_handler->find('default', 'block', $this->obj->getVar('bid'));
160 160
                     if (count($btemplate2) > 0) {
161 161
                         $this->addElement(new Xoops\Form\Label(
162 162
                             XoopsLocale::CONTENT,
163
-                            '<a href="' . \XoopsBaseConfig::get('url') . '/modules/system/admin.php?fct=tplsets&amp;op=edittpl&amp;id='
164
-                            . $btemplate2[0]->getVar('tpl_id') . '" rel="external">'
165
-                            . SystemLocale::EDIT_TEMPLATE . '</a>'
163
+                            '<a href="'.\XoopsBaseConfig::get('url').'/modules/system/admin.php?fct=tplsets&amp;op=edittpl&amp;id='
164
+                            . $btemplate2[0]->getVar('tpl_id').'" rel="external">'
165
+                            . SystemLocale::EDIT_TEMPLATE.'</a>'
166 166
                         ));
167 167
                     }
168 168
                 }
Please login to merge, or discard this patch.
htdocs/modules/system/class/form/user.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -145,7 +145,7 @@
 block discarded – undo
145 145
         } else {
146 146
             //add each user groups
147 147
             foreach ($groups as $key => $group) {
148
-                $group_select[] = new Xoops\Form\Hidden('groups[' . $key . ']', $group);
148
+                $group_select[] = new Xoops\Form\Hidden('groups['.$key.']', $group);
149 149
             }
150 150
         }
151 151
         foreach ($group_select as $group) {
Please login to merge, or discard this patch.
htdocs/modules/system/class/menu.php 2 patches
Indentation   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -132,10 +132,10 @@
 block discarded – undo
132 132
         $xoops->tpl()->assign('xo_module_menu_top', $this->_menutop);
133 133
         $this->_menutabs[$currentoption]['current'] = 1;
134 134
         $xoops->tpl()->assign('xo_module_menu_tab', $this->_menutabs);
135
-         //$xoops->tpl()->assign('xo_admin_help', $this->_help);
136
-         //if ($xoops->tpl()_name == '') {
137
-         //    $xoops->tpl()->display('admin:system/admin_tabs.tpl');
138
-         //}
135
+            //$xoops->tpl()->assign('xo_admin_help', $this->_help);
136
+            //if ($xoops->tpl()_name == '') {
137
+            //    $xoops->tpl()->display('admin:system/admin_tabs.tpl');
138
+            //}
139 139
         return;
140 140
 
141 141
         global $modversion;
Please login to merge, or discard this patch.
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -113,13 +113,13 @@  discard block
 block discarded – undo
113 113
 
114 114
         if ($num > 1) {
115 115
             foreach ($arr as $val) {
116
-                $return_str .= ' &gt; <a href="' . $site . $val . '/">' . $bc_label[$val] . '</a>';
117
-                $site .= $val . '/';
116
+                $return_str .= ' &gt; <a href="'.$site.$val.'/">'.$bc_label[$val].'</a>';
117
+                $site .= $val.'/';
118 118
             }
119 119
         } else {
120 120
             if ($num == 1) {
121 121
                 $arr = $str;
122
-                $return_str .= ' &gt; <a href="' . $bc_site . $arr . '/">' . $bc_label[$arr] . '</a>';
122
+                $return_str .= ' &gt; <a href="'.$bc_site.$arr.'/">'.$bc_label[$arr].'</a>';
123 123
             }
124 124
         }
125 125
 
@@ -161,12 +161,12 @@  discard block
 block discarded – undo
161 161
         $menu = substr($menu, 0, -1);
162 162
 
163 163
         $menu .= "</td>";
164
-        $menu .= "<td style='text-align: right;'><strong>" . $this->_obj->getVar('name') . "</strong> : " . $breadcrumb . "</td>";
164
+        $menu .= "<td style='text-align: right;'><strong>".$this->_obj->getVar('name')."</strong> : ".$breadcrumb."</td>";
165 165
         $menu .= "</tr>\n</table>\n";
166 166
         $menu .= "</div>\n";
167 167
         $menu .= "<div id='buttonbar_mod'><ul>";
168 168
         foreach ($this->_menutabs as $k => $v) {
169
-            $menu .= "<li id='" . $menuItems[$i] . "'><a href='" . \XoopsBaseConfig::get('url') . "/modules/" . $this->_obj->getVar('dirname') . "/" . $k . "'><span>$v</span></a></li>\n";
169
+            $menu .= "<li id='".$menuItems[$i]."'><a href='".\XoopsBaseConfig::get('url')."/modules/".$this->_obj->getVar('dirname')."/".$k."'><span>$v</span></a></li>\n";
170 170
             ++$i;
171 171
         }
172 172
         $menu .= "</ul>\n</div>\n";
@@ -174,18 +174,18 @@  discard block
 block discarded – undo
174 174
             $menu .= "<h4 class='admin_header'>";
175 175
             if (isset($modversion['name'])) {
176 176
                 if ($modversion['image'] && $this->_obj->getVar('mid') == 1) {
177
-                    $system_image = \XoopsBaseConfig::get('url') . '/modules/system/images/system/' . $modversion['image'];
177
+                    $system_image = \XoopsBaseConfig::get('url').'/modules/system/images/system/'.$modversion['image'];
178 178
                 } else {
179
-                    $system_image = \XoopsBaseConfig::get('url') . '/modules/' . $_dirname . '/images/' . $modversion['image'];
179
+                    $system_image = \XoopsBaseConfig::get('url').'/modules/'.$_dirname.'/images/'.$modversion['image'];
180 180
                 }
181 181
                 $menu .= "<img src='$system_image' align='middle' height='32' width='32' alt='' />";
182
-                $menu .= " " . $modversion['name'] . "</h4>\n";
182
+                $menu .= " ".$modversion['name']."</h4>\n";
183 183
             } else {
184
-                $menu .= " " . $this->_header . "</h4>\n";
184
+                $menu .= " ".$this->_header."</h4>\n";
185 185
             }
186 186
         }
187 187
         if ($this->_subheader) {
188
-            $menu .= "<div class='admin_subheader'>" . $this->_subheader . "</div>\n";
188
+            $menu .= "<div class='admin_subheader'>".$this->_subheader."</div>\n";
189 189
         }
190 190
         $menu .= '<div class="clear">&nbsp;</div>';
191 191
         unset($this->_obj);
Please login to merge, or discard this patch.
htdocs/modules/system/header.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -19,7 +19,7 @@  discard block
 block discarded – undo
19 19
  */
20 20
 
21 21
 // Include XOOPS control panel header
22
-include_once dirname(dirname(__DIR__)) . '/include/cp_header.php';
22
+include_once dirname(dirname(__DIR__)).'/include/cp_header.php';
23 23
 
24 24
 $xoops = Xoops::getInstance();
25 25
 
@@ -48,4 +48,4 @@  discard block
 block discarded – undo
48 48
 XoopsLoad::load('systembreadcrumb', 'system');
49 49
 
50 50
 $system_breadcrumb = SystemBreadcrumb::getInstance($fct);
51
-$system_breadcrumb->addLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url') . '/admin.php', true);
51
+$system_breadcrumb->addLink(SystemLocale::CONTROL_PANEL, \XoopsBaseConfig::get('url').'/admin.php', true);
Please login to merge, or discard this patch.
htdocs/modules/maintenance/admin/about.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include __DIR__ . '/header.php';
23
+include __DIR__.'/header.php';
24 24
 $xoops = Xoops::getInstance();
25 25
 $xoops->header();
26 26
 $aboutAdmin = new \Xoops\Module\Admin();
Please login to merge, or discard this patch.
htdocs/modules/maintenance/admin/dump.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -20,7 +20,7 @@  discard block
 block discarded – undo
20 20
  * @version         $Id$
21 21
  */
22 22
 
23
-include __DIR__ . '/header.php';
23
+include __DIR__.'/header.php';
24 24
 // Get main instance
25 25
 $system = System::getInstance();
26 26
 $xoops = Xoops::getInstance();
@@ -40,14 +40,14 @@  discard block
 block discarded – undo
40 40
 
41 41
     case 'list':
42 42
     default:
43
-        $files = glob(\XoopsBaseConfig::get('root-path') . '/modules/maintenance/dump/*.*');
43
+        $files = glob(\XoopsBaseConfig::get('root-path').'/modules/maintenance/dump/*.*');
44 44
         $count = 0;
45 45
         foreach ($files as $filename_path) {
46 46
             $filename = basename(strtolower($filename_path));
47 47
             if ($filename !== 'index.html') {
48 48
                 $file_arr[$count]['name'] = $filename;
49 49
                 $stat = stat($filename_path);
50
-                $file_arr[$count]['size'] = number_format($stat['size']/1024);
50
+                $file_arr[$count]['size'] = number_format($stat['size'] / 1024);
51 51
                 ++$count;
52 52
                 unset($filename);
53 53
             }
@@ -85,18 +85,18 @@  discard block
 block discarded – undo
85 85
         $dump = new Maintenance();
86 86
         $sql_text = "# \n";
87 87
         $sql_text .= "# Dump SQL, Generated by XOOPS \n";
88
-        $sql_text .= "# Date : " . \XoopsLocale::formatTimestamp(time(), 'medium') . " \n";
88
+        $sql_text .= "# Date : ".\XoopsLocale::formatTimestamp(time(), 'medium')." \n";
89 89
         $sql_text .= "# \n\n";
90 90
         if ($dump_tables != false) {
91 91
             $result_module = array();
92 92
             for ($i = 0; $i < count($dump_tables); ++$i) {
93 93
                 //structure
94
-                $result_tables[$i]['name'] = $db->prefix . '_' . $dump_tables[$i];
95
-                $result_structure = $dump->dumpTableStructure($db->prefix . '_' . $dump_tables[$i], $drop);
94
+                $result_tables[$i]['name'] = $db->prefix.'_'.$dump_tables[$i];
95
+                $result_structure = $dump->dumpTableStructure($db->prefix.'_'.$dump_tables[$i], $drop);
96 96
                 $sql_text .= $result_structure['sql_text'];
97 97
                 $result_tables[$i]['structure'] = $result_structure['structure'];
98 98
                 //data
99
-                $result_data = $dump->dumpTableData($db->prefix . '_' . $dump_tables[$i]);
99
+                $result_data = $dump->dumpTableData($db->prefix.'_'.$dump_tables[$i]);
100 100
                 $sql_text .= $result_data['sql_text'];
101 101
                 $result_tables[$i]['records'] = $result_data['records'];
102 102
             }
@@ -113,13 +113,13 @@  discard block
 block discarded – undo
113 113
                     $count = 0;
114 114
                     foreach ($modtables as $table) {
115 115
                         //structure
116
-                        $result_tables[$count]['name'] = $db->prefix . '_' . $table;
117
-                        $result_structure = $dump->dumpTableStructure($db->prefix . '_' . $table, $drop);
116
+                        $result_tables[$count]['name'] = $db->prefix.'_'.$table;
117
+                        $result_structure = $dump->dumpTableStructure($db->prefix.'_'.$table, $drop);
118 118
                         $sql_text .= $result_structure['sql_text'];
119 119
                         $result_tables[$count]['structure'] = $result_structure['structure'];
120 120
 
121 121
                         //data
122
-                        $result_data = $dump->dumpTableData($db->prefix . '_' . $table);
122
+                        $result_data = $dump->dumpTableData($db->prefix.'_'.$table);
123 123
                         $sql_text .= $result_data['sql_text'];
124 124
                         $result_tables[$count]['records'] = $result_data['records'];
125 125
                         ++$count;
@@ -143,12 +143,12 @@  discard block
 block discarded – undo
143 143
         if ($filename == '') {
144 144
             $xoops->redirect("dump.php", 2, _AM_MAINTENANCE_DUMP_NOFILE);
145 145
         }
146
-        unlink(\XoopsBaseConfig::get('root-path') . '/modules/maintenance/dump/' . $filename);
146
+        unlink(\XoopsBaseConfig::get('root-path').'/modules/maintenance/dump/'.$filename);
147 147
         $xoops->redirect("dump.php", 2, _AM_MAINTENANCE_DUMP_DELETED);
148 148
         break;
149 149
 
150 150
     case 'dump_deleteall':
151
-        $files = glob(\XoopsBaseConfig::get('root-path') . '/modules/maintenance/dump/*.*');
151
+        $files = glob(\XoopsBaseConfig::get('root-path').'/modules/maintenance/dump/*.*');
152 152
         $count = 0;
153 153
         foreach ($files as $filename_path) {
154 154
             if (basename(strtolower($filename_path)) !== 'index.html') {
Please login to merge, or discard this patch.