fix: [OCI8] getFieldData() returns incorrect default value

system/Database/OCI8/Connection.php

@@ -315,11 +315,7 @@ protected function _fieldData(string $table): array
 
             $retval[$i]->max_length = $length;
 
-            $default = $query[$i]->DATA_DEFAULT;
-            if ($default === null && $query[$i]->NULLABLE === 'N') {
-                $default = '';
-            }
-            $retval[$i]->default  = $default;
+            $retval[$i]->default  = $query[$i]->DATA_DEFAULT;
             $retval[$i]->nullable = $query[$i]->NULLABLE === 'Y';
         }
 

tests/system/Database/Live/ForgeTest.php

@@ -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);
@@ -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);
@@ -1058,7 +1058,7 @@
                     'name'       => 'username',
                     'type'       => 'VARCHAR2',
                     'max_length' => '255',
-                    'default'    => '',
+                    'default'    => null,
                     'nullable'   => false,
                 ],
                 2 => [
@@ -1182,7 +1182,7 @@
         $this->forge->dropTable('forge_test_1', true);
     }
 
     public function testSetKeyNames(): void
     {
         $this->forge->dropTable('forge_test_1', 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);

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

@@ -0,0 +1,115 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * This file is part of CodeIgniter 4 framework.
+ *
+ * (c) CodeIgniter Foundation <admin@codeigniter.com>
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
+namespace CodeIgniter\Database\Live\OCI8;
+
+use CodeIgniter\Database\Live\AbstractGetFieldDataTest;
+use Config\Database;
+
+/**
+ * @group DatabaseLive
+ *
+ * @internal
+ */
+final class GetFieldDataTest extends AbstractGetFieldDataTest
+{
+    protected function createForge(): void
+    {
+        if ($this->db->DBDriver !== 'OCI8') {
+            $this->markTestSkipped('This test is only for OCI8.');
+        }
+
+        $this->forge = Database::forge($this->db);
+    }
+
+    public function testGetFieldData(): void
+    {
+        $fields = $this->db->getFieldData('test1');
+
+        $data = [];
+
+        foreach ($fields as $obj) {
+            $data[$obj->name] = $obj;
+        }
+
+        $idDefault = $data['id']->default;
+        $this->assertMatchesRegularExpression('/"ORACLE"."ISEQ\$\$_[0-9]+".nextval/', $idDefault);
+
+        $expected = json_decode(json_encode([
+            (object) [
+                'name'       => 'id',
+                'type'       => 'NUMBER',
+                'max_length' => '11',
+                'default'    => $idDefault, // The default value is not defined.
+                // 'primary_key' => 1,
+                'nullable' => false,
+            ],
+            (object) [
+                'name'       => 'text_not_null',
+                'type'       => 'VARCHAR2',
+                'max_length' => '64',
+                'default'    => null, // The default value is not defined.
+                // 'primary_key' => 0,
+                'nullable' => false,
+            ],
+            (object) [
+                'name'       => 'text_null',
+                'type'       => 'VARCHAR2',
+                'max_length' => '64',
+                'default'    => null, // The default value is not defined.
+                // 'primary_key' => 0,
+                'nullable' => true,
+            ],
+            (object) [
+                'name'       => 'int_default_0',
+                'type'       => 'NUMBER',
+                'max_length' => '11',
+                'default'    => '0 ', // int 0
+                // 'primary_key' => 0,
+                'nullable' => false,
+            ],
+            (object) [
+                'name'       => 'text_default_null',
+                'type'       => 'VARCHAR2',
+                'max_length' => '64',
+                'default'    => 'NULL ', // NULL value
+                // 'primary_key' => 0,
+                'nullable' => true,
+            ],
+            (object) [
+                'name'       => 'text_default_text_null',
+                'type'       => 'VARCHAR2',
+                'max_length' => '64',
+                'default'    => "'null' ", // string "null"
+                // 'primary_key' => 0,
+                'nullable' => false,
+            ],
+            (object) [
+                'name'       => 'text_default_abc',
+                'type'       => 'VARCHAR2',
+                'max_length' => '64',
+                'default'    => "'abc' ", // string "abc"
+                // 'primary_key' => 0,
+                'nullable' => false,
+            ],
+        ]), true);
+        $names = array_column($expected, 'name');
+        array_multisort($names, SORT_ASC, $expected);
+
+        $fields = json_decode(json_encode($fields), true);
+        $names  = array_column($fields, 'name');
+        array_multisort($names, SORT_ASC, $fields);
+
+        $this->assertSame($expected, $fields);
+    }
+}