Passed
Push — master ( f54565...4ee54c )
by Goffy
03:32
created
class/SettingsHandler.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
      */
149 149
     public function setPrimarySetting($setId)
150 150
     {
151
-        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
151
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
152 152
         $settingsObj = null;
153 153
         $settingsHandler = $helper->getHandler('Settings');
154 154
         if (isset($setId)) {
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
         }
159 159
 
160 160
         // reset all
161
-        $strSQL = 'UPDATE ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . ' SET ' . $GLOBALS['xoopsDB']->prefix('wggithub_settings') . '.set_primary = 0';
161
+        $strSQL = 'UPDATE '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').' SET '.$GLOBALS['xoopsDB']->prefix('wggithub_settings').'.set_primary = 0';
162 162
         $GLOBALS['xoopsDB']->queryF($strSQL);
163 163
         // Set Vars
164 164
         $settingsObj->setVar('set_primary', 1);
Please login to merge, or discard this patch.
admin/github-api.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -30,7 +30,7 @@  discard block
 block discarded – undo
30 30
 use XoopsModules\Wggithub;
31 31
 use XoopsModules\Wggithub\Github;
32 32
 
33
-require __DIR__ . '/header.php';
33
+require __DIR__.'/header.php';
34 34
 // It recovered the value of argument op in URL$
35 35
 
36 36
 
@@ -41,8 +41,8 @@  discard block
 block discarded – undo
41 41
 
42 42
 $count = 0;
43 43
 foreach ($data as $key => $repo) {
44
-    echo "<br>key:" . $key . ' repo:' . $repo['name'];
44
+    echo "<br>key:".$key.' repo:'.$repo['name'];
45 45
     $count++;
46
-    if ($count>5) {break;}
46
+    if ($count > 5) {break; }
47 47
 }
48 48
 echo "<br>OK 2";
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/index.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -23,8 +23,8 @@  discard block
 block discarded – undo
23 23
 
24 24
 use XoopsModules\Wggithub\Common;
25 25
 
26
-include_once \dirname(__DIR__) . '/preloads/autoloader.php';
27
-require __DIR__ . '/header.php';
26
+include_once \dirname(__DIR__).'/preloads/autoloader.php';
27
+require __DIR__.'/header.php';
28 28
 
29 29
 // Template Index
30 30
 $templateMain = 'wggithub_admin_index.tpl';
@@ -40,12 +40,12 @@  discard block
 block discarded – undo
40 40
 // InfoBox Statistics
41 41
 $adminObject->addInfoBox(_AM_WGGITHUB_STATISTICS);
42 42
 // Info elements
43
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_SETTINGS . '</label>', $countSettings));
44
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_DIRECTORIES . '</label>', $countDirectories));
45
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_LOGS . '</label>', $countLogs));
46
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_REPOSITORIES . '</label>', $countRepositories));
47
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_READMES . '</label>', $countReadmes));
48
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . _AM_WGGITHUB_THEREARE_RELEASES . '</label>', $countReleases));
43
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_SETTINGS.'</label>', $countSettings));
44
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_DIRECTORIES.'</label>', $countDirectories));
45
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_LOGS.'</label>', $countLogs));
46
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_REPOSITORIES.'</label>', $countRepositories));
47
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_READMES.'</label>', $countReadmes));
48
+$adminObject->addInfoBoxLine(\sprintf('<label>'._AM_WGGITHUB_THEREARE_RELEASES.'</label>', $countReleases));
49 49
 
50 50
 // Upload Folders
51 51
 $configurator = new Common\Configurator();
@@ -65,12 +65,12 @@  discard block
 block discarded – undo
65 65
 // Test Data
66 66
 if ($helper->getConfig('displaySampleButton')) {
67 67
 	\xoops_loadLanguage('admin/modulesadmin', 'system');
68
-	include_once \dirname(__DIR__) . '/testdata/index.php';
69
-	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
70
-	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
68
+	include_once \dirname(__DIR__).'/testdata/index.php';
69
+	$adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
70
+	$adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
71 71
 //	$adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
72 72
 	$adminObject->displayButton('left');
73 73
 }
74 74
 $GLOBALS['xoopsTpl']->assign('index', $adminObject->displayIndex());
75 75
 // End Test Data
76
-require __DIR__ . '/footer.php';
76
+require __DIR__.'/footer.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.
index.php 1 patch
Spacing   +17 added lines, -17 removed lines patch added patch discarded remove patch
@@ -28,15 +28,15 @@  discard block
 block discarded – undo
