Always use ADD with COLUMN keyword in migrations for WP Playground

See: WordPress/sqlite-database-integration#145
mailpoet · Aug 6, 2024 · 8c1481f · 8c1481f
1 parent dcbdba1
commit 8c1481f
Show file tree

Hide file tree

Showing 5 changed files with 6 additions and 6 deletions.
diff --git a/mailpoet/lib/Migrations/Db/Migration_20221028_105818.php b/mailpoet/lib/Migrations/Db/Migration_20221028_105818.php
@@ -699,7 +699,7 @@ private function fixScheduledTasksSubscribersTimestampColumns() {
     if (empty($updatedAtColumnExists)) {
       $addUpdatedAtQuery = "
         ALTER TABLE `$scheduledTasksSubscribersTable`
-        ADD `updated_at` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP;
+        ADD COLUMN `updated_at` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP;
       ";
       $wpdb->query($addUpdatedAtQuery);
     }

diff --git a/mailpoet/lib/Migrations/Db/Migration_20230111_120000.php b/mailpoet/lib/Migrations/Db/Migration_20230111_120000.php
@@ -19,7 +19,7 @@ public function run(): void {
 
     $this->connection->executeStatement("
       ALTER TABLE {$segmentsTable}
-      ADD {$columnName} tinyint(1) NOT NULL DEFAULT 0
+      ADD COLUMN {$columnName} tinyint(1) NOT NULL DEFAULT 0
     ");
 
     $subscriptionSetting = $this->connection->fetchOne("

diff --git a/mailpoet/lib/Migrations/Db/Migration_20230111_130000.php b/mailpoet/lib/Migrations/Db/Migration_20230111_130000.php
@@ -11,7 +11,7 @@ public function run(): void {
     if (!$this->columnExists($tableName, 'method')) {
       $this->connection->executeStatement("
         ALTER TABLE {$tableName}
-        ADD method varchar(40) NOT NULL DEFAULT 'unknown'
+        ADD COLUMN method varchar(40) NOT NULL DEFAULT 'unknown'
       ");
     }
   }

diff --git a/mailpoet/lib/Migrations/Db/Migration_20230221_200520.php b/mailpoet/lib/Migrations/Db/Migration_20230221_200520.php
@@ -21,7 +21,7 @@ private function addRawMessagesToLogs() {
 
     $this->connection->executeStatement("
       ALTER TABLE {$logsTable}
-      ADD {$columnName} longtext DEFAULT NULL
+      ADD COLUMN {$columnName} longtext DEFAULT NULL
     ");
   }
 
@@ -35,7 +35,7 @@ private function addContextToLogs() {
 
     $this->connection->executeStatement("
       ALTER TABLE {$logsTable}
-      ADD {$columnName} longtext DEFAULT NULL
+      ADD COLUMN {$columnName} longtext DEFAULT NULL
     ");
   }
 }
diff --git a/mailpoet/lib/Migrations/Db/Migration_20230716_130221_Db.php b/mailpoet/lib/Migrations/Db/Migration_20230716_130221_Db.php
@@ -14,7 +14,7 @@ public function run(): void {
     if (!$this->columnExists($tableName, 'wp_post_id')) {
       $this->connection->executeStatement("
         ALTER TABLE {$tableName}
-        ADD wp_post_id int NULL
+        ADD COLUMN wp_post_id int NULL
       ");
     }