jaxon-php / jaxon-mono

jaxon-attributes/src/AttributeReader.php

Braces +1 added lines, -2 removed lines

@@ -258,8 +258,7 @@
             }
 
             return $this->xMetadata;
-        }
-        catch(Exception|Error $e)
+        } catch(Exception|Error $e)
         {
             throw new SetupException($e->getMessage());
         }

jaxon-core/src/Response/Manager/ResponseManager.php

Braces +1 added lines, -2 removed lines

@@ -214,8 +214,7 @@
         if($this->bOnConfirm)
         {
             $this->aConfirmCommands[] = $xCommand;
-        }
-        else
+        } else
         {
             $this->aCommands[] = $xCommand;
         }

jaxon-core/src/Plugin/Manager/PackageManager.php

Braces +1 added lines, -2 removed lines

@@ -218,8 +218,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]);

Switch Indentation +179 added lines, -179 removed lines

@@ -34,7 +34,7 @@ 
 
 class PackageManager
 {
-    /**
+/**
      * The constructor
      *
      * @param Container $di
@@ -46,48 +46,48 @@ 
      * @param CallbackManager $xCallbackManager
      * @param ComponentRegistry $xRegistry
      */
-    public function __construct(private Container $di, private Translator $xTranslator,
-        private PluginManager $xPluginManager, private ConfigManager $xConfigManager,
-        private CodeGenerator $xCodeGenerator, private ViewRenderer $xViewRenderer,
-        private CallbackManager $xCallbackManager, private ComponentRegistry $xRegistry)
-    {}
+public function __construct(private Container $di, private Translator $xTranslator,
+private PluginManager $xPluginManager, private ConfigManager $xConfigManager,
+private CodeGenerator $xCodeGenerator, private ViewRenderer $xViewRenderer,
+private CallbackManager $xCallbackManager, private ComponentRegistry $xRegistry)
+{}
 
-    /**
+/**
      * Save items in the DI container
      *
      * @param Config $xConfig
      *
      * @return void
      */
-    private function updateContainer(Config $xConfig): void
-    {
-        $aOptions = $xConfig->getOption('container.set', []);
-        foreach($aOptions as $xKey => $xValue)
-        {
-            // The key is the class name. It must be a string.
-            $this->di->set((string)$xKey, $xValue);
-        }
-        $aOptions = $xConfig->getOption('container.val', []);
-        foreach($aOptions as $xKey => $xValue)
-        {
-            // The key is the class name. It must be a string.
-            $this->di->val((string)$xKey, $xValue);
-        }
-        $aOptions = $xConfig->getOption('container.auto', []);
-        foreach($aOptions as $xValue)
-        {
-            // The key is the class name. It must be a string.
-            $this->di->auto((string)$xValue);
-        }
-        $aOptions = $xConfig->getOption('container.alias', []);
-        foreach($aOptions as $xKey => $xValue)
-        {
-            // The key is the class name. It must be a string.
-            $this->di->alias((string)$xKey, (string)$xValue);
-        }
-    }
+private function updateContainer(Config $xConfig): void
+{
+$aOptions = $xConfig->getOption('container.set', []);
+foreach($aOptions as $xKey => $xValue)
+{
+// The key is the class name. It must be a string.
+$this->di->set((string)$xKey, $xValue);
+}
+$aOptions = $xConfig->getOption('container.val', []);
+foreach($aOptions as $xKey => $xValue)
+{
+// The key is the class name. It must be a string.
+$this->di->val((string)$xKey, $xValue);
+}
+$aOptions = $xConfig->getOption('container.auto', []);
+foreach($aOptions as $xValue)
+{
+// The key is the class name. It must be a string.
+$this->di->auto((string)$xValue);
+}
+$aOptions = $xConfig->getOption('container.alias', []);
+foreach($aOptions as $xKey => $xValue)
+{
+// The key is the class name. It must be a string.
+$this->di->alias((string)$xKey, (string)$xValue);
+}
+}
 
-    /**
+/**
      * Register callables from a section of the config
      *
      * @param array $aOptions    The content of the config section
@@ -96,41 +96,41 @@ 
      * @return void
      * @throws SetupException
      */
-    private function registerCallables(array $aOptions, string $sCallableType): void
-    {
-        foreach($aOptions as $xKey => $xValue)
-        {
-            if(is_integer($xKey) && is_string($xValue))
-            {
-                // Register a function without options
-                $this->xPluginManager->registerCallable($sCallableType, $xValue);
-                continue;
-            }
+private function registerCallables(array $aOptions, string $sCallableType): void
+{
+foreach($aOptions as $xKey => $xValue)
+{
+if(is_integer($xKey) && is_string($xValue))
+{
+    // Register a function without options
+    $this->xPluginManager->registerCallable($sCallableType, $xValue);
+    continue;
+}
 
-            if(is_string($xKey) && (is_array($xValue) || is_string($xValue)))
-            {
-                // Register a function with options
-                $this->xPluginManager->registerCallable($sCallableType, $xKey, $xValue);
-            }
-        }
-    }
+if(is_string($xKey) && (is_array($xValue) || is_string($xValue)))
+{
+    // Register a function with options
+    $this->xPluginManager->registerCallable($sCallableType, $xKey, $xValue);
+}
+}
+}
 
-    /**
+/**
      * Register exceptions handlers
      *
      * @param Config $xConfig
      *
      * @return void
      */
-    private function registerExceptionHandlers(Config $xConfig): void
-    {
-        foreach($xConfig->getOption('exceptions', []) as $sExClass => $xExHandler)
-        {
-            $this->xCallbackManager->error($xExHandler, is_string($sExClass) ? $sExClass : '');
-        }
-    }
+private function registerExceptionHandlers(Config $xConfig): void
+{
+foreach($xConfig->getOption('exceptions', []) as $sExClass => $xExHandler)
+{
+$this->xCallbackManager->error($xExHandler, is_string($sExClass) ? $sExClass : '');
+}
+}
 
-    /**
+/**
      * Get a callable list from config
      *
      * @param Config $xConfig
@@ -140,36 +140,36 @@ 
      *
      * @return void
      */
-    private function registerCallablesFromConfig(Config $xConfig,
-        string $sOptionName, string $sOptionKey, string $sCallableType): void
-    {
-        // The callable (directory path, class or function name) can be used as the
-        // key of the array item, a string as the value of an entry without a key,
-        // or set with the key $sOptionKey in an array entry without a key.
-        $aCallables = [];
-        foreach($xConfig->getOption($sOptionName, []) as $xKey => $xValue)
-        {
-            if(is_string($xKey))
-            {
-                $aCallables[$xKey] = $xValue;
-                continue;
-            }
-            if(is_string($xValue))
-            {
-                $aCallables[] = $xValue;
-                continue;
-            }
+private function registerCallablesFromConfig(Config $xConfig,
+string $sOptionName, string $sOptionKey, string $sCallableType): void
+{
+// The callable (directory path, class or function name) can be used as the
+// key of the array item, a string as the value of an entry without a key,
+// or set with the key $sOptionKey in an array entry without a key.
+$aCallables = [];
+foreach($xConfig->getOption($sOptionName, []) as $xKey => $xValue)
+{
+if(is_string($xKey))
+{
+    $aCallables[$xKey] = $xValue;
+    continue;
+}
+if(is_string($xValue))
+{
+    $aCallables[] = $xValue;
+    continue;
+}
 
-            if(is_array($xValue) && isset($xValue[$sOptionKey]))
-            {
-                $aCallables[$xValue[$sOptionKey]] = $xValue;
-            }
-            // Invalid values are ignored.
-        }
-        $this->registerCallables($aCallables, $sCallableType);
-    }
+if(is_array($xValue) && isset($xValue[$sOptionKey]))
+{
+    $aCallables[$xValue[$sOptionKey]] = $xValue;
+}
+// Invalid values are ignored.
+}
+$this->registerCallables($aCallables, $sCallableType);
+}
 
-    /**
+/**
      * Read and set Jaxon options from a JSON config file
      *
      * @param Config $xConfig The config options
@@ -177,33 +177,33 @@ 
      * @return void
      * @throws SetupException
      */
-    private function registerItemsFromConfig(Config $xConfig): void
-    {
-        // Set the config for the registered callables.
-        $this->xRegistry->setPackageConfig($xConfig);
+private function registerItemsFromConfig(Config $xConfig): void
+{
+// Set the config for the registered callables.
+$this->xRegistry->setPackageConfig($xConfig);
 
-        // Register functions, classes and directories
-        $this->registerCallablesFromConfig($xConfig,
-            'functions', 'name', Jaxon::CALLABLE_FUNCTION);
-        $this->registerCallablesFromConfig($xConfig,
-            'classes', 'name', Jaxon::CALLABLE_CLASS);
-        $this->registerCallablesFromConfig($xConfig,
-            'directories', 'path', Jaxon::CALLABLE_DIR);
+// Register functions, classes and directories
+$this->registerCallablesFromConfig($xConfig,
+'functions', 'name', Jaxon::CALLABLE_FUNCTION);
+$this->registerCallablesFromConfig($xConfig,
+'classes', 'name', Jaxon::CALLABLE_CLASS);
+$this->registerCallablesFromConfig($xConfig,
+'directories', 'path', Jaxon::CALLABLE_DIR);
 
-        // Unset the current config.
-        $this->xRegistry->unsetPackageConfig();
+// Unset the current config.
+$this->xRegistry->unsetPackageConfig();
 
-        // Register the view namespaces
-        // Note: the $xUserConfig can provide a "template" option, which is used to customize
-        // the user defined view namespaces. That's why it is needed here.
-        $this->xViewRenderer->addNamespaces($xConfig);
-        // Save items in the DI container
-        $this->updateContainer($xConfig);
-        // Register the exception handlers
-        $this->registerExceptionHandlers($xConfig);
-    }
+// Register the view namespaces
+// Note: the $xUserConfig can provide a "template" option, which is used to customize
+// the user defined view namespaces. That's why it is needed here.
+$this->xViewRenderer->addNamespaces($xConfig);
+// Save items in the DI container
+$this->updateContainer($xConfig);
+// Register the exception handlers
+$this->registerExceptionHandlers($xConfig);
+}
 
-    /**
+/**
      * Get the options provided by the package library
      *
      * @param class-string $sClassName    The package class
@@ -211,27 +211,27 @@ 
      * @return Config
      * @throws SetupException
      */
-    private function getPackageLibConfig(string $sClassName): Config
-    {
-        // $this->aPackages contains packages config file paths.
-        $aLibOptions = $sClassName::config();
-        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))
-        {
-            // Otherwise, anything else than an array is not accepted.
-            $sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
-            throw new SetupException($sMessage);
-        }
-        // Add the package name to the config
-        $aLibOptions['package'] = $sClassName;
-        return $this->xConfigManager->newConfig($aLibOptions);
-    }
+private function getPackageLibConfig(string $sClassName): Config
+{
+// $this->aPackages contains packages config file paths.
+$aLibOptions = $sClassName::config();
+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))
+{
+// Otherwise, anything else than an array is not accepted.
+$sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
+throw new SetupException($sMessage);
+}
+// Add the package name to the config
+$aLibOptions['package'] = $sClassName;
+return $this->xConfigManager->newConfig($aLibOptions);
+}
 
