jaxon-php / jaxon-core

src/Plugin/Response/JQuery/DomSelector.php

Spacing +3 added lines, -3 removed lines

@@ -77,12 +77,12 @@ 
      */
     private function getPath(string $jQueryNs, string $sPath, $xContext)
     {
-        if(!$sPath)
+        if (!$sPath)
         {
             // If an empty selector is given, use javascript "this" instead
             return "$jQueryNs(this)";
         }
-        if(!$xContext)
+        if (!$xContext)
         {
             return "$jQueryNs('" . $sPath . "')";
         }
@@ -155,7 +155,7 @@ 
     public function getScript(): string
     {
         $sScript = $this->sPath;
-        if(count($this->aCalls) > 0)
+        if (count($this->aCalls) > 0)
         {
             $sScript .= '.' . implode('.', $this->aCalls);
         }

src/App/Dialog/Library/DialogLibraryHelper.php

Spacing +8 added lines, -8 removed lines

@@ -136,18 +136,18 @@ 
         $aOptions = $this->getOptionNames($sKeyPrefix);
         $sSpaces = str_repeat(' ', $nSpaces);
         $sScript = '';
-        foreach($aOptions as $sShortName => $sFullName)
+        foreach ($aOptions as $sShortName => $sFullName)
         {
             $value = $this->xConfigManager->getOption($sFullName);
-            if(is_string($value))
+            if (is_string($value))
             {
                 $value = "'$value'";
             }
-            elseif(is_bool($value))
+            elseif (is_bool($value))
             {
                 $value = ($value ? 'true' : 'false');
             }
-            elseif(!is_numeric($value))
+            elseif (!is_numeric($value))
             {
                 $value = json_encode($value);
             }
@@ -175,10 +175,10 @@ 
      */
     public function getJsCode(string $sFile): string
     {
-        if($this->hasOption('assets.js'))
+        if ($this->hasOption('assets.js'))
         {
             // If this expression evaluates to false, then the asset is not displayed.
-            if(!($sUri = $this->getOption('assets.js')))
+            if (!($sUri = $this->getOption('assets.js')))
             {
                 return '';
             }
@@ -200,10 +200,10 @@ 
      */
     public function getCssCode(string $sFile): string
     {
-        if($this->hasOption('assets.css'))
+        if ($this->hasOption('assets.css'))
         {
             // If this expression evaluates to false, then the asset is not displayed.
-            if(!($sUri = $this->getOption('assets.css')))
+            if (!($sUri = $this->getOption('assets.css')))
             {
                 return '';
             }

Braces +4 added lines, -8 removed lines

@@ -142,12 +142,10 @@ 
             if(is_string($value))
             {
                 $value = "'$value'";
-            }
-            elseif(is_bool($value))
+            } elseif(is_bool($value))
             {
                 $value = ($value ? 'true' : 'false');
-            }
-            elseif(!is_numeric($value))
+            } elseif(!is_numeric($value))
             {
                 $value = json_encode($value);
             }
@@ -182,8 +180,7 @@ 
             {
                 return '';
             }
-        }
-        else
+        } else
         {
             $sUri = $this->sUri . '/' . ($this->sSubDir ? $this->sSubDir . '/' : '') .
                 ($this->sVersion ? $this->sVersion . '/' : '') . $sFile;
@@ -207,8 +204,7 @@ 
             {
                 return '';
             }
-        }
-        else
+        } else
         {
             $sUri = $this->sUri . '/' . ($this->sSubDir ? $this->sSubDir . '/' : '') .
                 ($this->sVersion ? $this->sVersion . '/' : '') . $sFile;

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->bConfirm)
+        if ($this->bConfirm)
         {
             $sConfirmPhrase = $this->makePhrase($this->aConfirmArgs);
             $sScript = $this->xDialogLibraryManager->getQuestionLibrary()
                 ->confirm($sConfirmPhrase, $sScript, $sMessageScript);
         }
-        if($this->sCondition !== '')
+        if ($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 +1 added lines, -2 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];