[TypeDeclaration] Add AddReturnArrayDocblockBasedOnArrayMapRector

[TomasVotruba]

No description provided.

[samsonasik]

I actually still working on it similar to it at PR:

• [TypeDeclaration] Add AddParamArrayDocblockBasedOnCallableNativeFuncCallRector #6233

[samsonasik]

Oh, I see, it on return instead of param ...

[TomasVotruba]

@samsonasik Indeed :D I came across different but very similar case, so I'm trying to cover it

[samsonasik]

new fixture is needed to not change already valid docblock with valid class name from use statement:

namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\AddReturnArrayDocblockBasedOnArrayMapRector\Fixture;

use stdClass;

final class SkipAlreadyValidArrayClassNameReturn
{
    /**
     * @return stdClass[]
     */
    public function process(array $items)
    {
        return array_map(function ($item): stdClass {
            return $item;
        }, $items);
    }
}

[samsonasik]

new fixture for union return is needed:

namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\AddReturnArrayDocblockBasedOnArrayMapRector\Fixture;

final class UnionReturnMap
{
+   /**
+    * @return float[]|int[]
+    */
    public function run(array $items)
    {
        if (mt_rand(0, 1)) {
            return array_map(function ($item): float {
                return $item;
            }, $items);
        }

        return array_map(function ($item): int {
            return 1000;
        }, $items);
    }
}

[TomasVotruba]

Done 👍

[TomasVotruba]

Let's ship this 👍