-    /**
+/**
      * Register a package
      *
      * @param class-string $sClassName    The package class
@@ -240,27 +240,27 @@ 
      * @return void
      * @throws SetupException
      */
-    public function registerPackage(string $sClassName, array $aUserOptions = []): void
-    {
-        $sClassName = trim($sClassName, '\\ ');
-        if(!is_subclass_of($sClassName, AbstractPackage::class))
-        {
-            $sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
-            throw new SetupException($sMessage);
-        }
+public function registerPackage(string $sClassName, array $aUserOptions = []): void
+{
+$sClassName = trim($sClassName, '\\ ');
+if(!is_subclass_of($sClassName, AbstractPackage::class))
+{
+$sMessage = $this->xTranslator->trans('errors.register.invalid', ['name' => $sClassName]);
+throw new SetupException($sMessage);
+}
 
-        // Register the declarations in the package config.
-        $xAppConfig = $this->getPackageLibConfig($sClassName);
-        $this->registerItemsFromConfig($xAppConfig);
+// Register the declarations in the package config.
+$xAppConfig = $this->getPackageLibConfig($sClassName);
+$this->registerItemsFromConfig($xAppConfig);
 
-        // Register the package and its options in the DI
-        $this->di->registerPackage($sClassName, $aUserOptions);
+// Register the package and its options in the DI
+$this->di->registerPackage($sClassName, $aUserOptions);
 
-        // Register the package as a code generator.
-        $this->xCodeGenerator->addCodeGenerator($sClassName, 500);
-    }
+// Register the package as a code generator.
+$this->xCodeGenerator->addCodeGenerator($sClassName, 500);
+}
 
