Passed
Push — master ( bcd4f7...1bf514 )
by Goffy
03:22
created
xoops_version.php 1 patch
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -37,10 +37,10 @@  discard block
 block discarded – undo
37 37
     'license_url'         => 'http://www.gnu.org/licenses/gpl-3.0.en.html',
38 38
     'help'                => 'page=help',
39 39
     'release_info'        => 'release_info',
40
-    'release_file'        => XOOPS_URL . '/modules/wggithub/docs/release_info file',
40
+    'release_file'        => XOOPS_URL.'/modules/wggithub/docs/release_info file',
41 41
     'release_date'        => '2020/11/21',
42 42
     'manual'              => 'link to manual file',
43
-    'manual_file'         => XOOPS_URL . '/modules/wggithub/docs/install.txt',
43
+    'manual_file'         => XOOPS_URL.'/modules/wggithub/docs/install.txt',
44 44
     'min_php'             => '7.2',
45 45
     'min_xoops'           => '2.5.10',
46 46
     'min_admin'           => '1.2',
@@ -203,10 +203,10 @@  discard block
 block discarded – undo
203 203
 ];
204 204
 // Get groups
205 205
 $memberHandler = \xoops_getHandler('member');
206
-$xoopsGroups  = $memberHandler->getGroupList();
206
+$xoopsGroups = $memberHandler->getGroupList();
207 207
 $groups = [];
208 208
 foreach ($xoopsGroups as $key => $group) {
209
-    $groups[$group]  = $key;
209
+    $groups[$group] = $key;
210 210
 }
211 211
 // General access groups
