jaxon-php / jaxon-core

src/Request/Handler/RequestHandler.php

Braces +1 added lines, -2 removed lines

@@ -300,8 +300,7 @@
             $this->xResponseManager->error($e->getMessage());
             $this->onInvalid($e);
             throw $e;
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             $this->xResponseManager->error($e->getMessage());
             $this->onError($e);

Spacing +22 added lines, -22 removed lines

@@ -111,14 +111,14 @@ 
     {
         $xTarget = $this->xRequestPlugin->getTarget();
         // Call the user defined callback
-        foreach($this->xCallbackManager->getBeforeCallbacks() as $xCallback)
+        foreach ($this->xCallbackManager->getBeforeCallbacks() as $xCallback)
         {
             $xReturn = call_user_func_array($xCallback, [$xTarget, &$bEndRequest]);
-            if($xReturn instanceof ResponseInterface)
+            if ($xReturn instanceof ResponseInterface)
             {
                 $this->xResponseManager->append($xReturn);
             }
-            if($bEndRequest)
+            if ($bEndRequest)
             {
                 return;
             }
@@ -133,11 +133,11 @@ 
      */
     public function onAfter(bool $bEndRequest)
     {
-        foreach($this->xCallbackManager->getAfterCallbacks() as $xCallback)
+        foreach ($this->xCallbackManager->getAfterCallbacks() as $xCallback)
         {
             $xReturn = call_user_func_array($xCallback,
                 [$this->xRequestPlugin->getTarget(), $bEndRequest]);
-            if($xReturn instanceof ResponseInterface)
+            if ($xReturn instanceof ResponseInterface)
             {
                 $this->xResponseManager->append($xReturn);
             }
@@ -154,10 +154,10 @@ 
      */
     public function onInvalid(RequestException $xException)
     {
-        foreach($this->xCallbackManager->getInvalidCallbacks() as $xCallback)
+        foreach ($this->xCallbackManager->getInvalidCallbacks() as $xCallback)
         {
             $xReturn = call_user_func($xCallback, $xException);
-            if($xReturn instanceof ResponseInterface)
+            if ($xReturn instanceof ResponseInterface)
             {
                 $this->xResponseManager->append($xReturn);
             }
@@ -174,10 +174,10 @@ 
      */
     public function onError(Exception $xException)
     {
-        foreach($this->xCallbackManager->getErrorCallbacks() as $xCallback)
+        foreach ($this->xCallbackManager->getErrorCallbacks() as $xCallback)
         {
             $xReturn = call_user_func($xCallback, $xException);
-            if($xReturn instanceof ResponseInterface)
+            if ($xReturn instanceof ResponseInterface)
             {
                 $this->xResponseManager->append($xReturn);
             }
@@ -196,7 +196,7 @@ 
         $this->xUploadHandler = $this->di->getUploadHandler();
 
         // Return true if the request plugin was already found
-        if($this->xRequestPlugin !== null)
+        if ($this->xRequestPlugin !== null)
         {
             return true;
         }
@@ -204,9 +204,9 @@ 
         // The HTTP request
         $xRequest = $this->di->getRequest();
         // Find a plugin to process the request
-        foreach($this->xPluginManager->getRequestHandlers() as $sClassName)
+        foreach ($this->xPluginManager->getRequestHandlers() as $sClassName)
         {
-            if($sClassName::canProcessRequest($xRequest))
+            if ($sClassName::canProcessRequest($xRequest))
             {
                 $this->xRequestPlugin = $this->di->g($sClassName);
                 $this->xRequestPlugin->setTarget($xRequest);
@@ -215,7 +215,7 @@ 
         }
 
         // Check if the upload plugin is enabled
-        if($this->xUploadHandler === null)
+        if ($this->xUploadHandler === null)
         {
             return false;
         }
@@ -237,15 +237,15 @@ 
         // The HTTP request
         $xRequest = $this->di->getRequest();
         // Process uploaded files, if the upload plugin is enabled
-        if($this->xUploadHandler !== null && $this->xUploadHandler->canProcessRequest($xRequest))
+        if ($this->xUploadHandler !== null && $this->xUploadHandler->canProcessRequest($xRequest))
         {
             $this->xUploadHandler->processRequest($xRequest);
         }
         // Process the request
-        if(($this->xRequestPlugin))
+        if (($this->xRequestPlugin))
         {
             $xResponse = $this->xRequestPlugin->processRequest();
-            if(($xResponse))
+            if (($xResponse))
             {
                 $this->xResponseManager->append($xResponse);
             }
@@ -266,7 +266,7 @@ 
     public function processRequest()
     {
         // Check if there is a plugin to process this request
-        if(!$this->canProcessRequest())
+        if (!$this->canProcessRequest())
         {
             return;
         }
@@ -275,11 +275,11 @@ 
         {
             $bEndRequest = false;
             // Handle before processing event
-            if(($this->xRequestPlugin))
+            if (($this->xRequestPlugin))
             {
                 $this->onBefore($bEndRequest);
             }
-            if($bEndRequest)
+            if ($bEndRequest)
             {
                 return;
             }
@@ -287,7 +287,7 @@ 
             $this->_processRequest();
 
             // Handle after processing event
-            if(($this->xRequestPlugin))
+            if (($this->xRequestPlugin))
             {
                 $this->onAfter($bEndRequest);
             }
@@ -295,13 +295,13 @@ 
         // An exception was thrown while processing the request.
         // The request missed the corresponding handler function,
         // or an error occurred while attempting to execute the handler.
-        catch(RequestException $e)
+        catch (RequestException $e)
         {
             $this->xResponseManager->error($e->getMessage());
             $this->onInvalid($e);
             throw $e;
         }
-        catch(Exception $e)
+        catch (Exception $e)
         {
             $this->xResponseManager->error($e->getMessage());
             $this->onError($e);

src/Request/Handler/ParameterReader.php

Braces +5 added lines, -10 removed lines

@@ -110,8 +110,7 @@ 
         if(isset($aServerParams['CONTENT_TYPE']))
         {
             $sContentType = $aServerParams['CONTENT_TYPE'];
-        }
-        elseif(isset($aServerParams['HTTP_CONTENT_TYPE']))
+        } elseif(isset($aServerParams['HTTP_CONTENT_TYPE']))
         {
             $sContentType = $aServerParams['HTTP_CONTENT_TYPE'];
         }
@@ -142,20 +141,17 @@ 
             $this->cUtf8Decoder = function($sStr) use($sEncoding) {
                 return iconv("UTF-8", $sEncoding . '//TRANSLIT', $sStr);
             };
-        }
-        elseif(function_exists('mb_convert_encoding'))
+        } elseif(function_exists('mb_convert_encoding'))
         {
             $this->cUtf8Decoder = function($sStr) use($sEncoding) {
                 return mb_convert_encoding($sStr, $sEncoding, "UTF-8");
             };
-        }
-        elseif($sEncoding === "ISO-8859-1")
+        } elseif($sEncoding === "ISO-8859-1")
         {
             $this->cUtf8Decoder = function($sStr) {
                 return utf8_decode($sStr);
             };
-        }
-        else
+        } else
         {
             throw new RequestException($this->xTranslator->trans('errors.request.conversion'));
         }
@@ -251,8 +247,7 @@ 
             {
                 $aParams = $aBody['jxnargs'];
             }
-        }
-        else
+        } else
         {
             $aParams = $xRequest->getQueryParams();
             if(isset($aParams['jxnargs']))

Switch Indentation +9 added lines, -9 removed lines

@@ -196,15 +196,15 @@
         $sValue = substr($sValue, 1);
         switch($cType)
         {
-        case 'S':
-            return $sValue;
-        case 'B':
-            return $this->convertStringToBool($sValue);
-        case 'N':
-            return ($sValue == floor($sValue) ? (int)$sValue : (float)$sValue);
-        case '*':
-        default:
-            return null;
+            case 'S':
+                return $sValue;
+            case 'B':
+                return $this->convertStringToBool($sValue);
+            case 'N':
+                return ($sValue == floor($sValue) ? (int)$sValue : (float)$sValue);
+            case '*':
+            default:
+                return null;
         }
     }
 

Spacing +19 added lines, -19 removed lines

@@ -107,16 +107,16 @@ 
         // Parameters are url encoded when uploading files
         $aServerParams = $this->di->getRequest()->getServerParams();
         $sContentType = '';
-        if(isset($aServerParams['CONTENT_TYPE']))
+        if (isset($aServerParams['CONTENT_TYPE']))
         {
             $sContentType = $aServerParams['CONTENT_TYPE'];
         }
-        elseif(isset($aServerParams['HTTP_CONTENT_TYPE']))
+        elseif (isset($aServerParams['HTTP_CONTENT_TYPE']))
         {
             $sContentType = $aServerParams['HTTP_CONTENT_TYPE'];
         }
         $sType = 'multipart/form-data';
-        if(strncmp($sContentType, $sType, strlen($sType)) !== 0)
+        if (strncmp($sContentType, $sType, strlen($sType)) !== 0)
         {
             $this->cParamDecoder = function($sParam) { return $sParam; };
             return;
@@ -137,19 +137,19 @@ 
             return $sStr;
         };
         $sEncoding = $this->xConfigManager->getOption('core.encoding', '');
-        if(function_exists('iconv'))
+        if (function_exists('iconv'))
         {
             $this->cUtf8Decoder = function($sStr) use($sEncoding) {
                 return iconv("UTF-8", $sEncoding . '//TRANSLIT', $sStr);
             };
         }
-        elseif(function_exists('mb_convert_encoding'))
+        elseif (function_exists('mb_convert_encoding'))
         {
             $this->cUtf8Decoder = function($sStr) use($sEncoding) {
                 return mb_convert_encoding($sStr, $sEncoding, "UTF-8");
             };
         }
-        elseif($sEncoding === "ISO-8859-1")
+        elseif ($sEncoding === "ISO-8859-1")
         {
             $this->cUtf8Decoder = function($sStr) {
                 return utf8_decode($sStr);
@@ -170,11 +170,11 @@ 
      */
     private function convertStringToBool(string $sValue): bool
     {
-        if(strcasecmp($sValue, 'true') === 0)
+        if (strcasecmp($sValue, 'true') === 0)
         {
             return true;
         }
-        if(strcasecmp($sValue, 'false') === 0)
+        if (strcasecmp($sValue, 'false') === 0)
         {
             return false;
         }
@@ -194,14 +194,14 @@ 
     {
         $cType = substr($sValue, 0, 1);
         $sValue = substr($sValue, 1);
-        switch($cType)
+        switch ($cType)
         {
         case 'S':
             return $sValue;
         case 'B':
             return $this->convertStringToBool($sValue);
         case 'N':
-            return ($sValue == floor($sValue) ? (int)$sValue : (float)$sValue);
+            return ($sValue == floor($sValue) ? (int) $sValue : (float) $sValue);
         case '*':
         default:
             return null;
@@ -217,7 +217,7 @@ 
      */
     private function decodeRequestParameter(string $sParam)
     {
-        if($sParam === '')
+        if ($sParam === '')
         {
             return $sParam;
         }
@@ -225,7 +225,7 @@ 
         $sParam = call_user_func($this->cParamDecoder, $sParam);
 
         $xJson = json_decode($sParam, true);
-        if($xJson !== null && $sParam != $xJson)
+        if ($xJson !== null && $sParam != $xJson)
         {
             return $xJson;
         }
@@ -240,9 +240,9 @@ 
         $aParams = [];
         $xRequest = $this->di->getRequest();
         $aBody = $xRequest->getParsedBody();
-        if(is_array($aBody))
+        if (is_array($aBody))
         {
-            if(isset($aBody['jxnargs']))
+            if (isset($aBody['jxnargs']))
             {
                 $aParams = $aBody['jxnargs'];
             }
@@ -250,13 +250,13 @@ 
         else
         {
             $aParams = $xRequest->getQueryParams();
-            if(isset($aParams['jxnargs']))
+            if (isset($aParams['jxnargs']))
             {
                 $aParams = $aParams['jxnargs'];
             }
         }
         return array_map(function($sParam) {
-            return $this->decodeRequestParameter((string)$sParam);
+            return $this->decodeRequestParameter((string) $sParam);
         }, $aParams);
     }
 
@@ -269,7 +269,7 @@ 
      */
     private function decodeUtf8Parameter($xValue)
     {
-        if(is_string($xValue))
+        if (is_string($xValue))
         {
             return call_user_func($this->cUtf8Decoder, $xValue);
         }
@@ -289,7 +289,7 @@ 
     private function decodeUtf8Parameters(array $aParams): array
     {
         $aValues = [];
-        foreach($aParams as $sKey => $xValue)
+        foreach ($aParams as $sKey => $xValue)
         {
             // Decode the key
             $sKey = call_user_func($this->cUtf8Decoder, $sKey);
@@ -310,7 +310,7 @@ 
     {
         $this->setParamDecoder();
         $aParams = $this->getRequestParameters();
-        if(!$this->xConfigManager->getOption('core.decode_utf8'))
+        if (!$this->xConfigManager->getOption('core.decode_utf8'))
         {
             return $aParams;
         }

src/Plugin/Request/CallableFunction/CallableFunctionPlugin.php

Braces +1 added lines, -2 removed lines

@@ -125,8 +125,7 @@
         if(is_string($xOptions))
         {
             $xOptions = ['include' => $xOptions];
-        }
-        elseif(!is_array($xOptions))
+        } elseif(!is_array($xOptions))
         {
             throw new SetupException($this->xTranslator->trans('errors.objects.invalid-declaration'));
         }

Spacing +11 added lines, -11 removed lines

@@ -128,15 +128,15 @@ 
      */
     public function checkOptions(string $sCallable, $xOptions): array
     {
-        if(!$this->xValidator->validateFunction(trim($sCallable)))
+        if (!$this->xValidator->validateFunction(trim($sCallable)))
         {
             throw new SetupException($this->xTranslator->trans('errors.objects.invalid-declaration'));
         }
-        if(is_string($xOptions))
+        if (is_string($xOptions))
         {
             $xOptions = ['include' => $xOptions];
         }
-        elseif(!is_array($xOptions))
+        elseif (!is_array($xOptions))
         {
             throw new SetupException($this->xTranslator->trans('errors.objects.invalid-declaration'));
         }
@@ -157,9 +157,9 @@ 
         $sPhpFunction = trim($sCallable);
         $sFunction = $sPhpFunction;
         // Check if an alias is defined
-        if(isset($aOptions['alias']))
+        if (isset($aOptions['alias']))
         {
-            $sFunction = (string)$aOptions['alias'];
+            $sFunction = (string) $aOptions['alias'];
             unset($aOptions['alias']);
         }
         $this->aFunctions[$sFunction] = $sPhpFunction;
@@ -181,13 +181,13 @@ 
     public function getCallable(string $sCallable)
     {
         $sFunction = trim($sCallable);
-        if(!isset($this->aFunctions[$sFunction]))
+        if (!isset($this->aFunctions[$sFunction]))
         {
             return null;
         }
         $xCallable = new CallableFunction($this->di, $sFunction,
             $this->sPrefix . $sFunction, $this->aFunctions[$sFunction]);
-        foreach($this->aOptions[$sFunction] as $sName => $sValue)
+        foreach ($this->aOptions[$sFunction] as $sName => $sValue)
         {
             $xCallable->configure($sName, $sValue);
         }
@@ -216,7 +216,7 @@ 
     public function getScript(): string
     {
         $code = '';
-        foreach(array_keys($this->aFunctions) as $sFunction)
+        foreach (array_keys($this->aFunctions) as $sFunction)
         {
             $xFunction = $this->getCallable($sFunction);
             $code .= $this->getCallableScript($xFunction);
@@ -230,7 +230,7 @@ 
     public static function canProcessRequest(ServerRequestInterface $xRequest): bool
     {
         $aBody = $xRequest->getParsedBody();
-        if(is_array($aBody))
+        if (is_array($aBody))
         {
             return isset($aBody['jxnfun']);
         }
@@ -244,7 +244,7 @@ 
     public function setTarget(ServerRequestInterface $xRequest)
     {
         $aBody = $xRequest->getParsedBody();
-        if(is_array($aBody))
+        if (is_array($aBody))
         {
             $this->xTarget = Target::makeFunction(trim($aBody['jxnfun']));
             return;
@@ -262,7 +262,7 @@ 
         $sRequestedFunction = $this->xTarget->getFunctionName();
 
         // Security check: make sure the requested function was registered.
-        if(!$this->xValidator->validateFunction($sRequestedFunction) ||
+        if (!$this->xValidator->validateFunction($sRequestedFunction) ||
             !isset($this->aFunctions[$sRequestedFunction]))
         {
             // Unable to find the requested function

src/Plugin/Request/CallableFunction/CallableFunction.php

Switch Indentation +9 added lines, -9 removed lines

@@ -114,15 +114,15 @@
     {
         switch($sName)
         {
-        case 'class': // The user function is a method in the given class
-            $this->xPhpFunction = [$sValue, $this->xPhpFunction];
-            break;
-        case 'include':
-            $this->sInclude = $sValue;
-            break;
-        default:
-            $this->aOptions[$sName] = $sValue;
-            break;
+            case 'class': // The user function is a method in the given class
+                $this->xPhpFunction = [$sValue, $this->xPhpFunction];
+                break;
+            case 'include':
+                $this->sInclude = $sValue;
+                break;
+            default:
+                $this->aOptions[$sName] = $sValue;
+                break;
         }
     }
 

Spacing +3 added lines, -3 removed lines

@@ -123,7 +123,7 @@ 
      */
     public function configure(string $sName, string $sValue)
     {
-        switch($sName)
+        switch ($sName)
         {
         case 'class': // The user function is a method in the given class
             $this->xPhpFunction = [$sValue, $this->xPhpFunction];
@@ -147,12 +147,12 @@ 
      */
     public function call(array $aArgs = [])
     {
-        if(($this->sInclude))
+        if (($this->sInclude))
         {
             require_once $this->sInclude;
         }
         // If the function is an alias for a class method, then instantiate the class
-        if(is_array($this->xPhpFunction) && is_string($this->xPhpFunction[0]))
+        if (is_array($this->xPhpFunction) && is_string($this->xPhpFunction[0]))
         {
             $sClassName = $this->xPhpFunction[0];
             $this->xPhpFunction[0] = $this->di->h($sClassName) ?

src/Plugin/Request/CallableDir/CallableDirPlugin.php

Spacing +6 added lines, -6 removed lines

@@ -80,7 +80,7 @@ 
     private function checkDirectory(string $sDirectory): string
     {
         $sDirectory = rtrim(trim($sDirectory), '/\\');
-        if(!is_dir($sDirectory))
+        if (!is_dir($sDirectory))
         {
             throw new SetupException($this->xTranslator->trans('errors.objects.invalid-declaration'));
         }
@@ -93,11 +93,11 @@ 
      */
     public function checkOptions(string $sCallable, $xOptions): array
     {
-        if(is_string($xOptions))
+        if (is_string($xOptions))
         {
             $xOptions = ['namespace' => $xOptions];
         }
-        if(!is_array($xOptions))
+        if (!is_array($xOptions))
         {
             throw new SetupException($this->xTranslator->trans('errors.objects.invalid-declaration'));
         }
@@ -105,14 +105,14 @@ 
         $xOptions['directory'] = $this->checkDirectory($sCallable);
         // Check the namespace
         $sNamespace = $xOptions['namespace'] ?? '';
-        if(!($xOptions['namespace'] = trim($sNamespace, ' \\')))
+        if (!($xOptions['namespace'] = trim($sNamespace, ' \\')))
         {
             $xOptions['namespace'] = '';
         }
 
         // Change the keys in $xOptions to have "\" as separator
         $_aOptions = [];
-        foreach($xOptions as $sName => $aOption)
+        foreach ($xOptions as $sName => $aOption)
         {
             $sName = trim(str_replace('.', '\\', $sName), ' \\');
             $_aOptions[$sName] = $aOption;
@@ -125,7 +125,7 @@ 
      */
     public function register(string $sType, string $sCallable, array $aOptions): bool
     {
-        if(($aOptions['namespace']))
+        if (($aOptions['namespace']))
         {
             $this->xRegistry->addNamespace($aOptions['namespace'], $aOptions);
             return true;

src/Plugin/Response/DataBag/DataBagPlugin.php

Spacing +4 added lines, -5 removed lines

@@ -39,7 +39,7 @@ 
      */
     public function setDataBag()
     {
-        if($this->xDataBag !== null)
+        if ($this->xDataBag !== null)
         {
             return;
         }
@@ -48,8 +48,7 @@ 
         $aBody = $xRequest->getParsedBody();
         $aParams = $xRequest->getQueryParams();
         $aData = is_array($aBody) ?
-            $this->readData($aBody['jxnbags'] ?? []) :
-            $this->readData($aParams['jxnbags'] ?? []);
+            $this->readData($aBody['jxnbags'] ?? []) : $this->readData($aParams['jxnbags'] ?? []);
         $this->xDataBag = new DataBag($aData);
     }
 
@@ -69,7 +68,7 @@ 
     private function readData($xData): array
     {
         // Todo: clean input data.
-        if(is_string($xData))
+        if (is_string($xData))
         {
             return json_decode($xData, true) ?: [];
         }
@@ -105,7 +104,7 @@ 
     public function writeCommand()
     {
         $this->setDataBag();
-        if($this->xDataBag->touched())
+        if ($this->xDataBag->touched())
         {
             $this->addCommand(['cmd' => 'bags.set'], $this->xDataBag->getAll());
         }

src/Plugin/Response/Dialog/DialogPlugin.php

Spacing +1 added lines, -1 removed lines

@@ -126,7 +126,7 @@
      */
     private function getLibraries(): array
     {
-        if($this->aLibraries === null)
+        if ($this->aLibraries === null)
         {
             $this->aLibraries = $this->xLibraryManager->getLibraries();
         }

src/Plugin/Manager/PackageManager.php

Spacing +15 added lines, -15 removed lines

@@ -85,28 +85,28 @@ 
     private function updateContainer(Config $xConfig)
     {
         $aOptions = $xConfig->getOption('container.set', []);
-        foreach($aOptions as $xKey => $xValue)
+        foreach ($aOptions as $xKey => $xValue)
         {
             // The key is the class name. It must be a string.
-            $this->di->set((string)$xKey, $xValue);
+            $this->di->set((string) $xKey, $xValue);
         }
         $aOptions = $xConfig->getOption('container.val', []);
-        foreach($aOptions as $xKey => $xValue)
+        foreach ($aOptions as $xKey => $xValue)
         {
             // The key is the class name. It must be a string.
-            $this->di->val((string)$xKey, $xValue);
+            $this->di->val((string) $xKey, $xValue);
         }
         $aOptions = $xConfig->getOption('container.auto', []);
-        foreach($aOptions as $xValue)
+        foreach ($aOptions as $xValue)
         {
             // The key is the class name. It must be a string.
-            $this->di->auto((string)$xValue);
+            $this->di->auto((string) $xValue);
         }
         $aOptions = $xConfig->getOption('container.alias', []);
-        foreach($aOptions as $xKey => $xValue)
+        foreach ($aOptions as $xKey => $xValue)
         {
             // The key is the class name. It must be a string.
-            $this->di->alias((string)$xKey, (string)$xValue);
+            $this->di->alias((string) $xKey, (string) $xValue);
         }
     }
 
@@ -121,14 +121,14 @@ 
      */
     private function registerCallables(array $aOptions, string $sCallableType)
     {
-        foreach($aOptions as $xKey => $xValue)
+        foreach ($aOptions as $xKey => $xValue)
         {
-            if(is_integer($xKey) && is_string($xValue))
+            if (is_integer($xKey) && is_string($xValue))
             {
                 // Register a function without options
                 $this->xPluginManager->registerCallable($sCallableType, $xValue);
             }
-            elseif(is_string($xKey) && (is_array($xValue) || is_string($xValue)))
+            elseif (is_string($xKey) && (is_array($xValue) || is_string($xValue)))
             {
                 // Register a function with options
                 $this->xPluginManager->registerCallable($sCallableType, $xKey, $xValue);
@@ -171,12 +171,12 @@ 
     {
         // $this->aPackages contains packages config file paths.
         $aLibOptions = $sClassName::config();
-        if(is_string($aLibOptions))
+        if (is_string($aLibOptions))
         {
             // A string is supposed to be the path to a config file.
             $aLibOptions = $this->xConfigManager->read($aLibOptions);
         }
-        elseif(!is_array($aLibOptions))
+        elseif (!is_array($aLibOptions))
         {
             // Otherwise, anything else than an array is not accepted.
             $sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
@@ -197,7 +197,7 @@ 
     public function registerPackage(string $sClassName, array $aUserOptions)
     {
         $sClassName = trim($sClassName, '\\ ');
-        if(!is_subclass_of($sClassName, Package::class))
+        if (!is_subclass_of($sClassName, Package::class))
         {
             $sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
             throw new SetupException($sMessage);
@@ -241,7 +241,7 @@ 
 
         // Register packages
         $aPackageConfig = $xAppConfig->getOption('packages', []);
-        foreach($aPackageConfig as $sClassName => $aPkgOptions)
+        foreach ($aPackageConfig as $sClassName => $aPkgOptions)
         {
             $this->registerPackage($sClassName, $aPkgOptions);
         }

Braces +2 added lines, -4 removed lines

@@ -127,8 +127,7 @@ 
             {
                 // Register a function without options
                 $this->xPluginManager->registerCallable($sCallableType, $xValue);
-            }
-            elseif(is_string($xKey) && (is_array($xValue) || is_string($xValue)))
+            } elseif(is_string($xKey) && (is_array($xValue) || is_string($xValue)))
             {
                 // Register a function with options
                 $this->xPluginManager->registerCallable($sCallableType, $xKey, $xValue);
@@ -175,8 +174,7 @@ 
         {
             // A string is supposed to be the path to a config file.
             $aLibOptions = $this->xConfigManager->read($aLibOptions);
-        }
-        elseif(!is_array($aLibOptions))
+        } elseif(!is_array($aLibOptions))
         {
             // Otherwise, anything else than an array is not accepted.
             $sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);

src/Request/Factory/Factory.php

Spacing +2 added lines, -2 removed lines

@@ -62,12 +62,12 @@
     public function request(string $sClassName = ''): ?RequestFactory
     {
         $sClassName = trim($sClassName);
-        if(!$sClassName)
+        if (!$sClassName)
         {
             // There is a single request factory for all callable functions.
             return $this->xRequestFactory->noPrefix(false);
         }
-        if($sClassName === '.')
+        if ($sClassName === '.')
         {
             // The request factory is for a js function, not a Jaxon call.
             return $this->xRequestFactory->noPrefix(true);