-    /**
+/**
      * Get a package instance
      *
      * @template T of AbstractPackage
@@ -268,42 +268,42 @@ 
      *
      * @return T|null
      */
-    public function getPackage(string $sClassName): ?AbstractPackage
-    {
-        $sClassName = trim($sClassName, '\\ ');
-        return $this->di->h($sClassName) ? $this->di->g($sClassName) : null;
-    }
+public function getPackage(string $sClassName): ?AbstractPackage
+{
+$sClassName = trim($sClassName, '\\ ');
+return $this->di->h($sClassName) ? $this->di->g($sClassName) : null;
+}
 
-    /**
+/**
      * Read and set Jaxon options from the config
      *
      * @return void
      * @throws SetupException
      */
-    public function registerFromConfig(): void
-    {
-        $xAppConfig = $this->xConfigManager->getAppConfig();
-        $this->registerItemsFromConfig($xAppConfig);
+public function registerFromConfig(): void
+{
+$xAppConfig = $this->xConfigManager->getAppConfig();
+$this->registerItemsFromConfig($xAppConfig);
 
-        // Register packages
-        $aPackageConfig = $xAppConfig->getOption('packages', []);
-        foreach($aPackageConfig as $xKey => $xValue)
-        {
-            if(is_integer($xKey) && is_string($xValue))
-            {
-                // Register a package without options
-                $sClassName = $xValue;
-                $this->registerPackage($sClassName);
-                continue;
-            }
+// Register packages
+$aPackageConfig = $xAppConfig->getOption('packages', []);
+foreach($aPackageConfig as $xKey => $xValue)
+{
+if(is_integer($xKey) && is_string($xValue))
+{
+    // Register a package without options
+    $sClassName = $xValue;
+    $this->registerPackage($sClassName);
+    continue;
+}
 
-            if(is_string($xKey) && is_array($xValue))
-            {
-                // Register a package with options
-                $sClassName = $xKey;
-                $aPkgOptions = $xValue;
-                $this->registerPackage($sClassName, $aPkgOptions);
-            }
-        }
-    }
+if(is_string($xKey) && is_array($xValue))
+{
+    // Register a package with options
+    $sClassName = $xKey;
+    $aPkgOptions = $xValue;
+    $this->registerPackage($sClassName, $aPkgOptions);
+}
+}
+}
 }

