Fix phpstan/phpstan#12063: is_callable() false positive with union method names

- When ConstantArrayType::findTypeAndMethodNames() skips non-existent methods,
  isCallable() now accounts for those skipped entries
- Added doFindTypeAndMethodNames() private helper with out parameter tracking
  whether any method names were skipped due to not existing
- New regression test in tests/PHPStan/Rules/Comparison/data/bug-12063.php

Author: VincentLanglet

src/Type/Constant/ConstantArrayType.php

@@ -494,7 +494,8 @@ public function equals(Type $type): bool
 
 	public function isCallable(): TrinaryLogic
 	{
-		$typeAndMethods = $this->findTypeAndMethodNames();
+		$hasNonExistentMethod = false;
+		$typeAndMethods = $this->doFindTypeAndMethodNames($hasNonExistentMethod);
 		if ($typeAndMethods === []) {
 			return TrinaryLogic::createNo();
 		}
@@ -504,7 +505,13 @@ public function isCallable(): TrinaryLogic
 			$typeAndMethods,
 		);
 
-		return TrinaryLogic::createYes()->and(...$results);
+		$result = TrinaryLogic::createYes()->and(...$results);
+
+		if ($hasNonExistentMethod) {
+			$result = $result->and(TrinaryLogic::createMaybe());
+		}
+
+		return $result;
 	}
 
 	public function getCallableParametersAcceptors(ClassMemberAccessAnswerer $scope): array
@@ -537,6 +544,13 @@ public function getCallableParametersAcceptors(ClassMemberAccessAnswerer $scope)
 
 	/** @return ConstantArrayTypeAndMethod[] */
 	public function findTypeAndMethodNames(): array
+	{
+		$hasNonExistentMethod = false;
+		return $this->doFindTypeAndMethodNames($hasNonExistentMethod);
+	}
+
+	/** @return ConstantArrayTypeAndMethod[] */
+	private function doFindTypeAndMethodNames(bool &$hasNonExistentMethod): array
 	{
 		if (count($this->keyTypes) !== 2) {
 			return [];
@@ -578,6 +592,7 @@ public function findTypeAndMethodNames(): array
 		foreach ($methods->getConstantStrings() as $methodName) {
 			$has = $type->hasMethod($methodName->getValue());
 			if ($has->no()) {
+				$hasNonExistentMethod = true;
 				continue;
 			}
 

tests/PHPStan/Rules/Comparison/ImpossibleCheckTypeFunctionCallRuleTest.php

@@ -1214,4 +1214,10 @@ public function testBug13799(): void
 		]);
 	}
 
+	public function testBug12063(): void
+	{
+		$this->treatPhpDocTypesAsCertain = true;
+		$this->analyse([__DIR__ . '/data/bug-12063.php'], []);
+	}
+
 }

tests/PHPStan/Rules/Comparison/data/bug-12063.php

@@ -0,0 +1,38 @@
+<?php declare(strict_types=1);
+
+namespace Bug12063;
+
+use BadFunctionCallException;
+
+final class View
+{
+    public function existingMethod(): void
+    {
+    }
+}
+
+final class TwigExtension
+{
+    private View $viewFunctions;
+
+    public function __construct(View $viewFunctions)
+    {
+        $this->viewFunctions = $viewFunctions;
+    }
+
+    public function iterateFunctions(): void
+    {
+        $functionMappings = [
+            'i_exist' => 'existingMethod',
+			'i_dont_exist' => 'nonExistingMethod'
+        ];
+
+        $functions = [];
+        foreach ($functionMappings as $nameFrom => $nameTo) {
+            $callable = [$this->viewFunctions, $nameTo];
+            if (!is_callable($callable)) {
+                throw new BadFunctionCallException("Function $nameTo does not exist in view functions");
+            }
+        }
+    }
+}