Passed
Push — master ( 474df4...e81e63 )
by Goffy
03:32
created
class/Common/ServerStats.php 1 patch
Spacing   +12 added lines, -12 removed lines patch added patch discarded remove patch
@@ -36,19 +36,19 @@  discard block
 block discarded – undo
36 36
         //        $query = $GLOBALS['xoopsDB']->query($sql);
37 37
         //        list($meta) = $GLOBALS['xoopsDB']->fetchRow($query);
38 38
         $html .= '<fieldset>';
39
-        $html .= "<legend style='font-weight: bold; color: #900;'>" . \constant('CO_' . $moduleDirNameUpper . '_IMAGEINFO') . '</legend>';
39
+        $html .= "<legend style='font-weight: bold; color: #900;'>".\constant('CO_'.$moduleDirNameUpper.'_IMAGEINFO').'</legend>';
40 40
         $html .= "<div style='padding: 8px;'>";
41 41
         //        $html .= '<div>' . \constant('CO_' . $moduleDirNameUpper . '_METAVERSION') . $meta . "</div>";
42 42
         //        $html .= "<br>";
43 43
         //        $html .= "<br>";
44
-        $html .= '<div>' . \constant('CO_' . $moduleDirNameUpper . '_SPHPINI') . '</div>';
44
+        $html .= '<div>'.\constant('CO_'.$moduleDirNameUpper.'_SPHPINI').'</div>';
45 45
         $html .= '<ul>';
46 46
 
47
-        $gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
48
-        $html  .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
47
+        $gdlib = \function_exists('gd_info') ? '<span style="color: #008000;">'.\constant('CO_'.$moduleDirNameUpper.'_GDON').'</span>' : '<span style="color: #ff0000;">'.\constant('CO_'.$moduleDirNameUpper.'_GDOFF').'</span>';
48
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_GDLIBSTATUS').$gdlib;
49 49
         if (\function_exists('gd_info')) {
50 50
             if (true === ($gdlib = gd_info())) {
51
-                $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
51
+                $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_GDLIBVERSION').'<b>'.$gdlib['GD Version'].'</b>';
52 52
             }
53 53
         }
54 54
 
@@ -58,18 +58,18 @@  discard block
 block discarded – undo
58 58
         //    $registerglobals = (!\ini_get('register_globals')) ? "<span style=\"color: #008000;\">" . \constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>' : "<span style=\"color: #ff0000;\">" . \constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>';
59 59
         //    $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
60 60
 
61
-        $downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">' . \constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: #ff0000;">' . \constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
62
-        $html      .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
61
+        $downloads = \ini_get('file_uploads') ? '<span style="color: #008000;">'.\constant('CO_'.$moduleDirNameUpper.'_ON').'</span>' : '<span style="color: #ff0000;">'.\constant('CO_'.$moduleDirNameUpper.'_OFF').'</span>';
62
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_SERVERUPLOADSTATUS').$downloads;
63 63
 
64
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('upload_max_filesize') . '</span></b>';
65
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: #0000ff;">' . \ini_get('post_max_size') . '</span></b>';
66
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_MEMORYLIMIT') . ' <b><span style="color: #0000ff;">' . \ini_get('memory_limit') . '</span></b>';
64
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MAXUPLOADSIZE').' <b><span style="color: #0000ff;">'.\ini_get('upload_max_filesize').'</span></b>';
65
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MAXPOSTSIZE').' <b><span style="color: #0000ff;">'.\ini_get('post_max_size').'</span></b>';
66
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_MEMORYLIMIT').' <b><span style="color: #0000ff;">'.\ini_get('memory_limit').'</span></b>';
67 67
         $html .= '</ul>';
68 68
         $html .= '<ul>';
69
-        $html .= '<li>' . \constant('CO_' . $moduleDirNameUpper . '_SERVERPATH') . ' <b>' . \XOOPS_ROOT_PATH . '</b>';
69
+        $html .= '<li>'.\constant('CO_'.$moduleDirNameUpper.'_SERVERPATH').' <b>'.\XOOPS_ROOT_PATH.'</b>';
70 70
         $html .= '</ul>';