jaxon-core/src/Plugin/Request/CallableFunction/CallableFunctionPlugin.php

Braces +3 added lines, -6 removed lines

@@ -92,8 +92,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'));
         }
@@ -236,8 +235,7 @@ 
         {
             /** @var CallableFunction */
             $xFunction = $this->getCallable($sRequestedFunction);
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             // Unable to find the requested function
             $this->throwException($e, $this->xTranslator->trans('errors.functions.invalid',
@@ -246,8 +244,7 @@ 
         try
         {
             $xFunction->call($this->xTarget->args());
-        }
-        catch(Exception $e)
+        } catch(Exception $e)
         {
             // Unable to execute the requested function
             $this->throwException($e, $this->xTranslator->trans('errors.functions.call',

jaxon-core/src/App/Config/ConfigManager.php

Braces +8 added lines, -16 removed lines

@@ -69,23 +69,19 @@ 
         try
         {
             return $this->xConfigReader->read($sConfigFile);
-        }
-        catch(YamlExtension $e)
+        } catch(YamlExtension $e)
         {
             $sMessage = $this->xTranslator->trans('errors.yaml.install');
             throw new SetupException($sMessage);
-        }
-        catch(FileExtension $e)
+        } catch(FileExtension $e)
         {
             $sMessage = $this->xTranslator->trans('errors.file.extension', ['path' => $sConfigFile]);
             throw new SetupException($sMessage);
-        }
-        catch(FileAccess $e)
+        } catch(FileAccess $e)
         {
             $sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
             throw new SetupException($sMessage);
-        }
-        catch(FileContent $e)
+        } catch(FileContent $e)
         {
             $sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
             throw new SetupException($sMessage);
@@ -110,8 +106,7 @@ 
                 ->setOptions($this->xLibConfig, $this->read($sConfigFile), $sConfigSection);
             // Call the config change listeners.
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
-        }
-        catch(DataDepth $e)
+        } catch(DataDepth $e)
         {
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
                 'key' => $e->sPrefix,
@@ -139,8 +134,7 @@ 
             // Call the config change listeners.
             $this->xEventManager->libConfigChanged($this->xLibConfig, '');
             return $this->xLibConfig->changed();
-        }
-        catch(DataDepth $e)
+        } catch(DataDepth $e)
         {
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
                 'key' => $e->sPrefix,
@@ -246,8 +240,7 @@ 
             // Call the config change listeners.
             $this->xEventManager->appConfigChanged($this->xAppConfig, '');
             return $this->xAppConfig->changed();
-        }
-        catch(DataDepth $e)
+        } catch(DataDepth $e)
         {
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
                 'key' => $e->sPrefix,
@@ -296,8 +289,7 @@ 
         try
         {
             return $this->xConfigSetter->newConfig($aOptions, $sNamePrefix);
-        }
-        catch(DataDepth $e)
+        } catch(DataDepth $e)
         {
             $sMessage = $this->xTranslator->trans('errors.data.depth', [
                 'key' => $e->sPrefix,

Switch Indentation +180 added lines, -180 removed lines

@@ -29,17 +29,17 @@ 
 
 class ConfigManager
 {
-    /**
+/**
      * @var Config
      */
-    protected $xLibConfig;
+protected $xLibConfig;
 
-    /**
+/**
      * @var Config
      */
-    protected $xAppConfig;
+protected $xAppConfig;
 
-    /**
+/**
      * The constructor
      *
      * @param array $aDefaultOptions
@@ -48,15 +48,15 @@ 
      * @param ConfigSetter $xConfigSetter
      * @param ConfigEventManager $xEventManager
      */
-    public function __construct(array $aDefaultOptions, private Translator $xTranslator,
-        private ConfigReader $xConfigReader, private ConfigSetter $xConfigSetter,
-        private ConfigEventManager $xEventManager)
-    {
-        $this->xLibConfig = $xConfigSetter->newConfig($aDefaultOptions);
-        $this->xAppConfig = $xConfigSetter->newConfig();
-    }
+public function __construct(array $aDefaultOptions, private Translator $xTranslator,
+private ConfigReader $xConfigReader, private ConfigSetter $xConfigSetter,
+private ConfigEventManager $xEventManager)
+{
+$this->xLibConfig = $xConfigSetter->newConfig($aDefaultOptions);
+$this->xAppConfig = $xConfigSetter->newConfig();
+}
 
-    /**
+/**
      * Read a config file
      *
      * @param string $sConfigFile
@@ -64,35 +64,35 @@ 
      * @return array
      * @throws SetupException
      */
-    public function read(string $sConfigFile): array
-    {
-        try
-        {
-            return $this->xConfigReader->read($sConfigFile);
-        }
-        catch(YamlExtension $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.yaml.install');
-            throw new SetupException($sMessage);
-        }
-        catch(FileExtension $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.file.extension', ['path' => $sConfigFile]);
-            throw new SetupException($sMessage);
-        }
-        catch(FileAccess $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
-            throw new SetupException($sMessage);
-        }
-        catch(FileContent $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
-            throw new SetupException($sMessage);
-        }
-    }
+public function read(string $sConfigFile): array
+{
+try
+{
+return $this->xConfigReader->read($sConfigFile);
+}
+catch(YamlExtension $e)
+{
+$sMessage = $this->xTranslator->trans('errors.yaml.install');
+throw new SetupException($sMessage);
+}
+catch(FileExtension $e)
+{
+$sMessage = $this->xTranslator->trans('errors.file.extension', ['path' => $sConfigFile]);
+throw new SetupException($sMessage);
+}
+catch(FileAccess $e)
+{
+$sMessage = $this->xTranslator->trans('errors.file.access', ['path' => $sConfigFile]);
+throw new SetupException($sMessage);
+}
+catch(FileContent $e)
+{
+$sMessage = $this->xTranslator->trans('errors.file.content', ['path' => $sConfigFile]);
+throw new SetupException($sMessage);
+}
+}
 
-    /**
+/**
      * Read options from a config file and set the library config
      *
      * @param string $sConfigFile The full path to the config file
@@ -101,27 +101,27 @@ 
      * @return void
      * @throws SetupException
      */
-    public function load(string $sConfigFile, string $sConfigSection = ''): void
-    {
-        try
-        {
-            // Read the options and save in the config.
-            $this->xLibConfig = $this->xConfigSetter
-                ->setOptions($this->xLibConfig, $this->read($sConfigFile), $sConfigSection);
-            // Call the config change listeners.
-            $this->xEventManager->libConfigChanged($this->xLibConfig, '');
-        }
-        catch(DataDepth $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.data.depth', [
-                'key' => $e->sPrefix,
-                'depth' => $e->nDepth,
-            ]);
-            throw new SetupException($sMessage);
-        }
-    }
+public function load(string $sConfigFile, string $sConfigSection = ''): void
+{
+try
+{
+// Read the options and save in the config.
+$this->xLibConfig = $this->xConfigSetter
+    ->setOptions($this->xLibConfig, $this->read($sConfigFile), $sConfigSection);
+// Call the config change listeners.
+$this->xEventManager->libConfigChanged($this->xLibConfig, '');
+}
+catch(DataDepth $e)
+{
+$sMessage = $this->xTranslator->trans('errors.data.depth', [
+    'key' => $e->sPrefix,
+    'depth' => $e->nDepth,
+]);
+throw new SetupException($sMessage);
+}
+}
 
-    /**
+/**
      * Set the config options of the library
      *
      * @param array $aOptions
@@ -130,27 +130,27 @@ 
      * @return bool
      * @throws SetupException
      */
-    public function setOptions(array $aOptions, string $sNamePrefix = ''): bool
-    {
-        try
-        {
-            $this->xLibConfig = $this->xConfigSetter
-                ->setOptions($this->xLibConfig, $aOptions, $sNamePrefix);
-            // Call the config change listeners.
-            $this->xEventManager->libConfigChanged($this->xLibConfig, '');
-            return $this->xLibConfig->changed();
-        }
-        catch(DataDepth $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.data.depth', [
-                'key' => $e->sPrefix,
-                'depth' => $e->nDepth,
-            ]);
-            throw new SetupException($sMessage);
-        }
-    }
+public function setOptions(array $aOptions, string $sNamePrefix = ''): bool
+{
+try
+{
+$this->xLibConfig = $this->xConfigSetter
+    ->setOptions($this->xLibConfig, $aOptions, $sNamePrefix);
+// Call the config change listeners.
+$this->xEventManager->libConfigChanged($this->xLibConfig, '');
+return $this->xLibConfig->changed();
+}
+catch(DataDepth $e)
+{
+$sMessage = $this->xTranslator->trans('errors.data.depth', [
+    'key' => $e->sPrefix,
+    'depth' => $e->nDepth,
+]);
+throw new SetupException($sMessage);
+}
+}
 
