Upgrade psalm-5.6. (#517)

composer.json

@@ -33,7 +33,7 @@
         "rector/rector": "^0.14",
         "roave/infection-static-analysis-plugin": "^1.16",
         "spatie/phpunit-watcher": "^1.23",
-        "vimeo/psalm": "^4.18",
+        "vimeo/psalm": "^4.30|^5.6",
         "yiisoft/aliases": "^1.1|^2.0",
         "yiisoft/cache-file": "^2.0",
         "yiisoft/di": "^1.0",

psalm.xml

@@ -1,15 +1,17 @@
 <?xml version="1.0"?>
 <psalm
-    errorLevel="1"
-    resolveFromConfigFile="true"
-    xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
-    xmlns="https://getpsalm.org/schema/config"
-    xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
+  errorLevel="1"
+  findUnusedBaselineEntry="true"
+  findUnusedCode="false"
+  resolveFromConfigFile="true"
+  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  xmlns="https://getpsalm.org/schema/config"
+  xsi:schemaLocation="https://getpsalm.org/schema/config vendor/vimeo/psalm/config.xsd"
 >
-    <projectFiles>
-        <directory name="src" />
-        <ignoreFiles>
-            <directory name="vendor" />
-        </ignoreFiles>
-    </projectFiles>
+  <projectFiles>
+    <directory name="src" />
+      <ignoreFiles>
+        <directory name="vendor" />
+    </ignoreFiles>
+  </projectFiles>
 </psalm>

src/Expression/ArrayExpression.php

@@ -25,6 +25,9 @@
  *
  * Which, depending on DBMS, will result in a well-prepared condition. For example, in PostgresSQL it will be compiled
  * to `WHERE "items" @> ARRAY[1, 2, 3]::integer[]`.
+ *
+ * @template-implements ArrayAccess<int, mixed>
+ * @template-implements IteratorAggregate<int>
  */
 class ArrayExpression implements ExpressionInterface, ArrayAccess, Countable, IteratorAggregate
 {

src/Helper/ArrayHelper.php

@@ -205,10 +205,13 @@ public static function getValueByPath(object|array $array, Closure|string $key,
      * @throws \Exception
      *
      * @return array the indexed and/or grouped array
+     *
+     * @psalm-suppress MixedArrayAssignment
      */
     public static function index(array $array, string|null $key = null, array $groups = []): array
     {
         $result = [];
+
         foreach ($array as $element) {
             $lastArray = &$result;
 
@@ -229,12 +232,13 @@ public static function index(array $array, string|null $key = null, array $group
                 /** @psalm-var mixed $value */
                 $value = self::getValueByPath($element, $key);
                 if ($value !== null) {
-                    $lastArray[(string)$value] = $element;
+                    $lastArray[(string) $value] = $element;
                 }
             }
             unset($lastArray);
         }
 
+        /** @psalm-var array $result */
         return $result;
     }
 

src/Query/BatchQueryResultInterface.php

@@ -24,6 +24,8 @@
  * foreach ($query->each() as $user) {
  * }
  * ```
+ *
+ * @extends Iterator<int|string, mixed>
  */
 interface BatchQueryResultInterface extends Iterator
 {

src/Query/Data/DataReaderInterface.php

@@ -28,6 +28,8 @@
  *
  * Note that since DataReader is a forward-only stream, you can only traverse it once. Doing it the second time will
  * throw an exception.
+ *
+ * @extends Iterator<int|string, mixed>
  */
 interface DataReaderInterface extends Iterator, Countable
 {

src/QueryBuilder/AbstractDQLQueryBuilder.php

@@ -219,6 +219,7 @@ public function buildGroupBy(array $columns, array &$params = []): string
             }
         }
 
+        /** @psalm-var array<string, Expression|string> $columns */
         return 'GROUP BY ' . implode(', ', $columns);
     }
 
@@ -371,6 +372,7 @@ public function buildSelect(
             }
         }
 
+        /** @psalm-var array<string, Expression|string> $columns */
         return $select . ' ' . implode(', ', $columns);
     }