HexMakina / Crudites

src/Row.php

Spacing +4 added lines, -4 removed lines

@@ -45,9 +45,9 @@ 
 
     public function __toString()
     {
-        return PHP_EOL . 'load: '
+        return PHP_EOL.'load: '
             . json_encode($this->load)
-            . PHP_EOL . 'alterations: '
+            . PHP_EOL.'alterations: '
             . json_encode(array_keys($this->alterations));
     }
 
@@ -134,7 +134,7 @@ 
     {
         foreach (array_keys($datass) as $field_name) {
 
-            if($datass[$field_name] === $this->get($field_name)) {
+            if ($datass[$field_name] === $this->get($field_name)) {
                 continue;
             }
             
@@ -150,7 +150,7 @@ 
             }
 
             // Replace empty strings with null if the column is nullable
-            if (trim('' . $datass[$field_name]) === '' && $attributes->nullable()) {
+            if (trim(''.$datass[$field_name]) === '' && $attributes->nullable()) {
                 $datass[$field_name] = null;
             }
 

src/Crudites.php

Spacing +2 added lines, -2 removed lines

@@ -61,7 +61,7 @@ 
         $res = self::$connection->result($select);
         $res = $res->ret(\PDO::FETCH_COLUMN);
         if (is_array($res)) {
-            return (int) current($res);
+            return (int)current($res);
         }
 
         return null;
@@ -78,7 +78,7 @@ 
         $ret = [];
 
         $res = self::$connection->result($select);
-        if($res->ran()) {
+        if ($res->ran()) {
 
             $primary_keys = self::$connection->schema()->primaryKeys($select->table());
             $pk_name = implode('_', $primary_keys);

src/Grammar/Query/Query.php

Spacing +4 added lines, -4 removed lines

@@ -37,7 +37,7 @@ 
         if (empty($this->bindings)) {
             unset($dbg['bindings']);
         }
-        else{
+        else {
             $dbg['bindings'] = json_encode($dbg['bindings']);
         }
 
@@ -98,11 +98,11 @@ 
 
     public function bindings(): array
     {
-        if(empty($this->clauses)){
+        if (empty($this->clauses)) {
             return $this->bindings;
         }
 
-        return array_merge($this->bindings, array_reduce($this->clauses, function ($carry, $clause) {
+        return array_merge($this->bindings, array_reduce($this->clauses, function($carry, $clause) {
             return array_merge($carry, $clause->bindings());
         }, []));
     }
@@ -118,7 +118,7 @@ 
 
     public function bindLabel($field, $table_name = null): string
     {
-        return ':' . $this->tableLabel($table_name) . '_' . $field;
+        return ':'.$this->tableLabel($table_name).'_'.$field;
     }
 
     public function addBindings($assoc_data, $binding_label = null): array

src/Grammar/Query/Insert.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
 
     public function values(array $dat_ass): self
     {
-        $this->binding_names []= $this->addBindings($dat_ass);
+        $this->binding_names [] = $this->addBindings($dat_ass);
 
         return $this;
     }
@@ -37,12 +37,12 @@ 
 
         $fields_labels = array_shift($this->binding_names);
         $fields = array_keys($fields_labels);
-        $fields = '`' . implode('`,`', $fields) . '`';
+        $fields = '`'.implode('`,`', $fields).'`';
 
         $bindings = implode(',:', $fields_labels);
 
         $statement = sprintf('INSERT INTO `%s` (%s) VALUES (:%s)', $this->table, $fields, $bindings);
-        foreach($this->binding_names as $bindings){
+        foreach ($this->binding_names as $bindings) {
             $statement .= sprintf(',(:%s)', implode(',:', $bindings));
         }
 

src/Grammar/Query/Select.php

Spacing +7 added lines, -7 removed lines

@@ -30,7 +30,7 @@ 
     {
         $schema = Grammar::identifier($this->table());
         if (!empty($this->alias())) {
-            $schema .= ' ' . Grammar::identifier($this->alias());
+            $schema .= ' '.Grammar::identifier($this->alias());
         }
 
         $ret = sprintf('SELECT %s FROM %s', $this->deck, $schema);
@@ -46,10 +46,10 @@ 
                 Clause::LIMIT
             ] as $clause
         ) {
-            if($this->clause($clause) === null)
+            if ($this->clause($clause) === null)
                 continue;
 
-            $ret .= ' ' . $this->clause($clause);
+            $ret .= ' '.$this->clause($clause);
         }
 
         return $ret;
@@ -86,7 +86,7 @@ 
             if (is_int($alias)) {
                 $alias = null;
             }
-            if(!isset($this->deck)){
+            if (!isset($this->deck)) {
                 $this->deck = new Deck($column, $alias);
             } else {
                 $this->deck->add($column, $alias);
@@ -103,15 +103,15 @@ 
         return $where;
     }
 
-    public function join(string $table, ?string $alias=null): Join
+    public function join(string $table, ?string $alias = null): Join
     {
         $join = new Join($table, $alias);
 
-        if($this->clause(Clause::JOINS) === null){
+        if ($this->clause(Clause::JOINS) === null) {
             $joins = new Joins([$join]);
             $this->add($joins);
         }
-        else{
+        else {
             $this->clause(Clause::JOINS)->add($join);
         }
 

Braces +7 added lines, -6 removed lines

@@ -46,8 +46,9 @@ 
                 Clause::LIMIT
             ] as $clause
         ) {
-            if($this->clause($clause) === null)
-                continue;
+            if($this->clause($clause) === null) {
+                            continue;
+            }
 
             $ret .= ' ' . $this->clause($clause);
         }
@@ -78,8 +79,9 @@ 
      */
     public function selectAlso(array $setter): self
     {
-        if (empty($setter))
-            throw new \InvalidArgumentException('EMPTY_SETTER_ARRAY');
+        if (empty($setter)) {
+                    throw new \InvalidArgumentException('EMPTY_SETTER_ARRAY');
+        }
 
         foreach ($setter as $alias => $column) {
 
@@ -110,8 +112,7 @@ 
         if($this->clause(Clause::JOINS) === null){
             $joins = new Joins([$join]);
             $this->add($joins);
-        }
-        else{
+        } else{
             $this->clause(Clause::JOINS)->add($join);
         }
 

src/Grammar/Clause/Where.php

Braces +5 added lines, -4 removed lines

@@ -14,10 +14,11 @@
 
         if ($predicates !== null) {
             foreach ($predicates as $predicate) {
-                if (is_string($predicate))
-                    $this->and($predicate);
-                else
-                    $this->and($predicate, $predicate->bindings());
+                if (is_string($predicate)) {
+                                    $this->and($predicate);
+                } else {
+                                    $this->and($predicate, $predicate->bindings());
+                }
             }
         }
     }

src/Grammar/Clause/Join.php

Spacing +2 added lines, -2 removed lines

@@ -52,7 +52,7 @@ 
 
     public function __toString(): string
     {
-        if(isset($this->column, $this->referenced_table, $this->referenced_column)) {
+        if (isset($this->column, $this->referenced_table, $this->referenced_column)) {
             $on = (string)(new Predicate([$this->alias, $this->column], '=', [$this->referenced_table, $this->referenced_column]));
         }
         
@@ -92,7 +92,7 @@ 
      */
     public function referenced(): ?array
     {
-        if(isset($this->referenced_table, $this->referenced_column)) {
+        if (isset($this->referenced_table, $this->referenced_column)) {
             return [$this->referenced_table, $this->referenced_column];
         }
 

src/Result.php

Braces +9 added lines, -6 removed lines

@@ -72,8 +72,9 @@ 
             $this->statement = $statement;
         }
 
-        if ($this->statement instanceof \PDOStatement)
-            $this->prepared = $this->statement;
+        if ($this->statement instanceof \PDOStatement) {
+                    $this->prepared = $this->statement;
+        }
 
         $this->run($bindings);
     }
@@ -97,8 +98,9 @@ 
             $pdo_cascade = $this->pdo;
         }
         // two time testing is necessary, f.i. lastInsertId is a PDO method, not a PDOStatement method
-        if ($pdo_cascade === null || !method_exists($pdo_cascade, $method))
-            throw new \BadMethodCallException("__call($method) not possible in PDO or PDOStatement");
+        if ($pdo_cascade === null || !method_exists($pdo_cascade, $method)) {
+                    throw new \BadMethodCallException("__call($method) not possible in PDO or PDOStatement");
+        }
 
         return call_user_func_array([$pdo_cascade, $method], $args);
     }
@@ -130,8 +132,9 @@ 
             throw new CruditesException('No executed statement available for fetching results');
         }
 
-        if ($mode === \PDO::FETCH_CLASS)
-            return $this->executed->fetchAll($mode, $fetch_argument, $ctor_args);
+        if ($mode === \PDO::FETCH_CLASS) {
+                    return $this->executed->fetchAll($mode, $fetch_argument, $ctor_args);
+        }
 
         return $this->executed->fetchAll($mode);
     }

tests/Query/SelectTest.php

Spacing +2 added lines, -2 removed lines

@@ -100,13 +100,13 @@
         $this->assertEquals($expected, (string)$query);
 
         $query->join('products', 'p')->on('product_id', 'oi', 'product_id');
-        $this->assertEquals($expected .' JOIN `products` `p` ON `p`.`product_id` = `oi`.`product_id`', (string)$query);
+        $this->assertEquals($expected.' JOIN `products` `p` ON `p`.`product_id` = `oi`.`product_id`', (string)$query);
         
         $query->selectAlso(['productName' => ['p', 'name']]);
         $expected = 'SELECT quantity,price AS `sold_price`,`p`.`name` AS `productName` FROM `order_items` `oi` JOIN `products` `p` ON `p`.`product_id` = `oi`.`product_id`';
         $this->assertEquals($expected, (string)$query);
 
-        $query->join('orders', 'o')->on('order_id','oi', 'order_id');
+        $query->join('orders', 'o')->on('order_id', 'oi', 'order_id');
         $expected .= ' JOIN `orders` `o` ON `o`.`order_id` = `oi`.`order_id`';
         $this->assertEquals($expected, (string)$query);