[Php80] Handle parent with typed param on AddParamBasedOnParentClassM…

…ethodRector (#1455)

Co-authored-by: Tomas Votruba <tomas.vot@gmail.com>
Co-authored-by: GitHub Action <action@github.com>

rules-tests/Php80/Rector/ClassMethod/AddParamBasedOnParentClassMethodRector/Fixture/fixture_has_parent_typed_param.php.inc

@@ -0,0 +1,27 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\ClassMethod\AddParamBasedOnParentClassMethodRector\Fixture;
+
+use Rector\Tests\Php80\Rector\ClassMethod\AddParamBasedOnParentClassMethodRector\Source\ParentWithTypedParam;
+
+class B extends ParentWithTypedParam{
+    public function execute()
+    {
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\ClassMethod\AddParamBasedOnParentClassMethodRector\Fixture;
+
+use Rector\Tests\Php80\Rector\ClassMethod\AddParamBasedOnParentClassMethodRector\Source\ParentWithTypedParam;
+
+class B extends ParentWithTypedParam{
+    public function execute(int $foo)
+    {
+    }
+}
+
+?>

rules-tests/Php80/Rector/ClassMethod/AddParamBasedOnParentClassMethodRector/Source/ParentWithTypedParam.php

@@ -0,0 +1,10 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\ClassMethod\AddParamBasedOnParentClassMethodRector\Source;
+
+class ParentWithTypedParam
+{
+    public function execute(int $foo)
+    {
+    }
+}

rules/Php80/Rector/ClassMethod/AddParamBasedOnParentClassMethodRector.php

@@ -5,9 +5,11 @@
 namespace Rector\Php80\Rector\ClassMethod;
 
 use PhpParser\Node;
+use PhpParser\Node\ComplexType;
 use PhpParser\Node\Expr;
 use PhpParser\Node\Expr\ConstFetch;
 use PhpParser\Node\Expr\Variable;
+use PhpParser\Node\Identifier;
 use PhpParser\Node\Name;
 use PhpParser\Node\Param;
 use PhpParser\Node\Stmt\ClassMethod;
@@ -16,6 +18,7 @@
 use Rector\Core\Rector\AbstractRector;
 use Rector\Core\ValueObject\MethodName;
 use Rector\Core\ValueObject\PhpVersionFeature;
+use Rector\NodeTypeResolver\Node\AttributeKey;
 use Rector\VendorLocker\ParentClassMethodTypeOverrideGuard;
 use Rector\VersionBonding\Contract\MinPhpVersionInterface;
 use Symplify\RuleDocGenerator\ValueObject\CodeSample\CodeSample;
@@ -208,10 +211,12 @@ function (Node $subNode) use ($parentClassMethodParam): bool {
             }
 
             $paramName = $this->nodeNameResolver->getName($parentClassMethodParam);
+            $paramType = $this->resolveParamType($parentClassMethodParam);
+
             $node->params[$key] = new Param(
                 new Variable($paramName),
                 $paramDefault,
-                $parentClassMethodParam->type,
+                $paramType,
                 $parentClassMethodParam->byRef,
                 $parentClassMethodParam->variadic,
                 [],
@@ -223,6 +228,18 @@ function (Node $subNode) use ($parentClassMethodParam): bool {
         return $node;
     }
 
+    private function resolveParamType(Param $param): null|Identifier|Name|ComplexType
+    {
+        if ($param->type === null) {
+            return null;
+        }
+
+        $paramType = $param->type;
+        $paramType->setAttribute(AttributeKey::ORIGINAL_NODE, null);
+
+        return $paramType;
+    }
+
     /**
      * @return string[]
      */