skip function-like nodes when checking if a function is a generator

Author: sebkehr

src/Reflection/Php/PhpFunctionFromParserNodeReflection.php

@@ -281,7 +281,10 @@ private function nodeIsOrContainsYield(Node $node): bool
 		foreach ($node->getSubNodeNames() as $nodeName) {
 			$nodeProperty = $node->$nodeName;
 
-			if ($nodeProperty instanceof Node && $this->nodeIsOrContainsYield($nodeProperty)) {
+			if ($nodeProperty instanceof Node &&
+				!$nodeProperty instanceof FunctionLike &&
+				$this->nodeIsOrContainsYield($nodeProperty)
+			) {
 				return true;
 			}
 

tests/PHPStan/Rules/Functions/ReturnTypeRuleTest.php

@@ -341,4 +341,20 @@ public function testBug11301(): void
 		]);
 	}
 
+	public function testBug12462(): void
+	{
+		$this->checkExplicitMixed = true;
+		$this->checkNullables = true;
+		$this->analyse([__DIR__ . '/data/bug-12462.php'], [
+			[
+				'Function Bug12462\functionReturningYieldingClosure() should return int but returns Closure.',
+				7,
+			],
+			[
+				'Function Bug12462\functionReturningYieldingArrowFunction() should return int but returns Closure.',
+				12,
+			],
+		]);
+	}
+
 }

tests/PHPStan/Rules/Functions/data/bug-12462.php

@@ -0,0 +1,13 @@
+<?php declare(strict_types = 1);
+
+namespace Bug12462;
+
+function functionReturningYieldingClosure (): int
+{
+	return function () { yield ''; };
+}
+
+function functionReturningYieldingArrowFunction (): int
+{
+	return fn () => yield '';
+}

tests/PHPStan/Rules/Methods/ReturnTypeRuleTest.php

@@ -1227,4 +1227,18 @@ public function testBug1O580(): void
 		]);
 	}
 
+	public function testBug12462(): void
+	{
+		$this->analyse([__DIR__ . '/data/bug-12462.php'], [
+			[
+				'Method Bug12462\A::methodReturningYieldingClosure() should return int but returns Closure.',
+				8,
+			],
+			[
+				'Method Bug12462\A::methodReturningYieldingArrowFunction() should return int but returns Closure.',
+				13,
+			],
+		]);
+	}
+
 }

tests/PHPStan/Rules/Methods/data/bug-12462.php

@@ -0,0 +1,15 @@
+<?php declare(strict_types = 1);
+
+namespace Bug12462;
+
+class A {
+	function methodReturningYieldingClosure (): int
+	{
+		return function () { yield ''; };
+	}
+
+	function methodReturningYieldingArrowFunction (): int
+	{
+		return fn () => yield '';
+	}
+}