jaxon-php / jaxon-core

src/Script/Call/Parameter.php

Switch Indentation +22 added lines, -22 removed lines

@@ -161,28 +161,28 @@
     {
         switch($this->getType())
         {
-        case self::JS_VALUE:
-            return [
-                '_type' => 'expr',
-                'calls' => [['_type' => 'attr', '_name' => $this->getValue()]],
-            ];
-        case self::FORM_VALUES:
-            return ['_type' => 'form', '_name' => $this->getValue()];
-        case self::INPUT_VALUE:
-            return ['_type' => 'input', '_name' => $this->getValue()];
-        case self::CHECKED_VALUE:
-            return ['_type' => 'checked', '_name' => $this->getValue()];
-        case self::ELEMENT_INNERHTML:
-            return ['_type' => 'html', '_name' => $this->getValue()];
-        case self::PAGE_NUMBER:
-            return ['_type' => 'page', '_name' => ''];
-        case self::QUOTED_VALUE:
-        case self::BOOL_VALUE:
-        case self::NUMERIC_VALUE:
-        case self::JSON_VALUE:
-        default:
-            // Return the value as is.
-            return $this->getValue();
+            case self::JS_VALUE:
+                return [
+                    '_type' => 'expr',
+                    'calls' => [['_type' => 'attr', '_name' => $this->getValue()]],
+                ];
+            case self::FORM_VALUES:
+                return ['_type' => 'form', '_name' => $this->getValue()];
+            case self::INPUT_VALUE:
+                return ['_type' => 'input', '_name' => $this->getValue()];
+            case self::CHECKED_VALUE:
+                return ['_type' => 'checked', '_name' => $this->getValue()];
+            case self::ELEMENT_INNERHTML:
+                return ['_type' => 'html', '_name' => $this->getValue()];
+            case self::PAGE_NUMBER:
+                return ['_type' => 'page', '_name' => ''];
+            case self::QUOTED_VALUE:
+            case self::BOOL_VALUE:
+            case self::NUMERIC_VALUE:
+            case self::JSON_VALUE:
+            default:
+                // Return the value as is.
+                return $this->getValue();
         }
     }
 

Spacing +9 added lines, -9 removed lines

@@ -130,19 +130,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);
         }
