fix: primary_key value

Author: kenjis

system/Database/SQLite3/Connection.php

@@ -269,11 +269,14 @@ protected function _fieldData(string $table): array
         for ($i = 0, $c = count($query); $i < $c; $i++) {
             $retVal[$i] = new stdClass();
 
-            $retVal[$i]->name        = $query[$i]->name;
-            $retVal[$i]->type        = $query[$i]->type;
-            $retVal[$i]->max_length  = null;
-            $retVal[$i]->default     = $query[$i]->dflt_value;
-            $retVal[$i]->primary_key = $query[$i]->pk ?? false;
+            $retVal[$i]->name       = $query[$i]->name;
+            $retVal[$i]->type       = $query[$i]->type;
+            $retVal[$i]->max_length = null;
+            $retVal[$i]->default    = $query[$i]->dflt_value;
+            // "pk" (either zero for columns that are not part of the primary key,
+            // or the 1-based index of the column within the primary key).
+            // https://www.sqlite.org/pragma.html#pragma_table_info
+            $retVal[$i]->primary_key = ($query[$i]->pk === 0) ? 0 : 1;
             $retVal[$i]->nullable    = isset($query[$i]->notnull) && ! (bool) $query[$i]->notnull;
         }
 

tests/system/Database/Live/SQLite3/GetFieldDataTest.php

@@ -102,4 +102,63 @@ public function testGetFieldData(): void
             json_encode($fields)
         );
     }
+
+    protected function createTableCompositePrimaryKey()
+    {
+        $this->forge->dropTable('test1', true);
+
+        $this->forge->addField([
+            'pk1' => [
+                'type'       => 'VARCHAR',
+                'constraint' => 64,
+            ],
+            'pk2' => [
+                'type'       => 'VARCHAR',
+                'constraint' => 64,
+            ],
+            'text' => [
+                'type'       => 'VARCHAR',
+                'constraint' => 64,
+            ],
+        ]);
+        $this->forge->addPrimaryKey(['pk1', 'pk2']);
+        $this->forge->createTable('test1');
+    }
+
+    public function testGetFieldDataCompositePrimaryKey(): void
+    {
+        $this->createTableCompositePrimaryKey();
+
+        $fields = $this->db->getFieldData('test1');
+
+        $this->assertJsonStringEqualsJsonString(
+            json_encode([
+                (object) [
+                    'name'        => 'pk1',
+                    'type'        => 'VARCHAR',
+                    'max_length'  => null,
+                    'default'     => null,
+                    'primary_key' => 1,
+                    'nullable'    => false,
+                ],
+                (object) [
+                    'name'        => 'pk2',
+                    'type'        => 'VARCHAR',
+                    'max_length'  => null,
+                    'default'     => null,
+                    'primary_key' => 1,
+                    'nullable'    => false,
+                ],
+                (object) [
+                    'name'        => 'text',
+                    'type'        => 'VARCHAR',
+                    'max_length'  => null,
+                    'default'     => null,
+                    'primary_key' => 0,
+                    'nullable'    => false,
+                ],
+            ]),
+            json_encode($fields)
+        );
+    }
 }