Passed
Push — main ( 3607b0...fd5484 )
by Thierry
03:47
created
jaxon-core/src/App/Ajax/App.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -29,7 +29,7 @@  discard block
 block discarded – undo
29 29
      */
30 30
     public function setup(string $sConfigFile = ''): void
31 31
     {
32
-        if(!file_exists($sConfigFile))
32
+        if (!file_exists($sConfigFile))
33 33
         {
34 34
             throw new SetupException($this->translator()
35 35
                 ->trans('errors.file.access', ['path' => $sConfigFile]));
@@ -39,14 +39,14 @@  discard block
 block discarded – undo
39 39
         $aOptions = $this->config()->read($sConfigFile);
40 40
         $aLibOptions = $aOptions['lib'] ?? [];
41 41
         $aAppOptions = $aOptions['app'] ?? [];
42
-        if(!is_array($aLibOptions) || !is_array($aAppOptions))
42
+        if (!is_array($aLibOptions) || !is_array($aAppOptions))
43 43
         {
44 44
             throw new SetupException($sMessage = $this->translator()
45 45
                 ->trans('errors.file.content', ['path' => $sConfigFile]));
46 46
         }
47 47
 
48 48
         // The bootstrap set this to false. It needs to be changed.
49
-        if(!isset($aLibOptions['core']['response']['send']))
49
+        if (!isset($aLibOptions['core']['response']['send']))
50 50
         {
51 51
             $aLibOptions['core']['response']['send'] = true;
52 52
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Ajax/Traits/SendResponseTrait.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -59,20 +59,20 @@  discard block
 block discarded – undo
59 59
      */
60 60
     public function httpResponse(string $sCode = '200'): mixed
61 61
     {
62
-        if(!$this->config()->getOption('core.response.send', false))
62
+        if (!$this->config()->getOption('core.response.send', false))
63 63
         {
64 64
             return null;
65 65
         }
66 66
 
67 67
         // Check to see if headers have already been sent out, in which case we can't do our job
68
-        if(headers_sent($sFilename, $nLineNumber))
68
+        if (headers_sent($sFilename, $nLineNumber))
69 69
         {
70 70
             throw new RequestException($this->translator()
71 71
                 ->trans('errors.output.already-sent', [
72 72
                     'location' => "$sFilename:$nLineNumber",
73 73
                 ]) . "\n" . $this->translator()->trans('errors.output.advice'));
74 74
         }
75
-        if(empty($sContent = $this->getResponseManager()->getOutput()))
75
+        if (empty($sContent = $this->getResponseManager()->getOutput()))
76 76
         {
77 77
             return null;
78 78
         }
@@ -80,7 +80,7 @@  discard block
 block discarded – undo
80 80
         // Set the HTTP response code
81 81
         http_response_code(intval($sCode));
82 82
 
83
-        if($this->di()->getRequest()->getMethod() === 'GET')
83
+        if ($this->di()->getRequest()->getMethod() === 'GET')
84 84
         {
85 85
             header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
86 86
             header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
         header('Content-Type: ' . $this->getResponseManager()->getContentType());
91 91
 
92 92
         print $sContent;
93
-        if($this->config()->getOption('core.process.exit', false))
93
+        if ($this->config()->getOption('core.process.exit', false))
94 94
         {
95 95
             exit();
96 96
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Config/ConfigManager.php 2 patches
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -70,22 +70,22 @@  discard block
 block discarded – undo
70 70
         {
71 71
             return $this->xConfigReader->read($sConfigFile);
72 72
         }
73
-        catch(YamlExtension $e)
73
+        catch (YamlExtension $e)
74 74
         {
75 75
             $sMessage = $this->xTranslator->trans('errors.yaml.install');
76 76
             throw new SetupException($sMessage);
77 77
         }
78
-        catch(FileExtension $e)
78
+        catch (FileExtension $e)
79 79
         {
80 80
             $sMessage = $this->xTranslator->trans('errors.file.extension', ['path' => $sConfigFile]);
81 81
             throw new SetupException($sMessage);
82 82
         }
83
-        catch(FileAccess $e)
83
+        catch (FileAccess $e)
84 84
         {
85 85
             $sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
86 86
             throw new SetupException($sMessage);
87 87
         }
88
-        catch(FileContent $e)
88
+        catch (FileContent $e)
89 89
         {
90 90
             $sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
91 91
             throw new SetupException($sMessage);
@@ -111,7 +111,7 @@  discard block
 block discarded – undo
111 111
             // Call the config change listeners.
112 112
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
113 113
         }
114
-        catch(DataDepth $e)
114
+        catch (DataDepth $e)
115 115
         {
116 116
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
117 117
                 'key' => $e->sPrefix,
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
141 141
             return $this->xLibConfig->changed();
142 142
         }
143
-        catch(DataDepth $e)
143
+        catch (DataDepth $e)
144 144
         {
145 145
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
146 146
                 'key' => $e->sPrefix,
@@ -247,7 +247,7 @@  discard block
 block discarded – undo
247 247
             $this->xEventManager->appConfigChanged($this->xAppConfig, '');
248 248
             return $this->xAppConfig->changed();
249 249
         }
250
-        catch(DataDepth $e)
250
+        catch (DataDepth $e)
251 251
         {
252 252
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
253 253
                 'key' => $e->sPrefix,
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
297 297
         {
298 298
             return $this->xConfigSetter->newConfig($aOptions, $sNamePrefix);
299 299
         }
300
-        catch(DataDepth $e)
300
+        catch (DataDepth $e)
301 301
         {
302 302
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
303 303
                 'key' => $e->sPrefix,
@@ -336,7 +336,7 @@  discard block
 block discarded – undo
336 336
      */
337 337
     public function globals(bool $bForce = false): void
338 338
     {
339
-        if($bForce || $this->getAppOption('helpers.global', true))
339
+        if ($bForce || $this->getAppOption('helpers.global', true))
340 340
         {
341 341
             require_once dirname(__DIR__, 2) . '/globals.php';
342 342
         }
Please login to merge, or discard this patch.
Braces   +8 added lines, -16 removed lines patch added patch discarded remove patch
@@ -69,23 +69,19 @@  discard block
 block discarded – undo
69 69
         try
70 70
         {
71 71
             return $this->xConfigReader->read($sConfigFile);
72
-        }
73
-        catch(YamlExtension $e)
72
+        } catch(YamlExtension $e)
74 73
         {
75 74
             $sMessage = $this->xTranslator->trans('errors.yaml.install');
76 75
             throw new SetupException($sMessage);
77
-        }
78
-        catch(FileExtension $e)
76
+        } catch(FileExtension $e)
79 77
         {
80 78
             $sMessage = $this->xTranslator->trans('errors.file.extension', ['path' => $sConfigFile]);
81 79
             throw new SetupException($sMessage);
82
-        }
83
-        catch(FileAccess $e)
80
+        } catch(FileAccess $e)
84 81
         {
85 82
             $sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
86 83
             throw new SetupException($sMessage);
87
-        }
88
-        catch(FileContent $e)
84
+        } catch(FileContent $e)
89 85
         {
90 86
             $sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
91 87
             throw new SetupException($sMessage);
@@ -110,8 +106,7 @@  discard block
 block discarded – undo
110 106
                 ->setOptions($this->xLibConfig, $this->read($sConfigFile), $sConfigSection);
111 107
             // Call the config change listeners.
112 108
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
113
-        }
114
-        catch(DataDepth $e)
109
+        } catch(DataDepth $e)
115 110
         {
116 111
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
117 112
                 'key' => $e->sPrefix,
@@ -139,8 +134,7 @@  discard block
 block discarded – undo
139 134
             // Call the config change listeners.
140 135
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
141 136
             return $this->xLibConfig->changed();
142
-        }
143
-        catch(DataDepth $e)
137
+        } catch(DataDepth $e)
144 138
         {
145 139
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
146 140
                 'key' => $e->sPrefix,
@@ -246,8 +240,7 @@  discard block
 block discarded – undo
246 240
             // Call the config change listeners.
247 241
             $this->xEventManager->appConfigChanged($this->xAppConfig, '');
248 242
             return $this->xAppConfig->changed();
249
-        }
250
-        catch(DataDepth $e)
243
+        } catch(DataDepth $e)
251 244
         {
252 245
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
253 246
                 'key' => $e->sPrefix,
@@ -296,8 +289,7 @@  discard block
 block discarded – undo
296 289
         try
297 290
         {
298 291
             return $this->xConfigSetter->newConfig($aOptions, $sNamePrefix);
299
-        }
300
-        catch(DataDepth $e)
292
+        } catch(DataDepth $e)
301 293
         {
302 294
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
303 295
                 'key' => $e->sPrefix,
Please login to merge, or discard this patch.
jaxon-core/src/App/Config/ConfigEventManager.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@  discard block
 block discarded – undo
60 60
      */
61 61
     public function libConfigChanged(Config $xConfig, string $sName): void
62 62
     {
63
-        foreach($this->aLibConfigListeners as $sListener)
63
+        foreach ($this->aLibConfigListeners as $sListener)
64 64
         {
65 65
             $this->di->g($sListener)->onChange($xConfig, $sName);
66 66
         }
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
      */
72 72
     public function appConfigChanged(Config $xConfig, string $sName): void
73 73
     {
74
-        foreach($this->aAppConfigListeners as $sListener)
74
+        foreach ($this->aAppConfigListeners as $sListener)
75 75
         {
76 76
             $this->di->g($sListener)->onChange($xConfig, $sName);
77 77
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Metadata/Data/UploadData.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
      */
52 52
     protected function validateField(string $sField): void
53 53
     {
54
-        if(preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sField) > 0)
54
+        if (preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sField) > 0)
55 55
         {
56 56
             return;
57 57
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Metadata/Data/CallbackData.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -50,7 +50,7 @@
 block discarded – undo
50 50
      */
51 51
     protected function validateObjectName(string $sName): void
52 52
     {
53
-        if(preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sName) > 0)
53
+        if (preg_match('/^[a-zA-Z][a-zA-Z0-9_\-\.]*$/', $sName) > 0)
54 54
         {
55 55
             return;
56 56
         }
Please login to merge, or discard this patch.
jaxon-core/src/App/Dialog/Manager/DialogCommand.php 1 patch
Spacing   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -113,8 +113,7 @@  discard block
 block discarded – undo
113 113
     private function alert(string $sType, string $sMessage, array $aArgs): array
114 114
     {
115 115
         return [
116
-            'lib' => $this->getLibrary() ?:
117
-                ($this->xRegistry?->getAlertLibrary()->getName() ?? ''),
116
+            'lib' => $this->getLibrary() ?: ($this->xRegistry?->getAlertLibrary()->getName() ?? ''),
118 117
             'message' => [
119 118
                 'type' => $sType,
120 119
                 'title' => $this->getTitle(),
@@ -197,8 +196,7 @@  discard block
 block discarded – undo
197 196
     public function show(string $sTitle, string $sContent, array $aButtons, array $aOptions = []): array
198 197
     {
199 198
         return [
200
-            'lib' => $this->getLibrary() ?:
201
-                ($this->xRegistry?->getModalLibrary()->getName() ?? ''),
199
+            'lib' => $this->getLibrary() ?: ($this->xRegistry?->getModalLibrary()->getName() ?? ''),
202 200
             'dialog' => [
203 201
                 'title' => $sTitle,
204 202
                 'content' => $sContent,
@@ -216,8 +214,7 @@  discard block
 block discarded – undo
216 214
     public function hide(): array
217 215
     {
218 216
         return [
219
-            'lib' => $this->getLibrary() ?:
220
-                ($this->xRegistry?->getModalLibrary()->getName() ?? ''),
217
+            'lib' => $this->getLibrary() ?: ($this->xRegistry?->getModalLibrary()->getName() ?? ''),
221 218
         ];
222 219
     }
223 220
 
@@ -232,8 +229,7 @@  discard block
 block discarded – undo
232 229
     public function confirm(string $sQuestion, array $aArgs = []): array
233 230
     {
234 231
         return [
235
-            'lib' => $this->getLibrary() ?:
236
-                ($this->xRegistry?->getConfirmLibrary()->getName() ?? ''),
232
+            'lib' => $this->getLibrary() ?: ($this->xRegistry?->getConfirmLibrary()->getName() ?? ''),
237 233
             'question' => [
238 234
                 'title' => $this->getTitle(),
239 235
                 'phrase' => $this->phrase($sQuestion, $aArgs),
Please login to merge, or discard this patch.
jaxon-core/src/App/Pagination/Paginator.php 1 patch
Spacing   +17 added lines, -20 removed lines patch added patch discarded remove patch
@@ -128,9 +128,8 @@  discard block
 block discarded – undo
128 128
      */
129 129
     private function updatePagesCount(): Paginator
130 130
     {
131
-        $this->nPagesCount = ($this->nItemsPerPage === 0 ? 0 :
132
-            (int)ceil($this->nItemsCount / $this->nItemsPerPage));
133
-        if($this->nPageNumber > $this->nPagesCount)
131
+        $this->nPagesCount = ($this->nItemsPerPage === 0 ? 0 : (int)ceil($this->nItemsCount / $this->nItemsPerPage));
132
+        if ($this->nPageNumber > $this->nPagesCount)
134 133
         {
135 134
             $this->nPageNumber = $this->nPagesCount;
136 135
         }
@@ -184,8 +183,7 @@  discard block
 block discarded – undo
184 183
     protected function getPrevPage(): Page
185 184
     {
186 185
         return $this->nPageNumber <= 1 ?
187
-            new Page('disabled', $this->sPreviousText, 0) :
188
-            new Page('enabled', $this->sPreviousText, $this->nPageNumber - 1);
186
+            new Page('disabled', $this->sPreviousText, 0) : new Page('enabled', $this->sPreviousText, $this->nPageNumber - 1);
189 187
     }
190 188
 
191 189
     /**
@@ -196,8 +194,7 @@  discard block
 block discarded – undo
196 194
     protected function getNextPage(): Page
197 195
     {
198 196
         return $this->nPageNumber >= $this->nPagesCount ?
199
-            new Page('disabled', $this->sNextText, 0) :
200
-            new Page('enabled', $this->sNextText, $this->nPageNumber + 1);
197
+            new Page('disabled', $this->sNextText, 0) : new Page('enabled', $this->sNextText, $this->nPageNumber + 1);
201 198
     }
202 199
 
203 200
     /**
@@ -209,7 +206,7 @@  discard block
 block discarded – undo
209 206
      */
210 207
     protected function getPage(int $nNumber): Page
211 208
     {
212
-        if($nNumber < 1)
209
+        if ($nNumber < 1)
213 210
         {
214 211
             return new Page('disabled', $this->sEllipsysText, 0);
215 212
         }
@@ -228,9 +225,9 @@  discard block
 block discarded – undo
228 225
     {
229 226
         $aPageNumbers = [];
230 227
 
231
-        if($this->nPagesCount <= $this->nMaxPages)
228
+        if ($this->nPagesCount <= $this->nMaxPages)
232 229
         {
233
-            for($i = 0; $i < $this->nPagesCount; $i++)
230
+            for ($i = 0; $i < $this->nPagesCount; $i++)
234 231
             {
235 232
                 $aPageNumbers[] = $i + 1;
236 233
             }
@@ -243,7 +240,7 @@  discard block
 block discarded – undo
243 240
 
244 241
         $nSlidingStart = 1;
245 242
         $nSlidingEndOffset = $nNumAdjacents + 3 - $this->nPageNumber;
246
-        if($nSlidingEndOffset < 0)
243
+        if ($nSlidingEndOffset < 0)
247 244
         {
248 245
             $nSlidingStart = $this->nPageNumber - $nNumAdjacents;
249 246
             $nSlidingEndOffset = 0;
@@ -251,23 +248,23 @@  discard block
 block discarded – undo
251 248
 
252 249
         $nSlidingEnd = $this->nPagesCount;
253 250
         $nSlidingStartOffset = $this->nPageNumber + $nNumAdjacents + 2 - $this->nPagesCount;
254
-        if($nSlidingStartOffset < 0)
251
+        if ($nSlidingStartOffset < 0)
255 252
         {
256 253
             $nSlidingEnd = $this->nPageNumber + $nNumAdjacents;
257 254
             $nSlidingStartOffset = 0;
258 255
         }
259 256
 
260 257
         // Build the list of page numbers.
261
-        if($nSlidingStart > 1)
258
+        if ($nSlidingStart > 1)
262 259
         {
263 260
             $aPageNumbers[] = 1;
264 261
             $aPageNumbers[] = 0; // Ellipsys;
265 262
         }
266
-        for($i = $nSlidingStart - $nSlidingStartOffset; $i <= $nSlidingEnd + $nSlidingEndOffset; $i++)
263
+        for ($i = $nSlidingStart - $nSlidingStartOffset; $i <= $nSlidingEnd + $nSlidingEndOffset; $i++)
267 264
         {
268 265
             $aPageNumbers[] = $i;
269 266
         }
270
-        if($nSlidingEnd < $this->nPagesCount)
267
+        if ($nSlidingEnd < $this->nPagesCount)
271 268
         {
272 269
             $aPageNumbers[] = 0; // Ellipsys;
273 270
             $aPageNumbers[] = $this->nPagesCount;
@@ -293,7 +290,7 @@  discard block
 block discarded – undo
293 290
      */
294 291
     public function pages(): array
295 292
     {
296
-        if($this->nPagesCount < 2)
293
+        if ($this->nPagesCount < 2)
297 294
         {
298 295
             return [];
299 296
         }
@@ -344,7 +341,7 @@  discard block
 block discarded – undo
344 341
     private function renderLinks(): string
345 342
     {
346 343
         $aPages = $this->pages();
347
-        if(count($aPages) === 0)
344
+        if (count($aPages) === 0)
348 345
         {
349 346
             return '';
350 347
         }
@@ -365,7 +362,7 @@  discard block
 block discarded – undo
365 362
     {
366 363
         $sHtml = $this->renderLinks();
367 364
         // The HTML code must always be displayed, even if it is empty.
368
-        if(is_a($this->xPlugin->response(), Response::class))
365
+        if (is_a($this->xPlugin->response(), Response::class))
369 366
         {
370 367
             /** @var Response */
371 368
             $xResponse = $this->xPlugin->response();
@@ -388,13 +385,13 @@  discard block
 block discarded – undo
388 385
      */
389 386
     public function render(JsExpr $xCall, string $sWrapperId = ''): void
390 387
     {
391
-        if(($xFunc = $xCall->func()) === null)
388
+        if (($xFunc = $xCall->func()) === null)
392 389
         {
393 390
             return;
394 391
         }
395 392
 
396 393
         $aParams = $this->showLinks(trim($sWrapperId));
397
-        if($aParams !== null)
394
+        if ($aParams !== null)
398 395
         {
399 396
             // Set click handlers on the pagination links
400 397
             $aParams['func'] = $xFunc->withPage()->jsonSerialize();
Please login to merge, or discard this patch.
jaxon-core/src/App/View/ViewTrait.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -53,7 +53,7 @@
 block discarded – undo
53 53
     {
54 54
         $this->sDirectory = '';
55 55
         $this->sExtension = '';
56
-        if(isset($this->aDirectories[$sNamespace]))
56
+        if (isset($this->aDirectories[$sNamespace]))
57 57
         {
58 58
             // Make sure there's only one '/' at the end of the string
59 59
             $this->sDirectory = rtrim($this->aDirectories[$sNamespace]['path'], '/') . '/';
Please login to merge, or discard this patch.