jaxon-php / jaxon-upload

src/UploadHandler.php

Spacing +6 added lines, -6 removed lines

@@ -135,12 +135,12 @@ 
      */
     public function canProcessRequest(ServerRequestInterface $xRequest): bool
     {
-        if(count($xRequest->getUploadedFiles()) > 0)
+        if (count($xRequest->getUploadedFiles()) > 0)
         {
             return true;
         }
         $aBody = $xRequest->getParsedBody();
-        if(is_array($aBody))
+        if (is_array($aBody))
         {
             return isset($aBody['jxnupl']);
         }
@@ -158,7 +158,7 @@ 
     private function setTempFile(ServerRequestInterface $xRequest): bool
     {
         $aBody = $xRequest->getParsedBody();
-        if(is_array($aBody))
+        if (is_array($aBody))
         {
             $this->sTempFile = trim($aBody['jxnupl'] ?? '');
             return $this->sTempFile !== '';
@@ -178,7 +178,7 @@ 
      */
     public function processRequest(ServerRequestInterface $xRequest): bool
     {
-        if($this->setTempFile($xRequest))
+        if ($this->setTempFile($xRequest))
         {
             // Ajax request following a normal HTTP upload.
             // Copy the previously uploaded files' location from the temp file.
@@ -186,7 +186,7 @@ 
             return true;
         }
 
-        if($this->bIsAjaxRequest)
+        if ($this->bIsAjaxRequest)
         {
             // Ajax request with upload.
             // Copy the uploaded files from the HTTP request.
@@ -203,7 +203,7 @@ 
             $sTempFile = $this->xUploadManager->saveToTempFile($this->aUserFiles);
             $this->xResponseManager->append(new UploadResponse($this->xPsr17Factory, $sTempFile));
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             $this->xResponseManager->append(new UploadResponse($this->xPsr17Factory, '', $e->getMessage()));
         }

Braces +1 added lines, -2 removed lines

@@ -202,8 +202,7 @@
             $this->aUserFiles = $this->xUploadManager->readFromHttpData($xRequest);
             $sTempFile = $this->xUploadManager->saveToTempFile($this->aUserFiles);
             $this->xResponseManager->append(new UploadResponse($this->xPsr17Factory, $sTempFile));
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             $this->xResponseManager->append(new UploadResponse($this->xPsr17Factory, '', $e->getMessage()));
         }

src/Manager/Validator.php

Spacing +6 added lines, -6 removed lines

@@ -97,7 +97,7 @@ 
     {
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty);
         $aAllowed = $this->xConfigManager->getOption('upload.files.' . $sName . '.' . $sProperty, $xDefault);
-        if(is_array($aAllowed) && !in_array($sValue, $aAllowed))
+        if (is_array($aAllowed) && !in_array($sValue, $aAllowed))
         {
             $this->sErrorMessage = $this->xTranslator->trans('errors.upload.' . $sField, [$sField => $sValue]);
             return false;
@@ -118,7 +118,7 @@ 
     {
         $xDefault = $this->xConfigManager->getOption('upload.default.' . $sProperty, 0);
         $nSize = $this->xConfigManager->getOption('upload.files.' . $sName . '.' . $sProperty, $xDefault);
-        if($nSize > 0 && (
+        if ($nSize > 0 && (
             ($sProperty == 'max-size' && $nFileSize > $nSize) ||
             ($sProperty == 'min-size' && $nFileSize < $nSize)))
         {
@@ -140,25 +140,25 @@ 
     {
         $this->sErrorMessage = '';
         // Verify the file extension
-        if(!$this->validateFileProperty($sName, $xFile->type(), 'types', 'type'))
+        if (!$this->validateFileProperty($sName, $xFile->type(), 'types', 'type'))
         {
             return false;
         }
 
         // Verify the file extension
-        if(!$this->validateFileProperty($sName, $xFile->extension(), 'extensions', 'extension'))
+        if (!$this->validateFileProperty($sName, $xFile->extension(), 'extensions', 'extension'))
         {
             return false;
         }
 
         // Verify the max size
-        if(!$this->validateFileSize($sName, $xFile->size(), 'max-size'))
+        if (!$this->validateFileSize($sName, $xFile->size(), 'max-size'))
         {
             return false;
         }
 
         // Verify the min size
-        if(!$this->validateFileSize($sName, $xFile->size(), 'min-size'))
+        if (!$this->validateFileSize($sName, $xFile->size(), 'min-size'))
         {
             return false;
         }

src/Manager/UploadManager.php

Spacing +20 added lines, -20 removed lines

@@ -157,13 +157,13 @@ 
         try
         {
             $xFilesystem->createDirectory($sUploadDir);
-            if($xFilesystem->visibility($sUploadDir) !== Visibility::PUBLIC)
+            if ($xFilesystem->visibility($sUploadDir) !== Visibility::PUBLIC)
             {
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return $sUploadDir;
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -210,19 +210,19 @@ 
     {
         // Filename without the extension. Needs to be sanitized.
         $sName = pathinfo($xHttpFile->getClientFilename(), PATHINFO_FILENAME);
-        if($this->cNameSanitizer !== null)
+        if ($this->cNameSanitizer !== null)
         {
             $sName = (string)call_user_func($this->cNameSanitizer, $sName, $sVarName, $this->sUploadFieldId);
         }
         // Check the uploaded file validity
-        if($xHttpFile->getError())
+        if ($xHttpFile->getError())
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.failed', ['name' => $sVarName]));
         }
         // Set the user file data
         $xFile = File::fromHttpFile($xFilesystem, $sUploadDir, $sName, $xHttpFile);
         // Verify file validity (format, size)
-        if(!$this->xValidator->validateUploadedFile($sVarName, $xFile))
+        if (!$this->xValidator->validateUploadedFile($sVarName, $xFile))
         {
             throw new RequestException($this->xValidator->getErrorMessage());
         }
@@ -246,16 +246,16 @@ 
 
         $aUserFiles = [];
         $this->aAllFiles = []; // A flat list of all uploaded files
-        foreach($aTempFiles as $sVarName => $aFiles)
+        foreach ($aTempFiles as $sVarName => $aFiles)
         {
             $aUserFiles[$sVarName] = [];
             // Get the path to the upload dir
             [$xFilesystem, $sUploadDir] = $this->getUploadDir($sVarName);
-            if(!is_array($aFiles))
+            if (!is_array($aFiles))
             {
                 $aFiles = [$aFiles];
             }
-            foreach($aFiles as $xHttpFile)
+            foreach ($aFiles as $xHttpFile)
             {
                 $aUserFiles[$sVarName][] = $this->makeUploadedFile($xFilesystem, $sUploadDir, $sVarName, $xHttpFile);
             }
@@ -263,12 +263,12 @@ 
         // Copy the uploaded files from the temp dir to the user dir
         try
         {
-            foreach($this->aAllFiles as $aFiles)
+            foreach ($this->aAllFiles as $aFiles)
             {
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream()->getContents());
             }
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -287,10 +287,10 @@ 
     {
         // Convert uploaded file to an array
         $aFiles = [];
-        foreach($aUserFiles as $sVarName => $aVarFiles)
+        foreach ($aUserFiles as $sVarName => $aVarFiles)
         {
             $aFiles[$sVarName] = [];
-            foreach($aVarFiles as $aVarFile)
+            foreach ($aVarFiles as $aVarFile)
             {
                 $aFiles[$sVarName][] = $aVarFile->toTempData();
             }
@@ -302,7 +302,7 @@ 
         {
             $xFilesystem->write($sUploadDir . $sTempFile . '.json', json_encode($aFiles));
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -320,7 +320,7 @@ 
     private function getUploadTempFile(string $sTempFile): array
     {
         // Verify file name validity
-        if(!$this->xValidator->validateTempFileName($sTempFile))
+        if (!$this->xValidator->validateTempFileName($sTempFile))
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.invalid'));
         }
@@ -328,13 +328,13 @@ 
         $sUploadTempFile = $sUploadDir . $sTempFile . '.json';
         try
         {
-            if($xFilesystem->visibility($sUploadTempFile) !== Visibility::PUBLIC)
+            if ($xFilesystem->visibility($sUploadTempFile) !== Visibility::PUBLIC)
             {
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return [$xFilesystem, $sUploadTempFile];
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -356,15 +356,15 @@ 
         {
             $aFiles = json_decode($xFilesystem->read($sUploadTempFile), true);
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
         $aUserFiles = [];
-        foreach($aFiles as $sVarName => $aVarFiles)
+        foreach ($aFiles as $sVarName => $aVarFiles)
         {
             $aUserFiles[$sVarName] = [];
-            foreach($aVarFiles as $aVarFile)
+            foreach ($aVarFiles as $aVarFile)
             {
                 $aUserFiles[$sVarName][] = File::fromTempFile($xFilesystem, $aVarFile);
             }
@@ -373,7 +373,7 @@ 
         {
             $xFilesystem->delete($sUploadTempFile);
         }
-        catch(FilesystemException $e){}
+        catch (FilesystemException $e) {}
         return $aUserFiles;
     }
 }

Braces +6 added lines, -12 removed lines

@@ -162,8 +162,7 @@ 
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return $sUploadDir;
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -267,8 +266,7 @@ 
             {
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream()->getContents());
             }
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -301,8 +299,7 @@ 
         try
         {
             $xFilesystem->write($sUploadDir . $sTempFile . '.json', json_encode($aFiles));
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -333,8 +330,7 @@ 
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return [$xFilesystem, $sUploadTempFile];
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -355,8 +351,7 @@ 
         try
         {
             $aFiles = json_decode($xFilesystem->read($sUploadTempFile), true);
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -372,8 +367,7 @@ 
         try
         {
             $xFilesystem->delete($sUploadTempFile);
-        }
-        catch(FilesystemException $e){}
+        } catch(FilesystemException $e){}
         return $aUserFiles;
     }
 }

src/Manager/FileStorage.php

Spacing +2 added lines, -2 removed lines

@@ -57,11 +57,11 @@
     {
         // Default upload dir
         $sRootDir = $this->xConfigManager->getOption('upload.default.dir');
-        if(($sFieldId = trim($sFieldId)) !== '')
+        if (($sFieldId = trim($sFieldId)) !== '')
         {
             $sRootDir = $this->xConfigManager->getOption('upload.files.' . $sFieldId . '.dir', $sRootDir);
         }
-        if(!is_string($sRootDir))
+        if (!is_string($sRootDir))
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }

src/UploadResponse.php

Spacing +1 added lines, -2 removed lines

@@ -99,8 +99,7 @@
     public function getOutput(): string
     {
         $sResult = json_encode(($this->sUploadedFile) ?
-            ['code' => 'success', 'upl' => $this->sUploadedFile] :
-            ['code' => 'error', 'msg' => $this->sErrorMessage]) . ';';
+            ['code' => 'success', 'upl' => $this->sUploadedFile] : ['code' => 'error', 'msg' => $this->sErrorMessage]) . ';';
         $sConsoleLog = array_reduce($this->aDebugMessages, function($sJsLog, $sMessage) {
             return "$sJsLog\n\t" . 'console.log("' . addslashes($sMessage) . '");';
         }, '');

src/start.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
  */
 function register(Container $di, bool $bForce = false)
 {
-    if(!$bForce && $di->h(UploadHandler::class))
+    if (!$bForce && $di->h(UploadHandler::class))
     {
         return;
     }
@@ -80,7 +80,7 @@ 
         {
             public function onChanges(Config $xConfig)
             {
-                if($xConfig->getOption('core.upload.enabled'))
+                if ($xConfig->getOption('core.upload.enabled'))
                 {
                     register(jaxon()->di());
                 }
@@ -88,7 +88,7 @@ 
 
             public function onChange(Config $xConfig, string $sName)
             {
-                if($sName === 'core.upload.enabled' && $xConfig->getOption('core.upload.enabled'))
+                if ($sName === 'core.upload.enabled' && $xConfig->getOption('core.upload.enabled'))
                 {
                     register(jaxon()->di());
                 }