Passed
Push — main ( 99eb4f...cf0b02 )
by Thierry
04:07
created
jaxon-upload/src/register.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
 {
25 25
     $jaxon = jaxon();
26 26
     $di = $jaxon->di();
27
-    if($di->h(UploadHandler::class))
27
+    if ($di->h(UploadHandler::class))
28 28
     {
29 29
         return;
30 30
     }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
         $xUploadHandler = $di->g(UploadHandler::class);
77 77
         // The HTTP request
78 78
         $xRequest = $di->getRequest();
79
-        if($xUploadHandler->canProcessRequest($xRequest))
79
+        if ($xUploadHandler->canProcessRequest($xRequest))
80 80
         {
81 81
             $xUploadHandler->processRequest($xRequest);
82 82
         }
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
 {
93 93
     $jaxon = jaxon();
94 94
     $jaxon->callback()->boot(function() use($jaxon) {
95
-        if($jaxon->getOption('core.upload.enabled'))
95
+        if ($jaxon->getOption('core.upload.enabled'))
96 96
         {
97 97
             registerUpload();
98 98
         }
@@ -102,7 +102,7 @@  discard block
 block discarded – undo
102 102
 function register()
103 103
 {
104 104
     // Do nothing if running in cli.
105
-    if(php_sapi_name() !== 'cli')
105
+    if (php_sapi_name() !== 'cli')
106 106
     {
107 107
         _register();
108 108
     };
Please login to merge, or discard this patch.
jaxon-upload/src/Manager/FileStorage.php 1 patch
Spacing   +5 added lines, -6 removed lines patch added patch discarded remove patch
@@ -80,8 +80,7 @@  discard block
 block discarded – undo
80 80
     {
81 81
         // Local file system adapter
82 82
         $this->registerAdapter('local', function(string $sRootDir, $xOptions) {
83
-            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) :
84
-                new LocalFilesystemAdapter($sRootDir, $xOptions);
83
+            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) : new LocalFilesystemAdapter($sRootDir, $xOptions);
85 84
         });
86 85
 
87 86
         // In memory file system adapter
@@ -144,7 +143,7 @@  discard block
 block discarded – undo
144 143
     public function filesystem(string $sField = ''): Filesystem
145 144
     {
146 145
         $sField = trim($sField);
147
-        if(isset($this->aFilesystems[$sField]))
146
+        if (isset($this->aFilesystems[$sField]))
148 147
         {
149 148
             return $this->aFilesystems[$sField];
150 149
         }
@@ -154,18 +153,18 @@  discard block
 block discarded – undo
154 153
         $sRootDir = $this->xConfigManager->getOption('upload.default.dir', '');
155 154
         $aOptions = $this->xConfigManager->getOption('upload.default.options');
156 155
         $sConfigKey = "upload.files.$sField";
157
-        if($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
156
+        if ($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
158 157
         {
159 158
             $sStorage = $this->xConfigManager->getOption("$sConfigKey.storage", $sStorage);
160 159
             $sRootDir = $this->xConfigManager->getOption("$sConfigKey.dir", $sRootDir);
161 160
             $aOptions = $this->xConfigManager->getOption("$sConfigKey.options", $aOptions);
162 161
         }
163 162
 
164
-        if(!is_string($sRootDir))
163
+        if (!is_string($sRootDir))
165 164
         {
166 165
             throw new RequestException($this->xTranslator->trans('errors.upload.dir'));
167 166
         }
168
-        if(!isset($this->aAdapters[$sStorage]))
167
+        if (!isset($this->aAdapters[$sStorage]))
169 168
         {
170 169
             throw new RequestException($this->xTranslator->trans('errors.upload.adapter'));
171 170
         }
Please login to merge, or discard this patch.
jaxon-upload/src/Manager/Validator.php 1 patch
Spacing   +6 added lines, -6 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@  discard block
 block discarded – undo
97 97
     {
98 98
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty);
99 99
         $aAllowed = $this->xConfigManager->getOption('upload.files.' . $sVarName . '.' . $sProperty, $xDefault);
100
-        if(is_array($aAllowed) && !in_array($sValue, $aAllowed))
100
+        if (is_array($aAllowed) && !in_array($sValue, $aAllowed))
101 101
         {
102 102
             $this->sErrorMessage = $this->xTranslator->trans('errors.upload.' . $sField, [$sField => $sValue]);
103 103
             return false;
@@ -118,7 +118,7 @@  discard block
 block discarded – undo
118 118
     {
119 119
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty, 0);
120 120
         $nSize = $this->xConfigManager->getOption('upload.files.' . $sVarName . '.' . $sProperty, $xDefault);
121
-        if($nSize > 0 && (
121
+        if ($nSize > 0 && (
122 122
             ($sProperty == 'max-size' && $nFileSize > $nSize) ||
123 123
             ($sProperty == 'min-size' && $nFileSize < $nSize)))
124 124
         {
@@ -140,25 +140,25 @@  discard block
 block discarded – undo
140 140
     {
141 141
         $this->sErrorMessage = '';
142 142
         // Verify the file extension
143
-        if(!$this->validateFileProperty($sVarName, $xFile->type(), 'types', 'type'))
143
+        if (!$this->validateFileProperty($sVarName, $xFile->type(), 'types', 'type'))
144 144
         {
145 145
             return false;
146 146
         }
147 147
 
148 148
         // Verify the file extension
149
-        if(!$this->validateFileProperty($sVarName, $xFile->extension(), 'extensions', 'extension'))
149
+        if (!$this->validateFileProperty($sVarName, $xFile->extension(), 'extensions', 'extension'))
150 150
         {
151 151
             return false;
152 152
         }
153 153
 
154 154
         // Verify the max size
155
-        if(!$this->validateFileSize($sVarName, $xFile->size(), 'max-size'))
155
+        if (!$this->validateFileSize($sVarName, $xFile->size(), 'max-size'))
156 156
         {
157 157
             return false;
158 158
         }
159 159
 
160 160
         // Verify the min size
161
-        if(!$this->validateFileSize($sVarName, $xFile->size(), 'min-size'))
161
+        if (!$this->validateFileSize($sVarName, $xFile->size(), 'min-size'))
162 162
         {
163 163
             return false;
164 164
         }
Please login to merge, or discard this patch.
jaxon-upload/src/Manager/UploadManager.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -133,13 +133,13 @@  discard block
 block discarded – undo
133 133
         try
134 134
         {
135 135
             $xFilesystem->createDirectory($sUploadDir);
136
-            if($xFilesystem->visibility($sUploadDir) !== Visibility::PUBLIC)
136
+            if ($xFilesystem->visibility($sUploadDir) !== Visibility::PUBLIC)
137 137
             {
138 138
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
139 139
             }
140 140
             return $sUploadDir;
141 141
         }
142
-        catch(FilesystemException $e)
142
+        catch (FilesystemException $e)
143 143
         {
144 144
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
145 145
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
@@ -173,7 +173,7 @@  discard block
 block discarded – undo
173 173
     {
174 174
         // Check the uploaded file validity
175 175
         $nErrorCode = $xHttpFile->getError();
176
-        if($nErrorCode !== UPLOAD_ERR_OK)
176
+        if ($nErrorCode !== UPLOAD_ERR_OK)
177 177
         {
178 178
             $this->xLogger->error('File upload error.', [
179 179
                 'code' => $nErrorCode,
@@ -184,7 +184,7 @@  discard block
 block discarded – undo
184 184
 
185 185
         // Filename without the extension. Needs to be sanitized.
186 186
         $sName = pathinfo($xHttpFile->getClientFilename(), PATHINFO_FILENAME);
187
-        if($this->cNameSanitizer !== null)
187
+        if ($this->cNameSanitizer !== null)
188 188
         {
189 189
             $sName = (string)call_user_func($this->cNameSanitizer, $sName, $sField, $this->sUploadFieldId);
190 190
         }
@@ -192,7 +192,7 @@  discard block
 block discarded – undo
192 192
         // Set the user file data
193 193
         $xFile = File::fromHttpFile($this->xFileStorage->filesystem($sField), $xHttpFile, $sUploadDir, $sName);
194 194
         // Verify file validity (format, size)
195
-        if(!$this->xValidator->validateUploadedFile($sField, $xFile))
195
+        if (!$this->xValidator->validateUploadedFile($sField, $xFile))
196 196
         {
197 197
             throw new RequestException($this->xValidator->getErrorMessage());
198 198
         }
@@ -217,16 +217,16 @@  discard block
 block discarded – undo
217 217
 
218 218
         $aUserFiles = [];
219 219
         $this->aAllFiles = []; // A flat list of all uploaded files
220
-        foreach($aTempFiles as $sField => $aFiles)
220
+        foreach ($aTempFiles as $sField => $aFiles)
221 221
         {
222 222
             $aUserFiles[$sField] = [];
223 223
             // Get the path to the upload dir
224 224
             $sUploadDir = $this->getUploadDir($sField);
225
-            if(!is_array($aFiles))
225
+            if (!is_array($aFiles))
226 226
             {
227 227
                 $aFiles = [$aFiles];
228 228
             }
229
-            foreach($aFiles as $xHttpFile)
229
+            foreach ($aFiles as $xHttpFile)
230 230
             {
231 231
                 $aUserFiles[$sField][] = $this->makeUploadedFile($xHttpFile, $sUploadDir, $sField);
232 232
             }
@@ -234,12 +234,12 @@  discard block
 block discarded – undo
234 234
         // Copy the uploaded files from the temp dir to the user dir
235 235
         try
236 236
         {
237
-            foreach($this->aAllFiles as $aFiles)
237
+            foreach ($this->aAllFiles as $aFiles)
238 238
             {
239 239
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream());
240 240
             }
241 241
         }
242
-        catch(FilesystemException $e)
242
+        catch (FilesystemException $e)
243 243
         {
244 244
             $this->xLogger->error('Filesystem error.', ['message' => $e->getMessage()]);
245 245
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
Please login to merge, or discard this patch.
jaxon-core/src/App/Metadata/Data/DatabagData.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 validateName(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/Metadata/Data/HookData.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@  discard block
 block discarded – undo
54 54
      */
55 55
     protected function validateMethod(string $sMethod): void
56 56
     {
57
-        if(preg_match('/^[a-zA-Z][a-zA-Z0-9_]*$/', $sMethod) > 0)
57
+        if (preg_match('/^[a-zA-Z][a-zA-Z0-9_]*$/', $sMethod) > 0)
58 58
         {
59 59
             return;
60 60
         }
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
     public function encode(string $sVarName): array
82 82
     {
83 83
         $aCalls = [];
84
-        foreach($this->aCalls as $sMethod => $aParams)
84
+        foreach ($this->aCalls as $sMethod => $aParams)
85 85
         {
86 86
             $sParams = addslashes(json_encode($aParams));
87 87
             $aCalls[] = "{$sVarName}->addCall('$sMethod', json_decode(\"$sParams\", true));";
Please login to merge, or discard this patch.
jaxon-annotations/tests/TestAnnotation/AttrAnnotationTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
         $aFiles = scandir($this->sCacheDir);
46 46
         foreach ($aFiles as $sFile)
47 47
         {
48
-            if($sFile !== '.' && $sFile !== '..')
48
+            if ($sFile !== '.' && $sFile !== '..')
49 49
             {
50 50
                 @unlink($this->sCacheDir . DIRECTORY_SEPARATOR . $sFile);
51 51
             }
Please login to merge, or discard this patch.
jaxon-annotations/tests/TestAnnotation/DocBlockAnnotationTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
         $aFiles = scandir($this->sCacheDir);
46 46
         foreach ($aFiles as $sFile)
47 47
         {
48
-            if($sFile !== '.' && $sFile !== '..')
48
+            if ($sFile !== '.' && $sFile !== '..')
49 49
             {
50 50
                 @unlink($this->sCacheDir . DIRECTORY_SEPARATOR . $sFile);
51 51
             }
Please login to merge, or discard this patch.
jaxon-annotations/tests/TestAnnotation/TraitAnnotationTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -45,7 +45,7 @@
 block discarded – undo
45 45
         $aFiles = scandir($this->sCacheDir);
46 46
         foreach ($aFiles as $sFile)
47 47
         {
48
-            if($sFile !== '.' && $sFile !== '..')
48
+            if ($sFile !== '.' && $sFile !== '..')
49 49
             {
50 50
                 @unlink($this->sCacheDir . DIRECTORY_SEPARATOR . $sFile);
51 51
             }
Please login to merge, or discard this patch.