71 71
         $html .= '<br>';
72
-        $html .= \constant('CO_' . $moduleDirNameUpper . '_UPLOADPATHDSC') . '';
72
+        $html .= \constant('CO_'.$moduleDirNameUpper.'_UPLOADPATHDSC').'';
73 73
         $html .= '</div>';
74 74
         $html .= '</fieldset><br>';
75 75
 
Please login to merge, or discard this patch.
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.
class/Releases.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -136,9 +136,9 @@  discard block
 block discarded – undo
136 136
      */
137 137
     public function getValuesReleases($keys = null, $format = null, $maxDepth = null)
138 138
     {
139
-        $helper  = \XoopsModules\Wggithub\Helper::getInstance();
139
+        $helper = \XoopsModules\Wggithub\Helper::getInstance();
140 140
         $ret = $this->getValues($keys, $format, $maxDepth);
141
-        $ret['id']          = $this->getVar('rel_id');
141
+        $ret['id'] = $this->getVar('rel_id');
142 142
         $repositoriesHandler = $helper->getHandler('Repositories');
143 143
         $repositoriesObj = $repositoriesHandler->get($this->getVar('rel_repoid'));
144 144
         if (\is_object($repositoriesObj)) {
@@ -148,7 +148,7 @@  discard block
 block discarded – undo
148 148
         }
149 149
         $ret['type']        = $this->getVar('rel_type');
150 150
         $ret['name']        = $this->getVar('rel_name');
151
-        $ret['prerelease']  = (int)$this->getVar('rel_prerelease') > 0 ? _YES : _NO;
151
+        $ret['prerelease']  = (int) $this->getVar('rel_prerelease') > 0 ? _YES : _NO;
152 152
         $ret['publishedat'] = \formatTimestamp($this->getVar('rel_publishedat'), 'm');
153 153
         $ret['tarballurl']  = $this->getVar('rel_tarballurl');
154 154
         $ret['zipballurl']  = $this->getVar('rel_zipballurl');
Please login to merge, or discard this patch.
class/Directories.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -176,10 +176,10 @@
 block discarded – undo
176 176
                 break;
177 177
         }
178 178
         $ret['content_shorttext'] = Utility::truncateHtml($content_text);
179
-        $ret['autoupdate']        = (int)$this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
180
-        $ret['online']            = (int)$this->getVar('dir_online') > 0 ? _YES : _NO;
181
-        $ret['filterrelease']     = (int)$this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
182
-        $ret['weight']            = (int)$this->getVar('dir_weight');
179
+        $ret['autoupdate']        = (int) $this->getVar('dir_autoupdate') > 0 ? _YES : _NO;
180
+        $ret['online']            = (int) $this->getVar('dir_online') > 0 ? _YES : _NO;
181
+        $ret['filterrelease']     = (int) $this->getVar('dir_filterrelease') > 0 ? _YES : _NO;
182
+        $ret['weight']            = (int) $this->getVar('dir_weight');
183 183
         $ret['datecreated']       = \formatTimestamp($this->getVar('dir_datecreated'), 's');
184 184
         $ret['submitter']         = \XoopsUser::getUnameFromId($this->getVar('dir_submitter'));
185 185
         return $ret;
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
 use Xmf\Request;
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';
@@ -45,12 +45,12 @@  discard block
 block discarded – undo
45 45
 // InfoBox Statistics
46 46
 $adminObject->addInfoBox(\_AM_WGGITHUB_STATISTICS);
47 47
 // Info elements