28 28
     Github\GithubClient
29 29
 };
30 30
 
31
-require __DIR__ . '/header.php';
31
+require __DIR__.'/header.php';
32 32
 $GLOBALS['xoopsOption']['template_main'] = 'wggithub_index.tpl';
33
-include_once \XOOPS_ROOT_PATH . '/header.php';
33
+include_once \XOOPS_ROOT_PATH.'/header.php';
34 34
 
35 35
 // Permissions
36 36
 $permGlobalView = $permissionsHandler->getPermGlobalView();
37 37
 if (!$permGlobalView) {
38 38
     $GLOBALS['xoopsTpl']->assign('error', _NOPERM);
39
-    require __DIR__ . '/footer.php';
39
+    require __DIR__.'/footer.php';
40 40
 }
41 41
 $permGlobalRead   = $permissionsHandler->getPermGlobalRead();
42 42
 $permReadmeUpdate = $permissionsHandler->getPermReadmeUpdate();
@@ -50,7 +50,7 @@  discard block
 block discarded – undo
50 50
 
51 51
 // Define Stylesheet
52 52
 $GLOBALS['xoTheme']->addStylesheet($style, null);
53
-$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL . '/assets/css/tabs.css', null);
53
+$GLOBALS['xoTheme']->addStylesheet(WGGITHUB_URL.'/assets/css/tabs.css', null);
54 54
 $keywords = [];
55 55
 // 
56 56
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
@@ -120,13 +120,13 @@  discard block
 block discarded – undo
