gin0115 / pixie-wpdb

src/Binding.php

Spacing +2 added lines, -2 removed lines

@@ -125,7 +125,7 @@ 
     protected function verifyType(?string $type): void
     {
         $validTypes = [self::STRING, self::BOOL, self::FLOAT, self::INT, self::JSON, self::RAW];
-        if (null !== $type && !in_array($type, $validTypes, true)) {
+        if (null !== $type && ! in_array($type, $validTypes, true)) {
             throw new Exception("{$type} is not a valid type to use for Bindings.", 1);
         }
     }
@@ -137,7 +137,7 @@ 
      */
     public function hasTypeDefined(): bool
     {
-        return !\in_array($this->type, [null, self::RAW], true);
+        return ! \in_array($this->type, [null, self::RAW], true);
     }
 
     /**

src/Connection.php

Spacing +1 added lines, -1 removed lines

@@ -64,7 +64,7 @@
         }
 
         // Preserve the first database connection with a static property
-        if (!static::$storedConnection) {
+        if ( ! static::$storedConnection) {
             static::$storedConnection = $this;
         }
     }

src/QueryBuilder/WPDBAdapter.php

Spacing +42 added lines, -42 removed lines

@@ -49,9 +49,9 @@ 
      */
     public function select(array $statements): array
     {
-        if (!array_key_exists('tables', $statements)) {
+        if ( ! array_key_exists('tables', $statements)) {
             throw new Exception('No table specified.', 3);
-        } elseif (!array_key_exists('selects', $statements)) {
+        } elseif ( ! array_key_exists('selects', $statements)) {
             $statements['selects'][] = '*';
         }
 
@@ -65,7 +65,7 @@ 
         // Group bys
         $groupBys = '';
         if (isset($statements['groupBys']) && $groupBys = $this->arrayStr($statements['groupBys'], ', ')) {
-            $groupBys = 'GROUP BY ' . $groupBys;
+            $groupBys = 'GROUP BY '.$groupBys;
         }
 
         // Order bys
@@ -76,17 +76,17 @@ 
                 if ($field instanceof Closure) {
                     continue;
                 }
-                $orderBys .= $field . ' ' . $orderBy['type'] . ', ';
+                $orderBys .= $field.' '.$orderBy['type'].', ';
             }
 
             if ($orderBys = trim($orderBys, ', ')) {
-                $orderBys = 'ORDER BY ' . $orderBys;
+                $orderBys = 'ORDER BY '.$orderBys;
             }
         }
 
         // Limit and offset
-        $limit  = isset($statements['limit']) ? 'LIMIT ' . (int) $statements['limit'] : '';
-        $offset = isset($statements['offset']) ? 'OFFSET ' . (int) $statements['offset'] : '';
+        $limit  = isset($statements['limit']) ? 'LIMIT '.(int) $statements['limit'] : '';
+        $offset = isset($statements['offset']) ? 'OFFSET '.(int) $statements['offset'] : '';
 
         // Having
         list($havingCriteria, $havingBindings) = $this->buildCriteriaWithType($statements, 'havings', 'HAVING');
@@ -96,7 +96,7 @@ 
 
         /** @var string[] */
         $sqlArray = [
-            'SELECT' . (isset($statements['distinct']) ? ' DISTINCT' : ''),
+            'SELECT'.(isset($statements['distinct']) ? ' DISTINCT' : ''),
             $selects,
             'FROM',
             $tables,
@@ -130,7 +130,7 @@ 
     public function criteriaOnly(array $statements, bool $bindValues = true): array
     {
         $sql = $bindings = [];
-        if (!isset($statements['criteria'])) {
+        if ( ! isset($statements['criteria'])) {
             return compact('sql', 'bindings');
         }
 
@@ -152,7 +152,7 @@ 
      */
     private function doInsert(array $statements, array $data, string $type): array
     {
-        if (!isset($statements['tables'])) {
+        if ( ! isset($statements['tables'])) {
             throw new Exception('No table specified', 3);
         }
 
@@ -173,20 +173,20 @@ 
             if ($value instanceof Raw) {
                 $values[] = $this->parseRaw($value);
             } elseif ($isBindings) {
-                $values[]   =  $value->getType();
+                $values[]   = $value->getType();
                 $bindings[] = $value->getValue();
             } else {
-                $values[]   =  $this->inferType($value);
+                $values[]   = $this->inferType($value);
                 $bindings[] = $value;
             }
         }
 
         $sqlArray = [
-        $type . ' INTO',
+        $type.' INTO',
         $this->wrapSanitizer($table),
-        '(' . $this->arrayStr($keys, ',') . ')',
+        '('.$this->arrayStr($keys, ',').')',
         'VALUES',
-        '(' . $this->arrayStr($values, ',') . ')',
+        '('.$this->arrayStr($values, ',').')',
         ];
 
         if (isset($statements['onduplicate'])) {
@@ -194,7 +194,7 @@ 
                 throw new Exception('No data given.', 4);
             }
             list($updateStatement, $updateBindings) = $this->getUpdateStatement($statements['onduplicate']);
-            $sqlArray[]                             = 'ON DUPLICATE KEY UPDATE ' . $updateStatement;
+            $sqlArray[]                             = 'ON DUPLICATE KEY UPDATE '.$updateStatement;
             $bindings                               = array_merge($bindings, $updateBindings);
         }
 
@@ -302,12 +302,12 @@ 
             }
 
             if ($value instanceof Raw) {
-                $statement .= $this->stringifyValue($this->wrapSanitizer($key)) . '=' . $value . ',';
+                $statement .= $this->stringifyValue($this->wrapSanitizer($key)).'='.$value.',';
             } elseif ($isBindings) {
-                $statement .= $this->stringifyValue($this->wrapSanitizer($key)) . sprintf('=%s,', $value->getType());
+                $statement .= $this->stringifyValue($this->wrapSanitizer($key)).sprintf('=%s,', $value->getType());
                 $bindings[] = $value->getValue();
             } else {
-                $statement .= $this->stringifyValue($this->wrapSanitizer($key)) . sprintf('=%s,', $this->inferType($value));
+                $statement .= $this->stringifyValue($this->wrapSanitizer($key)).sprintf('=%s,', $this->inferType($value));
                 $bindings[] = $value;
             }
         }
@@ -329,7 +329,7 @@ 
      */
     public function update($statements, array $data)
     {
-        if (!isset($statements['tables'])) {
+        if ( ! isset($statements['tables'])) {
             throw new Exception('No table specified', 3);
         } elseif (count($data) < 1) {
             throw new Exception('No data given.', 4);
@@ -344,12 +344,12 @@ 
         list($whereCriteria, $whereBindings) = $this->buildCriteriaWithType($statements, 'wheres', 'WHERE');
 
         // Limit
-        $limit = isset($statements['limit']) ? 'LIMIT ' . $statements['limit'] : '';
+        $limit = isset($statements['limit']) ? 'LIMIT '.$statements['limit'] : '';
 
         $sqlArray = [
             'UPDATE',
             $this->wrapSanitizer($table),
-            'SET ' . $updateStatement,
+            'SET '.$updateStatement,
             $whereCriteria,
             $limit,
         ];
@@ -372,7 +372,7 @@ 
      */
     public function delete($statements)
     {
-        if (!isset($statements['tables'])) {
+        if ( ! isset($statements['tables'])) {
             throw new Exception('No table specified', 3);
         }
 
@@ -387,7 +387,7 @@ 
         list($whereCriteria, $whereBindings) = $this->buildCriteriaWithType($statements, 'wheres', 'WHERE');
 
         // Limit
-        $limit = isset($statements['limit']) ? 'LIMIT ' . $statements['limit'] : '';
+        $limit = isset($statements['limit']) ? 'LIMIT '.$statements['limit'] : '';
 
         $sqlArray = ['DELETE FROM', $table, $whereCriteria];
         $sql      = $this->concatenateQuery($sqlArray);
@@ -409,11 +409,11 @@ 
     {
         $str = '';
         foreach ($pieces as $key => $piece) {
-            if (!is_int($key)) {
-                $piece = $key . ' AS ' . $piece;
+            if ( ! is_int($key)) {
+                $piece = $key.' AS '.$piece;
             }
 
-            $str .= $piece . $glue;
+            $str .= $piece.$glue;
         }
 
         return trim($str, $glue);
@@ -430,7 +430,7 @@ 
     {
         $str = '';
         foreach ($pieces as $piece) {
-            $str = trim($str) . ' ' . trim($piece);
+            $str = trim($str).' '.trim($piece);
         }
 
         return trim($str);
@@ -445,7 +445,7 @@ 
     public function getType($value): string
     {
         return $value instanceof Binding && $value->getType() !== null
-            ? $value->getType() : $this->inferType($value) ;
+            ? $value->getType() : $this->inferType($value);
     }
 
     /**
@@ -488,7 +488,7 @@ 
 
 
         $bindings = array_map([$this, 'getValue'], $bindings);
-        $query = $this->connection->getDbInstance()->prepare($query, $bindings) ;
+        $query = $this->connection->getDbInstance()->prepare($query, $bindings);
         return is_string($query) ? $query : '';
     }
 
@@ -529,10 +529,10 @@ 
                 // Merge the bindings we get from nestedCriteria object
                 $bindings = array_merge($bindings, $queryObject->getBindings());
                 // Append the sql we get from the nestedCriteria object
-                $criteria .= $statement['joiner'] . ' (' . $queryObject->getSql() . ') ';
+                $criteria .= $statement['joiner'].' ('.$queryObject->getSql().') ';
             } elseif (is_array($value)) {
                 // where_in or between like query
-                $criteria .= $statement['joiner'] . ' ' . $key . ' ' . $statement['operator'];
+                $criteria .= $statement['joiner'].' '.$key.' '.$statement['operator'];
 
                 switch ($statement['operator']) {
                     case 'BETWEEN':
@@ -565,7 +565,7 @@ 
                         }
 
                         $valuePlaceholder = trim($valuePlaceholder, ', ');
-                        $criteria .= ' (' . $valuePlaceholder . ') ';
+                        $criteria .= ' ('.$valuePlaceholder.') ';
                         break;
                 }
             } elseif ($value instanceof Raw) {
@@ -573,20 +573,20 @@ 
                 $criteria .= "{$statement['joiner']} {$key} {$statement['operator']} $value ";
             } else {
                 // Usual where like criteria
-                if (!$bindValues) {
+                if ( ! $bindValues) {
                     // Specially for joins
                     // We are not binding values, lets sanitize then
                     $value = $this->stringifyValue($this->wrapSanitizer($value)) ?? '';
-                    $criteria .= $statement['joiner'] . ' ' . $key . ' ' . $statement['operator'] . ' ' . $value . ' ';
+                    $criteria .= $statement['joiner'].' '.$key.' '.$statement['operator'].' '.$value.' ';
                 } elseif ($statement['key'] instanceof Raw) {
-                    $criteria .= $statement['joiner'] . ' ' . $key . ' ';
+                    $criteria .= $statement['joiner'].' '.$key.' ';
                     $bindings = array_merge($bindings, $statement['key']->getBindings());
                 } else {
                     // For wheres
                     $bindings[] = $this->getValue($value);
 
-                    $criteria .= $statement['joiner'] . ' ' . $key . ' ' . $statement['operator'] . ' '
-                    . $this->getType($value) . ' ';
+                    $criteria .= $statement['joiner'].' '.$key.' '.$statement['operator'].' '
+                    . $this->getType($value).' ';
                 }
             }
         }
@@ -641,7 +641,7 @@ 
 
         foreach ($valueArr as $key => $subValue) {
             // Don't wrap if we have *, which is not a usual field
-            $valueArr[$key] = '*' == trim($subValue) ? $subValue : $this->sanitizer . $subValue . $this->sanitizer;
+            $valueArr[$key] = '*' == trim($subValue) ? $subValue : $this->sanitizer.$subValue.$this->sanitizer;
         }
 
         // Join these back with "." and return
@@ -668,7 +668,7 @@ 
             list($criteria, $bindings) = $this->buildCriteria($statements[$key], $bindValues);
 
             if ($criteria) {
-                $criteria = $type . ' ' . $criteria;
+                $criteria = $type.' '.$criteria;
             }
         }
 
@@ -686,7 +686,7 @@ 
     {
         $sql = '';
 
-        if (!array_key_exists('joins', $statements) || !is_array($statements['joins'])) {
+        if ( ! array_key_exists('joins', $statements) || ! is_array($statements['joins'])) {
             return $sql;
         }
 
@@ -694,7 +694,7 @@ 
             if (is_array($joinArr['table'])) {
                 $mainTable  = $this->stringifyValue($this->wrapSanitizer($joinArr['table'][0]));
                 $aliasTable = $this->stringifyValue($this->wrapSanitizer($joinArr['table'][1]));
-                $table      = $mainTable . ' AS ' . $aliasTable;
+                $table      = $mainTable.' AS '.$aliasTable;
             } else {
                 $table = $joinArr['table'] instanceof Raw
                     ? $this->parseRaw($joinArr['table'])

src/QueryBuilder/QueryObject.php

Spacing +1 added lines, -1 removed lines

@@ -28,7 +28,7 @@
      */
     public function __construct(string $sql, array $bindings, wpdb $dbInstance)
     {
-        $this->sql        = (string)$sql;
+        $this->sql        = (string) $sql;
         $this->bindings   = $bindings;
         $this->dbInstance = $dbInstance;
     }

src/QueryBuilder/Raw.php

Spacing +3 added lines, -3 removed lines

@@ -20,8 +20,8 @@ 
      */
     public function __construct($value, $bindings = [])
     {
-        $this->value    = (string)$value;
-        $this->bindings = (array)$bindings;
+        $this->value    = (string) $value;
+        $this->bindings = (array) $bindings;
     }
 
     /**
@@ -49,6 +49,6 @@ 
      */
     public function __toString()
     {
-        return (string)$this->value;
+        return (string) $this->value;
     }
 }

src/EventHandler.php

Spacing +2 added lines, -2 removed lines

@@ -86,7 +86,7 @@ 
             // Fire before events for :any table
             if ($action = $this->getEvent($event, $table)) {
                 // Make an event id, with event type and table
-                $eventId = $event . $table;
+                $eventId = $event.$table;
 
                 // Fire event
                 $handlerParams = func_get_args();
@@ -94,7 +94,7 @@ 
                 // Add to fired list
                 $this->firedEvents[] = $eventId;
                 $result              = call_user_func_array($action, $handlerParams);
-                if (!is_null($result)) {
+                if ( ! is_null($result)) {
                     return $result;
                 }
             }

src/Hydration/Hydrator.php

Spacing +1 added lines, -1 removed lines

@@ -195,6 +195,6 @@
     {
         return $underscore
             ? "set_{$property}"
-            : 'set' . ucfirst($property);
+            : 'set'.ucfirst($property);
     }
 }

src/QueryBuilder/QueryBuilderHandler.php

Spacing +28 added lines, -28 removed lines

@@ -201,7 +201,7 @@ 
         $start        = microtime(true);
         $sqlStatement = empty($bindings) ? $sql : $this->interpolateQuery($sql, $bindings);
 
-        if (!is_string($sqlStatement)) {
+        if ( ! is_string($sqlStatement)) {
             throw new Exception('Could not interpolate query', 1);
         }
 
@@ -218,7 +218,7 @@ 
     public function get()
     {
         $eventResult = $this->fireEvents('before-select');
-        if (!is_null($eventResult)) {
+        if ( ! is_null($eventResult)) {
             return $eventResult;
         }
         $executionTime = 0;
@@ -243,7 +243,7 @@ 
         $this->sqlStatement = null;
 
         // Ensure we have an array of results.
-        if (!is_array($result) && null !== $result) {
+        if ( ! is_array($result) && null !== $result) {
             $result = [$result];
         }
 
@@ -276,7 +276,7 @@ 
      */
     protected function useHydrator(): bool
     {
-        return !in_array($this->getFetchMode(), [\ARRAY_A, \ARRAY_N, \OBJECT, \OBJECT_K]);
+        return ! in_array($this->getFetchMode(), [\ARRAY_A, \ARRAY_N, \OBJECT, \OBJECT_K]);
     }
 
     /**
@@ -366,7 +366,7 @@ 
             ->select([$this->raw(sprintf('%s(%s) AS field', strtoupper($type), $field))])
             ->first();
 
-        return true === isset($count->field) ? (float)$count->field : 0;
+        return true === isset($count->field) ? (float) $count->field : 0;
     }
 
     /**
@@ -382,7 +382,7 @@ 
      */
     public function count(string $field = '*'): int
     {
-        return (int)$this->aggregate('count', $field);
+        return (int) $this->aggregate('count', $field);
     }
 
     /**
@@ -460,8 +460,8 @@ 
     public function getQuery(string $type = 'select', $dataToBePassed = [])
     {
         $allowedTypes = ['select', 'insert', 'insertignore', 'replace', 'delete', 'update', 'criteriaonly'];
-        if (!in_array(strtolower($type), $allowedTypes)) {
-            throw new Exception($type . ' is not a known type.', 2);
+        if ( ! in_array(strtolower($type), $allowedTypes)) {
+            throw new Exception($type.' is not a known type.', 2);
         }
 
         $queryArr = $this->adapterInstance->$type($this->statements, $dataToBePassed);
@@ -480,9 +480,9 @@ 
      */
     public function subQuery(QueryBuilderHandler $queryBuilder, ?string $alias = null)
     {
-        $sql = '(' . $queryBuilder->getQuery()->getRawSql() . ')';
+        $sql = '('.$queryBuilder->getQuery()->getRawSql().')';
         if (is_string($alias) && 0 !== mb_strlen($alias)) {
-            $sql = $sql . ' as ' . $alias;
+            $sql = $sql.' as '.$alias;
         }
 
         return $queryBuilder->raw($sql);
@@ -499,12 +499,12 @@ 
     private function doInsert(array $data, string $type)
     {
         $eventResult = $this->fireEvents('before-insert');
-        if (!is_null($eventResult)) {
+        if ( ! is_null($eventResult)) {
             return $eventResult;
         }
 
         // If first value is not an array () not a batch insert)
-        if (!is_array(current($data))) {
+        if ( ! is_array(current($data))) {
             $queryObject = $this->getQuery($type, $data);
 
             list($preparedQuery, $executionTime) = $this->statement($queryObject->getSql(), $queryObject->getBindings());
@@ -574,7 +574,7 @@ 
     public function update($data)
     {
         $eventResult = $this->fireEvents('before-update');
-        if (!is_null($eventResult)) {
+        if ( ! is_null($eventResult)) {
             return $eventResult;
         }
         $queryObject                         = $this->getQuery('update', $data);
@@ -620,7 +620,7 @@ 
     public function delete(): int
     {
         $eventResult = $this->fireEvents('before-delete');
-        if (!is_null($eventResult)) {
+        if ( ! is_null($eventResult)) {
             return $eventResult;
         }
 
@@ -642,7 +642,7 @@ 
      */
     public function table(...$tables): QueryBuilderHandler
     {
-        $instance =  $this->constructCurrentBuilderClass($this->connection);
+        $instance = $this->constructCurrentBuilderClass($this->connection);
         $this->setFetchMode($this->getFetchMode(), $this->hydratorConstructorArgs);
         $tables = $this->addTablePrefix($tables, false);
         $instance->addStatement('tables', $tables);
@@ -670,7 +670,7 @@ 
      */
     public function select($fields): self
     {
-        if (!is_array($fields)) {
+        if ( ! is_array($fields)) {
             $fields = func_get_args();
         }
 
@@ -763,7 +763,7 @@ 
      */
     public function orderBy($fields, string $defaultDirection = 'ASC'): self
     {
-        if (!is_array($fields)) {
+        if ( ! is_array($fields)) {
             $fields = [$fields];
         }
 
@@ -774,7 +774,7 @@ 
                 $field = $value;
                 $type  = $defaultDirection;
             }
-            if (!$field instanceof Raw) {
+            if ( ! $field instanceof Raw) {
                 $field = $this->addTablePrefix($field);
             }
             $this->statements['orderBys'][] = compact('field', 'type');
@@ -1114,7 +1114,7 @@ 
             throw new Exception('Key used for whereNull condition must be a string or raw exrpession.', 1);
         }
 
-        return $this->{$operator . 'Where'}($this->raw("{$key} IS{$prefix} NULL"));
+        return $this->{$operator.'Where'}($this->raw("{$key} IS{$prefix} NULL"));
     }
 
     /**
@@ -1166,8 +1166,8 @@ 
      */
     public function join($table, $key, ?string $operator = null, $value = null, $type = 'inner')
     {
-        if (!$key instanceof Closure) {
-            $key = function ($joinBuilder) use ($key, $operator, $value) {
+        if ( ! $key instanceof Closure) {
+            $key = function($joinBuilder) use ($key, $operator, $value) {
                 $joinBuilder->on($key, $operator, $value);
             };
         }
@@ -1322,7 +1322,7 @@ 
      */
     public function joinUsing(string $table, string $key, string $type = 'INNER'): self
     {
-        if (!array_key_exists('tables', $this->statements) || count($this->statements['tables']) !== 1) {
+        if ( ! array_key_exists('tables', $this->statements) || count($this->statements['tables']) !== 1) {
             throw new Exception("JoinUsing can only be used with a single table set as the base of the query", 1);
         }
         $baseTable = end($this->statements['tables']);
@@ -1411,7 +1411,7 @@ 
 
         // If supplied value is not an array then make it one
         $single = false;
-        if (!is_array($values)) {
+        if ( ! is_array($values)) {
             $values = [$values];
             // We had single value, so should return a single value
             $single = true;
@@ -1429,20 +1429,20 @@ 
             // If key is not integer, it is likely a alias mapping,
             // so we need to change prefix target
             $target = &$value;
-            if (!is_int($key)) {
+            if ( ! is_int($key)) {
                 $target = &$key;
             }
 
             // Do prefix if the target is an expression or function.
             if (
-                !$tableFieldMix
+                ! $tableFieldMix
                 || (
                     is_string($target) // Must be a string
                     && (bool) preg_match('/^[A-Za-z0-9_.]+$/', $target) // Can only contain letters, numbers, underscore and full stops
                     && 1 === \substr_count($target, '.') // Contains a single full stop ONLY.
                 )
             ) {
-                $target = $this->tablePrefix . $target;
+                $target = $this->tablePrefix.$target;
             }
 
             $return[$key] = $value;
@@ -1460,11 +1460,11 @@ 
      */
     protected function addStatement($key, $value)
     {
-        if (!is_array($value)) {
+        if ( ! is_array($value)) {
             $value = [$value];
         }
 
-        if (!array_key_exists($key, $this->statements)) {
+        if ( ! array_key_exists($key, $this->statements)) {
             $this->statements[$key] = $value;
         } else {
             $this->statements[$key] = array_merge($this->statements[$key], $value);