fix: add a primary key to an existing table

phpstan-baseline.php

@@ -1238,7 +1238,7 @@
 ];
 $ignoreErrors[] = [
 	'message' => '#^Construct empty\\(\\) is not allowed\\. Use more strict comparison\\.$#',
-	'count' => 16,
+	'count' => 13,
 	'path' => __DIR__ . '/system/Database/Forge.php',
 ];
 $ignoreErrors[] = [

system/Database/Forge.php

@@ -1073,7 +1073,7 @@ public function processIndexes(string $table): bool
         $sqls = [];
         $fk   = $this->foreignKeys;
 
-        if (empty($this->fields)) {
+        if ($this->fields === []) {
             $this->fields = array_flip(array_map(
                 static fn ($columnName) => $columnName->name,
                 $this->db->getFieldData($this->db->DBPrefix . $table)
@@ -1082,20 +1082,18 @@ public function processIndexes(string $table): bool
 
         $fields = $this->fields;
 
-        if (! empty($this->keys)) {
+        if ($this->keys !== []) {
             $sqls = $this->_processIndexes($this->db->DBPrefix . $table, true);
+        }
 
-            $pk = $this->_processPrimaryKeys($table, true);
-
-            if ($pk !== '') {
-                $sqls[] = $pk;
-            }
+        if ($this->primaryKeys !== []) {
+            $sqls[] = $this->_processPrimaryKeys($table, true);
         }
 
         $this->foreignKeys = $fk;
         $this->fields      = $fields;
 
-        if (! empty($this->foreignKeys)) {
+        if ($this->foreignKeys !== []) {
             $sqls = array_merge($sqls, $this->_processForeignKeys($table, true));
         }
 

tests/system/Database/Live/ForgeTest.php

@@ -57,7 +57,7 @@
         $this->assertTrue($databaseCreated);
     }
 
     public function testCreateDatabaseIfNotExists(): void
     {
         if ($this->db->DBDriver === 'OCI8') {
             $this->markTestSkipped('OCI8 does not support create database.');
@@ -72,7 +72,7 @@
         $this->assertTrue($databaseCreateIfNotExists);
     }
 
     public function testCreateDatabaseIfNotExistsWithDb(): void
     {
         if ($this->db->DBDriver === 'OCI8') {
             $this->markTestSkipped('OCI8 does not support create database.');
@@ -384,7 +384,7 @@
         $this->forge->createTable('forge_test_table');
     }
 
     public function testRenameTable(): void
     {
         $this->forge->dropTable('forge_test_table_dummy', true);
 
@@ -440,7 +440,7 @@
         $this->forge->dropTable('', true);
     }
 
     public function testForeignKey(): void
     {
         $this->forge->dropTable('forge_test_invoices', true);
         $this->forge->dropTable('forge_test_users', true);
@@ -544,7 +544,7 @@
     /**
      * @see https://github.com/codeigniter4/CodeIgniter4/issues/4310
      */
     public function testCompositeForeignKey(): void
     {
         $this->forge->dropTable('forge_test_invoices', true);
         $this->forge->dropTable('forge_test_users', true);
@@ -730,7 +730,7 @@
         $this->forge->createTable('forge_test_invoices', true, $attributes);
     }
 
     public function testDropForeignKey(): void
     {
         $this->forge->dropTable('forge_test_invoices', true);
         $this->forge->dropTable('forge_test_users', true);
@@ -1182,7 +1182,7 @@
         $this->forge->dropTable('forge_test_1', true);
     }
 
     public function testSetKeyNames(): void
     {
         $this->forge->dropTable('forge_test_1', true);
 
@@ -1318,7 +1318,7 @@
         $this->forge->dropTable('forge_test_three', true);
     }
 
     public function testModifyColumnNullTrue(): void
     {
         $this->forge->dropTable('forge_test_modify', true);
 
@@ -1430,7 +1430,7 @@
         }
     }
 
     public function testDropMultipleColumnWithArray(): void
     {
         $this->forge->dropTable('forge_test_two', true);
 
@@ -1506,7 +1506,7 @@
         $this->forge->dropTable('forge_test_four', true);
     }
 
     public function testDropKey(): void
     {
         $this->forge->dropTable('key_test_users', true);
         $keyName = 'key_test_users_id';
@@ -1593,7 +1593,7 @@
         $this->forge->dropTable('forge_test_users', true);
     }
 
     public function testProcessIndexes(): void
     {
         // make sure tables don't exist
         $this->forge->dropTable('actions', true);
@@ -1657,6 +1657,87 @@
         $this->forge->dropTable('user2', true);
     }
 
+    public function testProcessIndexesWithKeyOnly(): void
+    {
+        // make sure tables don't exist
+        $this->forge->dropTable('actions', true);
+
+        $this->createActionsTable();
+        $this->forge->addKey('name', false, false, 'db_actions_name');
+
+        // create indexes
+        $this->forge->processIndexes('actions');
+
+        // get a list of all indexes
+        $allIndexes = $this->db->getIndexData('actions');
+
+        // check that db_actions_name key exists
+        $indexes = array_filter(
+            $allIndexes,
+            static fn ($index) => ($index->name === 'db_actions_name')
+                && ($index->fields === [0 => 'name'])
+        );
+        $this->assertCount(1, $indexes);
+
+        // drop tables to avoid any future conflicts
+        $this->forge->dropTable('actions', true);
+    }
+
+    public function testProcessIndexesWithPrimaryKeyOnly(): void
+    {
+        // make sure tables don't exist
+        $this->forge->dropTable('actions', true);
+
+        $this->createActionsTable();
+        $this->forge->addPrimaryKey('id');
+
+        // create indexes
+        $this->forge->processIndexes('actions');
+
+        // get a list of all indexes
+        $allIndexes = $this->db->getIndexData('actions');
+
+        // check that the primary key exists
+        $indexes = array_filter(
+            $allIndexes,
+            static fn ($index) => $index->type === 'PRIMARY'
+        );
+        $this->assertCount(1, $indexes);
+
+        // drop tables to avoid any future conflicts
+        $this->forge->dropTable('actions', true);
+    }
+
+    public function testProcessIndexesWithForeignKeyOnly(): void
+    {
+        // make sure tables don't exist
+        $this->forge->dropTable('actions', true);
+        $this->forge->dropTable('user2', true);
+
+        $this->createUser2TableWithKeys();
+        $this->populateUser2Table();
+        $this->createActionsTable();
+
+        // SQLite does not support custom foreign key name
+        if ($this->db->DBDriver === 'SQLite3') {
+            $this->forge->addForeignKey('userid', 'user', 'id');
+            $this->forge->addForeignKey('userid2', 'user2', 'id');
+        } else {
+            $this->forge->addForeignKey('userid', 'user', 'id', '', '', 'db_actions_userid_foreign');
+            $this->forge->addForeignKey('userid2', 'user2', 'id', '', '', 'db_actions_userid2_foreign');
+        }
+
+        // create indexes
+        $this->forge->processIndexes('actions');
+
+        // check that the two foreign keys exist
+        $this->assertCount(2, $this->db->getForeignKeyData('actions'));
+
+        // drop tables to avoid any future conflicts
+        $this->forge->dropTable('actions', true);
+        $this->forge->dropTable('user2', true);
+    }
+
     private function createUser2TableWithKeys(): void
     {
         $fields = [

user_guide_src/source/changelogs/v4.4.2.rst

@@ -42,6 +42,8 @@ Bugs Fixed
   Page Not Found.
 - **Spark:** Fixed a bug that caused spark to not display exceptions in the
   production mode or to display backtrace in json when an exception occurred.
+- **Forge:** Fixed a bug where adding a Primary Key to an existing table was
+  ignored if there were no other Keys added too.
 
 See the repo's
 `CHANGELOG.md <https://github.com/codeigniter4/CodeIgniter4/blob/develop/CHANGELOG.md>`_