tmukherjee13 / yii2-reverse-migration

views/_createTable.php

Braces +3 added lines, -1 removed lines

@@ -13,9 +13,11 @@
     if (empty($field['decorators'])): ?>
             '<?= $field['property'] ?>',
 
-<?php else: ?>
+<?php else {
+    : ?>
             <?= "'{$field['property']}' => \$this->{$field['decorators']}" ?>,
 <?php endif;
+}
 
 endforeach; ?>
 

console/controllers/MigrationController.php

Braces +20 added lines, -34 removed lines

@@ -10,8 +10,7 @@ 
 use yii\helpers\Console;
 use yii\helpers\FileHelper;
 
-class MigrationController extends MigrateController
-{
+class MigrationController extends MigrateController {
     use \tmukherjee13\migration\console\components\Formatter;
 
     /**
@@ -60,16 +59,14 @@ 
     /**
      * @inheritdoc
      */
-    public function options($actionID)
-    {
+    public function options($actionID) {
         return array_merge(parent::options($actionID), ['migrationTable', 'db']);
     }
 
     /**
      * @inheritdoc
      */
-    public function beforeAction($action)
-    {
+    public function beforeAction($action) {
 
         if (parent::beforeAction($action)) {
             if (is_string($this->db)) {
@@ -103,8 +100,7 @@ 
      * Collects the foreign key column details for the given table.
      * @param TableSchema $table the table metadata
      */
-    protected function findConstraints($table)
-    {
+    protected function findConstraints($table) {
         $sql = <<<SQL
             SELECT
                 kcu.constraint_name,
@@ -172,20 +168,19 @@ 
      * @param TableSchema $table the table metadata
      * @return string $sql the result of 'SHOW CREATE TABLE'
      */
-    protected function getCreateTableSql($table)
-    {
+    protected function getCreateTableSql($table) {
         $row = $this->db->createCommand('SHOW CREATE TABLE ' . $this->quoteTableName($table->fullName))->queryOne();
         if (isset($row['Create Table'])) {
             $sql = $row['Create Table'];
-        } else {
+        }
+        else {
             $row = array_values($row);
             $sql = $row[1];
         }
         return $sql;
     }
 
-    public function quoteTableName($name)
-    {
+    public function quoteTableName($name) {
         return strpos($name, '`') !== false ? $name : "`$name`";
     }
 
@@ -197,8 +192,7 @@ 
      * @author Tarun Mukherjee (https://github.com/tmukherjee13)
      */
 
-    public function actionTable(array $tables)
-    {
+    public function actionTable(array $tables) {
 
         if ($this->confirm('Create the migration ' . "?")) {
 
@@ -276,8 +270,7 @@ 
      * @param array $tables the list of tables
      * @return integer|null
      */
-    public function actionData(array $tables)
-    {
+    public function actionData(array $tables) {
 
         if ($this->confirm('Create the migration ' . "?", true)) {
             foreach ($tables as $key => $args) {
@@ -319,7 +312,8 @@ 
 
                     if (empty($prepared_data)) {
                         $this->stdout("\nTable '{$table->name}' doesn't contain any data.\n\n", Console::FG_RED);
-                    } else {
+                    }
+                    else {
                         $pcolumns = $this->prepareColumns($pcolumns);
                         $prows    = $this->prepareData($prepared_data);
 
@@ -339,8 +333,7 @@ 
      * @param string $args the schema name
      * @return integer
      */
-    public function actionSchema($args)
-    {
+    public function actionSchema($args) {
 
         $schema      = $args;
         $this->class = 'dump_database_' . $schema;
@@ -368,18 +361,15 @@ 
 
     }
 
-    public function getFileName()
-    {
+    public function getFileName() {
         return 'm' . gmdate('ymd_His') . '_' . $this->class;
     }
 
-    public function setFileName()
-    {
+    public function setFileName() {
         $this->fileName = $this->getFileName();
     }
 
-    public function prepareFile($data)
-    {
+    public function prepareFile($data) {
         $file = $this->migrationPath . DIRECTORY_SEPARATOR . $this->getFileName() . '.php';
         try {
 
@@ -399,8 +389,7 @@ 
     /**
      * @inheritdoc
      */
-    protected function getMigrationHistory($limit)
-    {
+    protected function getMigrationHistory($limit) {
         if ($this->db->schema->getTableSchema($this->migrationTable, true) === null) {
             $this->createMigrationHistoryTable();
         }
@@ -420,8 +409,7 @@ 
     /**
      * @inheritdoc
      */
-    protected function addMigrationHistory($version)
-    {
+    protected function addMigrationHistory($version) {
         $command = $this->db->createCommand();
         $command->insert($this->migrationTable, [
             'version'    => $version,
@@ -432,8 +420,7 @@ 
     /**
      * @inheritdoc
      */
-    protected function removeMigrationHistory($version)
-    {
+    protected function removeMigrationHistory($version) {
         $command = $this->db->createCommand();
         $command->delete($this->migrationTable, [
             'version' => $version,
@@ -443,8 +430,7 @@ 
     /**
      * @inheritdoc
      */
-    protected function parseFields()
-    {
+    protected function parseFields() {
         $fields = [];
 
         foreach ($this->fields as $column => $schema) {

console/components/Formatter.php

Braces +23 added lines, -25 removed lines

@@ -5,8 +5,7 @@ 
 /**
  * Table class
  */
-trait Formatter
-{
+trait Formatter {
 
     /**
      * @var array column enclosing literal
@@ -41,15 +40,13 @@ 
      * @author Tarun Mukherjee (https://github.com/tmukherjee13)
      */
 
-    public function getTableName($table)
-    {
+    public function getTableName($table) {
         return '{{%' . str_replace($this->db->tablePrefix, '', $table->name) . '}}';
     }
 
 
     
-    public function prepareInsert($rows, $columns)
-    {
+    public function prepareInsert($rows, $columns) {
 
         
         return '$this->batchInsert("{{%test}}", ' . $rows . ', ' . $columns . ');';
@@ -60,8 +57,7 @@ 
      * @param string $data the column string|$trim the literal to trim
      * @return string
      */
-    public function prepareColumns($data, $trim = ',')
-    {
+    public function prepareColumns($data, $trim = ',') {
         return self::columnFormat($data, $trim);
     }
 
@@ -70,8 +66,7 @@ 
      * @param array $data the data array
      * @return string
      */
-    public function prepareData($data = [])
-    {
+    public function prepareData($data = []) {
         self::$rows = '';
         foreach ($data as $row) {
             $rows = '';
@@ -91,8 +86,7 @@ 
      * @param string $data the column string|$trim the literal to trim
      * @return string
      */
-    public function dataFormat($data, $trim = ',')
-    {
+    public function dataFormat($data, $trim = ',') {
         if (null !== $trim) {
             $data = rtrim($data, $trim);
         }
@@ -104,8 +98,7 @@ 
      * @param string $data the column string|$trim the literal to trim
      * @return string
      */
-    public function columnFormat($data, $trim = ',')
-    {
+    public function columnFormat($data, $trim = ',') {
         if (null !== $trim) {
             $data = rtrim($data, $trim);
         }
@@ -122,8 +115,7 @@ 
      * @author Tarun Mukherjee (https://github.com/tmukherjee13)
      */
 
-    public function getColType($col)
-    {
+    public function getColType($col) {
 
         if ($col->isPrimaryKey && $col->autoIncrement) {
             return 'pk';
@@ -135,11 +127,14 @@ 
         }
         if ($col->defaultValue != null && 'timestamp' != $col->dbType) {
             $result .= " DEFAULT '{$col->defaultValue}'";
-        } elseif ($col->defaultValue == 'CURRENT_TIMESTAMP' && 'timestamp' == $col->dbType) {
+        }
+        elseif ($col->defaultValue == 'CURRENT_TIMESTAMP' && 'timestamp' == $col->dbType) {
             $result .= " DEFAULT {$col->defaultValue}";
-        } elseif ($col->defaultValue != null && 'timestamp' == $col->dbType) {
+        }
+        elseif ($col->defaultValue != null && 'timestamp' == $col->dbType) {
             $result .= " DEFAULT '{$col->defaultValue}'";
-        } elseif ($col->allowNull) {
+        }
+        elseif ($col->allowNull) {
             $result .= ' DEFAULT NULL';
         }
         return $result;
@@ -167,19 +162,22 @@ 
      * @author Tarun Mukherjee (https://github.com/tmukherjee13)
      */
 
-    public function formatCol($col)
-    {
+    public function formatCol($col) {
         $decorator = [];
         if ($col->isPrimaryKey && $col->autoIncrement) {
             $decorator[] = 'primaryKey';
-        } elseif (in_array($col->type, self::$colTypes)) {
+        }
+        elseif (in_array($col->type, self::$colTypes)) {
             $decorator[] = "{$this->modifyColType($col->dbType)}";
-        } elseif ($col->type == 'decimal') {
+        }
+        elseif ($col->type == 'decimal') {
             $decorator[] = "{$col->dbType}";
-        } else {
+        }
+        else {
             if (!empty($col->size) && $col->size == 1 && $col->type != 'char') {
                 $column = "boolean";
-            } else {
+            }
+            else {
                 $column = "{$col->type}";
                 if (!empty($col->size)) {
                     $column .= "({$col->size})";