-    /**
+/**
      * Set the value of a config option
      *
      * @param string $sName The option name
@@ -158,15 +158,15 @@ 
      *
      * @return void
      */
-    public function setOption(string $sName, $xValue): void
-    {
-        $this->xLibConfig = $this->xConfigSetter
-            ->setOption($this->xLibConfig, $sName, $xValue);
-        // Call the config change listeners.
-        $this->xEventManager->libConfigChanged($this->xLibConfig, $sName);
-    }
+public function setOption(string $sName, $xValue): void
+{
+$this->xLibConfig = $this->xConfigSetter
+->setOption($this->xLibConfig, $sName, $xValue);
+// Call the config change listeners.
+$this->xEventManager->libConfigChanged($this->xLibConfig, $sName);
+}
 
-    /**
+/**
      * Get the value of a config option
      *
      * @param string $sName The option name
@@ -174,36 +174,36 @@ 
      *
      * @return mixed
      */
-    public function getOption(string $sName, $xDefault = null): mixed
-    {
-        return $this->xLibConfig->getOption($sName, $xDefault);
-    }
+public function getOption(string $sName, $xDefault = null): mixed
+{
+return $this->xLibConfig->getOption($sName, $xDefault);
+}
 
-    /**
+/**
      * Check the presence of a config option
      *
      * @param string $sName The option name
      *
      * @return bool
      */
