[TypeDeclaration] Skip nullable callable on TypedPropertyFromAssignsR…

…ector (#6308)

* [TypeDeclaration] Skip nullable callable on TypedPropertyFromAssignsRector

* [TypeDeclaration] Skip nullable callable on TypedPropertyFromAssignsRector

* Fix

rules-tests/TypeDeclaration/Rector/ClassMethod/ReturnUnionTypeRector/Fixture/union_callable_return.php.inc

@@ -0,0 +1,35 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\ReturnUnionTypeRector\Fixture;
+
+final class UnionCallableReturn
+{
+    public function run(callable $a)
+    {
+        if (rand(0, 1)) {
+            return $a;
+        }
+
+        return [];
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\ReturnUnionTypeRector\Fixture;
+
+final class UnionCallableReturn
+{
+    public function run(callable $a): callable|array
+    {
+        if (rand(0, 1)) {
+            return $a;
+        }
+
+        return [];
+    }
+}
+
+?>

rules-tests/TypeDeclaration/Rector/Property/TypedPropertyFromAssignsRector/Fixture/skip_nullable_callable.php.inc

@@ -0,0 +1,18 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\Property\TypedPropertyFromAssignsRector\Fixture;
+
+final class SkipNullableCallable
+{
+    private $prop;
+
+    public function set(callable $prop): void
+    {
+        $this->prop = $prop;
+    }
+
+    public function reset(): void
+    {
+        $this->prop = null;
+    }
+}

src/PHPStanStaticTypeMapper/TypeMapper/UnionTypeMapper.php

@@ -13,6 +13,7 @@
 use PhpParser\Node\NullableType;
 use PhpParser\Node\UnionType as PhpParserUnionType;
 use PHPStan\PhpDocParser\Ast\Type\TypeNode;
+use PHPStan\Type\CallableType;
 use PHPStan\Type\Type;
 use PHPStan\Type\UnionType;
 use Rector\BetterPhpDocParser\ValueObject\Type\BracketsAwareUnionTypeNode;
@@ -64,7 +65,7 @@ public function mapToPHPStanPhpDocTypeNode(Type $type): TypeNode
      */
     public function mapToPhpParserNode(Type $type, string $typeKind): ?Node
     {
-        $phpParserUnionType = $this->matchPhpParserUnionType($type);
+        $phpParserUnionType = $this->matchPhpParserUnionType($type, $typeKind);
         if ($phpParserUnionType instanceof PhpParserUnionType) {
             return $this->resolveUnionTypeNode($phpParserUnionType);
         }
@@ -171,7 +172,7 @@ private function hasObjectAndStaticType(PhpParserUnionType $phpParserUnionType):
     /**
      * @return Name|FullyQualified|ComplexType|Identifier|null
      */
-    private function matchPhpParserUnionType(UnionType $unionType): ?Node
+    private function matchPhpParserUnionType(UnionType $unionType, string $typeKind): ?Node
     {
         $phpParserUnionedTypes = [];
 
@@ -183,6 +184,11 @@ private function matchPhpParserUnionType(UnionType $unionType): ?Node
                 return null;
             }
 
+            // special callable type only not allowed on property
+            if ($typeKind === TypeKind::PROPERTY && $unionedType instanceof CallableType) {
+                return null;
+            }
+
             $phpParserUnionedTypes[] = $phpParserNode;
         }