jaxon-php / jaxon-core

src/Request/Plugin/FileUpload.php

Unused Use Statements -1 removed lines

@@ -16,7 +16,6 @@
 use Jaxon\Plugin\Request as RequestPlugin;
 use Jaxon\Request\Support\UploadedFile;
 use Jaxon\Response\UploadResponse;
-
 use Exception;
 use Closure;
 

Spacing +31 added lines, -31 removed lines

@@ -68,11 +68,11 @@ 
     {
         $this->sUploadSubdir = uniqid() . DIRECTORY_SEPARATOR;
 
-        if(array_key_exists('jxnupl', $_POST))
+        if (array_key_exists('jxnupl', $_POST))
         {
             $this->sTempFile = $_POST['jxnupl'];
         }
-        elseif(array_key_exists('jxnupl', $_GET))
+        elseif (array_key_exists('jxnupl', $_GET))
         {
             $this->sTempFile = $_GET['jxnupl'];
         }
@@ -100,7 +100,7 @@ 
      */
     protected function filterFilename($sFilename, $sVarName)
     {
-        if(($this->cFileFilter))
+        if (($this->cFileFilter))
         {
             $cFileFilter = $this->cFileFilter;
             $sFilename = (string)$cFileFilter($sFilename, $sVarName);
@@ -122,12 +122,12 @@ 
         $sUploadDir = $this->getOption('upload.files.' . $sFieldId . '.dir', $sDefaultUploadDir);
         $sUploadDir = rtrim(trim($sUploadDir), '/\\') . DIRECTORY_SEPARATOR;
         // Verify that the upload dir exists and is writable
-        if(!is_writable($sUploadDir))
+        if (!is_writable($sUploadDir))
         {
             throw new \Jaxon\Exception\Error($this->trans('errors.upload.access'));
         }
         $sUploadDir .= $this->sUploadSubdir;
-        if(!file_exists($sUploadDir) && !@mkdir($sUploadDir))
+        if (!file_exists($sUploadDir) && !@mkdir($sUploadDir))
         {
             throw new \Jaxon\Exception\Error($this->trans('errors.upload.access'));
         }
@@ -145,12 +145,12 @@ 
         $sUploadDir = $this->getOption('upload.default.dir');
         $sUploadDir = rtrim(trim($sUploadDir), '/\\') . DIRECTORY_SEPARATOR;
         // Verify that the upload dir exists and is writable
-        if(!is_writable($sUploadDir))
+        if (!is_writable($sUploadDir))
         {
             throw new \Jaxon\Exception\Error($this->trans('errors.upload.access'));
         }
         $sUploadDir .= 'tmp' . DIRECTORY_SEPARATOR;
-        if(!file_exists($sUploadDir) && !@mkdir($sUploadDir))
+        if (!file_exists($sUploadDir) && !@mkdir($sUploadDir))
         {
             throw new \Jaxon\Exception\Error($this->trans('errors.upload.access'));
         }
@@ -168,7 +168,7 @@ 
         $sUploadDir = rtrim(trim($sUploadDir), '/\\') . DIRECTORY_SEPARATOR;
         $sUploadDir .= 'tmp' . DIRECTORY_SEPARATOR;
         $sUploadTempFile = $sUploadDir . $this->sTempFile . '.json';
-        if(!is_readable($sUploadTempFile))
+        if (!is_readable($sUploadTempFile))
         {
             throw new \Jaxon\Exception\Error($this->trans('errors.upload.access'));
         }
@@ -184,18 +184,18 @@ 
     {
         // Check validity of the uploaded files
         $aTempFiles = [];
-        foreach($_FILES as $sVarName => $aFile)
+        foreach ($_FILES as $sVarName => $aFile)
         {
-            if(is_array($aFile['name']))
+            if (is_array($aFile['name']))
             {
                 $nFileCount = count($aFile['name']);
-                for($i = 0; $i < $nFileCount; $i++)
+                for ($i = 0; $i < $nFileCount; $i++)
                 {
-                    if(!$aFile['name'][$i])
+                    if (!$aFile['name'][$i])
                     {
                         continue;
                     }
-                    if(!array_key_exists($sVarName, $aTempFiles))
+                    if (!array_key_exists($sVarName, $aTempFiles))
                     {
                         $aTempFiles[$sVarName] = [];
                     }
@@ -215,11 +215,11 @@ 
             }
             else
             {
-                if(!$aFile['name'])
+                if (!$aFile['name'])
                 {
                     continue;
                 }
-                if(!array_key_exists($sVarName, $aTempFiles))
+                if (!array_key_exists($sVarName, $aTempFiles))
                 {
                     $aTempFiles[$sVarName] = [];
                 }
@@ -239,17 +239,17 @@ 
         }
 
         // Check uploaded files validity
-        foreach($aTempFiles as $sVarName => $aFiles)
+        foreach ($aTempFiles as $sVarName => $aFiles)
         {
-            foreach($aFiles as $aFile)
+            foreach ($aFiles as $aFile)
             {
                 // Verify upload result
-                if($aFile['error'] != 0)
+                if ($aFile['error'] != 0)
                 {
                     throw new \Jaxon\Exception\Error($this->trans('errors.upload.failed', $aFile));
                 }
                 // Verify file validity (format, size)
-                if(!$this->validateUploadedFile($sVarName, $aFile))
+                if (!$this->validateUploadedFile($sVarName, $aFile))
                 {
                     throw new \Jaxon\Exception\Error($this->getValidatorMessage());
                 }
@@ -259,10 +259,10 @@ 
         }
 
         // Copy the uploaded files from the temp dir to the user dir
-        foreach($aTempFiles as $sVarName => $_aTempFiles)
+        foreach ($aTempFiles as $sVarName => $_aTempFiles)
         {
             $this->aUserFiles[$sVarName] = [];
-            foreach($_aTempFiles as $aFile)
+            foreach ($_aTempFiles as $aFile)
             {
                 // Get the path to the upload dir
                 $sUploadDir = $this->getUploadDir($sVarName);
@@ -284,10 +284,10 @@ 
     {
         // Convert uploaded file to an array
         $aFiles = [];
-        foreach($this->aUserFiles as $sVarName => $aUserFiles)
+        foreach ($this->aUserFiles as $sVarName => $aUserFiles)
         {
             $aFiles[$sVarName] = [];
-            foreach($aUserFiles as $aUserFile)
+            foreach ($aUserFiles as $aUserFile)
             {
                  $aFiles[$sVarName][] = $aUserFile->toTempData();
             }
@@ -308,10 +308,10 @@ 
         // Upload temp file
         $sUploadTempFile = $this->getUploadTempFile();
         $aFiles = json_decode(file_get_contents($sUploadTempFile), true);
-        foreach($aFiles as $sVarName => $aUserFiles)
+        foreach ($aFiles as $sVarName => $aUserFiles)
         {
             $this->aUserFiles[$sVarName] = [];
-            foreach($aUserFiles as $aUserFile)
+            foreach ($aUserFiles as $aUserFile)
             {
                 $this->aUserFiles[$sVarName][] = UploadedFile::fromTempData($aUserFile);
             }
@@ -378,7 +378,7 @@ 
      */
     public function noRequestPluginFound()
     {
-        if(count($_FILES) > 0)
+        if (count($_FILES) > 0)
         {
             $this->bRequestIsHttpUpload = true;
         }
@@ -401,17 +401,17 @@ 
      */
     public function processRequest()
     {
-        if(!$this->canProcessRequest())
+        if (!$this->canProcessRequest())
         {
             return false;
         }
 
-        if(count($_FILES) > 0)
+        if (count($_FILES) > 0)
         {
             // Ajax request with upload
             $this->readFromHttpData();
 
-            if($this->bRequestIsHttpUpload)
+            if ($this->bRequestIsHttpUpload)
             {
                 // Process an HTTP upload request
                 // This requires to set the response to be returned.
@@ -421,14 +421,14 @@ 
                     $this->saveToTempFile();
                     $xResponse->setUploadedFile($this->sTempFile);
                 }
-                catch(Exception $e)
+                catch (Exception $e)
                 {
                     $xResponse->setErrorMessage($e->getMessage());
                 }
                 jaxon()->di()->getResponseManager()->append($xResponse);
             }
         }
-        elseif(($this->sTempFile))
+        elseif (($this->sTempFile))
         {
             // Ajax request following and HTTP upload
             $this->readFromTempFile();

Braces +4 added lines, -8 removed lines

@@ -71,8 +71,7 @@ 
         if(array_key_exists('jxnupl', $_POST))
         {
             $this->sTempFile = $_POST['jxnupl'];
-        }
-        elseif(array_key_exists('jxnupl', $_GET))
+        } elseif(array_key_exists('jxnupl', $_GET))
         {
             $this->sTempFile = $_GET['jxnupl'];
         }
@@ -212,8 +211,7 @@ 
                         'extension' => pathinfo($aFile['name'][$i], PATHINFO_EXTENSION),
                     ];
                 }
-            }
-            else
+            } else
             {
                 if(!$aFile['name'])
                 {
@@ -420,15 +418,13 @@ 
                 {
                     $this->saveToTempFile();
                     $xResponse->setUploadedFile($this->sTempFile);
-                }
-                catch(Exception $e)
+                } catch(Exception $e)
                 {
                     $xResponse->setErrorMessage($e->getMessage());
                 }
                 jaxon()->di()->getResponseManager()->append($xResponse);
             }
-        }
-        elseif(($this->sTempFile))
+        } elseif(($this->sTempFile))
         {
             // Ajax request following and HTTP upload
             $this->readFromTempFile();

src/Response/UploadResponse.php

Doc Comments +2 added lines

@@ -41,6 +41,7 @@ 
 
     /**
      * Set the path to the uploaded file
+     * @param string $sUploadedFile
      */
     public function setUploadedFile($sUploadedFile)
     {
@@ -49,6 +50,7 @@ 
 
     /**
      * Set the error message
+     * @param string $sErrorMessage
      */
     public function setErrorMessage($sErrorMessage)
     {

Spacing +2 added lines, -3 removed lines

@@ -86,7 +86,7 @@ 
     {
         $sCharacterSet = '';
         $sCharacterEncoding = trim($this->getOption('core.encoding'));
-        if(($sCharacterEncoding) && strlen($sCharacterEncoding) > 0)
+        if (($sCharacterEncoding) && strlen($sCharacterEncoding) > 0)
         {
             $sCharacterSet = '; charset="' . trim($sCharacterEncoding) . '"';
         }
@@ -102,8 +102,7 @@ 
     public function getOutput()
     {
         $aResponse = ($this->sUploadedFile) ?
-            ['code' => 'success', 'upl' => $this->sUploadedFile] :
-            ['code' => 'error', 'msg' => $this->sErrorMessage];
+            ['code' => 'success', 'upl' => $this->sUploadedFile] : ['code' => 'error', 'msg' => $this->sErrorMessage];
         return '<script>var res = ' . json_encode($aResponse) . '; </script>';
     }
 

src/Utils/View/Manager.php

Unused Use Statements -3 removed lines

@@ -3,9 +3,6 @@
 namespace Jaxon\Utils\View;
 
 use Jaxon\Utils\Config\Config;
-
-use stdClass;
-use Exception;
 use Closure;
 
 class Manager

Spacing +8 added lines, -8 removed lines

@@ -78,7 +78,7 @@ 
             'directory' => $sDirectory,
             'extension' => $sExtension,
         ];
-        if(key_exists($sRenderer, $this->aNamespaces))
+        if (key_exists($sRenderer, $this->aNamespaces))
         {
             $this->aNamespaces[$sRenderer][] = $aNamespace;
         }
@@ -99,12 +99,12 @@ 
     public function addNamespaces($xAppConfig)
     {
         $this->sDefaultNamespace = $xAppConfig->getOption('options.views.default', false);
-        if(is_array($namespaces = $xAppConfig->getOptionNames('views')))
+        if (is_array($namespaces = $xAppConfig->getOptionNames('views')))
         {
-            foreach($namespaces as $namespace => $option)
+            foreach ($namespaces as $namespace => $option)
             {
                 // If no default namespace is defined, use the first one as default.
-                if($this->sDefaultNamespace == false)
+                if ($this->sDefaultNamespace == false)
                 {
                     $this->sDefaultNamespace = $namespace;
                 }
@@ -126,7 +126,7 @@ 
      */
     public function getRenderer($sId = '')
     {
-        if(!$sId)
+        if (!$sId)
         {
             // Return the view renderer facade
             return jaxon()->di()->get(\Jaxon\Utils\View\Renderer::class);
@@ -149,14 +149,14 @@ 
         jaxon()->di()->set('jaxon.app.view.base.' . $sId, $xClosure);
 
         // Return the initialized view renderer
-        jaxon()->di()->set('jaxon.app.view.' . $sId, function ($c) use ($sId) {
+        jaxon()->di()->set('jaxon.app.view.' . $sId, function($c) use ($sId) {
             // Get the defined renderer
             $renderer = $c['jaxon.app.view.base.' . $sId];
 
             // Init the renderer with the template namespaces
-            if(key_exists($sId, $this->aNamespaces))
+            if (key_exists($sId, $this->aNamespaces))
             {
-                foreach($this->aNamespaces[$sId] as $ns)
+                foreach ($this->aNamespaces[$sId] as $ns)
                 {
                     $renderer->addNamespace($ns['namespace'], $ns['directory'], $ns['extension']);
                 }

Braces +1 added lines, -2 removed lines

@@ -81,8 +81,7 @@
         if(key_exists($sRenderer, $this->aNamespaces))
         {
             $this->aNamespaces[$sRenderer][] = $aNamespace;
-        }
-        else
+        } else
         {
             $this->aNamespaces[$sRenderer] = [$aNamespace];
         }

src/Utils/Session/Manager.php

Spacing +1 added lines, -1 removed lines

@@ -97,7 +97,7 @@
      */
     public function delete($sKey)
     {
-        if(key_exists($sKey, $_SESSION))
+        if (key_exists($sKey, $_SESSION))
         {
             unset($_SESSION[$sKey]);
         }

src/Utils/Pagination/Renderer.php

Spacing +4 added lines, -4 removed lines

@@ -46,7 +46,7 @@ 
      */
     protected function getPrevLink($xPaginator)
     {
-        if(!($sCall = $xPaginator->getPrevCall()))
+        if (!($sCall = $xPaginator->getPrevCall()))
         {
             return $this->xRenderer->render('pagination::links/disabled', ['text' => $xPaginator->getPreviousText()]);
         }
@@ -63,7 +63,7 @@ 
      */
     protected function getNextLink($xPaginator)
     {
-        if(!($sCall = $xPaginator->getNextCall()))
+        if (!($sCall = $xPaginator->getNextCall()))
         {
             return $this->xRenderer->render('pagination::links/disabled', ['text' => $xPaginator->getNextText()]);
         }
@@ -81,9 +81,9 @@ 
     protected function getLinks($xPaginator)
     {
         $sLinks = '';
-        foreach($xPaginator->getPages() as $page)
+        foreach ($xPaginator->getPages() as $page)
         {
-            if($page['call'])
+            if ($page['call'])
             {
                 $sTemplate = ($page['isCurrent'] ? 'pagination::links/current' : 'pagination::links/enabled');
                 $sLinks .= $this->xRenderer->render($sTemplate, ['call' => $page['call'], 'text' => $page['num']]);

Braces +1 added lines, -2 removed lines

@@ -87,8 +87,7 @@
             {
                 $sTemplate = ($page['isCurrent'] ? 'pagination::links/current' : 'pagination::links/enabled');
                 $sLinks .= $this->xRenderer->render($sTemplate, ['call' => $page['call'], 'text' => $page['num']]);
-            }
-            else
+            } else
             {
                 $sLinks .= $this->xRenderer->render('pagination::links/disabled', ['text' => $page['num']]);
             }

src/Utils/DI/Container.php

Spacing +32 added lines, -32 removed lines

@@ -124,19 +124,19 @@ 
          * Core library objects
          */
         // Global Response
-        $this->libContainer[Response::class] = function () {
+        $this->libContainer[Response::class] = function() {
             return new Response();
         };
         // Dialog
-        $this->libContainer[Dialog::class] = function () {
+        $this->libContainer[Dialog::class] = function() {
             return new Dialog();
         };
         // Jaxon App
-        $this->libContainer[App::class] = function () {
+        $this->libContainer[App::class] = function() {
             return new App();
         };
         // Jaxon App bootstrap
-        $this->libContainer[Bootstrap::class] = function () {
+        $this->libContainer[Bootstrap::class] = function() {
             return new Bootstrap();
         };
 
@@ -144,27 +144,27 @@ 
          * Plugins
          */
         // Callable objects repository
-        $this->libContainer[CallableRepository::class] = function () {
+        $this->libContainer[CallableRepository::class] = function() {
             return new CallableRepository();
         };
         // Callable class plugin
-        $this->libContainer[CallableClass::class] = function ($c) {
+        $this->libContainer[CallableClass::class] = function($c) {
             return new CallableClass($c[CallableRepository::class]);
         };
         // Callable dir plugin
-        $this->libContainer[CallableDir::class] = function ($c) {
+        $this->libContainer[CallableDir::class] = function($c) {
             return new CallableDir($c[CallableRepository::class]);
         };
         // Callable function plugin
-        $this->libContainer[CallableFunction::class] = function () {
+        $this->libContainer[CallableFunction::class] = function() {
             return new CallableFunction();
         };
         // File upload plugin
-        $this->libContainer[FileUpload::class] = function () {
+        $this->libContainer[FileUpload::class] = function() {
             return new FileUpload();
         };
         // JQuery response plugin
-        $this->libContainer[JQueryPlugin::class] = function () {
+        $this->libContainer[JQueryPlugin::class] = function() {
             return new JQueryPlugin();
         };
 
@@ -172,45 +172,45 @@ 
          * Managers
          */
         // Plugin Manager
-        $this->libContainer[PluginManager::class] = function () {
+        $this->libContainer[PluginManager::class] = function() {
             return new PluginManager();
         };
         // Request Handler
-        $this->libContainer[RequestHandler::class] = function ($c) {
+        $this->libContainer[RequestHandler::class] = function($c) {
             return new RequestHandler($c[PluginManager::class], $c[ResponseManager::class], $c[FileUpload::class]);
         };
         // Request Factory
-        $this->libContainer[RequestFactory::class] = function ($c) {
+        $this->libContainer[RequestFactory::class] = function($c) {
             return new RequestFactory($c[CallableRepository::class]);
         };
         // Parameter Factory
-        $this->libContainer[ParameterFactory::class] = function () {
+        $this->libContainer[ParameterFactory::class] = function() {
             return new ParameterFactory();
         };
         // Response Manager
-        $this->libContainer[ResponseManager::class] = function () {
+        $this->libContainer[ResponseManager::class] = function() {
             return new ResponseManager();
         };
         // Code Generator
-        $this->libContainer[CodeGenerator::class] = function ($c) {
+        $this->libContainer[CodeGenerator::class] = function($c) {
             return new CodeGenerator($c[PluginManager::class], $c[TemplateEngine::class]);
         };
         // View Manager
-        $this->libContainer[ViewManager::class] = function () {
+        $this->libContainer[ViewManager::class] = function() {
             return new ViewManager();
         };
         // View Renderer
-        $this->libContainer[ViewRenderer::class] = function ($c) {
+        $this->libContainer[ViewRenderer::class] = function($c) {
             return new ViewRenderer($c[ViewManager::class]);
         };
 
         /*
          * Config
          */
-        $this->libContainer[Config::class] = function () {
+        $this->libContainer[Config::class] = function() {
             return new Config();
         };
-        $this->libContainer[ConfigReader::class] = function () {
+        $this->libContainer[ConfigReader::class] = function() {
             return new ConfigReader();
         };
 
@@ -218,35 +218,35 @@ 
          * Services
          */
         // Minifier
-        $this->libContainer[Minifier::class] = function () {
+        $this->libContainer[Minifier::class] = function() {
             return new Minifier();
         };
         // Translator
-        $this->libContainer[Translator::class] = function ($c) {
+        $this->libContainer[Translator::class] = function($c) {
             return new Translator($c['jaxon.core.translation_dir'], $c[Config::class]);
         };
         // Template engine
-        $this->libContainer[TemplateEngine::class] = function ($c) {
+        $this->libContainer[TemplateEngine::class] = function($c) {
             return new TemplateEngine($c['jaxon.core.template_dir']);
         };
         // Template Renderer
-        $this->libContainer[TemplateRenderer::class] = function ($c) {
+        $this->libContainer[TemplateRenderer::class] = function($c) {
             return $c[TemplateEngine::class];
         };
         // Validator
-        $this->libContainer[Validator::class] = function ($c) {
+        $this->libContainer[Validator::class] = function($c) {
             return new Validator($c[Translator::class], $c[Config::class]);
         };
         // Pagination Paginator
-        $this->libContainer[Paginator::class] = function ($c) {
+        $this->libContainer[Paginator::class] = function($c) {
             return new Paginator($c[PaginationRenderer::class]);
         };
         // Pagination Renderer
-        $this->libContainer[PaginationRenderer::class] = function ($c) {
+        $this->libContainer[PaginationRenderer::class] = function($c) {
             return new PaginationRenderer($c[TemplateRenderer::class]);
         };
         // Event Dispatcher
-        $this->libContainer[EventDispatcher::class] = function () {
+        $this->libContainer[EventDispatcher::class] = function() {
             return new EventDispatcher();
         };
     }
@@ -258,7 +258,7 @@ 
      */
     public function get($sClass)
     {
-        if($this->appContainer != null && $this->appContainer->has($sClass))
+        if ($this->appContainer != null && $this->appContainer->has($sClass))
         {
             return $this->appContainer->get($sClass);
         }
@@ -288,7 +288,7 @@ 
      */
     public function alias($sClass, $sAlias)
     {
-        $this->libContainer[$sClass] = function ($c) use ($sAlias) {
+        $this->libContainer[$sClass] = function($c) use ($sAlias) {
             return $c[$sAlias];
         };
     }
@@ -558,7 +558,7 @@ 
      */
     public function setCallableClassRequestFactory($sClassName, CallableObject $xCallableObject)
     {
-        $this->libContainer[$sClassName . '_RequestFactory'] = function () use ($xCallableObject) {
+        $this->libContainer[$sClassName . '_RequestFactory'] = function() use ($xCallableObject) {
             // $xCallableObject = $c[CallableRepository::class]->getCallableObject($sClassName);
             return new CallableClassRequestFactory($xCallableObject);
         };
@@ -586,7 +586,7 @@ 
      */
     public function setCallableClassPaginatorFactory($sClassName, CallableObject $xCallableObject)
     {
-        $this->libContainer[$sClassName . '_PaginatorFactory'] = function () use ($xCallableObject) {
+        $this->libContainer[$sClassName . '_PaginatorFactory'] = function() use ($xCallableObject) {
             // $xCallableObject = $c[CallableRepository::class]->getCallableObject($sClassName);
             return new CallableClassPaginatorFactory($xCallableObject);
         };

src/Utils/Config/Json.php

Spacing +2 added lines, -2 removed lines

@@ -26,13 +26,13 @@
     public static function read($sConfigFile)
     {
         $sConfigFile = realpath($sConfigFile);
-        if(!is_readable($sConfigFile))
+        if (!is_readable($sConfigFile))
         {
             throw new \Jaxon\Utils\Config\Exception\File(jaxon_trans('config.errors.file.access', ['path' => $sConfigFile]));
         }
         $sFileContent = file_get_contents($sConfigFile);
         $aConfigOptions = json_decode($sFileContent, true);
-        if(!is_array($aConfigOptions))
+        if (!is_array($aConfigOptions))
         {
             throw new \Jaxon\Utils\Config\Exception\File(jaxon_trans('config.errors.file.content', ['path' => $sConfigFile]));
         }

src/Utils/Config/Config.php

Spacing +10 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
      */
     public function __construct(array $aOptions = [], $sKeys = '')
     {
-        if(count($aOptions) > 0)
+        if (count($aOptions) > 0)
         {
             $this->setOptions($aOptions, $sKeys);
         }
@@ -66,14 +66,14 @@ 
         $sPrefix = trim((string)$sPrefix);
         $nDepth = intval($nDepth);
         // Check the max depth
-        if($nDepth < 0 || $nDepth > 9)
+        if ($nDepth < 0 || $nDepth > 9)
         {
             throw new \Jaxon\Utils\Config\Exception\Data(jaxon_trans('config.errors.data.depth',
                 ['key' => $sPrefix, 'depth' => $nDepth]));
         }
-        foreach($aOptions as $sName => $xOption)
+        foreach ($aOptions as $sName => $xOption)
         {
-            if(is_int($sName))
+            if (is_int($sName))
             {
                 continue;
             }
@@ -83,7 +83,7 @@ 
             // Save the value of this option
             $this->aOptions[$sFullName] = $xOption;
             // Save the values of its sub-options
-            if(is_array($xOption))
+            if (is_array($xOption))
             {
                 // Recursively read the options in the array
                 $this->_setOptions($xOption, $sFullName, $nDepth + 1);
@@ -105,9 +105,9 @@ 
         $aKeys = explode('.', (string)$sKeys);
         foreach ($aKeys as $sKey)
         {
-            if(($sKey))
+            if (($sKey))
             {
-                if(!array_key_exists($sKey, $aOptions) || !is_array($aOptions[$sKey]))
+                if (!array_key_exists($sKey, $aOptions) || !is_array($aOptions[$sKey]))
                 {
                     return;
                 }
@@ -157,13 +157,12 @@ 
         $sPrefix = rtrim($sPrefix, '.') . '.';
         $sPrefixLen = strlen($sPrefix);
         $aOptions = [];
-        foreach($this->aOptions as $sName => $xValue)
+        foreach ($this->aOptions as $sName => $xValue)
         {
-            if(substr($sName, 0, $sPrefixLen) == $sPrefix)
+            if (substr($sName, 0, $sPrefixLen) == $sPrefix)
             {
                 $iNextDotPos = strpos($sName, '.', $sPrefixLen);
-                $sOptionName = $iNextDotPos === false ? substr($sName, $sPrefixLen) :
-                    substr($sName, $sPrefixLen, $iNextDotPos - $sPrefixLen);
+                $sOptionName = $iNextDotPos === false ? substr($sName, $sPrefixLen) : substr($sName, $sPrefixLen, $iNextDotPos - $sPrefixLen);
                 $aOptions[$sOptionName] = $sPrefix . $sOptionName;
             }
         }

src/Utils/Config/Php.php

Spacing +2 added lines, -2 removed lines

@@ -26,12 +26,12 @@
     public static function read($sConfigFile)
     {
         $sConfigFile = realpath($sConfigFile);
-        if(!is_readable($sConfigFile))
+        if (!is_readable($sConfigFile))
         {
             throw new \Jaxon\Utils\Config\Exception\File(jaxon_trans('config.errors.file.access', ['path' => $sConfigFile]));
         }
         $aConfigOptions = include($sConfigFile);
-        if(!is_array($aConfigOptions))
+        if (!is_array($aConfigOptions))
         {
             throw new \Jaxon\Utils\Config\Exception\File(jaxon_trans('config.errors.file.content', ['path' => $sConfigFile]));
         }