[TypeDeclaration] Skip param setter method intersection docblock not …

…autoload on ReturnNeverTypeRector (#4721) * Add test-case for #8121 * Closes #4717 Fixes rectorphp/rector#8121 --------- Co-authored-by: Matthias Krauser <matthias@krauser.eu>
rectorphp · Aug 8, 2023 · 17fe187 · 17fe187
1 parent 0a8ffe3
commit 17fe187
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/...turnNeverTypeRector/Fixture/skip_setters_param_intersection_docblock_not_autoload.php.inc b/...turnNeverTypeRector/Fixture/skip_setters_param_intersection_docblock_not_autoload.php.inc
@@ -0,0 +1,14 @@
+<?php
+
+namespace Rector\Tests\TypeDeclaration\Rector\ClassMethod\ReturnNeverTypeRector\Fixture;
+
+final class SkipSettersParamIntersectionDocblockNotAutoload
+{
+    /**
+     * @param Type&Parts[] $value
+     */
+    public function setter($value): void
+    {
+        $this->prop = $value;
+    }
+}
diff --git a/rules/TypeDeclaration/Rector/ClassMethod/ReturnNeverTypeRector.php b/rules/TypeDeclaration/Rector/ClassMethod/ReturnNeverTypeRector.php
@@ -141,7 +141,7 @@ private function hasNeverFuncCall(ClassMethod | Closure | Function_ $functionLik
                 continue;
             }
 
-            $stmtType = $this->getType($stmt);
+            $stmtType = $this->nodeTypeResolver->getNativeType($stmt);
             if ($stmtType instanceof NeverType) {
                 $hasNeverType = true;
             }