Fix PHPStan iterable type errors in diferent files issue (#8732)

Author: Chucho-Kun

admin/framework/composer.json

@@ -20,7 +20,7 @@
         "codeigniter/coding-standard": "^1.7",
         "fakerphp/faker": "^1.24",
         "friendsofphp/php-cs-fixer": "^3.47.1",
-        "kint-php/kint": "^6.1",
+        "kint-php/kint": "^6.0",
         "mikey179/vfsstream": "^1.6.12",
         "nexusphp/cs-config": "^3.6",
         "phpunit/phpunit": "^10.5.16 || ^11.2",

composer.json

@@ -19,7 +19,7 @@
     "require-dev": {
         "codeigniter/phpstan-codeigniter": "1.x-dev",
         "fakerphp/faker": "^1.24",
-        "kint-php/kint": "^6.1",
+        "kint-php/kint": "^6.0",
         "mikey179/vfsstream": "^1.6.12",
         "nexusphp/tachycardia": "^2.0",
         "phpstan/extension-installer": "^1.4",

system/BaseModel.php

@@ -436,7 +436,7 @@ abstract protected function doFindColumn(string $columnName);
      * @param int|null $limit  Limit
      * @param int      $offset Offset
      *
-     * @return array
+     * @return array<int,array<int|string,bool|float|int|object|string|null>|object>
      */
     abstract protected function doFindAll(?int $limit = null, int $offset = 0);
 
@@ -629,7 +629,7 @@ public function find($id = null)
      *
      * @param string $columnName Column Name
      *
-     * @return array|null The resulting row of data, or null if no data found.
+     * @return array<int,mixed>|null The resulting row of data, or null if no data found.
      *
      * @throws DataException
      */
@@ -650,7 +650,7 @@ public function findColumn(string $columnName)
      * @param int $limit  Limit
      * @param int $offset Offset
      *
-     * @return array
+     * @return array<mixed>
      */
     public function findAll(?int $limit = null, int $offset = 0)
     {
@@ -695,7 +695,7 @@ public function findAll(?int $limit = null, int $offset = 0)
     /**
      * Returns the first row of the result set.
      *
-     * @return array|object|null
+     * @return array<string,mixed>|object|null
      */
     public function first()
     {
@@ -1593,9 +1593,10 @@ public function getValidationMessages(): array
      * currently so that rules don't block updating when only updating
      * a partial row.
      *
-     * @param         array     $rules Array containing field name and rule
-     * @param         array     $row   Row data (@TODO Remove null in param type)
+     * @param array<string,string> $rules
+     * @param array<string,mixed>|null $row
      * @phpstan-param row_array $row
+     * @phpstan-return array<string,string>
      */
     protected function cleanValidationRules(array $rules, ?array $row = null): array
     {
@@ -1889,7 +1890,7 @@ protected function transformDataToArray($row, string $type): array
      *
      * @param string $name Name
      *
-     * @return array|bool|float|int|object|string|null
+     * @return array<mixed>|bool|float|int|object|string|null
      */
     public function __get(string $name)
     {

system/CLI/CLI.php

@@ -374,6 +374,10 @@ public static function promptByMultipleKeys(string $text, array $options): array
 
     /**
      * Validation for $options in promptByKey() and promptByMultipleKeys(). Return an error if $options is an empty array.
+     * 
+     * @param array<int,string> $options List of options
+     * 
+     * @return void
      */
     private static function isZeroOptions(array $options): void
     {

system/Commands/ListCommands.php

@@ -87,6 +87,8 @@ public function run(array $params)
 
     /**
      * Lists the commands with accompanying info.
+     * 
+     * @param array<string,array<string,string>> $commands Array of commands keyed by name, each with metadata
      *
      * @return int
      */

system/Commands/Translation/LocalizationFinder.php

@@ -304,6 +304,11 @@ private function replaceArraySyntax(string $code): string
 
     /**
      * Create multidimensional array from another keys
+     * 
+     * @param array<int,string> $fromKeys List of keys to build nested array
+     * @param string            $lastArrayValue Value to assign at the deepest key
+     * 
+     * @return array<string,mixed>   Multidimensional associative array
      */
     private function buildMultiArray(array $fromKeys, string $lastArrayValue = ''): array
     {
@@ -323,6 +328,10 @@ private function buildMultiArray(array $fromKeys, string $lastArrayValue = ''):
 
     /**
      * Convert multi arrays to specific CLI table rows (flat array)
+     * 
+     * @param array<int,mixed> $array Input translations (nested array or strings)
+     * 
+     * @return array<int,array{0:string,1:string}> Flat list of table rows [langFileNames, value]
      */
     private function arrayToTableRows(string $langFileName, array $array): array
     {

system/Commands/Utilities/Namespaces.php

@@ -91,6 +91,11 @@ public function run(array $params)
         CLI::table($tbody, $thead);
     }
 
+    /**
+     * @param array{m:int,r?:mixed} $params
+     * 
+     * @return array<int,array{0:string,1:string,2:string}>
+     */
     private function outputAllNamespaces(array $params): array
     {
         $maxLength = $params['m'];

system/Commands/Utilities/Routes/AutoRouterImproved/AutoRouteCollector.php

@@ -95,9 +95,10 @@ public function get(): array
     /**
      * Adding Filters
      *
-     * @param list<array<string, array|string>> $routes
+     * @param array<int,array<string,string|array<string>>>     $routes List of route definitions
+     * 
+     * @return array<int,array<string,string|array<string>>>    $Updated route definitions with filters
      *
-     * @return list<array<string, array|string>>
      */
     private function addFilters(array $routes): array
     {

system/Config/BaseService.php

@@ -304,6 +304,7 @@ public static function locator(bool $getShared = true)
      * Provides the ability to perform case-insensitive calling of service
      * names.
      *
+     * @param array<int,mixed> $arguments
      * @return object|null
      */
     public static function __callStatic(string $name, array $arguments)

system/Modules/Modules.php

@@ -60,6 +60,13 @@ public function shouldDiscover(string $alias): bool
         return in_array(strtolower($alias), $this->aliases, true);
     }
 
+    /**
+     * Restores the state of the object when exported with var_export()
+     * 
+     * @param array<string,mixed>   $array Properties and their values
+     * 
+     * @return static
+     */
     public static function __set_state(array $array)
     {
         $obj = new static();