XOOPS / XoopsCore

xoops_lib/Xoops/Core/Kernel/XoopsPersistableObjectHandler.php

Spacing +2 added lines, -2 removed lines

@@ -240,10 +240,10 @@
             if (!in_array($this->keyName, $fields)) {
                 $fields[] = $this->keyName;
             }
-            $first=true;
+            $first = true;
             foreach ($fields as $field) {
                 if ($first) {
-                    $first=false;
+                    $first = false;
                     $qb->select($field);
                 } else {
                     $qb->addSelect($field);

xoops_lib/Xoops/Core/Kernel/Criteria.php

Spacing +13 added lines, -13 removed lines

@@ -78,12 +78,12 @@ 
      */
     public function render()
     {
-        $clause = (!empty($this->prefix) ? "{$this->prefix}." : "") . $this->column;
+        $clause = (!empty($this->prefix) ? "{$this->prefix}." : "").$this->column;
         if (!empty($this->function)) {
             $clause = sprintf($this->function, $clause);
         }
         if (in_array(strtoupper($this->operator), array('IS NULL', 'IS NOT NULL'))) {
-            $clause .= ' ' . $this->operator;
+            $clause .= ' '.$this->operator;
         } else {
             if ('' === ($value = trim($this->value))) {
                 return '';
@@ -121,7 +121,7 @@ 
 
         if ($this->operator === '!=' || $this->operator === '<>') {
             $operator = '=';
-            $clause = "(!(" . $this->column . $operator . $this->value . "))";
+            $clause = "(!(".$this->column.$operator.$this->value."))";
         } else {
             if ($this->operator === 'IN') {
                 $newvalue = str_replace(array('(', ')'), '', $this->value);
@@ -129,9 +129,9 @@ 
                 foreach ($tab as $uid) {
                     $clause .= "({$this->column}={$uid})";
                 }
-                $clause = '(|' . $clause . ')';
+                $clause = '(|'.$clause.')';
             } else {
-                $clause = "(" . $this->column . ' ' . $this->operator . ' ' . $this->value . ")";
+                $clause = "(".$this->column.' '.$this->operator.' '.$this->value.")";
             }
         }
         return $clause;
@@ -159,7 +159,7 @@ 
      */
     public function renderQb(QueryBuilder $qb = null, $whereMode = '')
     {
-        if ($qb==null) { // initialize query builder if not passed in
+        if ($qb == null) { // initialize query builder if not passed in
             $qb = \Xoops::getInstance()->db()->createXoopsQueryBuilder();
             $whereMode = ''; // first entry in new instance must be where
         }
@@ -177,7 +177,7 @@ 
                 break;
         }
 
-        if ($this->limit!=0 || $this->start!=0) {
+        if ($this->limit != 0 || $this->start != 0) {
             $qb->setFirstResult($this->start)
                 ->setMaxResults($this->limit);
         }
@@ -209,14 +209,14 @@ 
     {
         $eb = $qb->expr();
 
-        $column = (empty($this->prefix) ? "" : $this->prefix.'.') . $this->column;
+        $column = (empty($this->prefix) ? "" : $this->prefix.'.').$this->column;
 
         // this should be done using portability functions
         if (!empty($this->function)) {
             $column = sprintf($this->function, $column);
         }
 
-        $value=trim($this->value);
+        $value = trim($this->value);
 
         $operator = strtolower($this->operator);
         $expr = '';
@@ -231,16 +231,16 @@ 
                     $expr = $eb->isNotNull($column);
                     break;
                 case 'in':
-                    if (!empty($value) && $value!=='()') {
-                        $expr = $column . ' IN ' . $value;
+                    if (!empty($value) && $value !== '()') {
+                        $expr = $column.' IN '.$value;
                     } else {
                         // odd case of a null set - this won't match anything
                         $expr = $eb->neq($column, $column);
                     }
                     break;
                 case 'not in':
-                    if (!empty($value) && $value!=='()') {
-                        $expr = $column . ' NOT IN ' . $value;
+                    if (!empty($value) && $value !== '()') {
+                        $expr = $column.' NOT IN '.$value;
                     }
                     break;
             }

xoops_lib/Xoops/Core/Kernel/Dtype/DtypeDateTime.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
         } elseif (is_string($value)) {
             $cleanValue = strtotime($value);
         } else {
-            $cleanValue = (int) $value;
+            $cleanValue = (int)$value;
         }
         return $cleanValue;
     }

xoops_lib/Xoops/Core/Kernel/Dtype/DtypeJson.php

Spacing +3 added lines, -3 removed lines

@@ -61,10 +61,10 @@
     public function cleanVar(XoopsObject $obj, $key)
     {
         $value = $obj->vars[$key]['value'];
-        $value = ($value===null || $value==='' || $value===false) ? null : $value;
-        if ($value!==null && null === json_decode($value, true)) {
+        $value = ($value === null || $value === '' || $value === false) ? null : $value;
+        if ($value !== null && null === json_decode($value, true)) {
             $value = json_encode($value, JSON_FORCE_OBJECT);
-            if ($value===false) {
+            if ($value === false) {
                 \Xoops::getInstance()->logger()->warning(
                     sprintf('Failed to encode to JSON - %s', json_last_error_msg())
                 );

xoops_lib/Xoops/Core/Kernel/Dtype/DtypeMoney.php

Spacing +1 added lines, -1 removed lines

@@ -93,6 +93,6 @@
         if (false === $decoded || !(isset($decoded['amount']) && isset($decoded['currency']))) {
             return null;
         }
-        return new Money((int) $decoded['amount'], new Currency($decoded['currency']));
+        return new Money((int)$decoded['amount'], new Currency($decoded['currency']));
     }
 }

xoops_lib/Xoops/Core/Kernel/Dtype/DtypeUrl.php

Spacing +1 added lines, -1 removed lines

@@ -42,7 +42,7 @@
             return $value;
         }
         if ($value != '' && !preg_match("/^http[s]*:\/\//i", $value)) {
-            $value = 'http://' . $value;
+            $value = 'http://'.$value;
         }
         return $value;
     }

xoops_lib/Xoops/Core/Kernel/Dtype/DtypeAbstract.php

Indentation +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
      * @param string      $format Dtype::FORMAT_* constant indicating desired formatting
      *
      * @return mixed
-    */
+     */
     public function getVar(XoopsObject $obj, $key, $format)
     {
         $value = $obj->vars[$key]['value'];

xoops_lib/Xoops/Core/Kernel/Handlers/XoopsOnlineHandler.php

Indentation +3 added lines, -3 removed lines

@@ -72,9 +72,9 @@
         $rows = $this->db2->updatePrefix(
             'system_online',
             array(
-               'online_uname'   => $uname,
-               'online_updated' => $time,
-               'online_module'  => $module,
+                'online_uname'   => $uname,
+                'online_updated' => $time,
+                'online_module'  => $module,
             ),
             $criteria
         );

Spacing +1 added lines, -1 removed lines

@@ -96,7 +96,7 @@
         if ($rows === false) {
             return false;
         }
-        return ($rows>0);
+        return ($rows > 0);
     }
 
     /**

xoops_lib/Xoops/Core/Kernel/Handlers/XoopsBlock.php

Spacing +2 added lines, -2 removed lines

@@ -414,7 +414,7 @@ 
                 return false;
             }
             $funcFile = $xoops->path(
-                'modules/' . $this->getVar('dirname') . '/blocks/' . $this->getVar('func_file')
+                'modules/'.$this->getVar('dirname').'/blocks/'.$this->getVar('func_file')
             );
             if (\XoopsLoad::fileExists($funcFile)) {
                 $xoops->loadLanguage('blocks', $this->getVar('dirname'));
@@ -468,7 +468,7 @@ 
             }
             if (!\XoopsLoad::fileExists(
                 $func_file = $xoops->path(
-                    'modules/' . $this->getVar('dirname') . '/blocks/' . $this->getVar('func_file')
+                    'modules/'.$this->getVar('dirname').'/blocks/'.$this->getVar('func_file')
                 )
             )) {
                 return false;