jaxon-php / jaxon-core

src/Utils/View/Manager.php

Unused Use Statements -3 removed lines

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

Spacing +8 added lines, -8 removed lines

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

Braces +1 added lines, -2 removed lines

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

src/Utils/Session/Manager.php

Spacing +1 added lines, -1 removed lines

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

src/Utils/Pagination/Renderer.php

Spacing +4 added lines, -4 removed lines

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

Braces +1 added lines, -2 removed lines

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

src/Utils/Config/Json.php

Spacing +2 added lines, -2 removed lines

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

src/Utils/Config/Config.php

Spacing +10 added lines, -11 removed lines

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

src/Utils/Config/Php.php

Spacing +2 added lines, -2 removed lines

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

src/Utils/Config/Yaml.php

Spacing +3 added lines, -3 removed lines

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

src/Utils/Config/Reader.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
     public function read($sConfigFile)
     {
         $sExt = pathinfo($sConfigFile, PATHINFO_EXTENSION);
-        switch($sExt)
+        switch ($sExt)
         {
         case 'php':
             $aConfigOptions = Php::read($sConfigFile);

src/Utils/View/Renderer.php

Spacing +3 added lines, -3 removed lines

@@ -40,7 +40,7 @@ 
      */
     protected function store()
     {
-        if(!$this->xStore)
+        if (!$this->xStore)
         {
             $this->xStore = new Store();
         }
@@ -93,12 +93,12 @@ 
         $sNamespace = $this->xManager->getDefaultNamespace();
         // Get the namespace from the view name
         $iSeparatorPosition = strrpos($sViewName, '::');
-        if($iSeparatorPosition !== false)
+        if ($iSeparatorPosition !== false)
         {
             $sNamespace = substr($sViewName, 0, $iSeparatorPosition);
         }
         $aRenderers = $this->xManager->getRenderers();
-        if(!key_exists($sNamespace, $aRenderers))
+        if (!key_exists($sNamespace, $aRenderers))
         {
             // Cannot render a view if there's no renderer corresponding to the namespace.
             return null;