212 212
 $modversion['config'][] = [
@@ -232,10 +232,10 @@  discard block
 block discarded – undo
232 232
 $crGroups = new \CriteriaCompo();
233 233
 $crGroups->add(new \Criteria('group_type', 'Admin'));
234 234
 $memberHandler = \xoops_getHandler('member');
235
-$adminXoopsGroups  = $memberHandler->getGroupList($crGroups);
235
+$adminXoopsGroups = $memberHandler->getGroupList($crGroups);
236 236
 $adminGroups = [];
237 237
 foreach ($adminXoopsGroups as $key => $adminGroup) {
238
-    $adminGroups[$adminGroup]  = $key;
238
+    $adminGroups[$adminGroup] = $key;
239 239
 }
240 240
 $modversion['config'][] = [
241 241
     'name'        => 'admin_groups',
@@ -335,8 +335,8 @@  discard block
 block discarded – undo
335 335
 // Make Sample button visible?
336 336
 $modversion['config'][] = [
337 337
     'name'        => 'displaySampleButton',
338
-    'title'       => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON',
339
-    'description' => 'CO_' . $moduleDirNameUpper . '_' . 'SHOW_SAMPLE_BUTTON_DESC',
338
+    'title'       => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON',
339
+    'description' => 'CO_'.$moduleDirNameUpper.'_'.'SHOW_SAMPLE_BUTTON_DESC',
340 340
     'formtype'    => 'yesno',
341 341
     'valuetype'   => 'int',
342 342
     'default'     => 1,
Please login to merge, or discard this patch.
include/functions.php 2 patches
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -133,7 +133,7 @@  discard block
 block discarded – undo
133 133
 
134 134
         case 'none':
135 135
             if($topic_name) {
136
-                 $topic_name = 'topic=' . $topic_name . '&';
136
+                    $topic_name = 'topic=' . $topic_name . '&';
137 137
             }
138 138
             $rewrite_base = '/modules/';
139 139
             $page = 'page=' . $array['content_alias'];
@@ -163,7 +163,7 @@  discard block
 block discarded – undo
163 163
             return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
164 164
             break;
165 165
 
166
-         case 'short':
166
+            case 'short':
167 167
             if($topic_name) {
168 168
                 $topic_name .= '/';
169 169
             }
Please login to merge, or discard this patch.
Spacing   +22 added lines, -22 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
         $cat_sql .= current($cats);
34 34
         array_shift($cats);
35 35
         foreach ($cats as $cat) {
36
-            $cat_sql .= ',' . $cat;
36
+            $cat_sql .= ','.$cat;
37 37
         }
38 38
     }
39 39
     $cat_sql .= ')';
@@ -72,9 +72,9 @@  discard block
 block discarded – undo
72 72
 {
73 73
     global $xoopsTpl, $xoTheme;
74 74
     $myts = MyTextSanitizer::getInstance();
75
-    $content= $myts->undoHtmlSpecialChars($myts->displayTarea($content));
76
-    if(isset($xoTheme) && \is_object($xoTheme)) {
77
-        $xoTheme->addMeta( 'meta', 'keywords', \strip_tags($content));
75
+    $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
76
+    if (isset($xoTheme) && \is_object($xoTheme)) {
77
+        $xoTheme->addMeta('meta', 'keywords', \strip_tags($content));
78 78
     } else {    // Compatibility for old Xoops versions
79 79
         $xoopsTpl->assign('xoops_meta_keywords', \strip_tags($content));
80 80
     }
@@ -91,8 +91,8 @@  discard block
 block discarded – undo
91 91
     global $xoopsTpl, $xoTheme;
92 92
     $myts = MyTextSanitizer::getInstance();
93 93
     $content = $myts->undoHtmlSpecialChars($myts->displayTarea($content));
94
-    if(isset($xoTheme) && \is_object($xoTheme)) {
95
-        $xoTheme->addMeta( 'meta', 'description', \strip_tags($content));
94
+    if (isset($xoTheme) && \is_object($xoTheme)) {
95
+        $xoTheme->addMeta('meta', 'description', \strip_tags($content));
96 96
     } else {    // Compatibility for old Xoops versions
97 97
         $xoopsTpl->assign('xoops_meta_description', \strip_tags($content));
98 98
     }
@@ -117,7 +117,7 @@  discard block
 block discarded – undo
117 117
     if (0 != $lenght_id) {
118 118
         $id = $array['content_id'];
119 119
         while (\strlen($id) < $lenght_id) {
120
-            $id = '0' . $id;
120
+            $id = '0'.$id;
121 121
         }
122 122
     } else {
123 123
         $id = $array['content_id'];
@@ -132,23 +132,23 @@  discard block
 block discarded – undo
132 132
     switch ($rewrite_url) {
133 133
 
134 134
         case 'none':
135
-            if($topic_name) {
136
-                 $topic_name = 'topic=' . $topic_name . '&amp;';
135
+            if ($topic_name) {
136
+                 $topic_name = 'topic='.$topic_name.'&amp;';
137 137
             }
138 138
             $rewrite_base = '/modules/';
139
-            $page = 'page=' . $array['content_alias'];
140
-            return XOOPS_URL . $rewrite_base . $module . '/' . $type . '.php?' . $topic_name . 'id=' . $id . '&amp;' . $page . $comment;
139
+            $page = 'page='.$array['content_alias'];
140
+            return XOOPS_URL.$rewrite_base.$module.'/'.$type.'.php?'.$topic_name.'id='.$id.'&amp;'.$page.$comment;
141 141
             break;
142 142
 
143 143
         case 'rewrite':
144
-            if($topic_name) {
144
+            if ($topic_name) {
145 145
                 $topic_name .= '/';
146 146
             }
147 147
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
148 148
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
149 149
             $module_name = '';
150
-            if(xoops_getModuleOption('rewrite_name', $module)) {
151
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
150
+            if (xoops_getModuleOption('rewrite_name', $module)) {
151
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
152 152
             }
153 153
             $page = $array['content_alias'];
154 154
             $type .= '/';
@@ -157,21 +157,21 @@  discard block
 block discarded – undo
157 157
                 $type = '';
158 158
             }
159 159
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
160
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
160
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
161 161
             }
162 162
 
163
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name  . $id . $page . $rewrite_ext;
163
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$id.$page.$rewrite_ext;
164 164
             break;
165 165
 
166 166
          case 'short':
167
-            if($topic_name) {
167
+            if ($topic_name) {
168 168
                 $topic_name .= '/';
169 169
             }
170 170
             $rewrite_base = xoops_getModuleOption('rewrite_mode', $module);
171 171
             $rewrite_ext = xoops_getModuleOption('rewrite_ext', $module);
172 172
             $module_name = '';
173
-            if(xoops_getModuleOption('rewrite_name', $module)) {
174
-                $module_name = xoops_getModuleOption('rewrite_name', $module) . '/';
173
+            if (xoops_getModuleOption('rewrite_name', $module)) {
174
+                $module_name = xoops_getModuleOption('rewrite_name', $module).'/';
175 175
             }
176 176
             $page = $array['content_alias'];
177 177
             $type .= '/';
@@ -179,10 +179,10 @@  discard block
 block discarded – undo
179 179
                 $type = '';
180 180
             }
181 181
             if ('comment-edit/' === $type || 'comment-reply/' === $type || 'comment-delete/' === $type) {
182
-                return XOOPS_URL . $rewrite_base . $module_name . $type . $id . '/';
182
+                return XOOPS_URL.$rewrite_base.$module_name.$type.$id.'/';
183 183
             }
184 184
 
185
-            return XOOPS_URL . $rewrite_base . $module_name . $type . $topic_name . $page . $rewrite_ext;
185
+            return XOOPS_URL.$rewrite_base.$module_name.$type.$topic_name.$page.$rewrite_ext;
186 186
             break;
187 187
     }
188 188
     return null;
@@ -207,6 +207,6 @@  discard block
 block discarded – undo
207 207
     $url .= htmlentities($url, ENT_COMPAT, 'utf-8');
208 208
     $url .= \preg_replace('`&([a-z])(acute|uml|circ|grave|ring|cedil|slash|tilde|caron|lig);`i', "\1", $url);
209 209
     $url .= \preg_replace(array($regular_expression, '`[-]+`'), '-', $url);
210
-    $url = ($url == '') ? $type : strtolower(    rim($url, '-'));
210
+    $url = ($url == '') ? $type : strtolower(rim($url, '-'));
211 211
     return $url;
212 212
 }
213 213
\ No newline at end of file
Please login to merge, or discard this patch.
include/common.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -20,29 +20,29 @@
 block discarded – undo
20 20
  * @author         Goffy - XOOPS Development Team - Email:<[email protected]> - Website:<https://wedega.com>
21 21
  */
22 22
 if (!\defined('XOOPS_ICONS32_PATH')) {
23
-    \define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH . '/Frameworks/moduleclasses/icons/32');
23
+    \define('XOOPS_ICONS32_PATH', XOOPS_ROOT_PATH.'/Frameworks/moduleclasses/icons/32');
24 24
 }
25 25
 if (!\defined('XOOPS_ICONS32_URL')) {
26
-    \define('XOOPS_ICONS32_URL', XOOPS_URL . '/Frameworks/moduleclasses/icons/32');
26
+    \define('XOOPS_ICONS32_URL', XOOPS_URL.'/Frameworks/moduleclasses/icons/32');
27 27
 }
28 28
 \define('WGGITHUB_DIRNAME', 'wggithub');
29
-\define('WGGITHUB_PATH', XOOPS_ROOT_PATH . '/modules/' . WGGITHUB_DIRNAME);
30
-\define('WGGITHUB_URL', XOOPS_URL . '/modules/' . WGGITHUB_DIRNAME);
31
-\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH . '/assets/icons');
32
-\define('WGGITHUB_ICONS_URL', WGGITHUB_URL . '/assets/icons');
33
-\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH . '/assets/images');
34
-\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL . '/assets/images');
35
-\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH . '/' . WGGITHUB_DIRNAME);
36
-\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL . '/' . WGGITHUB_DIRNAME);
37
-\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH . '/files');
38
-\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL . '/files');
39
-\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH . '/images');
40
-\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL . '/images');
41
-\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH . '/images/shots');
42
-\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL . '/images/shots');
43
-\define('WGGITHUB_ADMIN', WGGITHUB_URL . '/admin/index.php');
44
-$localLogo = WGGITHUB_IMAGE_URL . '/tdmxoops_logo.png';
29
+\define('WGGITHUB_PATH', XOOPS_ROOT_PATH.'/modules/'.WGGITHUB_DIRNAME);
30
+\define('WGGITHUB_URL', XOOPS_URL.'/modules/'.WGGITHUB_DIRNAME);
31
+\define('WGGITHUB_ICONS_PATH', WGGITHUB_PATH.'/assets/icons');
32
+\define('WGGITHUB_ICONS_URL', WGGITHUB_URL.'/assets/icons');
33
+\define('WGGITHUB_IMAGE_PATH', WGGITHUB_PATH.'/assets/images');
34
+\define('WGGITHUB_IMAGE_URL', WGGITHUB_URL.'/assets/images');
35
+\define('WGGITHUB_UPLOAD_PATH', XOOPS_UPLOAD_PATH.'/'.WGGITHUB_DIRNAME);
36
+\define('WGGITHUB_UPLOAD_URL', XOOPS_UPLOAD_URL.'/'.WGGITHUB_DIRNAME);
37
+\define('WGGITHUB_UPLOAD_FILES_PATH', WGGITHUB_UPLOAD_PATH.'/files');
38
+\define('WGGITHUB_UPLOAD_FILES_URL', WGGITHUB_UPLOAD_URL.'/files');
39
+\define('WGGITHUB_UPLOAD_IMAGE_PATH', WGGITHUB_UPLOAD_PATH.'/images');
40
+\define('WGGITHUB_UPLOAD_IMAGE_URL', WGGITHUB_UPLOAD_URL.'/images');
41
+\define('WGGITHUB_UPLOAD_SHOTS_PATH', WGGITHUB_UPLOAD_PATH.'/images/shots');
42
+\define('WGGITHUB_UPLOAD_SHOTS_URL', WGGITHUB_UPLOAD_URL.'/images/shots');
43
+\define('WGGITHUB_ADMIN', WGGITHUB_URL.'/admin/index.php');
44
+$localLogo = WGGITHUB_IMAGE_URL.'/tdmxoops_logo.png';
45 45
 // Module Information