48
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_SETTINGS . '</label>', $countSettings));
49
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_DIRECTORIES . '</label>', $countDirectories));
50
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_LOGS . '</label>', $countLogs));
51
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_REPOSITORIES . '</label>', $countRepositories));
52
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_READMES . '</label>', $countReadmes));
53
-$adminObject->addInfoBoxLine(\sprintf( '<label>' . \_AM_WGGITHUB_THEREARE_RELEASES . '</label>', $countReleases));
48
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_SETTINGS.'</label>', $countSettings));
49
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_DIRECTORIES.'</label>', $countDirectories));
50
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_LOGS.'</label>', $countLogs));
51
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_REPOSITORIES.'</label>', $countRepositories));
52
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_READMES.'</label>', $countReadmes));
53
+$adminObject->addInfoBoxLine(\sprintf('<label>'.\_AM_WGGITHUB_THEREARE_RELEASES.'</label>', $countReleases));
54 54
 
55 55
 // Upload Folders
56 56
 $configurator = new Common\Configurator();
@@ -70,12 +70,12 @@  discard block
 block discarded – undo
70 70
 // Test Data
71 71
 if ($helper->getConfig('displaySampleButton')) {
72 72
     \xoops_loadLanguage('admin/modulesadmin', 'system');
73
-    include_once \dirname(__DIR__) . '/testdata/index.php';
74
-    $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
75
-    $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
73
+    include_once \dirname(__DIR__).'/testdata/index.php';
74
+    $adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_ADD_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=load', 'add');
75
+    $adminObject->addItemButton(\constant('CO_'.$moduleDirNameUpper.'_SAVE_SAMPLEDATA'), '__DIR__ . /../../testdata/index.php?op=save', 'add');
76 76
 //    $adminObject->addItemButton(\constant('CO_' . $moduleDirNameUpper . '_EXPORT_SCHEMA'), '__DIR__ . /../../testdata/index.php?op=exportschema', 'add');
77 77
     $adminObject->displayButton('left');
78 78
 }
79 79
 $GLOBALS['xoopsTpl']->assign('index', $adminObject->displayIndex());
80 80
 // End Test Data
81
-require __DIR__ . '/footer.php';
81
+require __DIR__.'/footer.php';
Please login to merge, or discard this patch.
class/Readmes.php 1 patch
Spacing   +14 added lines, -14 removed lines patch added patch discarded remove patch
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
         $helper  = \XoopsModules\Wggithub\Helper::getInstance();
138 138
         $utility = new \XoopsModules\Wggithub\Utility();
139 139
         $ret = $this->getValues($keys, $format, $maxDepth);
140
-        $ret['id']            = $this->getVar('rm_id');
140
+        $ret['id'] = $this->getVar('rm_id');
141 141
         $repositoriesHandler = $helper->getHandler('Repositories');
142 142
         $repositoriesObj = $repositoriesHandler->get($this->getVar('rm_repoid'));
143 143
         $repoName = '*****missing repo_name*****';
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
         $ret['submitter']     = \XoopsUser::getUnameFromId($this->getVar('rm_submitter'));
170 170
         $ret['gitbook_link']  = '';
171 171
         if (\strpos($ret['downloadurl'], 'XoopsDoc') > 0) {
172
-            $ret['gitbook_link']  = 'https://xoops.gitbook.io/' . $repoName . '/';
172
+            $ret['gitbook_link'] = 'https://xoops.gitbook.io/'.$repoName.'/';
173 173
         }
174 174
         return $ret;
175 175
     }
@@ -214,22 +214,22 @@  discard block
 block discarded – undo
214 214
         $arrReplace = [
215 215
             'src="https://xoops.org/images/logoXoopsPhp8.png',
216 216
             "src='https://xoops.org/images/logoXoopsPhp8.png",
217
-            'src="' . $baseUrl . '.gitbook/assets/',
218
-            "src='" . $baseUrl . '.gitbook/assets/',
219
-            'src="' . $baseUrl . 'en/assets/',
220
-            "src='" . $baseUrl . 'en/assets/',
221
-            'src="' . $baseUrl . 'assets/',
222
-            "src='" . $baseUrl . 'assets/'
217
+            'src="'.$baseUrl.'.gitbook/assets/',
218
+            "src='".$baseUrl.'.gitbook/assets/',
219
+            'src="'.$baseUrl.'en/assets/',
220
+            "src='".$baseUrl.'en/assets/',
221
+            'src="'.$baseUrl.'assets/',
222
+            "src='".$baseUrl.'assets/'
223 223
         ];
