Resolve deprecated methods (#287)

* Resolve deprecated methods

* Add line to CHANGELOG.md

* Fix test

* Fix psalm issues

---------

Co-authored-by: Sergei Predvoditelev <sergei@predvoditelev.ru>

Author: Tigrov

CHANGELOG.md

@@ -6,6 +6,7 @@
 - Enh #282: Change property `Schema::$typeMap` to constant `Schema::TYPE_MAP` (@Tigrov)
 - Enh #283: Remove unnecessary check for array type in `Schema::loadTableIndexes()` (@Tigrov)
 - Enh #288: Minor refactoring of `DDLQueryBuilder` and `Schema` (@Tigrov)
+- Enh #287: Resolve deprecated methods (@Tigrov) 
 
 ## 1.1.0 November 12, 2023
 

src/Schema.php

@@ -21,6 +21,7 @@
 use Yiisoft\Db\Schema\TableSchemaInterface;
 
 use function array_column;
+use function array_map;
 use function array_merge;
 use function count;
 use function explode;
@@ -201,7 +202,7 @@ protected function loadTableForeignKeys(string $tableName): array
 
         $foreignKeysList = $this->getPragmaForeignKeyList($tableName);
         /** @psalm-var ForeignKeyInfo[] $foreignKeysList */
-        $foreignKeysList = $this->normalizeRowKeyCase($foreignKeysList, true);
+        $foreignKeysList = array_map('array_change_key_case', $foreignKeysList);
         $foreignKeysList = DbArrayHelper::index($foreignKeysList, null, ['table']);
         DbArrayHelper::multisort($foreignKeysList, 'seq');
 
@@ -553,7 +554,7 @@ private function loadTableColumnsInfo(string $tableName): array
     {
         $tableColumns = $this->getPragmaTableInfo($tableName);
         /** @psalm-var ColumnInfo[] $tableColumns */
-        $tableColumns = $this->normalizeRowKeyCase($tableColumns, true);
+        $tableColumns = array_map('array_change_key_case', $tableColumns);
 
         /** @psalm-var ColumnInfo[] */
         return DbArrayHelper::index($tableColumns, 'cid');
@@ -575,7 +576,7 @@ private function loadTableConstraints(string $tableName, string $returnType): Co
     {
         $indexList = $this->getPragmaIndexList($tableName);
         /** @psalm-var IndexListInfo[] $indexes */
-        $indexes = $this->normalizeRowKeyCase($indexList, true);
+        $indexes = array_map('array_change_key_case', $indexList);
         $result = [
             self::PRIMARY_KEY => null,
             self::INDEXES => [],
@@ -642,9 +643,12 @@ private function createColumnSchema(string $name): ColumnSchemaInterface
      * @throws Exception
      * @throws InvalidConfigException
      * @throws Throwable
+     *
+     * @psalm-return ForeignKeyInfo[]
      */
     private function getPragmaForeignKeyList(string $tableName): array
     {
+        /** @psalm-var ForeignKeyInfo[] */
         return $this->db->createCommand(
             'PRAGMA FOREIGN_KEY_LIST(' . $this->db->getQuoter()->quoteSimpleTableName($tableName) . ')'
         )->queryAll();
@@ -662,7 +666,7 @@ private function getPragmaIndexInfo(string $name): array
         $column = $this->db
             ->createCommand('PRAGMA INDEX_INFO(' . (string) $this->db->getQuoter()->quoteValue($name) . ')')
             ->queryAll();
-        $column = $this->normalizeRowKeyCase($column, true);
+        $column = array_map('array_change_key_case', $column);
         DbArrayHelper::multisort($column, 'seqno');
 
         /** @psalm-var IndexInfo[] $column */
@@ -673,9 +677,12 @@ private function getPragmaIndexInfo(string $name): array
      * @throws Exception
      * @throws InvalidConfigException
      * @throws Throwable
+     *
+     * @psalm-return IndexListInfo[]
      */
     private function getPragmaIndexList(string $tableName): array
     {
+        /** @psalm-var IndexListInfo[] */
         return $this->db
             ->createCommand('PRAGMA INDEX_LIST(' . (string) $this->db->getQuoter()->quoteValue($tableName) . ')')
             ->queryAll();
@@ -723,6 +730,8 @@ protected function findViewNames(string $schema = ''): array
      * @param string $name the table name.
      *
      * @return array The cache key.
+     *
+     * @psalm-suppress DeprecatedMethod
      */
     protected function getCacheKey(string $name): array
     {

tests/SchemaTest.php

@@ -352,11 +352,11 @@ public function testWorkWithPrimaryKeyConstraint(): void
     public function testNotConnectionPDO(): void
     {
         $db = $this->createMock(ConnectionInterface::class);
-        $schema = new Schema($db, DbHelper::getSchemaCache(), 'system');
+        $schema = new Schema($db, DbHelper::getSchemaCache());
 
         $this->expectException(NotSupportedException::class);
         $this->expectExceptionMessage('Only PDO connections are supported.');
 
-        $schema->refreshTableSchema('customer');
+        $schema->refresh();
     }
 }