120 120
             foreach (\array_keys($directoriesAll) as $i) {
121 121
                 $directories[$i] = $directoriesAll[$i]->getValuesDirectories();
122 122
                 $dirName = $directoriesAll[$i]->getVar('dir_name');
123
-                $dirFilterRelease = (bool)$directoriesAll[$i]->getVar('dir_filterrelease');
123
+                $dirFilterRelease = (bool) $directoriesAll[$i]->getVar('dir_filterrelease');
124 124
                 $repos = [];
125 125
                 $crRepositories = new \CriteriaCompo();
126 126
                 $crRepositories->add(new \Criteria('repo_user', $dirName));
127 127
                 $repositoriesCountTotal = $repositoriesHandler->getCount($crRepositories);
128 128
                 if ('any' === $filterRelease && $dirFilterRelease) {
129
-                    $crRepositories->add(new \Criteria('repo_prerelease', 1) );
129
+                    $crRepositories->add(new \Criteria('repo_prerelease', 1));
130 130
                     $crRepositories->add(new \Criteria('repo_release', 1), 'OR');
131 131
                 }
132 132
                 if ('final' === $filterRelease && $dirFilterRelease) {
@@ -181,9 +181,9 @@  discard block
 block discarded – undo
181 181
                 }
182 182
                 $directories[$i]['repos'] = $repos;
183 183
                 $directories[$i]['previousRepos'] = $start > 0;
184
-                $directories[$i]['previousOp'] = '&amp;start=' . ($start - $limit) . '&amp;limit=' . $limit . '&amp;release=' . $filterRelease . '&amp;sortby=' . $filterSortby;
184
+                $directories[$i]['previousOp'] = '&amp;start='.($start - $limit).'&amp;limit='.$limit.'&amp;release='.$filterRelease.'&amp;sortby='.$filterSortby;
185 185
                 $directories[$i]['nextRepos'] = ($repositoriesCount - $start) > $limit;
186
-                $directories[$i]['nextOp'] = '&amp;start=' . ($start + $limit) . '&amp;limit=' . $limit . '&amp;release=' . $filterRelease . '&amp;sortby=' . $filterSortby;
186
+                $directories[$i]['nextOp'] = '&amp;start='.($start + $limit).'&amp;limit='.$limit.'&amp;release='.$filterRelease.'&amp;sortby='.$filterSortby;
187 187
                 $GLOBALS['xoopsTpl']->assign('start', $start);
188 188
                 $GLOBALS['xoopsTpl']->assign('limit', $limit);
189 189
                 $GLOBALS['xoopsTpl']->assign('menu', $menu);
@@ -194,8 +194,8 @@  discard block
 block discarded – undo
194 194
             unset($crDirectories, $directories);
195 195
             // Display Navigation
196 196
             if ($directoriesCount > $limit) {
197
-                require_once \XOOPS_ROOT_PATH . '/class/pagenav.php';
198
-                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit=' . $limit);
197
+                require_once \XOOPS_ROOT_PATH.'/class/pagenav.php';
198
+                $pagenav = new \XoopsPageNav($directoriesCount, $limit, $start, 'start', 'op=list&limit='.$limit);
199 199
                 $GLOBALS['xoopsTpl']->assign('pagenav', $pagenav->renderNav(4));
200 200
             }
201 201
             $GLOBALS['xoopsTpl']->assign('lang_thereare', \sprintf(\_MA_WGGITHUB_INDEX_THEREARE, $directoriesCount));
@@ -208,11 +208,11 @@  discard block
 block discarded – undo
208 208
         // Permissions
209 209
         if (!$permGlobalRead) {
210 210
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
211
-            require __DIR__ . '/footer.php';
211
+            require __DIR__.'/footer.php';
212 212
         }
213 213
         $dirName = Request::getString('dir_name', '');
214 214
         $result = executeUpdate($dirName);
215
-        $redir = 'index.php?op=list_afterupdate&amp;start=' . $start . '&amp;limit=' . $limit . '&amp;release=' . $filterRelease . '&amp;sortby=' . $filterSortby;
215
+        $redir = 'index.php?op=list_afterupdate&amp;start='.$start.'&amp;limit='.$limit.'&amp;release='.$filterRelease.'&amp;sortby='.$filterSortby;
216 216
         if ($result) {
217 217
             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);
218 218
         } else {
@@ -224,13 +224,13 @@  discard block
 block discarded – undo
224 224
         // Permissions
225 225
         if (!$permReadmeUpdate) {
226 226
             $GLOBALS['xoopsTpl']->assign('error', \_NOPERM);
227
-            require __DIR__ . '/footer.php';
227
+            require __DIR__.'/footer.php';
228 228
         }
229 229
         $repoId = Request::getInt('repo_id', 0);
230 230
         $repoUser  = Request::getString('repo_user', 'none');
231 231
         $repoName  = Request::getString('repo_name', 'none');
232 232
         $res = $helper->getHandler('Readmes')->updateReadmes($repoId, $repoUser, $repoName);
233
-        $redir = 'index.php?op=list_afterupdate&amp;start=' . $start . '&amp;limit=' . $limit . '&amp;release=' . $filterRelease . '&amp;sortby=' . $filterSortby;
233
+        $redir = 'index.php?op=list_afterupdate&amp;start='.$start.'&amp;limit='.$limit.'&amp;release='.$filterRelease.'&amp;sortby='.$filterSortby;
234 234
         if ($result) {
235 235
             \redirect_header($redir, 2, \_MA_WGGITHUB_READGH_SUCCESS);
236 236
         } else {
@@ -243,14 +243,14 @@  discard block
 block discarded – undo
243 243
 // Breadcrumbs
244 244
 $xoBreadcrumbs[] = ['title' => \_MA_WGGITHUB_INDEX];
245 245
 // Keywords
246
-wggithubMetaKeywords($helper->getConfig('keywords') . ', ' . \implode(',', $keywords));
246
+wggithubMetaKeywords($helper->getConfig('keywords').', '.\implode(',', $keywords));
247 247
 unset($keywords);
248 248
 // Description
249 249
 wggithubMetaDescription(\_MA_WGGITHUB_INDEX_DESC);
250 250
 $GLOBALS['xoopsTpl']->assign('xoops_mpageurl', WGGITHUB_URL.'/index.php');
251 251
 $GLOBALS['xoopsTpl']->assign('xoops_icons32_url', XOOPS_ICONS32_URL);
252 252
 $GLOBALS['xoopsTpl']->assign('wggithub_upload_url', WGGITHUB_UPLOAD_URL);
253
-require __DIR__ . '/footer.php';
253
+require __DIR__.'/footer.php';
254 254
 
255 255
 
256 256
 /**
@@ -258,7 +258,7 @@  discard block
 block discarded – undo
258 258
  * @param string $dirName
259 259
  * @return bool
260 260
  */
261
-function executeUpdate($dirName = ''){
261
+function executeUpdate($dirName = '') {
262 262
 
263 263
     $githubClient = GithubClient::getInstance();
264 264
     $helper = Helper::getInstance();
Please login to merge, or discard this patch.
class/Github/Paginator.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -148,7 +148,7 @@
 block discarded – undo
148 148
 	 */
149 149
 	public static function parseLink($link, $rel)
150 150
 	{
151
-		if (!preg_match('(<([^>]+)>;\s*rel="' . preg_quote($rel) . '")', $link, $match)) {
151
+		if (!preg_match('(<([^>]+)>;\s*rel="'.preg_quote($rel).'")', $link, $match)) {
152 152
 			return NULL;
153 153
 		}
154 154
 
Please login to merge, or discard this patch.
class/Github/Sanity.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -13,7 +13,7 @@  discard block
 block discarded – undo
13 13
 	 */
14 14
 	public function & __get($name)
15 15
 	{
16
-		throw new LogicException('Cannot read an undeclared property ' . get_class($this) . "::\$$name.");
16
+		throw new LogicException('Cannot read an undeclared property '.get_class($this)."::\$$name.");
17 17
 	}
18 18
 
19 19
 
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
 	 */
23 23
 	public function __set($name, $value)
24 24
 	{
25
-		throw new LogicException('Cannot write to an undeclared property ' . get_class($this) . "::\$$name.");
25
+		throw new LogicException('Cannot write to an undeclared property '.get_class($this)."::\$$name.");
26 26
 	}
27 27
 
28 28
 }
Please login to merge, or discard this patch.
class/Github/Api.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -254,7 +254,7 @@  discard block
 block discarded – undo
254 254
 			$urlPath = $this->expandUriTemplate($urlPath, $parameters, $this->defaultParameters);
255 255
 		}
256 256
 
257
-		$url = rtrim($baseUrl, '/') . '/' . ltrim($urlPath, '/');
257
+		$url = rtrim($baseUrl, '/').'/'.ltrim($urlPath, '/');
258 258
 
259 259
 		if ($content !== NULL && (is_array($content) || is_object($content))) {
260 260
 			$headers['Content-Type'] = 'application/json; charset=utf-8';
@@ -394,13 +394,13 @@  discard block
 block discarded – undo
394 394
 			}
395 395
 			$parameter = $parameters[$m[2]];
396 396
 			unset($parameters[$m[2]]);
397
-			return $m[1] . rawurlencode($parameter);
397
+			return $m[1].rawurlencode($parameter);
398 398
 		}, $url);
399 399
 
400 400
 		$url = rtrim($url, '/');
401 401
 
402 402
 		if (count($parameters)) {
403
-			$url .= '?' . http_build_query($parameters);
403
+			$url .= '?'.http_build_query($parameters);
404 404
 		}
405 405
 
406 406
 		return $url;
@@ -421,14 +421,14 @@  discard block
 block discarded – undo
421 421
 		$parameters += $defaultParameters;
422 422
 
423 423
 		static $operatorFlags = [
424
-			''  => ['prefix' => '',  'separator' => ',', 'named' => FALSE, 'ifEmpty' => '',  'reserved' => FALSE],
425
-			'+' => ['prefix' => '',  'separator' => ',', 'named' => FALSE, 'ifEmpty' => '',  'reserved' => TRUE],
426
-			'#' => ['prefix' => '#', 'separator' => ',', 'named' => FALSE, 'ifEmpty' => '',  'reserved' => TRUE],
427
-			'.' => ['prefix' => '.', 'separator' => '.', 'named' => FALSE, 'ifEmpty' => '',  'reserved' => FALSE],
428
-			'/' => ['prefix' => '/', 'separator' => '/', 'named' => FALSE, 'ifEmpty' => '',  'reserved' => FALSE],
429
-			';' => ['prefix' => ';', 'separator' => ';', 'named' => TRUE,  'ifEmpty' => '',  'reserved' => FALSE],
430
-			'?' => ['prefix' => '?', 'separator' => '&', 'named' => TRUE,  'ifEmpty' => '=', 'reserved' => FALSE],
431
-			'&' => ['prefix' => '&', 'separator' => '&', 'named' => TRUE,  'ifEmpty' => '=', 'reserved' => FALSE],
424
+			''  => ['prefix' => '', 'separator' => ',', 'named' => FALSE, 'ifEmpty' => '', 'reserved' => FALSE],
425
+			'+' => ['prefix' => '', 'separator' => ',', 'named' => FALSE, 'ifEmpty' => '', 'reserved' => TRUE],
426
+			'#' => ['prefix' => '#', 'separator' => ',', 'named' => FALSE, 'ifEmpty' => '', 'reserved' => TRUE],
427
+			'.' => ['prefix' => '.', 'separator' => '.', 'named' => FALSE, 'ifEmpty' => '', 'reserved' => FALSE],
428
+			'/' => ['prefix' => '/', 'separator' => '/', 'named' => FALSE, 'ifEmpty' => '', 'reserved' => FALSE],
429
+			';' => ['prefix' => ';', 'separator' => ';', 'named' => TRUE, 'ifEmpty' => '', 'reserved' => FALSE],
430
+			'?' => ['prefix' => '?', 'separator' => '&', 'named' => TRUE, 'ifEmpty' => '=', 'reserved' => FALSE],
431
+			'&' => ['prefix' => '&', 'separator' => '&', 'named' => TRUE, 'ifEmpty' => '=', 'reserved' => FALSE],
432 432
 		];
433 433
 
434 434
 		return preg_replace_callback('~{([+#./;?&])?([^}]+?)}~', function($m) use ($url, & $parameters, $operatorFlags) {
@@ -463,17 +463,17 @@  discard block
 block discarded – undo
463 463
 					if ($explode) {
464 464
 						$parts = [];
465 465
 						if ($isAssoc) {
466
-							$this->walk($value, function ($v, $k) use (& $parts, $flags, $maxLength) {
466
+							$this->walk($value, function($v, $k) use (& $parts, $flags, $maxLength) {
467 467
 								$parts[] = $this->prefix(['named' => TRUE] + $flags, $k, $this->escape($flags, $v, $maxLength));
468 468
 							});
469 469
 
470 470
 						} elseif ($flags['named']) {
471
-							$this->walk($value, function ($v) use (& $parts, $flags, $name, $maxLength) {
471
+							$this->walk($value, function($v) use (& $parts, $flags, $name, $maxLength) {
472 472
 								$parts[] = $this->prefix($flags, $name, $this->escape($flags, $v, $maxLength));
473 473
 							});
474 474
 
475 475
 						} else {
476
-							$this->walk($value, function ($v) use (& $parts, $flags, $maxLength) {
476
+							$this->walk($value, function($v) use (& $parts, $flags, $maxLength) {
477 477
 								$parts[] = $this->escape($flags, $v, $maxLength);
478 478
 							});
479 479
 						}
@@ -504,7 +504,7 @@  discard block
 block discarded – undo
504 504
 			}
505 505
 
506 506
 			if (isset($translated[0])) {
507
-				return $flags['prefix'] . implode($flags['separator'], $translated);
507
+				return $flags['prefix'].implode($flags['separator'], $translated);
508 508
 			}
509 509
 
510 510
 			return '';
@@ -530,7 +530,7 @@  discard block
 block discarded – undo
530 530
 			}
531 531
 		}
532 532
 
533
-		return $prefix . $value;
533
+		return $prefix.$value;
534 534
 	}
535 535
 
536 536
 
@@ -545,7 +545,7 @@  discard block
 block discarded – undo
545 545
 		$value = (string) $value;
546 546
 
547 547
 		if ($maxLength !== NULL) {
548
-			if (preg_match('~^(.{' . $maxLength . '}).~u', $value, $m)) {
548
+			if (preg_match('~^(.{'.$maxLength.'}).~u', $value, $m)) {
549 549
 				$value = $m[1];
550 550
 			} elseif (strlen($value) > $maxLength) {  # when malformed UTF-8
551 551
 				$value = substr($value, 0, $maxLength);
@@ -558,7 +558,7 @@  discard block
 block discarded – undo
558 558
 
559 559
 			$escaped = '';
560 560
 			for ($i = 0, $count = count($parts); $i < $count; $i += 2) {
561
-				$escaped .= rawurlencode($parts[$i]) . $parts[$i + 1];
561
+				$escaped .= rawurlencode($parts[$i]).$parts[$i + 1];
562 562
 			}
563 563
 
564 564
 			return $escaped;
@@ -596,7 +596,7 @@  discard block
 block discarded – undo
596 596
 
597 597
 		if (isset($content->errors)) {
598 598
 			$message .= implode(', ', array_map(function($error) {
599
-				return '[' . implode(':', (array) $error) . ']';
599
+				return '['.implode(':', (array) $error).']';
600 600
 			}, $content->errors));
601 601
 		}
602 602
 
Please login to merge, or discard this patch.