Passed
Push — main ( 5fbd1e...a5917e )
by Thierry
10:54
created
src/Response/Response.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@  discard block
 block discarded – undo
86 86
      */
87 87
     public function getOutput(): string
88 88
     {
89
-        if($this->getCommandCount() === 0)
89
+        if ($this->getCommandCount() === 0)
90 90
         {
91 91
             return '';
92 92
         }
@@ -145,7 +145,7 @@  discard block
 block discarded – undo
145 145
      */
146 146
     public function bag(string $sName): DataBagContext
147 147
     {
148
-        return $this->plugin('bags')->bag($sName);;
148
+        return $this->plugin('bags')->bag($sName); ;
149 149
     }
150 150
 
151 151
     /**
@@ -159,7 +159,7 @@  discard block
 block discarded – undo
159 159
     public function addCommand(array $aAttributes, $mData): Response
160 160
     {
161 161
         $aAttributes = array_map(function($xAttribute) {
162
-            return is_integer($xAttribute) ? $xAttribute : trim((string)$xAttribute, " \t");
162
+            return is_integer($xAttribute) ? $xAttribute : trim((string) $xAttribute, " \t");
163 163
         }, $aAttributes);
164 164
         $aAttributes['data'] = $mData;
165 165
         $this->aCommands[] = $aAttributes;
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
     protected function _addCommand(string $sName, array $aAttributes, $mData, bool $bRemoveEmpty = false): Response
180 180
     {
181 181
         $mData = is_array($mData) ? array_map(function($sData) {
182
-            return trim((string)$sData, " \t\n");
183
-        }, $mData) : trim((string)$mData, " \t\n");
184
-        if($bRemoveEmpty)
182
+            return trim((string) $sData, " \t\n");
183
+        }, $mData) : trim((string) $mData, " \t\n");
184
+        if ($bRemoveEmpty)
185 185
         {
186 186
             $aAttributes = array_filter($aAttributes, function($xValue) {
187 187
                 return $xValue === '';
@@ -214,7 +214,7 @@  discard block
 block discarded – undo
214 214
     public function toPsr(): PsrResponseInterface
215 215
     {
216 216
         $xPsrResponse = $this->xPsr17Factory->createResponse(200);
217
-        if($this->xRequest->getMethod() === 'GET')
217
+        if ($this->xRequest->getMethod() === 'GET')
218 218
         {
219 219
             $xPsrResponse = $xPsrResponse
220 220
                 ->withHeader('Expires', 'Mon, 26 Jul 1997 05:00:00 GMT')
Please login to merge, or discard this patch.
src/Response/Manager/ResponseManager.php 1 patch
Spacing   +8 added lines, -9 removed lines patch added patch discarded remove patch
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
      */
80 80
     public function clear()
81 81
     {
82
-        if($this->xResponse !== null)
82
+        if ($this->xResponse !== null)
83 83
         {
84 84
             $this->xResponse->clearCommands();
85 85
         }
@@ -93,7 +93,7 @@  discard block
 block discarded – undo
93 93
      */
94 94
     public function getResponse(): ResponseInterface
95 95
     {
96
-        if(!$this->xResponse)
96
+        if (!$this->xResponse)
97 97
         {
98 98
             $this->xResponse = $this->di->getResponse();
99 99
         }
@@ -114,21 +114,21 @@  discard block
 block discarded – undo
114 114
      */
115 115
     public function append(ResponseInterface $xResponse)
116 116
     {
117
-        if(!$this->xResponse)
117
+        if (!$this->xResponse)
118 118
         {
119 119
             $this->xResponse = $this->di->getResponse();
120 120
         }
121
-        if($this->xResponse->getCommandCount() === 0)
121
+        if ($this->xResponse->getCommandCount() === 0)
122 122
         {
123 123
             $this->xResponse = $xResponse;
124 124
             return;
125 125
         }
126
-        if(get_class($this->xResponse) !== get_class($xResponse))
126
+        if (get_class($this->xResponse) !== get_class($xResponse))
127 127
         {
128 128
             throw new RequestException($this->xTranslator->trans('errors.mismatch.types',
129 129
                 ['class' => get_class($xResponse)]));
130 130
         }
131
-        if($this->xResponse !== $xResponse)
131
+        if ($this->xResponse !== $xResponse)
132 132
         {
133 133
             $this->xResponse->appendResponse($xResponse);
134 134
         }
@@ -169,7 +169,7 @@  discard block
 block discarded – undo
169 169
      */
170 170
     public function printDebug()
171 171
     {
172
-        foreach($this->aDebugMessages as $sMessage)
172
+        foreach ($this->aDebugMessages as $sMessage)
173 173
         {
174 174
             $this->getResponse()->debug($sMessage);
175 175
         }
@@ -183,8 +183,7 @@  discard block
 block discarded – undo
183 183
      */
184 184
     public function getContentType(): string
185 185
     {
186
-        return empty($this->sCharacterEncoding) ? $this->getResponse()->getContentType() :
187
-            $this->getResponse()->getContentType() . '; charset="' . $this->sCharacterEncoding . '"';
186
+        return empty($this->sCharacterEncoding) ? $this->getResponse()->getContentType() : $this->getResponse()->getContentType() . '; charset="' . $this->sCharacterEncoding . '"';
188 187
     }
189 188
 
190 189
     /**
Please login to merge, or discard this patch.
src/Response/Traits/JsTrait.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -94,7 +94,7 @@
 block discarded – undo
94 94
     public function redirect(string $sURL, int $nDelay = 0): Response
95 95
     {
96 96
         $sURL = $this->xPluginManager->getParameterReader()->parseUrl($sURL);
97
-        if($nDelay <= 0)
97
+        if ($nDelay <= 0)
98 98
         {
99 99
             return $this->script("window.location = '$sURL';");
100 100
         }
Please login to merge, or discard this patch.
src/Plugin/Code/AssetManager.php 1 patch
Spacing   +9 added lines, -9 removed lines patch added patch discarded remove patch
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
     public function shallIncludeAssets(Plugin $xPlugin): bool
72 72
     {
73 73
         $sPluginOptionName = 'assets.include.' . $xPlugin->getName();
74
-        if($this->xConfigManager->hasOption($sPluginOptionName))
74
+        if ($this->xConfigManager->hasOption($sPluginOptionName))
75 75
         {
76 76
             return $this->xConfigManager->getOption($sPluginOptionName);
77 77
         }
@@ -90,7 +90,7 @@  discard block
 block discarded – undo
90 90
         $sJsLibUri = rtrim($this->xConfigManager->getOption('js.lib.uri', self::JS_LIB_URL), '/') . '/';
91 91
         // Add component files to the javascript file array;
92 92
         $aJsFiles = [$sJsLibUri . 'jaxon.core' . $sJsExtension];
93
-        if($this->xConfigManager->getOption('core.debug.on'))
93
+        if ($this->xConfigManager->getOption('core.debug.on'))
94 94
         {
95 95
             $sLanguage = $this->xConfigManager->getOption('core.language');
96 96
             $aJsFiles[] = $sJsLibUri . 'jaxon.debug' . $sJsExtension;
@@ -107,7 +107,7 @@  discard block
 block discarded – undo
107 107
      */
108 108
     public function getOptionVars(): array
109 109
     {
110
-        if(!$this->xConfigManager->hasOption('core.request.uri'))
110
+        if (!$this->xConfigManager->hasOption('core.request.uri'))
111 111
         {
112 112
             $this->xConfigManager->setOption('core.request.uri', $this->xParameterReader->uri());
113 113
         }
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
         // Check config options
141 141
         // - The js.app.export option must be set to true
142 142
         // - The js.app.uri and js.app.dir options must be set to non null values
143
-        if(!$this->xConfigManager->getOption('js.app.export', false) ||
143
+        if (!$this->xConfigManager->getOption('js.app.export', false) ||
144 144
             !$this->xConfigManager->getOption('js.app.uri') ||
145 145
             !$this->xConfigManager->getOption('js.app.dir'))
146 146
         {
@@ -158,7 +158,7 @@  discard block
 block discarded – undo
158 158
      */
159 159
     public function createJsFiles(CodeGenerator $xCodeGenerator): string
160 160
     {
161
-        if(!$this->shallCreateJsFiles())
161
+        if (!$this->shallCreateJsFiles())
162 162
         {
163 163
             return '';
164 164
         }
@@ -167,7 +167,7 @@  discard block
 block discarded – undo
167 167
         $sJsFileName = $this->xConfigManager->getOption('js.app.file') ?: $xCodeGenerator->getHash();
168 168
         $sJsDirectory = rtrim($this->xConfigManager->getOption('js.app.dir'), '\/') . DIRECTORY_SEPARATOR;
169 169
         // - The js.app.dir must be writable
170
-        if(!$sJsFileName || !is_dir($sJsDirectory) || !is_writable($sJsDirectory))
170
+        if (!$sJsFileName || !is_dir($sJsDirectory) || !is_writable($sJsDirectory))
171 171
         {
172 172
             return '';
173 173
         }
@@ -175,15 +175,15 @@  discard block
 block discarded – undo
175 175
         $sJsFilePath = $sJsDirectory . $sJsFileName . '.js';
176 176
         $sJsMinFilePath = $sJsDirectory . $sJsFileName . '.min.js';
177 177
         $sJsFileUri = rtrim($this->xConfigManager->getOption('js.app.uri'), '/') . "/$sJsFileName";
178
-        if(!is_file($sJsFilePath) && !@file_put_contents($sJsFilePath, $xCodeGenerator->getJsScript()))
178
+        if (!is_file($sJsFilePath) && !@file_put_contents($sJsFilePath, $xCodeGenerator->getJsScript()))
179 179
         {
180 180
             return '';
181 181
         }
182
-        if(!$this->xConfigManager->getOption('js.app.minify', false))
182
+        if (!$this->xConfigManager->getOption('js.app.minify', false))
183 183
         {
184 184
             return $sJsFileUri . '.js';
185 185
         }
186
-        if(!is_file($sJsMinFilePath) && !$this->xMinifier->minify($sJsFilePath, $sJsMinFilePath))
186
+        if (!is_file($sJsMinFilePath) && !$this->xMinifier->minify($sJsFilePath, $sJsMinFilePath))
187 187
         {
188 188
             // If the file cannot be minified, return the plain js file.
189 189
             return $sJsFileUri . '.js';
Please login to merge, or discard this patch.
src/Plugin/Response/JQuery/JQueryPlugin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -87,7 +87,7 @@
 block discarded – undo
87 87
     public function selector(string $sPath = '', $xContext = null): DomSelector
88 88
     {
89 89
         $xSelector = new DomSelector($this->jQueryNs, $sPath, $xContext);
90
-        if($this->bCommand && $this->response() !== null)
90
+        if ($this->bCommand && $this->response() !== null)
91 91
         {
92 92
             $this->addCommand(['cmd' => 'jquery'], $xSelector);
93 93
         }
Please login to merge, or discard this patch.
src/App/Dialog/Library/DialogLibraryHelper.php 2 patches
Spacing   +8 added lines, -8 removed lines patch added patch discarded remove patch
@@ -136,18 +136,18 @@  discard block
 block discarded – undo
136 136
         $aOptions = $this->getOptionNames($sKeyPrefix);
137 137
         $sSpaces = str_repeat(' ', $nSpaces);
138 138
         $sScript = '';
139
-        foreach($aOptions as $sShortName => $sFullName)
139
+        foreach ($aOptions as $sShortName => $sFullName)
140 140
         {
141 141
             $value = $this->xConfigManager->getOption($sFullName);
142
-            if(is_string($value))
142
+            if (is_string($value))
143 143
             {
144 144
                 $value = "'$value'";
145 145
             }
146
-            elseif(is_bool($value))
146
+            elseif (is_bool($value))
147 147
             {
148 148
                 $value = ($value ? 'true' : 'false');
149 149
             }
150
-            elseif(!is_numeric($value))
150
+            elseif (!is_numeric($value))
151 151
             {
152 152
                 $value = json_encode($value);
153 153
             }
@@ -175,10 +175,10 @@  discard block
 block discarded – undo
175 175
      */
176 176
     public function getJsCode(string $sFile): string
177 177
     {
178
-        if($this->hasOption('assets.js'))
178
+        if ($this->hasOption('assets.js'))
179 179
         {
180 180
             // If this expression evaluates to false, then the asset is not displayed.
181
-            if(!($sUri = $this->getOption('assets.js')))
181
+            if (!($sUri = $this->getOption('assets.js')))
182 182
             {
183 183
                 return '';
184 184
             }
@@ -200,10 +200,10 @@  discard block
 block discarded – undo
200 200
      */
201 201
     public function getCssCode(string $sFile): string
202 202
     {
203
-        if($this->hasOption('assets.css'))
203
+        if ($this->hasOption('assets.css'))
204 204
         {
205 205
             // If this expression evaluates to false, then the asset is not displayed.
206
-            if(!($sUri = $this->getOption('assets.css')))
206
+            if (!($sUri = $this->getOption('assets.css')))
207 207
             {
208 208
                 return '';
209 209
             }
Please login to merge, or discard this patch.
Braces   +4 added lines, -8 removed lines patch added patch discarded remove patch
@@ -142,12 +142,10 @@  discard block
 block discarded – undo
142 142
             if(is_string($value))
143 143
             {
144 144
                 $value = "'$value'";
145
-            }
146
-            elseif(is_bool($value))
145
+            } elseif(is_bool($value))
147 146
             {
148 147
                 $value = ($value ? 'true' : 'false');
149
-            }
150
-            elseif(!is_numeric($value))
148
+            } elseif(!is_numeric($value))
151 149
             {
152 150
                 $value = json_encode($value);
153 151
             }
@@ -182,8 +180,7 @@  discard block
 block discarded – undo
182 180
             {
183 181
                 return '';
184 182
             }
185
-        }
186
-        else
183
+        } else
187 184
         {
188 185
             $sUri = $this->sUri . '/' . ($this->sSubDir ? $this->sSubDir . '/' : '') .
189 186
                 ($this->sVersion ? $this->sVersion . '/' : '') . $sFile;
@@ -207,8 +204,7 @@  discard block
 block discarded – undo
207 204
             {
208 205
                 return '';
209 206
             }
210
-        }
211
-        else
207
+        } else
212 208
         {
213 209
             $sUri = $this->sUri . '/' . ($this->sSubDir ? $this->sSubDir . '/' : '') .
214 210
                 ($this->sVersion ? $this->sVersion . '/' : '') . $sFile;
Please login to merge, or discard this patch.
src/Plugin/Request/CallableClass/CallableRepository.php 1 patch
Spacing   +15 added lines, -15 removed lines patch added patch discarded remove patch
@@ -110,7 +110,7 @@  discard block
 block discarded – undo
110 110
 
111 111
         // The methods of the CallableClass class must not be exported
112 112
         $xCallableClass = new ReflectionClass(CallableClass::class);
113
-        foreach($xCallableClass->getMethods(ReflectionMethod::IS_PUBLIC) as $xMethod)
113
+        foreach ($xCallableClass->getMethods(ReflectionMethod::IS_PUBLIC) as $xMethod)
114 114
         {
115 115
             $this->aProtectedMethods[] = $xMethod->getName();
116 116
         }
@@ -181,14 +181,14 @@  discard block
 block discarded – undo
181 181
      */
182 182
     public function makeClassOptions(string $sClassName, array $aClassOptions, array $aDirectoryOptions): array
183 183
     {
184
-        foreach($this->aDefaultClassOptions as $sOption => $xValue)
184
+        foreach ($this->aDefaultClassOptions as $sOption => $xValue)
185 185
         {
186
-            if(!isset($aClassOptions[$sOption]))
186
+            if (!isset($aClassOptions[$sOption]))
187 187
             {
188 188
                 $aClassOptions[$sOption] = $xValue;
189 189
             }
190 190
         }
191
-        if(is_string($aClassOptions['protected']))
191
+        if (is_string($aClassOptions['protected']))
192 192
         {
193 193
             $aClassOptions['protected'] = [$aClassOptions['protected']]; // Convert to array.
194 194
         }
@@ -199,21 +199,21 @@  discard block
 block discarded – undo
199 199
             $aDirectoryOptions['classes']['*'] ?? [], // Options for all classes
200 200
             $aDirectoryOptions['classes'][$sClassName] ?? [], // Options for this specific class
201 201
         ];
202
-        foreach($aOptionGroups as $aOptionGroup)
202
+        foreach ($aOptionGroups as $aOptionGroup)
203 203
         {
204
-            if(isset($aOptionGroup['separator']))
204
+            if (isset($aOptionGroup['separator']))
205 205
             {
206 206
                 $aClassOptions['separator'] = $aOptionGroup['separator'];
207 207
             }
208
-            if(isset($aOptionGroup['protected']))
208
+            if (isset($aOptionGroup['protected']))
209 209
             {
210
-                if(is_string($aOptionGroup['protected']))
210
+                if (is_string($aOptionGroup['protected']))
211 211
                 {
212 212
                     $aOptionGroup['protected'] = [$aOptionGroup['protected']]; // Convert to array.
213 213
                 }
214 214
                 $aClassOptions['protected'] = array_merge($aClassOptions['protected'], $aOptionGroup['protected']);
215 215
             }
216
-            if(isset($aOptionGroup['functions']))
216
+            if (isset($aOptionGroup['functions']))
217 217
             {
218 218
                 $aClassOptions['functions'] = array_merge($aClassOptions['functions'], $aOptionGroup['functions']);
219 219
             }
@@ -259,10 +259,10 @@  discard block
 block discarded – undo
259 259
     private function getNamespaceClassOptions(string $sClassName)
260 260
     {
261 261
         // Find the corresponding namespace
262
-        foreach($this->aNamespaceOptions as $sNamespace => $aOptions)
262
+        foreach ($this->aNamespaceOptions as $sNamespace => $aOptions)
263 263
         {
264 264
             // Check if the namespace matches the class.
265
-            if(strncmp($sClassName, $sNamespace . '\\', strlen($sNamespace) + 1) === 0)
265
+            if (strncmp($sClassName, $sNamespace . '\\', strlen($sNamespace) + 1) === 0)
266 266
             {
267 267
                 // Save the class options
268 268
                 $this->aClasses[$sClassName] = $this->makeClassOptions($sClassName,
@@ -283,17 +283,17 @@  discard block
 block discarded – undo
283 283
     public function getClassOptions(string $sClassName): array
284 284
     {
285 285
         // Find options for a class registered with namespace.
286
-        if(!isset($this->aClasses[$sClassName]))
286
+        if (!isset($this->aClasses[$sClassName]))
287 287
         {
288 288
             $this->getNamespaceClassOptions($sClassName);
289
-            if(!isset($this->aClasses[$sClassName]))
289
+            if (!isset($this->aClasses[$sClassName]))
290 290
             {
291 291
                 // Find options for a class registered without namespace.
292 292
                 // We then need to parse all classes to be able to find one.
293 293
                 $this->di->getCallableRegistry()->parseDirectories();
294 294
             }
295 295
         }
296
-        if(isset($this->aClasses[$sClassName]))
296
+        if (isset($this->aClasses[$sClassName]))
297 297
         {
298 298
             return $this->aClasses[$sClassName];
299 299
         }
@@ -322,7 +322,7 @@  discard block
 block discarded – undo
322 322
     public function getCallableObjects(): array
323 323
     {
324 324
         $aCallableObjects = [];
325
-        foreach($this->aClasses as $sClassName => $aOptions)
325
+        foreach ($this->aClasses as $sClassName => $aOptions)
326 326
         {
327 327
             $this->di->registerCallableClass($sClassName, $aOptions);
328 328
             $aCallableObjects[$sClassName] = $this->di->getCallableObject($sClassName);
Please login to merge, or discard this patch.
src/Plugin/Request/CallableClass/CallableRegistry.php 1 patch
Spacing   +19 added lines, -19 removed lines patch added patch discarded remove patch
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
         $this->xRepository = $xRepository;
90 90
 
91 91
         // Set the composer autoloader
92
-        if(file_exists(($sAutoloadFile = __DIR__ . '/../../../../../../autoload.php')) ||
92
+        if (file_exists(($sAutoloadFile = __DIR__ . '/../../../../../../autoload.php')) ||
93 93
             file_exists(($sAutoloadFile = __DIR__ . '/../../../../vendor/autoload.php')))
94 94
         {
95 95
             $this->xAutoloader = require($sAutoloadFile);
@@ -106,7 +106,7 @@  discard block
 block discarded – undo
106 106
     public function addDirectory(string $sDirectory, array $aOptions)
107 107
     {
108 108
         // Set the autoload option default value
109
-        if(!isset($aOptions['autoload']))
109
+        if (!isset($aOptions['autoload']))
110 110
         {
111 111
             $aOptions['autoload'] = true;
112 112
         }
@@ -123,26 +123,26 @@  discard block
 block discarded – undo
123 123
     public function addNamespace(string $sNamespace, array $aOptions)
124 124
     {
125 125
         // Separator default value
126
-        if(!isset($aOptions['separator']))
126
+        if (!isset($aOptions['separator']))
127 127
         {
128 128
             $aOptions['separator'] = '.';
129 129
         }
130 130
         $aOptions['separator'] = trim($aOptions['separator']);
131
-        if(!in_array($aOptions['separator'], ['.', '_']))
131
+        if (!in_array($aOptions['separator'], ['.', '_']))
132 132
         {
133 133
             $aOptions['separator'] = '.';
134 134
         }
135
-        if($aOptions['separator'] === '_')
135
+        if ($aOptions['separator'] === '_')
136 136
         {
137 137
             $this->bUsingUnderscore = true;
138 138
         }
139 139
         // Set the autoload option default value
140
-        if(!isset($aOptions['autoload']))
140
+        if (!isset($aOptions['autoload']))
141 141
         {
142 142
             $aOptions['autoload'] = true;
143 143
         }
144 144
         // Register the dir with PSR4 autoloading
145
-        if(($aOptions['autoload']) && $this->xAutoloader != null)
145
+        if (($aOptions['autoload']) && $this->xAutoloader != null)
146 146
         {
147 147
             $this->xAutoloader->setPsr4($sNamespace . '\\', $aOptions['directory']);
148 148
         }
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
     public function parseDirectories()
158 158
     {
159 159
         // This is to be done only once.
160
-        if($this->bDirectoriesParsed)
160
+        if ($this->bDirectoriesParsed)
161 161
         {
162 162
             return;
163 163
         }
@@ -165,14 +165,14 @@  discard block
 block discarded – undo
165 165
 
166 166
         // Browse directories without namespaces and read all the files.
167 167
         $aClassMap = [];
168
-        foreach($this->xRepository->getDirectoryOptions() as $sDirectory => $aOptions)
168
+        foreach ($this->xRepository->getDirectoryOptions() as $sDirectory => $aOptions)
169 169
         {
170 170
             $itFile = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($sDirectory));
171 171
             // Iterate on dir content
172
-            foreach($itFile as $xFile)
172
+            foreach ($itFile as $xFile)
173 173
             {
174 174
                 // Skip everything except PHP files
175
-                if(!$xFile->isFile() || $xFile->getExtension() != 'php')
175
+                if (!$xFile->isFile() || $xFile->getExtension() != 'php')
176 176
                 {
177 177
                     continue;
178 178
                 }
@@ -180,7 +180,7 @@  discard block
 block discarded – undo
180 180
                 $sClassName = $xFile->getBasename('.php');
181 181
                 $aClassOptions = ['timestamp' => $xFile->getMTime()];
182 182
                 // No more custom classmap autoloading. The file will be included when needed.
183
-                if(($aOptions['autoload']))
183
+                if (($aOptions['autoload']))
184 184
                 {
185 185
                     $aClassMap[$sClassName] = $xFile->getPathname();
186 186
                 }
@@ -188,7 +188,7 @@  discard block
 block discarded – undo
188 188
             }
189 189
         }
190 190
         // Set classmap autoloading
191
-        if(($aClassMap) && $this->xAutoloader !== null)
191
+        if (($aClassMap) && $this->xAutoloader !== null)
192 192
         {
193 193
             $this->xAutoloader->addClassMap($aClassMap);
194 194
         }
@@ -202,7 +202,7 @@  discard block
 block discarded – undo
202 202
     public function parseNamespaces()
203 203
     {
204 204
         // This is to be done only once.
205
-        if($this->bNamespacesParsed)
205
+        if ($this->bNamespacesParsed)
206 206
         {
207 207
             return;
208 208
         }
@@ -210,17 +210,17 @@  discard block
 block discarded – undo
210 210
 
211 211
         // Browse directories with namespaces and read all the files.
212 212
         $sDS = DIRECTORY_SEPARATOR;
213
-        foreach($this->xRepository->getNamespaceOptions() as $sNamespace => $aOptions)
213
+        foreach ($this->xRepository->getNamespaceOptions() as $sNamespace => $aOptions)
214 214
         {
215 215
             $this->xRepository->addNamespace($sNamespace, ['separator' => $aOptions['separator']]);
216 216
 
217 217
             // Iterate on dir content
218 218
             $sDirectory = $aOptions['directory'];
219 219
             $itFile = new RecursiveIteratorIterator(new RecursiveDirectoryIterator($sDirectory));
220
-            foreach($itFile as $xFile)
220
+            foreach ($itFile as $xFile)
221 221
             {
222 222
                 // skip everything except PHP files
223
-                if(!$xFile->isFile() || $xFile->getExtension() != 'php')
223
+                if (!$xFile->isFile() || $xFile->getExtension() != 'php')
224 224
                 {
225 225
                     continue;
226 226
                 }
@@ -229,7 +229,7 @@  discard block
 block discarded – undo
229 229
                 $sClassPath = $sNamespace;
230 230
                 $sRelativePath = substr($xFile->getPath(), strlen($sDirectory));
231 231
                 $sRelativePath = trim(str_replace($sDS, '\\', $sRelativePath), '\\');
232
-                if($sRelativePath !== '')
232
+                if ($sRelativePath !== '')
233 233
                 {
234 234
                     $sClassPath .= '\\' . $sRelativePath;
235 235
                 }
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
256 256
         // Replace all separators ('.' and '_') with antislashes, and remove the antislashes
257 257
         // at the beginning and the end of the class name.
258 258
         $sClassName = trim(str_replace('.', '\\', $sClassName), '\\');
259
-        if($this->bUsingUnderscore)
259
+        if ($this->bUsingUnderscore)
260 260
         {
261 261
             $sClassName = trim(str_replace('_', '\\', $sClassName), '\\');
262 262
         }
Please login to merge, or discard this patch.
src/Di/Traits/RegisterTrait.php 2 patches
Spacing   +13 added lines, -13 removed lines patch added patch discarded remove patch
@@ -36,15 +36,15 @@  discard block
 block discarded – undo
36 36
     private function getCallableObjectOptions(array $aConfigOptions, array $aAnnotationOptions): array
37 37
     {
38 38
         $aOptions = [];
39
-        foreach($aConfigOptions as $sNames => $aFunctionOptions)
39
+        foreach ($aConfigOptions as $sNames => $aFunctionOptions)
40 40
         {
41 41
             $aFunctionNames = explode(',', $sNames); // Names are in comma-separated list.
42
-            foreach($aFunctionNames as $sFunctionName)
42
+            foreach ($aFunctionNames as $sFunctionName)
43 43
             {
44 44
                 $aOptions[$sFunctionName] = array_merge($aOptions[$sFunctionName] ?? [], $aFunctionOptions);
45 45
             }
46 46
         }
47
-        foreach($aAnnotationOptions as $sFunctionName => $aFunctionOptions)
47
+        foreach ($aAnnotationOptions as $sFunctionName => $aFunctionOptions)
48 48
         {
49 49
             $aOptions[$sFunctionName] = array_merge($aOptions[$sFunctionName] ?? [], $aFunctionOptions);
50 50
         }
@@ -66,7 +66,7 @@  discard block
 block discarded – undo
66 66
         $aMethods = $xCallableObject->getPublicMethods($aProtectedMethods);
67 67
         $aProperties = $xCallableObject->getProperties();
68 68
         [$bExcluded, $aAnnotationOptions, $aAnnotationProtected] = $xAnnotationReader->getAttributes($sClassName, $aMethods, $aProperties);
69
-        if($bExcluded)
69
+        if ($bExcluded)
70 70
         {
71 71
             $xCallableObject->configure('excluded', true);
72 72
             return;
@@ -78,11 +78,11 @@  discard block
 block discarded – undo
78 78
         // Functions options
79 79
         $aCallableOptions = [];
80 80
         $aOptions = $this->getCallableObjectOptions($aOptions['functions'], $aAnnotationOptions);
81
-        foreach($aOptions as $sFunctionName => $aFunctionOptions)
81
+        foreach ($aOptions as $sFunctionName => $aFunctionOptions)
82 82
         {
83
-            foreach($aFunctionOptions as $sOptionName => $xOptionValue)
83
+            foreach ($aFunctionOptions as $sOptionName => $xOptionValue)
84 84
             {
85
-                if(substr($sOptionName, 0, 2) !== '__')
85
+                if (substr($sOptionName, 0, 2) !== '__')
86 86
                 {
87 87
                     // Options for javascript code.
88 88
                     $aCallableOptions[$sFunctionName][$sOptionName] = $xOptionValue;
@@ -111,13 +111,13 @@  discard block
 block discarded – undo
111 111
         $sReflectionClass = $sClassName . '_ReflectionClass';
112 112
 
113 113
         // Prevent duplication
114
-        if($this->h($sReflectionClass)) // It's important not to use the class name here.
114
+        if ($this->h($sReflectionClass)) // It's important not to use the class name here.
115 115
         {
116 116
             return;
117 117
         }
118 118
 
119 119
         // Make sure the registered class exists
120
-        if(isset($aOptions['include']))
120
+        if (isset($aOptions['include']))
121 121
         {
122 122
             require_once($aOptions['include']);
123 123
         }
@@ -126,7 +126,7 @@  discard block
 block discarded – undo
126 126
         {
127 127
             $this->val($sReflectionClass, new ReflectionClass($sClassName));
128 128
         }
129
-        catch(ReflectionException $e)
129
+        catch (ReflectionException $e)
130 130
         {
131 131
             $xTranslator = $this->g(Translator::class);
132 132
             $sMessage = $xTranslator->trans('errors.class.invalid', ['name' => $sClassName]);
@@ -149,7 +149,7 @@  discard block
 block discarded – undo
149 149
         });
150 150
 
151 151
         // Register the user class, but only if the user already didn't.
152
-        if(!$this->h($sClassName))
152
+        if (!$this->h($sClassName))
153 153
         {
154 154
             $this->set($sClassName, function($c) use($sReflectionClass) {
155 155
                 return $this->make($c->g($sReflectionClass));
@@ -157,7 +157,7 @@  discard block
 block discarded – undo
157 157
         }
158 158
         // Initialize the user class instance
159 159
         $this->extend($sClassName, function($xRegisteredObject, $c) use($sClassName) {
160
-            if($xRegisteredObject instanceof CallableClass)
160
+            if ($xRegisteredObject instanceof CallableClass)
161 161
             {
162 162
                 // Set the protected attributes of the object
163 163
                 $cSetter = function($c, $sClassName) {
@@ -170,7 +170,7 @@  discard block
 block discarded – undo
170 170
 
171 171
             // Run the callbacks for class initialisation
172 172
             $aCallbacks = $c->g(CallbackManager::class)->getInitCallbacks();
173
-            foreach($aCallbacks as $xCallback)
173
+            foreach ($aCallbacks as $xCallback)
174 174
             {
175 175
                 call_user_func($xCallback, $xRegisteredObject);
176 176
             }
Please login to merge, or discard this patch.
Braces   +4 added lines, -3 removed lines patch added patch discarded remove patch
@@ -111,10 +111,12 @@  discard block
 block discarded – undo
111 111
         $sReflectionClass = $sClassName . '_ReflectionClass';
112 112
 
113 113
         // Prevent duplication
114
-        if($this->h($sReflectionClass)) // It's important not to use the class name here.
114
+        if($this->h($sReflectionClass)) {
115
+            // It's important not to use the class name here.
115 116
         {
116 117
             return;
117 118
         }
119
+        }
118 120
 
119 121
         // Make sure the registered class exists
120 122
         if(isset($aOptions['include']))
@@ -125,8 +127,7 @@  discard block
 block discarded – undo
125 127
         try
126 128
         {
127 129
             $this->val($sReflectionClass, new ReflectionClass($sClassName));
128
-        }
129
-        catch(ReflectionException $e)
130
+        } catch(ReflectionException $e)
130 131
         {
131 132
             $xTranslator = $this->g(Translator::class);
132 133
             $sMessage = $xTranslator->trans('errors.class.invalid', ['name' => $sClassName]);
Please login to merge, or discard this patch.