jaxon-php / jaxon-core

src/Di/Container.php

Spacing +7 added lines, -7 removed lines

@@ -141,7 +141,7 @@ 
      */
     public function has(string $sClass): bool
     {
-        if($this->xContainer != null && $this->xContainer->has($sClass))
+        if ($this->xContainer != null && $this->xContainer->has($sClass))
         {
             return true;
         }
@@ -172,13 +172,13 @@ 
     {
         try
         {
-            if($this->xContainer != null && $this->xContainer->has($sClass))
+            if ($this->xContainer != null && $this->xContainer->has($sClass))
             {
                 return $this->xContainer->get($sClass);
             }
             return $this->offsetGet($sClass);
         }
-        catch(Exception|Throwable $e)
+        catch (Exception | Throwable $e)
         {
             $xTranslator = $this->g(Translator::class);
             $sMessage = $xTranslator->trans('errors.class.container', ['name' => $sClass]);
@@ -238,22 +238,22 @@ 
      */
     public function make($xClass)
     {
-        if(is_string($xClass))
+        if (is_string($xClass))
         {
             $xClass = new ReflectionClass($xClass); // Create the reflection class instance
         }
-        if(!($xClass instanceof ReflectionClass))
+        if (!($xClass instanceof ReflectionClass))
         {
             return null;
         }
         // Use the Reflection class to get the parameters of the constructor
-        if(($constructor = $xClass->getConstructor()) === null)
+        if (($constructor = $xClass->getConstructor()) === null)
         {
             return $xClass->newInstance();
         }
         $parameters = $constructor->getParameters();
         $parameterInstances = [];
-        foreach($parameters as $parameter)
+        foreach ($parameters as $parameter)
         {
             // Get the parameter instance from the DI
             $parameterInstances[] = $this->get($parameter->getClass()->getName());

Braces +1 added lines, -2 removed lines

@@ -177,8 +177,7 @@
                 return $this->xContainer->get($sClass);
             }
             return $this->offsetGet($sClass);
-        }
-        catch(Exception|Throwable $e)
+        } catch(Exception|Throwable $e)
         {
             $xTranslator = $this->g(Translator::class);
             $sMessage = $xTranslator->trans('errors.class.container', ['name' => $sClass]);

src/Di/Traits/RegisterTrait.php

Braces +1 added lines, -2 removed lines

@@ -118,8 +118,7 @@
         try
         {
             $this->val($sReflectionClass, new ReflectionClass($sClassName));
-        }
-        catch(ReflectionException $e)
+        } catch(ReflectionException $e)
         {
             $xTranslator = $this->g(Translator::class);
             $sMessage = $xTranslator->trans('errors.class.invalid', ['name' => $sClassName]);

Spacing +11 added lines, -11 removed lines

@@ -36,15 +36,15 @@ 
     private function getCallableObjectOptions(array $aConfigOptions, array $aAnnotationOptions): array
     {
         $aOptions = [];
-        foreach($aConfigOptions as $sNames => $aFunctionOptions)
+        foreach ($aConfigOptions as $sNames => $aFunctionOptions)
         {
             $aFunctionNames = explode(',', $sNames); // Names are in comma-separated list.
-            foreach($aFunctionNames as $sFunctionName)
+            foreach ($aFunctionNames as $sFunctionName)
             {
                 $aOptions[$sFunctionName] = array_merge($aOptions[$sFunctionName] ?? [], $aFunctionOptions);
             }
         }
-        foreach($aAnnotationOptions as $sFunctionName => $aFunctionOptions)
+        foreach ($aAnnotationOptions as $sFunctionName => $aFunctionOptions)
         {
             $aOptions[$sFunctionName] = array_merge($aOptions[$sFunctionName] ?? [], $aFunctionOptions);
         }
@@ -66,7 +66,7 @@ 
         $aMethods = $xCallableObject->getPublicMethods($aProtectedMethods);
         $aProperties = $xCallableObject->getProperties();
         [$bExcluded, $aAnnotationOptions, $aAnnotationProtected] = $xAnnotationReader->getAttributes($sClassName, $aMethods, $aProperties);
-        if($bExcluded)
+        if ($bExcluded)
         {
             $xCallableObject->configure('excluded', true);
             return;
@@ -78,11 +78,11 @@ 
         // Functions options
         $aCallableOptions = [];
         $aOptions = $this->getCallableObjectOptions($aOptions['functions'], $aAnnotationOptions);
-        foreach($aOptions as $sFunctionName => $aFunctionOptions)
+        foreach ($aOptions as $sFunctionName => $aFunctionOptions)
         {
-            foreach($aFunctionOptions as $sOptionName => $xOptionValue)
+            foreach ($aFunctionOptions as $sOptionName => $xOptionValue)
             {
-                if(substr($sOptionName, 0, 2) !== '__')
+                if (substr($sOptionName, 0, 2) !== '__')
                 {
                     // Options for javascript code.
                     $aCallableOptions[$sFunctionName][$sOptionName] = $xOptionValue;
@@ -111,7 +111,7 @@ 
         $sReflectionClass = $sClassName . '_ReflectionClass';
 
         // Make sure the registered class exists
-        if(isset($aOptions['include']))
+        if (isset($aOptions['include']))
         {
             require_once($aOptions['include']);
         }
@@ -120,7 +120,7 @@ 
         {
             $this->val($sReflectionClass, new ReflectionClass($sClassName));
         }
-        catch(ReflectionException $e)
+        catch (ReflectionException $e)
         {
             $xTranslator = $this->g(Translator::class);
             $sMessage = $xTranslator->trans('errors.class.invalid', ['name' => $sClassName]);
@@ -146,7 +146,7 @@ 
         $this->set($sClassName, function($c) use($sClassName, $sReflectionClass) {
             $xRegisteredObject = $this->make($c->g($sReflectionClass));
             // Initialize the object
-            if($xRegisteredObject instanceof CallableClass)
+            if ($xRegisteredObject instanceof CallableClass)
             {
                 // Set the protected attributes of the object
                 $cSetter = function($c, $sClassName) {
@@ -159,7 +159,7 @@ 
 
             // Run the callback for class initialisation
             $aCallbacks = $c->g(CallbackManager::class)->getInitCallbacks();
-            foreach($aCallbacks as $xCallback)
+            foreach ($aCallbacks as $xCallback)
             {
                 call_user_func($xCallback, $xRegisteredObject);
             }

src/Request/Call/JsCall.php

Spacing +2 added lines, -2 removed lines

@@ -105,9 +105,9 @@
      */
     public function addParameters(array $aParameters): JsCall
     {
-        foreach($aParameters as $xParameter)
+        foreach ($aParameters as $xParameter)
         {
-            if($xParameter instanceof JsCall)
+            if ($xParameter instanceof JsCall)
             {
                 $this->addParameter(Parameter::JS_VALUE, 'function(){' . $xParameter->getScript() . ';}');
             }

Braces +1 added lines, -2 removed lines

@@ -159,8 +159,7 @@
             if($xParameter instanceof JsCall)
             {
                 $this->addParameter(Parameter::JS_VALUE, 'function(){' . $xParameter->getScript() . ';}');
-            }
-            else
+            } else
             {
                 $this->pushParameter(Parameter::make($xParameter));
             }

src/Request/Call/Call.php

Spacing +16 added lines, -17 removed lines

@@ -79,10 +79,10 @@ 
      */
     private function _makeUniqueJsVar(ParameterInterface $xParameter): ParameterInterface
     {
-        if($xParameter instanceof DomSelector)
+        if ($xParameter instanceof DomSelector)
         {
             $sParameterStr = $xParameter->getScript();
-            if(!isset($this->aVariables[$sParameterStr]))
+            if (!isset($this->aVariables[$sParameterStr]))
             {
                 // The value is not yet defined. A new variable is created.
                 $sVarName = 'jxnVar' . $this->nVarId;
@@ -109,18 +109,18 @@ 
      */
     private function makePhrase(array $aArgs): string
     {
-        if(empty($aArgs))
+        if (empty($aArgs))
         {
             return '';
         }
         // The first array entry is the message.
         $sPhrase = array_shift($aArgs);
-        if(empty($aArgs))
+        if (empty($aArgs))
         {
             return $sPhrase;
         }
         $nParamId = 1;
-        foreach($aArgs as &$xParameter)
+        foreach ($aArgs as &$xParameter)
         {
             $xParameter = $this->_makeUniqueJsVar($xParameter);
             $xParameter = "'$nParamId':" . $xParameter->getScript();
@@ -137,7 +137,7 @@ 
      */
     private function makeMessage(): string
     {
-        if(!($sPhrase = $this->makePhrase($this->aMessageArgs)))
+        if (!($sPhrase = $this->makePhrase($this->aMessageArgs)))
         {
             return '';
         }
@@ -166,23 +166,22 @@ 
         // This array will avoid declaring multiple variables with the same value.
         // The array key is the variable value, while the array value is the variable name.
         $this->aVariables = []; // Array of local variables.
-        foreach($this->aParameters as &$xParameter)
+        foreach ($this->aParameters as &$xParameter)
         {
             $xParameter = $this->_makeUniqueJsVar($xParameter);
         }
 
         $sMessageScript = $this->makeMessage();
         $sScript = parent::getScript();
-        if($this->sCondition === '__confirm__')
+        if ($this->sCondition === '__confirm__')
         {
             $sConfirmPhrase = $this->makePhrase($this->aConfirmArgs);
             $sScript = $this->xDialogLibraryManager->getQuestionLibrary()
                 ->confirm($sConfirmPhrase, $sScript, $sMessageScript);
         }
-        elseif($this->sCondition !== '')
+        elseif ($this->sCondition !== '')
         {
-            $sScript = empty($sMessageScript) ? 'if(' . $this->sCondition . '){' . $sScript . ';}' :
-                'if(' . $this->sCondition . '){' . $sScript . ';}else{' . $sMessageScript . ';}';
+            $sScript = empty($sMessageScript) ? 'if(' . $this->sCondition . '){' . $sScript . ';}' : 'if(' . $this->sCondition . '){' . $sScript . ';}else{' . $sMessageScript . ';}';
         }
         return $this->sVars . $sScript;
     }
@@ -194,9 +193,9 @@ 
      */
     public function hasPageNumber(): bool
     {
-        foreach($this->aParameters as $xParameter)
+        foreach ($this->aParameters as $xParameter)
         {
-            if($xParameter->getType() === Parameter::PAGE_NUMBER)
+            if ($xParameter->getType() === Parameter::PAGE_NUMBER)
             {
                 return true;
             }
@@ -214,9 +213,9 @@ 
     public function setPageNumber(int $nPageNumber): Call
     {
         // Set the value of the Parameter::PAGE_NUMBER parameter
-        foreach($this->aParameters as $xParameter)
+        foreach ($this->aParameters as $xParameter)
         {
-            if($xParameter->getType() === Parameter::PAGE_NUMBER)
+            if ($xParameter->getType() === Parameter::PAGE_NUMBER)
             {
                 $xParameter->setValue($nPageNumber);
                 break;
@@ -237,7 +236,7 @@ 
     public function pages(int $nCurrentPage, int $nItemsPerPage, int $nItemsTotal): array
     {
         // Append the page number to the parameter list, if not yet given.
-        if(!$this->hasPageNumber())
+        if (!$this->hasPageNumber())
         {
             $this->addParameter(Parameter::PAGE_NUMBER, 0);
         }
@@ -256,7 +255,7 @@ 
     public function paginate(int $nCurrentPage, int $nItemsPerPage, int $nItemsTotal): Paginator
     {
         // Append the page number to the parameter list, if not yet given.
-        if(!$this->hasPageNumber())
+        if (!$this->hasPageNumber())
         {
             $this->addParameter(Parameter::PAGE_NUMBER, 0);
         }

Braces +2 added lines, -4 removed lines

@@ -89,8 +89,7 @@ 
                 $this->aVariables[$sParameterStr] = $sVarName;
                 $this->sVars .= "$sVarName=$xParameter;";
                 $this->nVarId++;
-            }
-            else
+            } else
             {
                 // The value is already defined. The corresponding variable is assigned.
                 $sVarName = $this->aVariables[$sParameterStr];
@@ -178,8 +177,7 @@ 
             $sConfirmPhrase = $this->makePhrase($this->aConfirmArgs);
             $sScript = $this->xDialogLibraryManager->getQuestionLibrary()
                 ->confirm($sConfirmPhrase, $sScript, $sMessageScript);
-        }
-        elseif($this->sCondition !== '')
+        } elseif($this->sCondition !== '')
         {
             $sScript = empty($sMessageScript) ? 'if(' . $this->sCondition . '){' . $sScript . ';}' :
                 'if(' . $this->sCondition . '){' . $sScript . ';}else{' . $sMessageScript . ';}';

src/Request/Call/Paginator.php

Spacing +15 added lines, -15 removed lines

@@ -145,7 +145,7 @@ 
      */
     protected function updateTotalPages(): Paginator
     {
-        $this->nTotalPages = ($this->nItemsPerPage === 0 ? 0 : (int)ceil($this->nTotalItems / $this->nItemsPerPage));
+        $this->nTotalPages = ($this->nItemsPerPage === 0 ? 0 : (int) ceil($this->nTotalItems / $this->nItemsPerPage));
         return $this;
     }
 
@@ -220,7 +220,7 @@ 
      */
     protected function getPrevLink(): array
     {
-        if($this->nCurrentPage <= 1)
+        if ($this->nCurrentPage <= 1)
         {
             return ['disabled', $this->sPreviousText, ''];
         }
@@ -234,7 +234,7 @@ 
      */
     protected function getNextLink(): array
     {
-        if($this->nCurrentPage >= $this->nTotalPages)
+        if ($this->nCurrentPage >= $this->nTotalPages)
         {
             return ['disabled', $this->sNextText, ''];
         }
@@ -250,7 +250,7 @@ 
      */
     protected function getPageLink(int $nNumber): array
     {
-        if($nNumber < 1)
+        if ($nNumber < 1)
         {
             return ['disabled', $this->sEllipsysText, ''];
         }
@@ -269,9 +269,9 @@ 
     {
         $aPageNumbers = [];
 
-        if($this->nTotalPages <= $this->nMaxPages)
+        if ($this->nTotalPages <= $this->nMaxPages)
         {
-            for($i = 0; $i < $this->nTotalPages; $i++)
+            for ($i = 0; $i < $this->nTotalPages; $i++)
             {
                 $aPageNumbers[] = $i + 1;
             }
@@ -280,11 +280,11 @@ 
         }
 
         // Determine the sliding range, centered around the current page.
-        $nNumAdjacents = (int)floor(($this->nMaxPages - 4) / 2);
+        $nNumAdjacents = (int) floor(($this->nMaxPages - 4) / 2);
 
         $nSlidingStart = 1;
         $nSlidingEndOffset = $nNumAdjacents + 3 - $this->nCurrentPage;
-        if($nSlidingEndOffset < 0)
+        if ($nSlidingEndOffset < 0)
         {
             $nSlidingStart = $this->nCurrentPage - $nNumAdjacents;
             $nSlidingEndOffset = 0;
@@ -292,23 +292,23 @@ 
 
         $nSlidingEnd = $this->nTotalPages;
         $nSlidingStartOffset = $this->nCurrentPage + $nNumAdjacents + 2 - $this->nTotalPages;
-        if($nSlidingStartOffset < 0)
+        if ($nSlidingStartOffset < 0)
         {
             $nSlidingEnd = $this->nCurrentPage + $nNumAdjacents;
             $nSlidingStartOffset = 0;
         }
 
         // Build the list of page numbers.
-        if($nSlidingStart > 1)
+        if ($nSlidingStart > 1)
         {
             $aPageNumbers[] = 1;
             $aPageNumbers[] = 0; // Ellipsys;
         }
-        for($i = $nSlidingStart - $nSlidingStartOffset; $i <= $nSlidingEnd + $nSlidingEndOffset; $i++)
+        for ($i = $nSlidingStart - $nSlidingStartOffset; $i <= $nSlidingEnd + $nSlidingEndOffset; $i++)
         {
             $aPageNumbers[] = $i;
         }
-        if($nSlidingEnd < $this->nTotalPages)
+        if ($nSlidingEnd < $this->nTotalPages)
         {
             $aPageNumbers[] = 0; // Ellipsys;
             $aPageNumbers[] = $this->nTotalPages;
@@ -341,13 +341,13 @@ 
      */
     public function pages(): array
     {
-        if($this->nTotalPages < 2)
+        if ($this->nTotalPages < 2)
         {
             return [];
         }
 
         return array_map(function($aPage) {
-            return (object)['type' => $aPage[0], 'text' => $aPage[1], 'call' => $aPage[2]];
+            return (object) ['type' => $aPage[0], 'text' => $aPage[1], 'call' => $aPage[2]];
         }, $this->links());
     }
 
@@ -375,7 +375,7 @@ 
      */
     public function __toString()
     {
-        if($this->nTotalPages < 2)
+        if ($this->nTotalPages < 2)
         {
             return '';
         }

src/Request/Call/Parameter.php

Spacing +8 added lines, -8 removed lines

@@ -116,19 +116,19 @@ 
      */
     public static function make($xValue): ParameterInterface
     {
-        if($xValue instanceof ParameterInterface)
+        if ($xValue instanceof ParameterInterface)
         {
             return $xValue;
         }
-        if(is_numeric($xValue))
+        if (is_numeric($xValue))
         {
             return new Parameter(self::NUMERIC_VALUE, $xValue);
         }
-        if(is_string($xValue))
+        if (is_string($xValue))
         {
             return new Parameter(self::QUOTED_VALUE, $xValue);
         }
-        if(is_bool($xValue))
+        if (is_bool($xValue))
         {
             return new Parameter(self::BOOL_VALUE, $xValue);
         }
@@ -230,7 +230,7 @@ 
      */
     protected function getNumericValueScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -240,7 +240,7 @@ 
      */
     protected function getUnquotedValueScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -262,7 +262,7 @@ 
      */
     protected function getPageNumberScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -273,7 +273,7 @@ 
     public function getScript(): string
     {
         $sMethodName = 'get' . $this->sType . 'Script';
-        if(!method_exists($this, $sMethodName))
+        if (!method_exists($this, $sMethodName))
         {
             return '';
         }

src/Request/Handler/Psr/PsrRequestHandler.php

Spacing +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
         // and the other classes will get this request from there.
         $this->di->val(ServerRequestInterface::class, $request);
 
-        if(!$this->xRequestHandler->canProcessRequest())
+        if (!$this->xRequestHandler->canProcessRequest())
         {
             // Unable to find a plugin to process the request
             throw new RequestException($this->xTranslator->trans('errors.request.plugin'));

src/Request/Handler/Psr/PsrAjaxMiddleware.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
         // and the other classes will get this request from there.
         $this->di->val(ServerRequestInterface::class, $request);
 
-        if(!$this->xRequestHandler->canProcessRequest())
+        if (!$this->xRequestHandler->canProcessRequest())
         {
             // Unable to find a plugin to process the request
             return $handler->handle($request);

src/Request/Handler/CallbackManager.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
      */
     public function popBootCallbacks(): array
     {
-        if(empty($this->aBootCallbacks))
+        if (empty($this->aBootCallbacks))
         {
             return [];
         }