refactor: remove deprecations in Database

paulbalandan · paulbalandan · commit 847e72dc5c42 · 2026-02-23T22:09:01.000+08:00
diff --git a/system/Database/BaseBuilder.php b/system/Database/BaseBuilder.php
@@ -2260,24 +2260,6 @@ protected function _insertBatch(string $table, array $keys, array $values): stri
         return str_replace('{:_table_:}', $data, $sql);
     }
 
-    /**
-     * Allows key/value pairs to be set for batch inserts
-     *
-     * @param mixed $key
-     *
-     * @return $this|null
-     *
-     * @deprecated
-     */
-    public function setInsertBatch($key, string $value = '', ?bool $escape = null)
-    {
-        if (! is_array($key)) {
-            $key = [[$key => $value]];
-        }
-
-        return $this->setData($key, $escape);
-    }
-
     /**
      * Compiles an insert query and returns the sql
      *
@@ -2709,28 +2691,6 @@ protected function _updateBatch(string $table, array $keys, array $values): stri
         return str_replace('{:_table_:}', $data, $sql);
     }
 
-    /**
-     * Allows key/value pairs to be set for batch updating
-     *
-     * @param array|object $key
-     *
-     * @return $this
-     *
-     * @throws DatabaseException
-     *
-     * @deprecated
-     */
-    public function setUpdateBatch($key, string $index = '', ?bool $escape = null)
-    {
-        if ($index !== '') {
-            $this->onConstraint($index);
-        }
-
-        $this->setData($key, $escape);
-
-        return $this;
-    }
-
     /**
      * Compiles a delete string and runs "DELETE FROM table"
      *
@@ -3550,18 +3510,6 @@ protected function setBind(string $key, $value = null, bool $escape = true): str
         return $key . '.' . $count;
     }
 
-    /**
-     * Returns a clone of a Base Builder with reset query builder values.
-     *
-     * @return $this
-     *
-     * @deprecated
-     */
-    protected function cleanClone()
-    {
-        return (clone $this)->from([], true)->resetQuery();
-    }
-
     /**
      * @param mixed $value
      */
diff --git a/system/Database/BaseConnection.php b/system/Database/BaseConnection.php
@@ -193,17 +193,6 @@ abstract class BaseConnection implements ConnectionInterface
      */
     protected $compress = false;
 
-    /**
-     * Strict ON flag
-     *
-     * Whether we're running in strict SQL mode.
-     *
-     * @var bool|null
-     *
-     * @deprecated 4.5.0 Will move to MySQLi\Connection.
-     */
-    protected $strictOn;
-
     /**
      * Settings for a failover connection.
      *
diff --git a/system/Database/Forge.php b/system/Database/Forge.php
@@ -108,15 +108,6 @@ class Forge
      */
     protected $createTableStr = "%s %s (%s\n)";
 
-    /**
-     * CREATE TABLE IF statement
-     *
-     * @var bool|string
-     *
-     * @deprecated This is no longer used.
-     */
-    protected $createTableIfStr = 'CREATE TABLE IF NOT EXISTS';
-
     /**
      * CREATE TABLE keys flag
      *
@@ -565,7 +556,7 @@ public function createTable(string $table, bool $ifNotExists = false, array $att
             return true;
         }
 
-        $sql = $this->_createTable($table, false, $attributes);
+        $sql = $this->_createTable($table, $attributes);
 
         if (($result = $this->db->query($sql)) !== false) {
             if (isset($this->db->dataCache['table_names']) && ! in_array($table, $this->db->dataCache['table_names'], true)) {
@@ -586,13 +577,11 @@ public function createTable(string $table, bool $ifNotExists = false, array $att
     }
 
     /**
-     * @param array $attributes Table attributes
+     * @param array<string, mixed> $attributes Table attributes
      *
      * @return string SQL string
-     *
-     * @deprecated $ifNotExists is no longer used, and will be removed.
      */
