jaxon-php / jaxon-upload

src/Manager/FileStorage.php

Spacing +5 added lines, -6 removed lines

@@ -80,8 +80,7 @@ 
     {
         // Local file system adapter
         $this->registerAdapter('local', function(string $sRootDir, $xOptions) {
-            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) :
-                new LocalFilesystemAdapter($sRootDir, $xOptions);
+            return empty($xOptions) ? new LocalFilesystemAdapter($sRootDir) : new LocalFilesystemAdapter($sRootDir, $xOptions);
         });
 
         // In memory file system adapter
@@ -144,7 +143,7 @@ 
     public function filesystem(string $sField = ''): Filesystem
     {
         $sField = trim($sField);
-        if(isset($this->aFilesystems[$sField]))
+        if (isset($this->aFilesystems[$sField]))
         {
             return $this->aFilesystems[$sField];
         }
@@ -154,18 +153,18 @@ 
         $sRootDir = $this->xConfigManager->getOption('upload.default.dir', '');
         $aOptions = $this->xConfigManager->getOption('upload.default.options');
         $sConfigKey = "upload.files.$sField";
-        if($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
+        if ($sField !== '' && $this->xConfigManager->hasOption($sConfigKey))
         {
             $sStorage = $this->xConfigManager->getOption("$sConfigKey.storage", $sStorage);
             $sRootDir = $this->xConfigManager->getOption("$sConfigKey.dir", $sRootDir);
             $aOptions = $this->xConfigManager->getOption("$sConfigKey.options", $aOptions);
         }
 
-        if(!is_string($sRootDir))
+        if (!is_string($sRootDir))
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.dir'));
         }
-        if(!isset($this->aAdapters[$sStorage]))
+        if (!isset($this->aAdapters[$sStorage]))
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.adapter'));
         }

src/Manager/UploadManager.php

Spacing +20 added lines, -20 removed lines

@@ -153,13 +153,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'));
         }
@@ -202,20 +202,20 @@ 
     private function makeUploadedFile(string $sUploadDir, string $sField, UploadedFile $xHttpFile): File
     {
         // Check the uploaded file validity
-        if($xHttpFile->getError())
+        if ($xHttpFile->getError())
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.failed', ['name' => $sField]));
         }
         // Set the user file data
         $xFile = File::fromHttpFile($this->xFileStorage->filesystem($sField), $sUploadDir, $xHttpFile);
         // Verify file validity (format, size)
-        if(!$this->xValidator->validateUploadedFile($sField, $xFile))
+        if (!$this->xValidator->validateUploadedFile($sField, $xFile))
         {
             throw new RequestException($this->xValidator->getErrorMessage());
         }
         // 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, $sField, $this->sUploadFieldId);
         }
@@ -240,16 +240,16 @@ 
 
         $aUserFiles = [];
         $this->aAllFiles = []; // A flat list of all uploaded files
-        foreach($aTempFiles as $sField => $aFiles)
+        foreach ($aTempFiles as $sField => $aFiles)
         {
             $aUserFiles[$sField] = [];
             // Get the path to the upload dir
             $sUploadDir = $this->getUploadDir($sField);
-            if(!is_array($aFiles))
+            if (!is_array($aFiles))
             {
                 $aFiles = [$aFiles];
             }
-            foreach($aFiles as $xHttpFile)
+            foreach ($aFiles as $xHttpFile)
             {
                 $aUserFiles[$sField][] = $this->makeUploadedFile($sUploadDir, $sField, $xHttpFile);
             }
@@ -257,12 +257,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());
             }
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -281,10 +281,10 @@ 
     {
         // Convert uploaded file to an array
         $aFiles = [];
-        foreach($aUserFiles as $sField => $aFieldFiles)
+        foreach ($aUserFiles as $sField => $aFieldFiles)
         {
             $aFiles[$sField] = [];
-            foreach($aFieldFiles as $aFieldFile)
+            foreach ($aFieldFiles as $aFieldFile)
             {
                 $aFiles[$sField][] = $aFieldFile->toTempData();
             }
@@ -296,7 +296,7 @@ 
         {
             $this->xFileStorage->filesystem()->write($sUploadDir . $sTempFile . '.json', json_encode($aFiles));
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -314,7 +314,7 @@ 
     private function getUploadTempFile(string $sTempFile): string
     {
         // Verify file name validity
-        if(!$this->xValidator->validateTempFileName($sTempFile))
+        if (!$this->xValidator->validateTempFileName($sTempFile))
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.invalid'));
         }
@@ -322,13 +322,13 @@ 
         $sUploadTempFile = $sUploadDir . $sTempFile . '.json';
         try
         {
-            if($this->xFileStorage->filesystem()->visibility($sUploadTempFile) !== Visibility::PUBLIC)
+            if ($this->xFileStorage->filesystem()->visibility($sUploadTempFile) !== Visibility::PUBLIC)
             {
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return $sUploadTempFile;
         }
-        catch(FilesystemException $e)
+        catch (FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -351,15 +351,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 $sField => $aFieldFiles)
+        foreach ($aFiles as $sField => $aFieldFiles)
         {
             $aUserFiles[$sField] = [];
-            foreach($aFieldFiles as $aFieldFile)
+            foreach ($aFieldFiles as $aFieldFile)
             {
                 $aUserFiles[$sField][] = File::fromTempFile($this->xFileStorage->filesystem($sField), $aFieldFile);
             }
@@ -368,7 +368,7 @@ 
         {
             $xFileSystem->delete($sUploadTempFile);
         }
-        catch(FilesystemException $e){/* Not a big deal if the file cannot be deleted. */}
+        catch (FilesystemException $e) {/* Not a big deal if the file cannot be deleted. */}
         return $aUserFiles;
     }
 }

Braces +6 added lines, -12 removed lines

@@ -158,8 +158,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'));
         }
@@ -261,8 +260,7 @@ 
             {
                 $aFiles['user']->filesystem()->write($aFiles['user']->path(), $aFiles['temp']->getStream());
             }
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -295,8 +293,7 @@ 
         try
         {
             $this->xFileStorage->filesystem()->write($sUploadDir . $sTempFile . '.json', json_encode($aFiles));
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -327,8 +324,7 @@ 
                 throw new RequestException($this->xTranslator->trans('errors.upload.access'));
             }
             return $sUploadTempFile;
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -350,8 +346,7 @@ 
         try
         {
             $aFiles = json_decode($xFileSystem->read($sUploadTempFile), true);
-        }
-        catch(FilesystemException $e)
+        } catch(FilesystemException $e)
         {
             throw new RequestException($this->xTranslator->trans('errors.upload.access'));
         }
@@ -367,8 +362,7 @@ 
         try
         {
             $xFileSystem->delete($sUploadTempFile);
-        }
-        catch(FilesystemException $e){/* Not a big deal if the file cannot be deleted. */}
+        } catch(FilesystemException $e){/* Not a big deal if the file cannot be deleted. */}
         return $aUserFiles;
     }
 }