-    public function hasOption(string $sName): bool
-    {
-        return $this->xLibConfig->hasOption($sName);
-    }
+public function hasOption(string $sName): bool
+{
+return $this->xLibConfig->hasOption($sName);
+}
 
-    /**
+/**
      * Get the names of the options matching a given prefix
      *
      * @param string $sPrefix The prefix to match
      *
      * @return array
      */
-    public function getOptionNames(string $sPrefix): array
-    {
-        return $this->xLibConfig->getOptionNames($sPrefix);
-    }
+public function getOptionNames(string $sPrefix): array
+{
+return $this->xLibConfig->getOptionNames($sPrefix);
+}
 
-    /**
+/**
      * Set the value of a config option
      *
      * @param string $sName The option name
@@ -211,25 +211,25 @@ 
      *
      * @return void
      */
-    public function setAppOption(string $sName, $xValue): void
-    {
-        $this->xAppConfig = $this->xConfigSetter
-            ->setOption($this->xAppConfig, $sName, $xValue);
-        // Call the config change listeners.
-        $this->xEventManager->appConfigChanged($this->xAppConfig, $sName);
-    }
+public function setAppOption(string $sName, $xValue): void
+{
+$this->xAppConfig = $this->xConfigSetter
+->setOption($this->xAppConfig, $sName, $xValue);
+// Call the config change listeners.
+$this->xEventManager->appConfigChanged($this->xAppConfig, $sName);
+}
 
-    /**
+/**
      * Get the application config
      *
      * @return Config
      */
-    public function getAppConfig(): Config
-    {
-        return $this->xAppConfig;
-    }
+public function getAppConfig(): Config
+{
+return $this->xAppConfig;
+}
 
-    /**
+/**
      * Set the application config options
      *
      * @param array $aAppOptions
@@ -237,27 +237,27 @@ 
      *
      * @return bool
      */
-    public function setAppOptions(array $aAppOptions, string $sNamePrefix = ''): bool
-    {
-        try
-        {
-            $this->xAppConfig = $this->xConfigSetter
-                ->setOptions($this->xAppConfig, $aAppOptions, $sNamePrefix);
-            // Call the config change listeners.
-            $this->xEventManager->appConfigChanged($this->xAppConfig, '');
-            return $this->xAppConfig->changed();
-        }
-        catch(DataDepth $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.data.depth', [
-                'key' => $e->sPrefix,
-                'depth' => $e->nDepth,
-            ]);
-            throw new SetupException($sMessage);
-        }
-    }
+public function setAppOptions(array $aAppOptions, string $sNamePrefix = ''): bool
+{
+try
+{
+$this->xAppConfig = $this->xConfigSetter
+    ->setOptions($this->xAppConfig, $aAppOptions, $sNamePrefix);
+// Call the config change listeners.
+$this->xEventManager->appConfigChanged($this->xAppConfig, '');
+return $this->xAppConfig->changed();
+}
+catch(DataDepth $e)
+{
+$sMessage = $this->xTranslator->trans('errors.data.depth', [
+    'key' => $e->sPrefix,
+    'depth' => $e->nDepth,
+]);
+throw new SetupException($sMessage);
+}
+}
 
-    /**
+/**
      * Get the value of an application config option
      *
      * @param string $sName The option name
@@ -265,24 +265,24 @@ 
      *
      * @return mixed
      */