224 224
         $contentClean = \str_replace($arrSearch, $arrReplace, $contentClean);
225 225
 
226 226
         //replace known missing images
227 227
         $arrSearch = [
228
-            'src="' . \XOOPS_URL . '/screenshot.jpg',
229
-            'src="' . \XOOPS_URL . '/sd-099-theme-surprice.gif',
230
-            'src="' . \XOOPS_URL . '/shot.gif',
231
-            'src="' . \XOOPS_URL . '/shot.jpg',
232
-            'src="' . \XOOPS_URL . '/theme_preview.png',
228
+            'src="'.\XOOPS_URL.'/screenshot.jpg',
229
+            'src="'.\XOOPS_URL.'/sd-099-theme-surprice.gif',
230
+            'src="'.\XOOPS_URL.'/shot.gif',
231
+            'src="'.\XOOPS_URL.'/shot.jpg',
232
+            'src="'.\XOOPS_URL.'/theme_preview.png',
233 233
             'src="https://github.com/xoops/newbb-tutorial/tree/54ef6104e66b74e8c6dea683d3cce70ceafdd739/assets/image001.jpg',
234 234
             'src="https://github.com/xoops/repairshop-tutorial/tree/719d6d32dea221b3e980fd3d6222f179ac898394/en/assets/img_2.jpg',
235 235
             'src="https://github.com/xoops/xoops-mylinks-tutorial/tree/d30c30645d48c5ead51399ff711c25935cbd0495/en/assets/forkit.png',
@@ -300,7 +300,7 @@  discard block
 block discarded – undo
300 300
             'src="https://insight.sensiolabs.com/projects/7ada0220-a68e-4369-aef0-950172630ff8/mini.png',
301 301
             'src="https://xoops.org/images/translations-github-blue.svg',
302 302
         ];
303
-        $contentClean = \str_replace($arrSearch, 'src="' . \WGGITHUB_IMAGE_URL . '/blank.gif', $contentClean);
303
+        $contentClean = \str_replace($arrSearch, 'src="'.\WGGITHUB_IMAGE_URL.'/blank.gif', $contentClean);
304 304
 
305 305
         return $contentClean;
306 306
     }
Please login to merge, or discard this patch.
include/comment_functions.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -32,9 +32,9 @@  discard block
 block discarded – undo
32 32
     // Get instance of module
33 33
     $helper = \XoopsModules\Wggithub\Helper::getInstance();
34 34
     $repositoriesHandler = $helper->getHandler('Repositories');
35
-    $repoId = (int)$itemId;
35
+    $repoId = (int) $itemId;
36 36
     $repositoriesObj = $repositoriesHandler->get($repoId);
37
-    $repositoriesObj->setVar('repo_comments', (int)$itemNumb);
37
+    $repositoriesObj->setVar('repo_comments', (int) $itemNumb);
38 38
     if ($repositoriesHandler->insert($repositoriesObj)) {
39 39
         return true;
40 40
     }
@@ -59,7 +59,7 @@  discard block
 block discarded – undo
59 59
 
60 60
     $tags = [];
61 61
     $tags['ITEM_NAME'] = $repoName;
62
-    $tags['ITEM_URL']  = \XOOPS_URL . '/modules/wggithub/repositories.php?op=show&repo_id=' . $repoId;
62
+    $tags['ITEM_URL']  = \XOOPS_URL.'/modules/wggithub/repositories.php?op=show&repo_id='.$repoId;
63 63
     $notificationHandler = \xoops_getHandler('notification');
64 64
     // Event modify notification
65 65
     $notificationHandler->triggerEvent('global', 0, 'global_comment', $tags);
Please login to merge, or discard this patch.
class/Common/FileChecker.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
25 25
 
26 26
 //\defined('XOOPS_ROOT_PATH') || die('XOOPS root path not defined');
27 27
 
28
-require_once \dirname(__DIR__, 4) . '/mainfile.php';
28
+require_once \dirname(__DIR__, 4).'/mainfile.php';
29 29
 $moduleDirName      = \basename(\dirname(__DIR__, 2));
