Do not parse the same method multiple times

Author: janedbal

src/Type/Doctrine/QueryBuilder/OtherMethodQueryBuilderParser.php

@@ -35,6 +35,9 @@ class OtherMethodQueryBuilderParser
 	/** @var Container */
 	private $container;
 
+	/** @var array<string, array<string, list<QueryBuilderType>>> */
+	private $cache = [];
+
 	public function __construct(bool $descendIntoOtherMethods, Parser $parser, Container $container)
 	{
 		$this->descendIntoOtherMethods = $descendIntoOtherMethods;
@@ -51,18 +54,23 @@ public function findQueryBuilderTypesInCalledMethod(Scope $scope, MethodReflecti
 			return [];
 		}
 
+		$methodName = $methodReflection->getName();
 		$fileName = $methodReflection->getDeclaringClass()->getFileName();
 		if ($fileName === null) {
 			return [];
 		}
 
+		if (isset($this->cache[$fileName][$methodName])) {
+			return $this->cache[$fileName][$methodName];
+		}
+
 		$nodes = $this->parser->parseFile($fileName);
 		$classNode = $this->findClassNode($methodReflection->getDeclaringClass()->getName(), $nodes);
 		if ($classNode === null) {
 			return [];
 		}
 
-		$methodNode = $this->findMethodNode($methodReflection->getName(), $classNode->stmts);
+		$methodNode = $this->findMethodNode($methodName, $classNode->stmts);
 		if ($methodNode === null || $methodNode->stmts === null) {
 			return [];
 		}
@@ -100,6 +108,8 @@ public function findQueryBuilderTypesInCalledMethod(Scope $scope, MethodReflecti
 			});
 		});
 
+		$this->cache[$fileName][$methodName] = $queryBuilderTypes;
+
 		return $queryBuilderTypes;
 	}