-    public function getAppOption(string $sName, $xDefault = null): mixed
-    {
-        return $this->xAppConfig->getOption($sName, $xDefault);
-    }
+public function getAppOption(string $sName, $xDefault = null): mixed
+{
+return $this->xAppConfig->getOption($sName, $xDefault);
+}
 
-    /**
+/**
      * Check the presence of an application config option
      *
      * @param string $sName The option name
      *
      * @return bool
      */
-    public function hasAppOption(string $sName): bool
-    {
-        return $this->xAppConfig->hasOption($sName);
-    }
+public function hasAppOption(string $sName): bool
+{
+return $this->xAppConfig->hasOption($sName);
+}
 
-    /**
+/**
      * Create a new the config object
      *
      * @param array $aOptions     The options array
@@ -291,54 +291,54 @@ 
      * @return Config
      * @throws SetupException
      */
-    public function newConfig(array $aOptions = [], string $sNamePrefix = ''): Config
-    {
-        try
-        {
-            return $this->xConfigSetter->newConfig($aOptions, $sNamePrefix);
-        }
-        catch(DataDepth $e)
-        {
-            $sMessage = $this->xTranslator->trans('errors.data.depth', [
-                'key' => $e->sPrefix,
-                'depth' => $e->nDepth,
-            ]);
-            throw new SetupException($sMessage);
-        }
-    }
+public function newConfig(array $aOptions = [], string $sNamePrefix = ''): Config
+{
+try
+{
+return $this->xConfigSetter->newConfig($aOptions, $sNamePrefix);
+}
+catch(DataDepth $e)
+{
+$sMessage = $this->xTranslator->trans('errors.data.depth', [
+    'key' => $e->sPrefix,
+    'depth' => $e->nDepth,
+]);
+throw new SetupException($sMessage);
+}
+}
 
-    /**
+/**
      * @param string $sClassName
      *
      * @return void
      */
-    public function addLibEventListener(string $sClassName): void
-    {
-        $this->xEventManager->addLibConfigListener($sClassName);
-    }
+public function addLibEventListener(string $sClassName): void
+{
+$this->xEventManager->addLibConfigListener($sClassName);
+}
 
-    /**
+/**
      * @param string $sClassName
      *
      * @return void
      */
-    public function addAppEventListener(string $sClassName): void
-    {
-        $this->xEventManager->addAppConfigListener($sClassName);
-    }
+public function addAppEventListener(string $sClassName): void
+{
+$this->xEventManager->addAppConfigListener($sClassName);
+}
 
-    /**
+/**
      * Make the helpers functions available in the global namespace.
      *
      * @param bool $bForce
      *
      * @return void
      */
-    public function globals(bool $bForce = false): void
-    {
-        if($bForce || $this->getAppOption('helpers.global', true))
-        {
-            require_once dirname(__DIR__, 2) . '/globals.php';
-        }
-    }
+public function globals(bool $bForce = false): void
+{
+if($bForce || $this->getAppOption('helpers.global', true))
+{
+require_once dirname(__DIR__, 2) . '/globals.php';
+}
+}
 }

jaxon-core/src/App/PageComponent.php

Switch Indentation +23 added lines, -23 removed lines

@@ -6,53 +6,53 @@
 
 abstract class PageComponent extends NodeComponent
 {
-    /**
+/**
      * The current page number.
      *
      * @var int
      */
-    private int $currentPage = 1;
+private int $currentPage = 1;
 
-    /**
+/**
      * Get the total number of items to paginate.
      *
      * @return int
      */
-    abstract protected function count(): int;
+abstract protected function count(): int;
 
-    /**
+/**
      * Get the max number of items per page.
      *
      * @return int
      */
-    abstract protected function limit(): int;
+abstract protected function limit(): int;
 
-    /**
+/**
      * Get the paginator for the component.
      *
      * @param int $pageNumber
      *
      * @return Paginator
      */
-    protected function paginator(int $pageNumber): Paginator
-    {
-        return $this->cl(Component\Pagination::class)
-            // Use the js class name as component item identifier.
-            ->item($this->rq()->_class())
-            ->paginator($pageNumber > 0 ? $pageNumber : 1, $this->limit(), $this->count())
-            // This callback will receive the final value of the current page number.
-            ->page(function(int $currentPage) {
-                $this->currentPage = $currentPage;
-            });
-    }
+protected function paginator(int $pageNumber): Paginator
+{
+return $this->cl(Component\Pagination::class)
+// Use the js class name as component item identifier.
+->item($this->rq()->_class())
+->paginator($pageNumber > 0 ? $pageNumber : 1, $this->limit(), $this->count())
+// This callback will receive the final value of the current page number.
+->page(function(int $currentPage) {
+    $this->currentPage = $currentPage;
+});
+}
 
-    /**
+/**
      * Get the current page number
      *
      * @return int
      */
-    protected function currentPage(): int
-    {
-        return $this->currentPage;
-    }
+protected function currentPage(): int
+{
+return $this->currentPage;
+}
 }