30 30
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
31 31
 \xoops_loadLanguage('filechecker', $moduleDirName);
@@ -57,24 +57,24 @@  discard block
 block discarded – undo
57 57
         if (null === $original_file_path) {
58 58
             if (self::fileExists($file_path)) {
59 59
                 $path_status = "<img src='$pathIcon16/1.png' >";
60
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
60
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_AVAILABLE').') ';
61 61
             } else {
62 62
                 $path_status = "<img src='$pathIcon16/0.png' >";
63
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
63
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE').') ';
64 64
             }
65 65
         } else {
66 66
             if (self::compareFiles($file_path, $original_file_path)) {
67 67
                 $path_status = "<img src='$pathIcon16/1.png' >";
68
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_AVAILABLE') . ') ';
68
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_AVAILABLE').') ';
69 69
             } else {
70 70
                 $path_status = "<img src='$pathIcon16/0.png' >";
71
-                $path_status .= "$file_path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_NOTAVAILABLE') . ') ';
72
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
71
+                $path_status .= "$file_path (".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_NOTAVAILABLE').') ';
72
+                $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
73 73
                 $path_status .= "<input type='hidden' name='op' value='copyfile'>";
74 74
                 $path_status .= "<input type='hidden' name='file_path' value='$file_path'>";
75 75
                 $path_status .= "<input type='hidden' name='original_file_path' value='$original_file_path'>";
76 76
                 $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
77
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_CREATETHEFILE') . '</button>';
77
+                $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'FC_CREATETHEFILE').'</button>';
78 78
                 $path_status .= '</form>';
79 79
             }
80 80
         }
@@ -139,7 +139,7 @@  discard block
 block discarded – undo
139 139
     {
140 140
         $target = \str_replace('..', '', $target);
141 141
 
142
-        return @\chmod($target, (int)$mode);
142
+        return @\chmod($target, (int) $mode);
143 143
     }
144 144
 }
145 145
 
@@ -155,7 +155,7 @@  discard block
 block discarded – undo
155 155
         if (\Xmf\Request::hasVar('redirect', 'POST')) {
156 156
             $redirect = $_POST['redirect'];
157 157
         }
158
-        $msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILECOPIED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'FC_FILENOTCOPIED');
159
-        \redirect_header($redirect, 2, $msg . ': ' . $file_path);
158
+        $msg = FileChecker::copyFile($original_file_path, $file_path) ? \constant('CO_'.$moduleDirNameUpper.'_'.'FC_FILECOPIED') : \constant('CO_'.$moduleDirNameUpper.'_'.'FC_FILENOTCOPIED');
159
+        \redirect_header($redirect, 2, $msg.': '.$file_path);
160 160
         break;
161 161
 }
Please login to merge, or discard this patch.
class/Common/DirectoryChecker.php 1 patch
Spacing   +16 added lines, -16 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 use XoopsModules\Wggithub;
25 25
 
26 26
 
27
-require_once \dirname(__DIR__, 4) . '/mainfile.php';
27
+require_once \dirname(__DIR__, 4).'/mainfile.php';
28 28
 $moduleDirName      = \basename(\dirname(__DIR__, 2));
29 29
 $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
30 30
 \xoops_loadLanguage('directorychecker', $moduleDirName);
@@ -56,38 +56,38 @@  discard block
 block discarded – undo
56 56
         $moduleDirNameUpper = \mb_strtoupper($moduleDirName);
57 57
         if (!@\is_dir($path)) {
58 58
             $path_status = "<img src='$pathIcon16/0.png' >";
59
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTAVAILABLE') . ') ';
60
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
59
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTAVAILABLE').') ';
60
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
61 61
             $path_status .= "<input type='hidden' name='op' value='createdir'>";
62 62
             $path_status .= "<input type='hidden' name='path' value='$path'>";
63 63
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
64
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_CREATETHEDIR') . '</button>';
64
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_CREATETHEDIR').'</button>';
65 65
             $path_status .= '</form>';