46
-$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='" . $localLogo . "' alt='XOOPS on Wedega' /></a>";
47
-include_once XOOPS_ROOT_PATH . '/class/xoopsrequest.php';
48
-include_once WGGITHUB_PATH . '/include/functions.php';
46
+$copyright = "<a href='https://wedega.com' title='XOOPS on Wedega' target='_blank'><img src='".$localLogo."' alt='XOOPS on Wedega' /></a>";
47
+include_once XOOPS_ROOT_PATH.'/class/xoopsrequest.php';
48
+include_once WGGITHUB_PATH.'/include/functions.php';
Please login to merge, or discard this patch.
include/update.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
     $ret = wggithub_check_db($module);
40 40
 
41 41
     //check upload directory
42
-    include_once __DIR__ . '/install.php';
42
+    include_once __DIR__.'/install.php';
43 43
     $ret = xoops_module_install_wggithub($module);
44 44
 
45 45
     $errors = $module->getErrors();
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
 {
62 62
     global $xoopsDB;
63 63
     $result = $xoopsDB->query(
64
-        'SELECT t1.tpl_id FROM ' . $xoopsDB->prefix('tplfile') . ' t1, ' . $xoopsDB->prefix('tplfile') . ' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
64
+        'SELECT t1.tpl_id FROM '.$xoopsDB->prefix('tplfile').' t1, '.$xoopsDB->prefix('tplfile').' t2 WHERE t1.tpl_refid = t2.tpl_refid AND t1.tpl_module = t2.tpl_module AND t1.tpl_tplset=t2.tpl_tplset AND t1.tpl_file = t2.tpl_file AND t1.tpl_type = t2.tpl_type AND t1.tpl_id > t2.tpl_id'
65 65
     );
66 66
     $tplids = [];
67 67
     while (false !== (list($tplid) = $xoopsDB->fetchRow($result))) {
@@ -69,7 +69,7 @@  discard block
 block discarded – undo
69 69
     }
70 70
     if (\count($tplids) > 0) {
71 71
         $tplfileHandler  = \xoops_getHandler('tplfile');
72
-        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '(' . \implode(',', $tplids) . ')', 'IN'));
72
+        $duplicate_files = $tplfileHandler->getObjects(new \Criteria('tpl_id', '('.\implode(',', $tplids).')', 'IN'));
73 73
 