jaxon-core/src/App/PageDatabagTrait.php

Switch Indentation +37 added lines, -37 removed lines

@@ -6,81 +6,81 @@
 
 trait PageDatabagTrait
 {
-    /**
+/**
      * The current page number.
      *
      * @var int
      */
-    private int $currentPage = 1;
+private int $currentPage = 1;
 
-    /**
+/**
      * Get the pagination databag name.
      *
      * @return string
      */
-    abstract protected function bagName(): string;
+abstract protected function bagName(): string;
 
-    /**
+/**
      * Get the pagination databag attribute.
      *
      * @return string
      */
-    abstract protected function bagAttr(): string;
+abstract protected function bagAttr(): string;
 
-    /**
+/**
      * Get the page number.
      *
      * @param int $pageNumber
      *
      * @return int
      */
-    private function getPageNumber(int $pageNumber): int
-    {
-        // If no page number is provided, then get the value from the databag.
-        return $pageNumber > 0 ? $pageNumber :
-            (int)$this->bag($this->bagName())->get($this->bagAttr(), 1);
-    }
+private function getPageNumber(int $pageNumber): int
+{
+// If no page number is provided, then get the value from the databag.
+return $pageNumber > 0 ? $pageNumber :
+(int)$this->bag($this->bagName())->get($this->bagAttr(), 1);
+}
 
-    /**
+/**
      * Set the page number.
      *
      * @param int $currentPage
      *
      * @return void
      */
-    private function setCurrentPage(int $currentPage): void
-    {
-        // Save the current page in the databag.
-        $this->bag($this->bagName())->set($this->bagAttr(), $currentPage);
-        $this->currentPage = $currentPage;
-    }
+private function setCurrentPage(int $currentPage): void
+{
+// Save the current page in the databag.
+$this->bag($this->bagName())->set($this->bagAttr(), $currentPage);
+$this->currentPage = $currentPage;
+}
 
-    /**
+/**
      * Get the paginator for the component.
      *
      * @param int $pageNumber
      *
      * @return Paginator
      */
-    protected function paginator(int $pageNumber): Paginator
-    {
-        return $this->cl(Component\Pagination::class)
-            // Use the js class name as component item identifier.
-            ->item($this->rq()->_class())
-            ->paginator($this->getPageNumber($pageNumber), $this->limit(), $this->count())
-            // This callback will receive the final value of the current page number.
-            ->page(function(int $currentPage) {
-                $this->setCurrentPage($currentPage);
-            });
-    }
+protected function paginator(int $pageNumber): Paginator
+{
+return $this->cl(Component\Pagination::class)
+// Use the js class name as component item identifier.
+->item($this->rq()->_class())
+->paginator($this->getPageNumber($pageNumber), $this->limit(), $this->count())
+// This callback will receive the final value of the current page number.
+->page(function(int $currentPage) {
+    $this->setCurrentPage($currentPage);
+});
+}
 
-    /**
+/**
      * Get the current page number
      *
      * @return int
      */
-    protected function currentPage(): int
-    {
-        return $this->currentPage;
-    }
+protected function currentPage(): int
+{
+return $this->currentPage;
+}
 }

jaxon-core/src/globals.php

Switch Indentation +6 added lines, -6 removed lines

@@ -32,7 +32,7 @@ 
  */
 function cl(string $sClassName): mixed
 {
-    return \Jaxon\cl($sClassName);
+return \Jaxon\cl($sClassName);
 }
 
 /**
@@ -44,7 +44,7 @@ 
  */
 function rq(string $sClassName = ''): JxnCall
 {
-    return \Jaxon\rq($sClassName);
+return \Jaxon\rq($sClassName);
 }
 
 /**
@@ -56,7 +56,7 @@ 
  */
 function jo(string $sJsObject = ''): JsObjectCall
 {
-    return \Jaxon\jo($sJsObject);
+return \Jaxon\jo($sJsObject);
 }
 
 /**
@@ -69,7 +69,7 @@ 
  */
 function jq(string $sPath = '', $xContext = null): JqSelectorCall
 {
-    return \Jaxon\jq($sPath, $xContext);
+return \Jaxon\jq($sPath, $xContext);
 }
 
 /**
@@ -81,7 +81,7 @@ 
  */
 function je(string $sElementId = ''): JsSelectorCall
 {
-    return \Jaxon\je($sElementId);
+return \Jaxon\je($sElementId);
 }
 
 /**
@@ -91,5 +91,5 @@ 
  */
 function attr(): HtmlAttrHelper
 {
-    return \Jaxon\attr();
+return \Jaxon\attr();
 }

jaxon-core/src/Request/Handler/RequestHandler.php

Braces +1 added lines, -2 removed lines

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