Completed
Push — master ( 825da9...adb11b )
by Michael
02:15
created
preloads/autoloader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,7 @@
 block discarded – undo
3 3
 /**
4 4
  * @see http://www.php-fig.org/psr/psr-4/examples/
5 5
  */
6
-spl_autoload_register(function ($class) {
6
+spl_autoload_register(function($class) {
7 7
     // project-specific namespace prefix
8 8
     $prefix = 'Xoopsmodules\\instruction\\';
9 9
 
Please login to merge, or discard this patch.
index.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -17,9 +17,9 @@  discard block
 block discarded – undo
17 17
 // Стили
18 18
 $xoTheme->addStylesheet(XOOPS_URL . '/modules/' . $moduleDirName . '/assets/css/style.css');
19 19
 //
20
-$cid = isset($_GET['cid']) ? (int)$_GET['cid'] : 0;
20
+$cid = isset($_GET['cid']) ? (int) $_GET['cid'] : 0;
21 21
 //
22
-$start = isset($_GET['start']) ? (int)$_GET['start'] : 0;
22
+$start = isset($_GET['start']) ? (int) $_GET['start'] : 0;
23 23
 //
24 24
 
25 25
 //mb $limit = xoops_getModuleOption('perpagemain', 'instruction');
@@ -63,7 +63,7 @@  discard block
 block discarded – undo
63 63
 // Если есть категория
64 64
 if ($cid) {
65 65
     // Если нельзя просматривать эту категорию
66
-    if (!in_array($cid, $categories)) {
66
+    if ( ! in_array($cid, $categories)) {
67 67
         redirect_header('index.php', 3, _MD_INSTRUCTION_NOPERM_CAT);
68 68
     }
69 69
     $criteria->add(new \Criteria('cid', $cid, '='));
Please login to merge, or discard this patch.
class/common/VersionChecks.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,7 @@
 block discarded – undo
51 51
                     break;
52 52
                 }
53 53
             } else {
54
-                if ((int)$v > 0) { // handles versions like x.x.x.0_RC2
54
+                if ((int) $v > 0) { // handles versions like x.x.x.0_RC2
55 55
                     $success = false;
56 56
                     break;
57 57
                 }
Please login to merge, or discard this patch.
class/common/ServerStats.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -43,7 +43,7 @@  discard block
 block discarded – undo
43 43
         $html .= "<ul>\n";
44 44
         //
45 45
         $gdlib = function_exists('gd_info') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_GDON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_GDOFF') . '</span>';
46
-        $html  .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
46
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBSTATUS') . $gdlib;
47 47
         if (function_exists('gd_info')) {
48 48
             if ($gdlib = (true === gd_info())) {
49 49
                 $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_GDLIBVERSION') . '<b>' . $gdlib['GD Version'] . '</b>';
@@ -57,7 +57,7 @@  discard block
 block discarded – undo
57 57
         //    $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_REGISTERGLOBALS . $registerglobals;
58 58
         //
59 59
         $downloads = ini_get('file_uploads') ? '<span style="color: green;">' . constant('CO_' . $moduleDirNameUpper . '_ON') . '</span>' : '<span style="color: red;">' . constant('CO_' . $moduleDirNameUpper . '_OFF') . '</span>';
60
-        $html      .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
60
+        $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_SERVERUPLOADSTATUS') . $downloads;
61 61
         //
62 62
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXUPLOADSIZE') . ' <b><span style="color: blue;">' . ini_get('upload_max_filesize') . "</span></b>\n";
63 63
         $html .= '<li>' . constant('CO_' . $moduleDirNameUpper . '_MAXPOSTSIZE') . ' <b><span style="color: blue;">' . ini_get('post_max_size') . "</span></b>\n";
Please login to merge, or discard this patch.
class/common/FilesManagement.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -28,8 +28,8 @@  discard block
 block discarded – undo
28 28
     public static function createFolder($folder)
29 29
     {
30 30
         try {
31
-            if (!file_exists($folder)) {
32
-                if (!mkdir($folder) && !is_dir($folder)) {
31
+            if ( ! file_exists($folder)) {
32
+                if ( ! mkdir($folder) && ! is_dir($folder)) {
33 33
                     throw new \RuntimeException(sprintf('Unable to create the %s directory', $folder));
34 34
                 }
35 35
 
@@ -85,7 +85,7 @@  discard block
 block discarded – undo
85 85
     public static function deleteDirectory($src)
86 86
     {
87 87
         // Only continue if user is a 'global' Admin
88
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
88
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
89 89
             return false;
90 90
         }
91 91
 
@@ -99,12 +99,12 @@  discard block
 block discarded – undo
99 99
                 $fileInfo = new \SplFileInfo("{$src}/{$v}");
100 100
                 if ($fileInfo->isDir()) {
101 101
                     // recursively handle subdirectories
102
-                    if (!$success = self::deleteDirectory($fileInfo->getRealPath())) {
102
+                    if ( ! $success = self::deleteDirectory($fileInfo->getRealPath())) {
103 103
                         break;
104 104
                     }
105 105
                 } else {
106 106
                     // delete the file
107
-                    if (!($success = unlink($fileInfo->getRealPath()))) {
107
+                    if ( ! ($success = unlink($fileInfo->getRealPath()))) {
108 108
                         break;
109 109
                     }
110 110
                 }
@@ -133,12 +133,12 @@  discard block
 block discarded – undo
133 133
     public static function rrmdir($src)
134 134
     {
135 135
         // Only continue if user is a 'global' Admin
136
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
136
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
137 137
             return false;
138 138
         }
139 139
 
140 140
         // If source is not a directory stop processing
141
-        if (!is_dir($src)) {
141
+        if ( ! is_dir($src)) {
142 142
             return false;
143 143
         }
144 144
 
@@ -150,15 +150,15 @@  discard block
 block discarded – undo
150 150
             if ($fObj->isFile()) {
151 151
                 $filename = $fObj->getPathname();
152 152
                 $fObj     = null; // clear this iterator object to close the file
153
-                if (!unlink($filename)) {
153
+                if ( ! unlink($filename)) {
154 154
                     return false; // couldn't delete the file
155 155
                 }
156
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
156
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
157 157
                 // Try recursively on directory
158 158
                 self::rrmdir($fObj->getPathname());
159 159
             }
160 160
         }
161
-        $iterator = null;   // clear iterator Obj to close file/directory
161
+        $iterator = null; // clear iterator Obj to close file/directory
162 162
         return rmdir($src); // remove the directory & return results
163 163
     }
164 164
 
@@ -173,17 +173,17 @@  discard block
 block discarded – undo
173 173
     public static function rmove($src, $dest)
174 174
     {
175 175
         // Only continue if user is a 'global' Admin
176
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
176
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
177 177
             return false;
178 178
         }
179 179
 
180 180
         // If source is not a directory stop processing
181
-        if (!is_dir($src)) {
181
+        if ( ! is_dir($src)) {
182 182
             return false;
183 183
         }
184 184
 
185 185
         // If the destination directory does not exist and could not be created stop processing
186
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
186
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
187 187
             return false;
188 188
         }
189 189
 
@@ -192,13 +192,13 @@  discard block
 block discarded – undo
192 192
         foreach ($iterator as $fObj) {
193 193
             if ($fObj->isFile()) {
194 194
                 rename($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
195
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
195
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
196 196
                 // Try recursively on directory
197 197
                 self::rmove($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
198 198
                 //                rmdir($fObj->getPath()); // now delete the directory
199 199
             }
200 200
         }
201
-        $iterator = null;   // clear iterator Obj to close file/directory
201
+        $iterator = null; // clear iterator Obj to close file/directory
202 202
         return rmdir($src); // remove the directory & return results
203 203
     }
204 204
 
@@ -216,17 +216,17 @@  discard block
 block discarded – undo
216 216
     public static function rcopy($src, $dest)
217 217
     {
218 218
         // Only continue if user is a 'global' Admin
219
-        if (!($GLOBALS['xoopsUser'] instanceof XoopsUser) || !$GLOBALS['xoopsUser']->isAdmin()) {
219
+        if ( ! ($GLOBALS['xoopsUser'] instanceof XoopsUser) || ! $GLOBALS['xoopsUser']->isAdmin()) {
220 220
             return false;
221 221
         }
222 222
 
223 223
         // If source is not a directory stop processing
224
-        if (!is_dir($src)) {
224
+        if ( ! is_dir($src)) {
225 225
             return false;
226 226
         }
227 227
 
228 228
         // If the destination directory does not exist and could not be created stop processing
229
-        if (!is_dir($dest) && !mkdir($dest, 0755)) {
229
+        if ( ! is_dir($dest) && ! mkdir($dest, 0755)) {
230 230
             return false;
231 231
         }
232 232
 
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
         foreach ($iterator as $fObj) {
236 236
             if ($fObj->isFile()) {
237 237
                 copy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
238
-            } elseif (!$fObj->isDot() && $fObj->isDir()) {
238
+            } elseif ( ! $fObj->isDot() && $fObj->isDir()) {
239 239
                 self::rcopy($fObj->getPathname(), "{$dest}/" . $fObj->getFilename());
240 240
             }
241 241
         }
Please login to merge, or discard this patch.
class/common/Breadcrumb.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -63,7 +63,7 @@
 block discarded – undo
63 63
      */
64 64
     public function render()
65 65
     {
66
-        if (!isset($GLOBALS['xoTheme']) || !is_object($GLOBALS['xoTheme'])) {
66
+        if ( ! isset($GLOBALS['xoTheme']) || ! is_object($GLOBALS['xoTheme'])) {
67 67
             require_once $GLOBALS['xoops']->path('class/theme.php');
68 68
             $GLOBALS['xoTheme'] = new xos_opal_Theme();
69 69
         }
Please login to merge, or discard this patch.
class/Category.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         //        $categoryHandler = xoops_getModuleHandler('category', 'instruction');
93 93
         $criteria = new \CriteriaCompo();
94 94
         // Если мы редактируем, то убрать текущую категорию из списка выбора родительской
95
-        if (!$this->isNew()) {
95
+        if ( ! $this->isNew()) {
96 96
             $criteria->add(new \Criteria('cid', $this->getVar('cid'), '<>'));
97 97
         }
98 98
         $criteria->setSort('weight ASC, title');
@@ -138,7 +138,7 @@  discard block
 block discarded – undo
138 138
         // Права на просмотр
139 139
         $groups_ids = [];
140 140
         // Если мы редактируем
141
-        if (!$this->isNew()) {
141
+        if ( ! $this->isNew()) {
142 142
             $groups_ids        = $gpermHandler->getGroupIds('instruction_view', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
143 143
             $groups_ids        = array_values($groups_ids);
144 144
             $groups_instr_view = new \XoopsFormCheckBox(_AM_INSTRUCTION_PERM_VIEW, 'groups_instr_view', $groups_ids);
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
 
151 151
         // Права на отправку
152 152
         $groups_ids = [];
153
-        if (!$this->isNew()) {
153
+        if ( ! $this->isNew()) {
154 154
             $groups_ids          = $gpermHandler->getGroupIds('instruction_submit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
155 155
             $groups_ids          = array_values($groups_ids);
156 156
             $groups_instr_submit = new \XoopsFormCheckBox(_AM_INSTRUCTION_PERM_SUBMIT, 'groups_instr_submit', $groups_ids);
@@ -162,7 +162,7 @@  discard block
 block discarded – undo
162 162
 
163 163
         // Права на редактирование
164 164
         $groups_ids = [];
165
-        if (!$this->isNew()) {
165
+        if ( ! $this->isNew()) {
166 166
             $groups_ids        = $gpermHandler->getGroupIds('instruction_edit', $this->getVar('cid'), $GLOBALS['xoopsModule']->getVar('mid'));
167 167
             $groups_ids        = array_values($groups_ids);
168 168
             $groups_instr_edit = new \XoopsFormCheckBox(_AM_INSTRUCTION_PERM_EDIT, 'groups_instr_edit', $groups_ids);
@@ -176,7 +176,7 @@  discard block
 block discarded – undo
176 176
         // ==========================================================
177 177
 
178 178
         // Если мы редактируем категорию
179
-        if (!$this->isNew()) {
179
+        if ( ! $this->isNew()) {
180 180
             $form->addElement(new \XoopsFormHidden('cid', $this->getVar('cid')));
181 181
             //$form->addElement( new \XoopsFormHidden( 'catmodify', true));
182 182
         }
Please login to merge, or discard this patch.
class/Page.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
         // Находим все страницы данной инструкции
87 87
         $criteria->add(new \Criteria('instrid', $instrid_page, '='));
88 88
         // Если мы редактируем, то убрать текущую страницу из списка выбора родительской
89
-        if (!$this->isNew()) {
89
+        if ( ! $this->isNew()) {
90 90
             $criteria->add(new \Criteria('pageid', $this->getVar('pageid'), '<>'));
91 91
         }
92 92
         $criteria->setSort('weight');
@@ -151,7 +151,7 @@  discard block
 block discarded – undo
151 151
         $form->addElement($option_tray);
152 152
 
153 153
         // Если мы редактируем страницу
154
-        if (!$this->isNew()) {
154
+        if ( ! $this->isNew()) {
155 155
             $form->addElement(new \XoopsFormHidden('pageid', $this->getVar('pageid')));
156 156
         } else {
157 157
             $form->addElement(new \XoopsFormHidden('pageid', 0));
Please login to merge, or discard this patch.
class/Utility.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -64,7 +64,7 @@
 block discarded – undo
64 64
                 break;
65 65
             case 'int':
66 66
             default:
67
-                $ret = isset($global[$key]) ? (int)$global[$key] : (int)$default;
67
+                $ret = isset($global[$key]) ? (int) $global[$key] : (int) $default;
68 68
                 break;
69 69
         }
70 70
         if (false === $ret) {
Please login to merge, or discard this patch.