74 74
         if (\count($duplicate_files) > 0) {
75 75
             foreach (\array_keys($duplicate_files) as $i) {
@@ -77,9 +77,9 @@  discard block
 block discarded – undo
77 77
             }
78 78
         }
79 79
     }
80
-    $sql = 'SHOW INDEX FROM ' . $xoopsDB->prefix('tplfile') . " WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
80
+    $sql = 'SHOW INDEX FROM '.$xoopsDB->prefix('tplfile')." WHERE KEY_NAME = 'tpl_refid_module_set_file_type'";
81 81
     if (!$result = $xoopsDB->queryF($sql)) {
82
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
82
+        xoops_error($xoopsDB->error().'<br>'.$sql);
83 83
 
84 84
         return false;
85 85
     }
@@ -92,9 +92,9 @@  discard block
 block discarded – undo
92 92
 
93 93
         return true;
94 94
     }
95
-    $sql = 'ALTER TABLE ' . $xoopsDB->prefix('tplfile') . ' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
95
+    $sql = 'ALTER TABLE '.$xoopsDB->prefix('tplfile').' ADD UNIQUE tpl_refid_module_set_file_type ( tpl_refid, tpl_module, tpl_tplset, tpl_file, tpl_type )';
96 96
     if (!$result = $xoopsDB->queryF($sql)) {
97
-        xoops_error($xoopsDB->error() . '<br>' . $sql);
97
+        xoops_error($xoopsDB->error().'<br>'.$sql);
98 98
         $module->setErrors("'tpl_refid_module_set_file_type' unique index is not added to 'tplfile' table. Warning: do not use XOOPS until you add this unique index.");
99 99
 
100 100
         return false;
@@ -118,12 +118,12 @@  discard block
 block discarded – undo
118 118
     // Example: update table (add new field)
119 119
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
120 120
     $field   = 'repo_release';
121
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
121
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
122 122
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
123 123
     if (!$numRows) {
124 124
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
125 125
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
126
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
126
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
127 127
             $module->setErrors("Error when adding '$field' to table '$table'.");
128 128
             $ret = false;
129 129
         }
@@ -132,12 +132,12 @@  discard block
 block discarded – undo
132 132
     // Example: update table (add new field)
133 133
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
134 134
     $field   = 'repo_prerelease';
135
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
135
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
136 136
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
137 137
     if (!$numRows) {
138 138
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
139 139
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
140
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
140
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
141 141
             $module->setErrors("Error when adding '$field' to table '$table'.");
142 142
             $ret = false;
143 143
         }
@@ -145,12 +145,12 @@  discard block
 block discarded – undo
145 145
     // Example: update table (add new field)
146 146
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_repositories');
147 147
     $field   = 'repo_readme';
148
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
148
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
149 149
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
150 150
     if (!$numRows) {
151 151
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `repo_htmlurl`;";
152 152
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
153
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
153
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
154 154
             $module->setErrors("Error when adding '$field' to table '$table'.");
155 155
             $ret = false;
156 156
         }
@@ -159,12 +159,12 @@  discard block
 block discarded – undo
159 159
 // Example: update table (add new field)
160 160
     $table   = $GLOBALS['xoopsDB']->prefix('wggithub_directories');
161 161
     $field   = 'dir_filterrelease';
162
-    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `' . $table . "` LIKE '" . $field . "'");
162
+    $check   = $GLOBALS['xoopsDB']->queryF('SHOW COLUMNS FROM `'.$table."` LIKE '".$field."'");
163 163
     $numRows = $GLOBALS['xoopsDB']->getRowsNum($check);
164 164
     if (!$numRows) {
165 165
         $sql = "ALTER TABLE `$table` ADD `$field` INT(1) NOT NULL DEFAULT '0' AFTER `dir_online`;";
166 166
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
167
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
167
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
168 168
             $module->setErrors("Error when adding '$field' to table '$table'.");
169 169
             $ret = false;
170 170
         }
@@ -186,7 +186,7 @@  discard block
 block discarded – undo
186 186
               PRIMARY KEY (`log_id`)
187 187
                 ) ENGINE=InnoDB;";