-    protected function _createTable(string $table, bool $ifNotExists, array $attributes)
+    protected function _createTable(string $table, array $attributes)
     {
         $processedFields = $this->_processFields(true);
 
diff --git a/system/Database/OCI8/Forge.php b/system/Database/OCI8/Forge.php
@@ -34,15 +34,6 @@ class Forge extends BaseForge
      */
     protected $createDatabaseStr = false;
 
-    /**
-     * CREATE TABLE IF statement
-     *
-     * @var false
-     *
-     * @deprecated This is no longer used.
-     */
-    protected $createTableIfStr = false;
-
     /**
      * DROP TABLE IF EXISTS statement
      *
diff --git a/system/Database/SQLSRV/Connection.php b/system/Database/SQLSRV/Connection.php
@@ -537,40 +537,6 @@ protected function execute(string $sql)
         return $stmt;
     }
 
-    /**
-     * Returns the last error encountered by this connection.
-     *
-     * @return array<string, int|string>
-     *
-     * @deprecated Use `error()` instead.
-     */
-    public function getError()
-    {
-        $error = [
-            'code'    => '00000',
-            'message' => '',
-        ];
-
-        $sqlsrvErrors = sqlsrv_errors(SQLSRV_ERR_ERRORS);
-
-        if (! is_array($sqlsrvErrors)) {
-            return $error;
-        }
-
-        $sqlsrvError = array_shift($sqlsrvErrors);
-        if (isset($sqlsrvError['SQLSTATE'])) {
-            $error['code'] = isset($sqlsrvError['code']) ? $sqlsrvError['SQLSTATE'] . '/' . $sqlsrvError['code'] : $sqlsrvError['SQLSTATE'];
-        } elseif (isset($sqlsrvError['code'])) {
-            $error['code'] = $sqlsrvError['code'];
-        }
-
-        if (isset($sqlsrvError['message'])) {
-            $error['message'] = $sqlsrvError['message'];
-        }
-
-        return $error;
-    }
-
     /**
      * The name of the platform in use (MySQLi, mssql, etc)
      */
diff --git a/system/Database/SQLSRV/Forge.php b/system/Database/SQLSRV/Forge.php
@@ -94,15 +94,6 @@ class Forge extends BaseForge
      */
     protected $fkAllowActions = ['CASCADE', 'SET NULL', 'NO ACTION', 'RESTRICT', 'SET DEFAULT'];
 
-    /**
-     * CREATE TABLE IF statement
-     *
-     * @var string
-     *
-     * @deprecated This is no longer used.
-     */
-    protected $createTableIfStr;
-
     /**
      * CREATE TABLE statement
      *
diff --git a/system/Database/Seeder.php b/system/Database/Seeder.php
@@ -16,8 +16,6 @@
 use CodeIgniter\CLI\CLI;
 use CodeIgniter\Exceptions\InvalidArgumentException;
 use Config\Database;
-use Faker\Factory;
-use Faker\Generator;
 
 /**
  * Class Seeder
@@ -66,13 +64,6 @@ class Seeder
      */
     protected $silent = false;
 
-    /**
-     * Faker Generator instance.
-     *
-     * @deprecated
-     */
-    private static ?Generator $faker = null;
-
     /**
      * Seeder constructor.
      */
@@ -104,20 +95,6 @@ public function __construct(Database $config, ?BaseConnection $db = null)
         }
     }
 
