Fixed naming issue:

Nullable tax category ID migration now runs the correct script.

app/Database/Migrations/20250716170000_MissingConfigKeys.php

@@ -11,12 +11,22 @@ class Migration_MissingConfigKeys extends Migration
      */
     public function up(): void
     {
-        error_log('Migrating config table');
+        error_log('Starting transaction...');
+        $db = db_connect();
+        $db->transStart();
 
         helper('migration');
-        execute_script(APPPATH . 'Database/Migrations/sqlscripts/3.4.2_missing_config_keys.sql');
 
-        error_log('Migrating config table');
+        // execute_script returns whether everything executed successfully
+        if (execute_script(APPPATH . 'Database/Migrations/sqlscripts/3.4.2_missing_config_keys.sql')) {
+            error_log('Migrated config table.');
+        }
+        else {
+            error_log('Failed to migrate config table.');
+        }
+
+        error_log('Transaction completed.');
+        $db->transComplete();
     }
 
     /**

app/Database/Migrations/20250729170000_NullableTaxCategoryId.php

@@ -11,12 +11,12 @@ class Migration_NullableTaxCategoryId extends Migration
      */
     public function up(): void
     {
-        error_log('Migrating config table');
+        error_log('Migrating nullable tax category ID');
 
         helper('migration');
-        execute_script(APPPATH . 'Database/Migrations/sqlscripts/3.4.2_missing_config_keys.sql');
+        execute_script(APPPATH . 'Database/Migrations/sqlscripts/3.4.2_nullable_tax_category_id.sql');
 
-        error_log('Migrating config table');
+        error_log('Migrated nullable tax category ID');
     }
 
     /**

app/Helpers/migration_helper.php

@@ -3,9 +3,11 @@
 use Config\Database;
 
 /**
- * Migration helper
+ * Migration helper.
+ * @param string $path Path to migration script.
+ * @return bool Whether the migration executed successfully.
  */
-function execute_script(string $path): void
+function execute_script(string $path): bool
 {
     $version = preg_replace("/(.*_)?(.*).sql/", "$2", $path);
     error_log("Migrating to $version (file: $path)");
@@ -16,17 +18,27 @@ function execute_script(string $path): void
 
     $db = Database::connect();
 
+    $success = true; // whether *all* queries succeeded
     foreach ($sqls as $statement) {
         $statement = "$statement;";
+        $hadError = !$db->simpleQuery($statement);
 
-        if (!$db->simpleQuery($statement)) {
+        if ($hadError) {
+            $success = false;
             foreach ($db->error() as $error) {
                 error_log("error: $error");
             }
         }
     }
 
-    error_log("Migrated to $version");
+    if ($success) {
+        error_log("Successfully migrated to $version");
+    }
+    else {
+        error_log("Could not migrate to $version.");
+    }
+
+    return $success;
 }
 
 /**
@@ -228,9 +240,9 @@ function dropColumnIfExists(string $table, string $column): void
 
     // Check if the column exists in the table
     $builder->select('COLUMN_NAME')
-        ->where('TABLE_SCHEMA', $db->database)
-        ->where('TABLE_NAME', $prefix . $table)
-        ->where('COLUMN_NAME', $column);
+            ->where('TABLE_SCHEMA', $db->database)
+            ->where('TABLE_NAME', $prefix . $table)
+            ->where('COLUMN_NAME', $column);
 
     $query = $builder->get();