188 188
         if (!$result = $GLOBALS['xoopsDB']->queryF($sql)) {
189
-            xoops_error($GLOBALS['xoopsDB']->error() . '<br>' . $sql);
189
+            xoops_error($GLOBALS['xoopsDB']->error().'<br>'.$sql);
190 190
             $module->setErrors("Error when creating table '$table'.");
191 191
             $ret = false;
192 192
         }
Please login to merge, or discard this patch.
include/install.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
  */
30 30
 function xoops_module_pre_install_wggithub(\XoopsModule $module)
31 31
 {
32
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
32
+    require \dirname(__DIR__).'/preloads/autoloader.php';
33 33
 
34 34
     $utility = new Wggithub\Utility();
35 35
 
@@ -42,7 +42,7 @@  discard block
 block discarded – undo
42 42
     if ($xoopsSuccess && $phpSuccess) {
43 43
         $moduleTables = &$module->getInfo('tables');
44 44
         foreach ($moduleTables as $table) {
45
-            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS ' . $GLOBALS['xoopsDB']->prefix($table) . ';');
45
+            $GLOBALS['xoopsDB']->queryF('DROP TABLE IF EXISTS '.$GLOBALS['xoopsDB']->prefix($table).';');
46 46
         }
47 47
     }
48 48
 
@@ -55,7 +55,7 @@  discard block
 block discarded – undo
55 55
  */
56 56
 function xoops_module_install_wggithub(\XoopsModule $module)
57 57
 {
58
-    require \dirname(__DIR__) . '/preloads/autoloader.php';
58
+    require \dirname(__DIR__).'/preloads/autoloader.php';
59 59
 
60 60
     /** @var Wggithub\Helper $helper */ 
61 61
     /** @var Wggithub\Utility $utility */
@@ -79,14 +79,14 @@  discard block
 block discarded – undo
79 79
 
80 80
     //  ---  COPY blank.gif FILES ---------------
81 81
     if ($configurator->copyBlankFiles && \is_array($configurator->copyBlankFiles)) {
82
-        $file = \dirname(__DIR__) . '/assets/images/blank.gif';
82
+        $file = \dirname(__DIR__).'/assets/images/blank.gif';
83 83
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
84
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.gif';
84
+            $dest = $configurator->copyBlankFiles[$i].'/blank.gif';
85 85
             $utility::copyFile($file, $dest);
86 86
         }
87
-        $file = \dirname(__DIR__) . '/assets/images/blank.png';
87
+        $file = \dirname(__DIR__).'/assets/images/blank.png';
88 88
         foreach (\array_keys($configurator->copyBlankFiles) as $i) {
89
-            $dest = $configurator->copyBlankFiles[$i] . '/blank.png';
89
+            $dest = $configurator->copyBlankFiles[$i].'/blank.png';
90 90
             $utility::copyFile($file, $dest);
91 91
         }
92 92
     }
Please login to merge, or discard this patch.
include/search.inc.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -47,7 +47,7 @@  discard block
 block discarded – undo
47 47
     }
