Resolve StaticType coming from dynamic return type extensions

phpstan · Feb 8, 2021 · 76b65b4 · 76b65b4
1 parent ec4e7ed
commit 76b65b4
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 11 deletions.
diff --git a/src/Analyser/MutatingScope.php b/src/Analyser/MutatingScope.php
@@ -4637,9 +4637,8 @@ private function methodCallReturnType(Type $calledOnType, Type $typeWithMethod,
 
 		$methodReflection = $typeWithMethod->getMethod($methodName, $this);
 
+		$resolvedTypes = [];
 		if ($typeWithMethod instanceof TypeWithClassName) {
-			$resolvedTypes = [];
-
 			if ($methodCall instanceof MethodCall) {
 				foreach ($this->dynamicReturnTypeExtensionRegistry->getDynamicMethodReturnTypeExtensionsForClass($typeWithMethod->getClassName()) as $dynamicMethodReturnTypeExtension) {
 					if (!$dynamicMethodReturnTypeExtension->isMethodSupported($methodReflection)) {
@@ -4657,16 +4656,17 @@ private function methodCallReturnType(Type $calledOnType, Type $typeWithMethod,
 					$resolvedTypes[] = $dynamicStaticMethodReturnTypeExtension->getTypeFromStaticMethodCall($methodReflection, $methodCall, $this);
 				}
 			}
-			if (count($resolvedTypes) > 0) {
-				return TypeCombinator::union(...$resolvedTypes);
-			}
 		}
 
-		$methodReturnType = ParametersAcceptorSelector::selectFromArgs(
-			$this,
-			$methodCall->args,
-			$methodReflection->getVariants()
-		)->getReturnType();
+		if (count($resolvedTypes) > 0) {
+			$methodReturnType = TypeCombinator::union(...$resolvedTypes);
+		} else {
+			$methodReturnType = ParametersAcceptorSelector::selectFromArgs(
+				$this,
+				$methodCall->args,
+				$methodReflection->getVariants()
+			)->getReturnType();
+		}
 
 		if ($methodCall instanceof MethodCall) {
 			$calledOnThis = $calledOnType instanceof StaticType && $this->isInClass();

diff --git a/tests/PHPStan/Analyser/NodeScopeResolverTest.php b/tests/PHPStan/Analyser/NodeScopeResolverTest.php
@@ -3235,7 +3235,7 @@ public function dataBinaryOperations(): array
 				'$simpleXMLWrongXpath',
 			],
 			[
-				'array<static(SimpleXMLElement)>|false',
+				'array<SimpleXMLElement>|false',
 				'$simpleXMLUnknownXpath',
 			],
 		];