hfw / db

src/DB/Select.php

Braces +6 added lines, -12 removed lines

@@ -132,8 +132,7 @@ 
         if ($table instanceof Select) {
             $this->_table = $table->toSubquery();
             $this->alias = "_anon{$autoAlias}_{$table->alias}";
-        }
-        else {
+        } else {
             if (is_string($table)) {
                 $table = $db[$table];
                 assert(isset($table));
@@ -265,8 +264,7 @@ 
     public function group (string $column) {
         if (!strlen($this->_group)) {
             $this->_group = " GROUP BY {$column}";
-        }
-        else {
+        } else {
             $this->_group .= ", {$column}";
         }
         return $this;
@@ -283,8 +281,7 @@ 
         $conditions = implode(' AND ', $conditions);
         if (!strlen($this->_having)) {
             $this->_having = " HAVING {$conditions}";
-        }
-        else {
+        } else {
             $this->_having .= " AND {$conditions}";
         }
         return $this;
@@ -376,8 +373,7 @@ 
     public function limit (int $limit, int $offset = 0) {
         if ($limit == 0) {
             $this->_limit = '';
-        }
-        else {
+        } else {
             $this->_limit = " LIMIT {$limit}";
             if ($offset > 1) {
                 $this->_limit .= " OFFSET {$offset}";
@@ -452,8 +448,7 @@ 
             $name = $match['name'] ?? null;
             if (is_int($alias)) {
                 $alias = $name;
-            }
-            elseif ($alias !== $name) {
+            } elseif ($alias !== $name) {
                 $expr .= " AS {$alias}";
             }
             if (isset($alias)) {
@@ -539,8 +534,7 @@ 
         $conditions = implode(' AND ', $conditions);
         if (!strlen($this->_where)) {
             $this->_where = " WHERE {$conditions}";
-        }
-        else {
+        } else {
             $this->_where .= " AND {$conditions}";
         }
         return $this;

src/DB/Junction.php

Braces +1 added lines, -2 removed lines

@@ -114,8 +114,7 @@
             $slots = implode(',', $this->db->slots(array_keys($this->columns)));
             if ($this->db->isSQLite()) {
                 $sql = "INSERT OR IGNORE INTO {$this} ({$columns}) VALUES ({$slots})";
-            }
-            else {
+            } else {
                 $sql = "INSERT IGNORE INTO {$this} ({$columns}) VALUES ({$slots})";
             }
             return $this->db->prepare($sql);

bin/helix.db.migrate.php

Braces +1 added lines, -10 removed lines

@@ -301,13 +301,4 @@
              */
             public function down (\$schema)
             {
-        {$down}
-            }
-
-        };
-
-        MIGRATION
-        );
-    }
-
-})->_exec();
+        {$down}
\ No newline at end of file

src/DB/Schema.php

Braces +1 added lines, -2 removed lines

@@ -356,8 +356,7 @@
             if ($this->db->isSQLite()) {
                 $info = $this->db->query("PRAGMA table_info({$name})")->fetchAll();
                 $cols = array_column($info, 'name');
-            }
-            else {
+            } else {
                 $cols = $this->db->query("SELECT column_name FROM information_schema.tables WHERE table_name = \"{$name}\"")->fetchAll(DB::FETCH_COLUMN);
             }
             if (!$cols) {

src/DB/AttributesTrait.php

Braces +2 added lines, -4 removed lines

@@ -63,12 +63,10 @@
             if (isset($value)) {
                 assert(is_scalar($value));
                 $this->attributes[$attr] = $value;
-            }
-            else {
+            } else {
                 $this->offsetUnset($attr);
             }
-        }
-        else {
+        } else {
             // appending must not be null.
             // even though missing numeric offsets would yield null when fetched individually,
             // getAttributes() would not have them.

src/DB/Record.php

Braces +3 added lines, -6 removed lines

@@ -147,8 +147,7 @@ 
             $doc = $rProp->getDocComment();
             if (preg_match(static::RX_IS_COLUMN, $doc)) {
                 $columns[] = $rProp->getName();
-            }
-            elseif (preg_match(static::RX_EAV, $doc, $eav)) {
+            } elseif (preg_match(static::RX_EAV, $doc, $eav)) {
                 preg_match(static::RX_EAV_VAR, $doc, $var);
                 $type = $var['type'] ?? 'string';
                 $type = static::SCALARS[$type] ?? 'string';
@@ -513,8 +512,7 @@ 
     public function save (EntityInterface $entity): int {
         if (!$entity->getId()) {
             $this->saveInsert($entity);
-        }
-        else {
+        } else {
             $this->saveUpdate($entity);
         }
         $this->saveEav($entity);
@@ -637,8 +635,7 @@ 
         foreach ($values as $name => $value) {
             if (isset($this->properties[$name])) {
                 $this->properties[$name]->setValue($entity, $this->setType($name, $value));
-            }
-            else {
+            } else {
                 // attempt to set unknown fields directly on the instance.
                 $entity->{$name} = $value;
             }

src/DB.php

Braces +4 added lines, -8 removed lines

@@ -166,8 +166,7 @@ 
 
         if ($this->isMySQL()) {
             $this->exec("SET time_zone = 'UTC'");
-        }
-        elseif ($this->isSQLite()) {
+        } elseif ($this->isSQLite()) {
             // polyfill sqlite functions
             $this->sqliteCreateFunctions([ // deterministic functions
                 // https://www.sqlite.org/lang_mathfunc.html
@@ -222,8 +221,7 @@ 
         if ($this->transactions === 0) {
             $this->logger->__invoke("BEGIN TRANSACTION");
             parent::beginTransaction();
-        }
-        else {
+        } else {
             $this->exec("SAVEPOINT SAVEPOINT_{$this->transactions}");
         }
         $this->transactions++;
@@ -242,8 +240,7 @@ 
         if ($this->transactions === 1) {
             $this->logger->__invoke("COMMIT TRANSACTION");
             parent::commit();
-        }
-        else {
+        } else {
             $savepoint = $this->transactions - 1;
             $this->exec("RELEASE SAVEPOINT SAVEPOINT_{$savepoint}");
         }
@@ -536,8 +533,7 @@ 
         if ($this->transactions === 1) {
             $this->logger->__invoke("ROLLBACK TRANSACTION");
             parent::rollBack();
-        }
-        else {
+        } else {
             $savepoint = $this->transactions - 1;
             $this->exec("ROLLBACK TO SAVEPOINT SAVEPOINT_{$savepoint}");
         }

src/DB/Fluent/Value/ComparisonTrait.php

Braces +1 added lines, -2 removed lines

@@ -64,8 +64,7 @@
         }
         if ($arg === null or is_bool($arg)) {
             $arg = ['' => 'NULL', 1 => 'TRUE', 0 => 'FALSE'][$arg];
-        }
-        else {
+        } else {
             $arg = $this->db->quote($arg);
         }
         if ($this->db->isMySQL()) {

src/DB/Fluent/Choice.php

Braces +3 added lines, -1 removed lines

@@ -85,8 +85,10 @@
      * @param string|ValueInterface $else
      * @return $this
      */
-    public function else ($else) {
+    public function else {
+        ($else) {
         $this->else = isset($else) ? $this->db->quote($else) : null;
+    }
         return $this;
     }