48 48
     if ($elementCount > 0) {
49 49
         $crKeywords = new \CriteriaCompo();
50
-        for ($i = 0; $i  <  $elementCount; $i++) {
50
+        for ($i = 0; $i < $elementCount; $i++) {
51 51
             $crKeyword = new \CriteriaCompo();
52 52
             unset($crKeyword);
53 53
         }
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
     if ($userid && \is_array($userid)) {
57 57
         $userid = array_map('intval', $userid);
58 58
         $crUser = new \CriteriaCompo();
59
-        $crUser->add(new \Criteria('repo_submitter', '(' . \implode(',', $userid) . ')', 'IN'), 'OR');
59
+        $crUser->add(new \Criteria('repo_submitter', '('.\implode(',', $userid).')', 'IN'), 'OR');
60 60
     } elseif (is_numeric($userid) && $userid > 0) {
61 61
         $crUser = new \CriteriaCompo();
62 62
         $crUser->add(new \Criteria('repo_submitter', $userid), 'OR');
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
     foreach (\array_keys($repositoriesAll) as $i) {
77 77
         $ret[] = [
78 78
             'image'  => 'assets/icons/16/repositories.png',
79
-            'link'   => 'repositories.php?op=show&amp;repo_id=' . $repositoriesAll[$i]->getVar('repo_id'),
79
+            'link'   => 'repositories.php?op=show&amp;repo_id='.$repositoriesAll[$i]->getVar('repo_id'),
80 80
             'title'  => $repositoriesAll[$i]->getVar('repo_name'),
81 81
             'time'   => $repositoriesAll[$i]->getVar('repo_datecreated')
82 82
         ];
Please login to merge, or discard this patch.
include/notification.inc.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,7 +32,7 @@  discard block
 block discarded – undo
32 32
     global $xoopsDB;
33 33
 
34 34
     if (!\defined('WGGITHUB_URL')) {
35
-        \define('WGGITHUB_URL', \XOOPS_URL . '/modules/wggithub');
35
+        \define('WGGITHUB_URL', \XOOPS_URL.'/modules/wggithub');
36 36
     }
37 37
 
38 38
     switch ($category) {
@@ -42,11 +42,11 @@  discard block
 block discarded – undo
42 42
             return $item;
43 43
             break;
44 44
         case 'repositories':
45
-            $sql          = 'SELECT repo_name FROM ' . $xoopsDB->prefix('wggithub_repositories') . ' WHERE repo_id = '. $item_id;
45
+            $sql          = 'SELECT repo_name FROM '.$xoopsDB->prefix('wggithub_repositories').' WHERE repo_id = '.$item_id;
46 46
             $result       = $xoopsDB->query($sql);
47 47
             $result_array = $xoopsDB->fetchArray($result);
48 48
             $item['name'] = $result_array['repo_name'];
49
-            $item['url']  = WGGITHUB_URL . '/repositories.php?repo_id=' . $item_id;
49
+            $item['url']  = WGGITHUB_URL.'/repositories.php?repo_id='.$item_id;
50 50
             return $item;
51 51
             break;
52 52
     }
Please login to merge, or discard this patch.
admin/logs.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 use XoopsModules\Wggithub\Constants;
26 26
 use XoopsModules\Wggithub\Common;
27 27
 
28
-require __DIR__ . '/header.php';
28
+require __DIR__.'/header.php';
29 29
 // It recovered the value of argument op in URL$
30 30
 $op = Request::getCmd('op', 'list');
31 31
 // Request log_id
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
             }
56 56
             // Display Navigation
57 57
             if ($logsCount > $limit) {
58
-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
59
-                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
58
+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
59
+                $pagenav = new \XoopsPageNav($logsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
60 60
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
61 61
             }
62 62
         } else {
@@ -134,4 +134,4 @@  discard block
 block discarded – undo
134 134
         }
135 135
         break;
136 136
 }
137
-require __DIR__ . '/footer.php';
137
+require __DIR__.'/footer.php';
Please login to merge, or discard this patch.
admin/settings.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 use XoopsModules\Wggithub\Constants;
26 26
 use XoopsModules\Wggithub\Common;
27 27
 
28
-require __DIR__ . '/header.php';
28
+require __DIR__.'/header.php';
29 29
 // It recovered the value of argument op in URL$
30 30
 $op = Request::getCmd('op', 'list');
31 31
 // Request set_id
@@ -55,8 +55,8 @@  discard block
 block discarded – undo
55 55
             }
56 56
             // Display Navigation
57 57
             if ($settingsCount > $limit) {
58
-                include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
59
-                $pagenav = new \XoopsPageNav($settingsCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
58
+                include_once XOOPS_ROOT_PATH.'/class/pagenav.php';
59
+                $pagenav = new \XoopsPageNav($settingsCount, $limit, $start, 'start', 'op=list&limit='.$limit);
60 60
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
61 61
             }
62 62
         } else {
@@ -140,4 +140,4 @@  discard block
 block discarded – undo
140 140
         }
141 141
         break;
142 142
 }
143
-require __DIR__ . '/footer.php';
143
+require __DIR__.'/footer.php';
Please login to merge, or discard this patch.