-    /**
-     * Gets the Faker Generator instance.
-     *
-     * @deprecated
-     */
-    public static function faker(): ?Generator
-    {
-        if (! self::$faker instanceof Generator && class_exists(Factory::class)) {
-            self::$faker = Factory::create();
-        }
-
-        return self::$faker;
-    }
-
     /**
      * Loads the specified seeder and runs it.
      *
diff --git a/tests/_support/Database/Seeds/CITestSeeder.php b/tests/_support/Database/Seeds/CITestSeeder.php
@@ -185,6 +185,8 @@ public function run(): void
         }
 
         if ($this->db->DBDriver === 'MySQLi') {
+            $data['type_test'][0]['type_time'] = '15:22:00';
+
             $data['ci_sessions'][] = [
                 'id'         => 'ci_session:1f5o06b43phsnnf8if6bo33b635e4p2o',
                 'ip_address' => '127.0.0.1',
diff --git a/tests/system/Database/BaseConnectionTest.php b/tests/system/Database/BaseConnectionTest.php
@@ -84,7 +84,6 @@ public function testSavesConfigOptions(): void
         $this->assertSame('', $db->swapPre);
         $this->assertFalse($db->encrypt);
         $this->assertFalse($db->compress);
-        $this->assertTrue($db->strictOn);
         $this->assertSame([], $db->failover);
         $this->assertSame([
             'date'        => 'Y-m-d',
diff --git a/tests/system/Database/Builder/UpdateTest.php b/tests/system/Database/Builder/UpdateTest.php
@@ -253,9 +253,7 @@ public function testUpdateBatch(): void
     public function testSetUpdateBatchWithoutEscape(): void
     {
         $builder = new BaseBuilder('jobs', $this->db);
-        $escape  = false;
-
-        $builder->setUpdateBatch([
+        $builder->setData([
             [
                 'id'          => 2,
                 'name'        => 'SUBSTRING(name, 1)',
@@ -266,7 +264,7 @@ public function testSetUpdateBatchWithoutEscape(): void
                 'name'        => 'SUBSTRING(name, 2)',
                 'description' => 'SUBSTRING(description, 4)',
             ],
-        ], 'id', $escape);
+        ], false);
 
         $this->db->shouldReturn('execute', new class () {});
         $builder->updateBatch(null, 'id');
diff --git a/tests/system/Database/ConfigTest.php b/tests/system/Database/ConfigTest.php
@@ -143,7 +143,6 @@ public function testConnectionGroupWithDSN(): void
         $this->assertTrue($this->getPrivateProperty($conn, 'pConnect'));
         $this->assertSame('latin1', $this->getPrivateProperty($conn, 'charset'));
         $this->assertSame('latin1_swedish_ci', $this->getPrivateProperty($conn, 'DBCollat'));
-        $this->assertTrue($this->getPrivateProperty($conn, 'strictOn'));
         $this->assertSame([], $this->getPrivateProperty($conn, 'failover'));
     }
 
@@ -163,7 +162,6 @@ public function testConnectionGroupWithDSNPostgre(): void
         $this->assertFalse($this->getPrivateProperty($conn, 'pConnect'));
         $this->assertSame('utf8mb4', $this->getPrivateProperty($conn, 'charset'));
         $this->assertSame('utf8mb4_general_ci', $this->getPrivateProperty($conn, 'DBCollat'));
-        $this->assertTrue($this->getPrivateProperty($conn, 'strictOn'));
         $this->assertSame([], $this->getPrivateProperty($conn, 'failover'));
         $this->assertSame('5', $this->getPrivateProperty($conn, 'connect_timeout'));
         $this->assertSame('1', $this->getPrivateProperty($conn, 'sslmode'));
@@ -191,7 +189,6 @@ public function testConnectionGroupWithDSNPostgreNative(): void
         $this->assertFalse($this->getPrivateProperty($conn, 'pConnect'));
         $this->assertSame('utf8mb4', $this->getPrivateProperty($conn, 'charset'));
         $this->assertSame('utf8mb4_general_ci', $this->getPrivateProperty($conn, 'DBCollat'));
-        $this->assertTrue($this->getPrivateProperty($conn, 'strictOn'));
         $this->assertSame([], $this->getPrivateProperty($conn, 'failover'));
     }
 
diff --git a/tests/system/Database/DatabaseSeederTest.php b/tests/system/Database/DatabaseSeederTest.php
@@ -15,7 +15,6 @@
 
 use CodeIgniter\Test\CIUnitTestCase;
 use Config\Database;
-use Faker\Generator;
 use PHPUnit\Framework\Attributes\Group;
 use Tests\Support\Database\Seeds\SeederWithDBGroup;
 use Tests\Support\Database\Seeds\SeederWithoutDBGroup;
@@ -52,14 +51,6 @@ public function testInstantiateNotDirSeedPath(): void
         new Seeder($config);
     }
 
-    /**
-     * @TODO remove this when Seeder::faker() is removed
-     */
-    public function testFakerGet(): void
-    {
-        $this->assertInstanceOf(Generator::class, Seeder::faker());
-    }
-
     public function testCallOnEmptySeeder(): void
     {
         $this->expectException('InvalidArgumentException');
diff --git a/tests/system/Database/Live/ForgeTest.php b/tests/system/Database/Live/ForgeTest.php
@@ -373,7 +373,7 @@ public function testCreateTableWithNullableFieldsGivesNullDataType(): void
 
         $createTable = self::getPrivateMethodInvoker($this->forge, '_createTable');
 
-        $sql = $createTable('forge_nullable_table', false, []);
+        $sql = $createTable('forge_nullable_table', []);
 
         if ($this->db->DBDriver !== 'SQLSRV') {
             // @see https://regex101.com/r/bIHVNw/1
@@ -1621,6 +1621,10 @@ public function testAddTextColumnWithConstraint(): void
             $this->forge->addColumn('user', [
                 'text_with_constraint' => ['type' => 'nvarchar(max)', 'default' => ''],
             ]);
+        } elseif (in_array($this->db->DBDriver, ['MySQLi', 'Postgre', 'SQLite3'], true)) {
+            $this->forge->addColumn('user', [
+                'text_with_constraint' => ['type' => 'text'],
+            ]);
         } else {
             $this->forge->addColumn('user', [
                 'text_with_constraint' => ['type' => 'text', 'constraint' => 255, 'default' => ''],
diff --git a/tests/system/Database/Live/MySQLi/RawSqlTest.php b/tests/system/Database/Live/MySQLi/RawSqlTest.php
diff --git a/user_guide_src/source/changelogs/v4.8.0.rst b/user_guide_src/source/changelogs/v4.8.0.rst
diff --git a/utils/phpstan-baseline/loader.neon b/utils/phpstan-baseline/loader.neon
diff --git a/utils/phpstan-baseline/missingType.iterableValue.neon b/utils/phpstan-baseline/missingType.iterableValue.neon
diff --git a/utils/phpstan-baseline/property.phpDocType.neon b/utils/phpstan-baseline/property.phpDocType.neon

Original file line number	Diff line number	Diff line change
`@@ -185,6 +185,8 @@ public function run(): void`
`185`	`185`	`}`
`186`	`186`
`187`	`187`	`if ($this->db->DBDriver === 'MySQLi') {`
	`188`	`+ $data['type_test'][0]['type_time'] = '15:22:00';`
	`189`	`+`
`188`	`190`	`$data['ci_sessions'][] = [`
`189`	`191`	`'id' => 'ci_session:1f5o06b43phsnnf8if6bo33b635e4p2o',`
`190`	`192`	`'ip_address' => '127.0.0.1',`