Merge pull request #359 from bowphp/refactor/code-base

Fix migration query builder

Author: papac

src/Database/Migration/Migration.php

@@ -203,10 +203,16 @@ final public function alter(string $table, callable $cb, bool $displayInfo = tru
             $generator = new Table($table, $this->adapter->getName(), 'alter')
         ]);
 
+        $sql_definition = $generator->make();
+
         if ($this->adapter->getName() === 'pgsql') {
-            $sql = sprintf('ALTER TABLE %s %s;', $table, $generator->make());
+            $sql = sprintf('ALTER TABLE %s %s;', $table, $sql_definition);
         } else {
-            $sql = sprintf('ALTER TABLE `%s` %s;', $table, $generator->make());
+            $sql = sprintf('ALTER TABLE `%s` %s;', $table, $sql_definition);
+        }
+
+        if (empty($sql_definition)) {
+            return $this;
         }
 
         return $this->executeSqlQuery($sql, $displayInfo);
@@ -218,12 +224,25 @@ final public function alter(string $table, callable $cb, bool $displayInfo = tru
      * @param  string $sql
      * @return Migration
      * @throws MigrationException
+     * @deprecated Use sql() instead.
      */
     final public function addSql(string $sql): Migration
     {
         return $this->executeSqlQuery($sql);
     }
 
+    /**
+     * Execute SQL query
+     *
+     * @param  string $sql
+     * @return Migration
+     * @throws MigrationException
+     */
+    final public function sql(string $sql): Migration
+    {
+        return $this->executeSqlQuery($sql);
+    }
+
     /**
      * Rename table
      *