geekwright / XoopsCore

xoops_lib/Xoops/Core/Kernel/Model/Write.php

Spacing +5 added lines, -5 removed lines

@@ -66,21 +66,21 @@ 
     {
         if (!(class_exists($this->handler->className) && $object instanceof $this->handler->className)) {
             trigger_error(
-                "Object '" . get_class($object) . "' is not an instance of '" . $this->handler->className . "'",
+                "Object '".get_class($object)."' is not an instance of '".$this->handler->className."'",
                 E_USER_NOTICE
             );
             return false;
         }
         if (!$object->isDirty()) {
             trigger_error(
-                "Data entry is not inserted - the object '" . get_class($object) . "' is not dirty",
+                "Data entry is not inserted - the object '".get_class($object)."' is not dirty",
                 E_USER_NOTICE
             );
             return false;
         }
         if (!$this->cleanVars($object)) {
             trigger_error(
-                "Insert failed in method 'cleanVars' of object '" . get_class($object) . "'" . $object->getHtmlErrors(),
+                "Insert failed in method 'cleanVars' of object '".get_class($object)."'".$object->getHtmlErrors(),
                 E_USER_WARNING
             );
             return false;
@@ -89,7 +89,7 @@ 
         if ($object->isNew()) {
             if (empty($object->cleanVars)) {
                 trigger_error(
-                    "Data entry is not inserted - no variable is changed in object of '" . get_class($object) . "'",
+                    "Data entry is not inserted - no variable is changed in object of '".get_class($object)."'",
                     E_USER_NOTICE
                 );
                 return false;
@@ -131,7 +131,7 @@ 
     {
         if (!(class_exists($this->handler->className) && $object instanceof $this->handler->className)) {
             trigger_error(
-                "Object '" . get_class($object) . "' is not an instance of '" . $this->handler->className . "'",
+                "Object '".get_class($object)."' is not an instance of '".$this->handler->className."'",
                 E_USER_NOTICE
             );
             return false;

xoops_lib/Xoops/Core/Kernel/Model/Joint.php

Spacing +6 added lines, -6 removed lines

@@ -90,8 +90,8 @@ 
 
         $qb = $this->handler->db2->createXoopsQueryBuilder();
         if (is_array($fields) && count($fields)) {
-            if (!in_array("o." . $this->handler->keyName, $fields)) {
-                $fields[] = "o." . $this->handler->keyName;
+            if (!in_array("o.".$this->handler->keyName, $fields)) {
+                $fields[] = "o.".$this->handler->keyName;
             }
             $first = true;
             foreach ($fields as $field) {
@@ -230,13 +230,13 @@ 
 
         $set = array();
         foreach ($data as $key => $val) {
-            $set[] = "o.{$key}=" . $this->handler->db2->quote($val);
+            $set[] = "o.{$key}=".$this->handler->db2->quote($val);
         }
-        $sql = " UPDATE {$this->handler->table} AS o" . " SET " . implode(", ", $set)
+        $sql = " UPDATE {$this->handler->table} AS o"." SET ".implode(", ", $set)
             . " LEFT JOIN {$this->handler->table_link} AS l "
             . "ON o.{$this->handler->field_object} = l.{$this->handler->field_link}";
         if (isset($criteria) && ($criteria instanceof CriteriaElement)) {
-            $sql .= " " . $criteria->renderWhere();
+            $sql .= " ".$criteria->renderWhere();
         }
 
         return $this->handler->db2->executeUpdate($sql);
@@ -265,7 +265,7 @@ 
             . "LEFT JOIN {$this->handler->table_link} AS l "
             . "ON o.{$this->handler->field_object} = l.{$this->handler->field_link}";
         if (isset($criteria) && ($criteria instanceof CriteriaElement)) {
-            $sql .= " " . $criteria->renderWhere();
+            $sql .= " ".$criteria->renderWhere();
         }
 
         return $this->handler->db2->executeUpdate($sql);

xoops_lib/Xoops/Core/Kernel/Model/Read.php

Spacing +3 added lines, -3 removed lines

@@ -45,10 +45,10 @@ 
             if (!in_array($this->handler->keyName, $fields)) {
                 $fields[] = $this->handler->keyName;
             }
-            $first=true;
+            $first = true;
             foreach ($fields as $field) {
                 if ($first) {
-                    $first=false;
+                    $first = false;
                     $qb->select($field);
                 } else {
                     $qb->addSelect($field);
@@ -131,7 +131,7 @@ 
             $qb->addSelect($this->handler->identifierName);
         }
         $qb->from($this->handler->table, null);
-        if ($limit!=0 || $start!=0) {
+        if ($limit != 0 || $start != 0) {
             $qb->setFirstResult($start)
                 ->setMaxResults($limit);
         }

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;
     }