Merge pull request #23 from sartor/master

Invalid date (1970-01-01) fix in migration table
bashkarev · Sep 10, 2020 · f56d6a2 · f56d6a2
2 parents 4ac95e4 + 57e6ec0
commit f56d6a2
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 18 deletions.
diff --git a/console/controllers/MigrateController.php b/console/controllers/MigrateController.php
@@ -122,25 +122,25 @@ protected function getMigrationHistory($limit)
      */
     protected function addMigrationHistory($version)
     {
-        $command = $this->db->createCommand();
-        $command->insert($this->migrationTable, [
-            'version' => $version,
-            'apply_time' => time(),
-            'is_deleted' => 0
-        ])->execute();
+        $this->insertHistory($version);
     }
 
     /**
      * @inheritdoc
      */
     protected function removeMigrationHistory($version)
     {
-        $command = $this->db->createCommand();
-        $command->insert($this->migrationTable, [
+        $this->insertHistory($version, true);
+    }
+
+    private function insertHistory(string $version, bool $isDelete = false): void
+    {
+        $time = time();
+        $this->db->createCommand()->insert($this->migrationTable, [
             'version' => $version,
-            'apply_time' => time(),
-            'is_deleted' => 1
+            'date' => date('Y-m-d', $time),
+            'apply_time' => $time,
+            'is_deleted' => (int) $isDelete
         ])->execute();
     }
-
 }
diff --git a/tests/SchemaBuilderTest.php b/tests/SchemaBuilderTest.php
@@ -229,9 +229,9 @@ public function testDefaultExpression()
         $this->assertEquals(1, $createResult);
 
         $expected = [
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST(100, 'Int32')"],
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST(100, 'Nullable(UInt32)')"],
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST('qweasdzxc1', 'Nullable(FixedString(10))')"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "100"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "100"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "'qweasdzxc1'"],
             ['default_kind' => 'DEFAULT', 'default_expression' => "'str1'"],
             ['default_kind' => 'DEFAULT', 'default_expression' => "concat('str1', 'str2')"],
             ['default_kind' => 'DEFAULT', 'default_expression' => "today()"],
@@ -258,10 +258,10 @@ public function testDefaultValue()
         $this->assertEquals(1, $createResult);
 
         $expected = [
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST('100', 'Int32')"],
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST(100, 'Int32')"],
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST(100, 'Nullable(UInt32)')"],
-            ['default_kind' => 'DEFAULT', 'default_expression' => "CAST('qweasdzxc1', 'Nullable(FixedString(10))')"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "'100'"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "100"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "100"],
+            ['default_kind' => 'DEFAULT', 'default_expression' => "'qweasdzxc1'"],
             ['default_kind' => 'DEFAULT', 'default_expression' => "'str1'"],
         ];