66 66
         } elseif (@\is_writable($path)) {
67 67
             $path_status = "<img src='$pathIcon16/1.png' >";
68
-            $path_status .= "$path (" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_AVAILABLE') . ') ';
68
+            $path_status .= "$path (".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_AVAILABLE').') ';
69 69
             $currentMode = \mb_substr(\decoct(\fileperms($path)), 2);
70 70
             if ($currentMode != \decoct($mode)) {
71 71
                 $path_status = "<img src='$pathIcon16/0.png' >";
72
-                $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
73
-                $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
72
+                $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
73
+                $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
74 74
                 $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
75 75
                 $path_status .= "<input type='hidden' name='mode' value='$mode'>";
76 76
                 $path_status .= "<input type='hidden' name='path' value='$path'>";
77 77
                 $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
78
-                $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
78
+                $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
79 79
                 $path_status .= '</form>';
80 80
             }
81 81
         } else {
82 82
             $currentMode = \mb_substr(\decoct(\fileperms($path)), 2);
83 83
             $path_status = "<img src='$pathIcon16/0.png' >";
84
-            $path_status .= $path . \sprintf(\constant('CO_' . $moduleDirNameUpper . '_' . 'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
85
-            $path_status .= "<form action='" . $_SERVER['SCRIPT_NAME'] . "' method='post'>";
84
+            $path_status .= $path.\sprintf(\constant('CO_'.$moduleDirNameUpper.'_'.'DC_NOTWRITABLE'), \decoct($mode), $currentMode);
85
+            $path_status .= "<form action='".$_SERVER['SCRIPT_NAME']."' method='post'>";
86 86
             $path_status .= "<input type='hidden' name='op' value='setdirperm'>";
87 87
             $path_status .= "<input type='hidden' name='mode' value='$mode'>";
88 88
             $path_status .= "<input type='hidden' name='path' value='$path'>";
89 89
             $path_status .= "<input type='hidden' name='redirect' value='$redirectFile'>";
90
-            $path_status .= "<button class='submit' onClick='this.form.submit();'>" . \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_SETMPERM') . '</button>';
90
+            $path_status .= "<button class='submit' onClick='this.form.submit();'>".\constant('CO_'.$moduleDirNameUpper.'_'.'DC_SETMPERM').'</button>';
91 91
             $path_status .= '</form>';
92 92
         }
93 93
 
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
     {
119 119
         $target = \str_replace('..', '', $target);
120 120
 
121
-        return @\chmod($target, (int)$mode);
121
+        return @\chmod($target, (int) $mode);
122 122
     }
123 123
 
124 124
     /**
@@ -141,8 +141,8 @@  discard block
 block discarded – undo
141 141
         if (\Xmf\Request::hasVar('redirect', 'POST')) {
142 142
             $redirect = $_POST['redirect'];
143 143
         }
144
-        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRCREATED') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_DIRNOTCREATED');
145
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
144
+        $msg = DirectoryChecker::createDirectory($path) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRCREATED') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_DIRNOTCREATED');
145
+        \redirect_header($redirect, 2, $msg.': '.$path);
146 146
         break;
147 147
     case 'setdirperm':
148 148
         if (\Xmf\Request::hasVar('path', 'POST')) {
@@ -154,7 +154,7 @@  discard block
 block discarded – undo
154 154
         if (\Xmf\Request::hasVar('mode', 'POST')) {
155 155
             $mode = $_POST['mode'];
156 156
         }
157
-        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMSET') : \constant('CO_' . $moduleDirNameUpper . '_' . 'DC_PERMNOTSET');
158
-        \redirect_header($redirect, 2, $msg . ': ' . $path);
157
+        $msg = DirectoryChecker::setDirectoryPermissions($path, $mode) ? \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMSET') : \constant('CO_'.$moduleDirNameUpper.'_'.'DC_PERMNOTSET');
158
+        \redirect_header($redirect, 2, $msg.': '.$path);
159 159
         break;
160 160
 }
Please login to merge, or discard this patch.