@@ -159,7 +159,7 @@ 
      */
     public function jsonSerialize(): mixed
     {
-        switch($this->getType())
+        switch ($this->getType())
         {
         case self::JS_VALUE:
             return [
@@ -278,7 +278,7 @@ 
      */
     protected function getNumericValueScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -288,7 +288,7 @@ 
      */
     protected function getUnquotedValueScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -310,7 +310,7 @@ 
      */
     protected function getPageNumberScript(): string
     {
-        return (string)$this->xValue;
+        return (string) $this->xValue;
     }
 
     /**
@@ -321,7 +321,7 @@ 
     public function getScript(): string
     {
         $sMethodName = 'get' . $this->sType . 'Script';
-        if(!method_exists($this, $sMethodName))
+        if (!method_exists($this, $sMethodName))
         {
             return '';
         }

src/Request/Handler/CallbackManager.php

Spacing +9 added lines, -9 removed lines

@@ -98,7 +98,7 @@ 
      */
     public function popBootCallbacks(): array
     {
-        if(empty($this->aBootCallbacks))
+        if (empty($this->aBootCallbacks))
         {
             return [];
         }
@@ -117,9 +117,9 @@ 
     private function getExceptionCallbacks(Exception $xException): array
     {
         $aExceptionCallbacks = [];
-        foreach($this->aExceptionCallbacks as $sExClass => $aCallbacks)
+        foreach ($this->aExceptionCallbacks as $sExClass => $aCallbacks)
         {
-            if(is_a($xException, $sExClass))
+            if (is_a($xException, $sExClass))
             {
                 $aExceptionCallbacks = array_merge($aExceptionCallbacks, $aCallbacks);
             }
@@ -189,13 +189,13 @@ 
      */
     public function error(callable $xCallable, string $sExClass = ''): CallbackManager
     {
-        if($sExClass === '' || $sExClass === Exception::class)
+        if ($sExClass === '' || $sExClass === Exception::class)
         {
             $this->aErrorCallbacks[] = $xCallable;
             return $this;
         }
         // Callback for a given exception class
-        if(isset($this->aExceptionCallbacks[$sExClass]))
+        if (isset($this->aExceptionCallbacks[$sExClass]))
         {
             $this->aExceptionCallbacks[$sExClass][] = $xCallable;
             return $this;
@@ -236,7 +236,7 @@ 
      */
     private function executeCallbacks(array $aCallbacks, array $aParameters)
     {
-        foreach($aCallbacks as $xCallback)
+        foreach ($aCallbacks as $xCallback)
         {
             $this->executeCallback($xCallback, $aParameters);
         }
@@ -266,10 +266,10 @@ 
     public function onBefore(Target $xTarget, bool &$bEndRequest)
     {
         // Call the user defined callback
-        foreach($this->aBeforeCallbacks as $xCallback)
+        foreach ($this->aBeforeCallbacks as $xCallback)
         {
             $this->executeCallback($xCallback, [$xTarget, &$bEndRequest]);
-            if($bEndRequest)
+            if ($bEndRequest)
             {
                 return;
             }
@@ -316,7 +316,7 @@ 
     {
         $aExceptionCallbacks = $this->getExceptionCallbacks($xException);
         $this->executeCallbacks($aExceptionCallbacks, [$xException]);
-        if(count($aExceptionCallbacks) > 0)
+        if (count($aExceptionCallbacks) > 0)
         {
             // Do not throw the exception if a custom handler is defined
             return;

src/Di/Traits/AttributeTrait.php

Spacing +2 added lines, -2 removed lines

@@ -36,7 +36,7 @@ 
         $this->set('metadata_reader_null', function() {
             return new class implements CallableMetadataInterface
             {
-                public function getAttributes(ReflectionClass|string $xReflectionClass,
+                public function getAttributes(ReflectionClass | string $xReflectionClass,
                     array $aMethods = [], array $aProperties = []): array
                 {
                     return [false, [], []];
@@ -54,7 +54,7 @@ 
      */
     public function getMetadataReader(string $sReaderId): CallableMetadataInterface
     {
-        if(($sReaderId === 'attributes' || $sReaderId === 'annotations')
+        if (($sReaderId === 'attributes' || $sReaderId === 'annotations')
             && $this->h("metadata_reader_$sReaderId"))
         {
             return $this->g("metadata_reader_$sReaderId");

src/Di/Traits/PluginTrait.php

Spacing +1 added lines, -1 removed lines

@@ -157,7 +157,7 @@
     public function registerPackage(string $sClassName, Config $xPkgConfig)
     {
         // Register the user class, but only if the user didn't already.
-        if(!$this->h($sClassName))
+        if (!$this->h($sClassName))
         {
             $this->set($sClassName, function() use($sClassName) {
                 return $this->make($sClassName);

src/Di/ClassContainer.php

Indentation +2 added lines, -2 removed lines

@@ -118,7 +118,7 @@ 
      */
     public function set(string $sClass, Closure $xClosure)
     {
-       $this->xContainer->offsetSet($sClass, function() use($xClosure) {
+        $this->xContainer->offsetSet($sClass, function() use($xClosure) {
             return $xClosure($this);
         });
     }
@@ -133,7 +133,7 @@ 
      */
     public function val(string $sKey, $xValue)
     {
-       $this->xContainer->offsetSet($sKey, $xValue);
+        $this->xContainer->offsetSet($sKey, $xValue);
     }
 
     /**

Spacing +17 added lines, -17 removed lines

@@ -159,19 +159,19 @@ 
     private function getClassOptions(string $sClassName): array
     {
         // Find options for a class registered with namespace.
-        if(!isset($this->aClasses[$sClassName]))
+        if (!isset($this->aClasses[$sClassName]))
         {
             /** @var CallableRegistry */
             $xRegistry = $this->di->g(CallableRegistry::class);
             $xRegistry->setNamespaceClassOptions($sClassName);
-            if(!isset($this->aClasses[$sClassName]))
+            if (!isset($this->aClasses[$sClassName]))
             {
                 // Find options for a class registered without namespace.
                 // We then need to parse all classes to be able to find one.
                 $xRegistry->parseDirectories();
             }
         }
-        if(isset($this->aClasses[$sClassName]))
+        if (isset($this->aClasses[$sClassName]))
         {
             return $this->aClasses[$sClassName];
         }
@@ -189,7 +189,7 @@ 
     public function getCallableObjects(): array
     {
         $aCallableObjects = [];
-        foreach($this->aClasses as $sClassName => $_)
+        foreach ($this->aClasses as $sClassName => $_)
         {
             $this->registerCallableClass($sClassName);
             $aCallableObjects[$sClassName] = $this->getCallableObject($sClassName);
@@ -208,15 +208,15 @@ 
     {
         $xType = $xParameter->getType();
         // Check the parameter class first.
-        if($xType instanceof ReflectionNamedType)
+        if ($xType instanceof ReflectionNamedType)
         {
             // Check the class + the name
-            if($this->di->has($xType->getName() . ' $' . $xParameter->getName()))
+            if ($this->di->has($xType->getName() . ' $' . $xParameter->getName()))
             {
                 return $this->di->get($xType->getName() . ' $' . $xParameter->getName());
             }
             // Check the class only
-            if($this->di->has($xType->getName()))
+            if ($this->di->has($xType->getName()))
             {
                 return $this->di->get($xType->getName());
             }
@@ -236,16 +236,16 @@ 
      */
     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();
         }
@@ -312,7 +312,7 @@ 
     {
         $sCallableObject = $this->getCallableObjectKey($sClassName);
         // Prevent duplication. It's important not to use the class name here.
-        if($this->has($sCallableObject))
+        if ($this->has($sCallableObject))
         {
             return;
         }
@@ -335,7 +335,7 @@ 
             isset($aOptions['include']) && require_once($aOptions['include']);
             $this->val($this->getReflectionClassKey($sClassName), new ReflectionClass($sClassName));
         }
-        catch(ReflectionException $e)
+        catch (ReflectionException $e)
         {
             $xTranslator = $this->di->g(Translator::class);
             throw new SetupException($xTranslator->trans('errors.class.invalid', ['name' => $sClassName]));
@@ -351,7 +351,7 @@ 
         });
 
         // Register the user class, but only if the user didn't already.
-        if(!$this->has($sClassName))
+        if (!$this->has($sClassName))
         {
             $this->set($sClassName, function() use($sClassName) {
                 return $this->make($this->get($this->getReflectionClassKey($sClassName)));
@@ -359,7 +359,7 @@ 
         }
         // Initialize the user class instance
         $this->xContainer->extend($sClassName, function($xClassInstance) use($sClassName) {
-            if($xClassInstance instanceof AbstractCallable)
+            if ($xClassInstance instanceof AbstractCallable)
             {
                 $xClassInstance->_initCallable($this->di, $this->get($this->getCallableHelperKey($sClassName)));
             }
@@ -403,7 +403,7 @@ 
         // Replace all separators ('.' and '_') with antislashes, and remove the antislashes
         // at the beginning and the end of the class name.
         $sClassName = trim(str_replace('.', '\\', $sClassName), '\\');
-        if($this->bUsingUnderscore)
+        if ($this->bUsingUnderscore)
         {
             $sClassName = trim(str_replace('_', '\\', $sClassName), '\\');
         }
@@ -470,7 +470,7 @@ 
     private function registerRequestFactory(string $sClassName, string $sFactoryKey)
     {
         $this->xContainer->offsetSet($sFactoryKey, function() use($sClassName) {
-            if(!($xCallable = $this->makeCallableObject($sClassName)))
+            if (!($xCallable = $this->makeCallableObject($sClassName)))
             {
                 return null;
             }
@@ -491,7 +491,7 @@ 
     {
         $sClassName = trim($sClassName, " \t") ?: JxnCall::class;
         $sFactoryKey = $this->getRequestFactoryKey($sClassName);
-        if(!$this->has($sFactoryKey))
+        if (!$this->has($sFactoryKey))
         {
             $this->registerRequestFactory($sClassName, $sFactoryKey);
         }

Braces +1 added lines, -2 removed lines

@@ -334,8 +334,7 @@
             // Make sure the registered class exists
             isset($aOptions['include']) && require_once($aOptions['include']);
             $this->val($this->getReflectionClassKey($sClassName), new ReflectionClass($sClassName));
-        }
-        catch(ReflectionException $e)
+        } catch(ReflectionException $e)
         {
             $xTranslator = $this->di->g(Translator::class);
             throw new SetupException($xTranslator->trans('errors.class.invalid', ['name' => $sClassName]));

src/App/Cache/Cache.php

Spacing +1 added lines, -1 removed lines

@@ -31,7 +31,7 @@
     public function get(string $key, mixed $default = null): mixed
     {
         $value = $this->values[$key] ?? $default;
-        if(is_callable($value))
+        if (is_callable($value))
         {
             $value = $value();
             // Save the value returned by the callback in the cache.

src/App/Ajax/SendTrait.php

Spacing +5 added lines, -5 removed lines

@@ -33,24 +33,24 @@ 
      */
     public function sendResponse()
     {
-        if(!$this->xConfigManager->getOption('core.response.send', false))
+        if (!$this->xConfigManager->getOption('core.response.send', false))
         {
             return;
         }
 
         // Check to see if headers have already been sent out, in which case we can't do our job
-        if(headers_sent($sFilename, $nLineNumber))
+        if (headers_sent($sFilename, $nLineNumber))
         {
             throw new RequestException($this->xTranslator->trans('errors.output.already-sent',
                     ['location' => $sFilename . ':' . $nLineNumber]) . "\n" .
                 $this->xTranslator->trans('errors.output.advice'));
         }
-        if(empty($sContent = $this->xResponseManager->getOutput()))
+        if (empty($sContent = $this->xResponseManager->getOutput()))
         {
             return;
         }
 
-        if($this->di()->getRequest()->getMethod() === 'GET')
+        if ($this->di()->getRequest()->getMethod() === 'GET')
         {
             header("Expires: Mon, 26 Jul 1997 05:00:00 GMT");
             header("Last-Modified: " . gmdate("D, d M Y H:i:s") . " GMT");
@@ -61,7 +61,7 @@ 
 
         print $sContent;
 
-        if($this->xConfigManager->getOption('core.process.exit', false))
+        if ($this->xConfigManager->getOption('core.process.exit', false))
         {
             exit();
         }

src/App/Ajax/Bootstrap.php

Spacing +3 added lines, -3 removed lines

@@ -117,11 +117,11 @@ 
         // Jaxon library settings
         $this->xConfigManager->setOption('js.app.export', $bExport);
         $this->xConfigManager->setOption('js.app.minify', $bMinify);
-        if($sUri !== '')
+        if ($sUri !== '')
         {
             $this->xConfigManager->setOption('js.app.uri', $sUri);
         }
-        if($sDir !== '')
+        if ($sDir !== '')
         {
             $this->xConfigManager->setOption('js.app.dir', $sDir);
         }
@@ -173,7 +173,7 @@ 
     {
         // Only call the callbacks that aren't called yet.
         $aBootCallbacks = $this->xCallbackManager->popBootCallbacks();
-        foreach($aBootCallbacks as $aBootCallback)
+        foreach ($aBootCallbacks as $aBootCallback)
         {
             call_user_func($aBootCallback);
         }

src/App/Ajax/App.php

Spacing +3 added lines, -3 removed lines

@@ -32,7 +32,7 @@ 
      */
     public function setup(string $sConfigFile = '')
     {
-        if(!file_exists($sConfigFile))
+        if (!file_exists($sConfigFile))
         {
             $sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
             throw new SetupException($sMessage);
@@ -42,13 +42,13 @@ 
         $aOptions = $this->xConfigManager->read($sConfigFile);
         $aLibOptions = $aOptions['lib'] ?? [];
         $aAppOptions = $aOptions['app'] ?? [];
-        if(!is_array($aLibOptions) || !is_array($aAppOptions))
+        if (!is_array($aLibOptions) || !is_array($aAppOptions))
         {
             $sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
             throw new SetupException($sMessage);
         }
         // The bootstrap set this to false. It needs to be changed.
-        if(!isset($aLibOptions['core']['response']['send']))
+        if (!isset($aLibOptions['core']['response']['send']))
         {
             $aLibOptions['core']['response']['send'] = true;
         }