ExportedNodeFetcher - use PHPStan's Parser interface

Author: ondrejmirtes

conf/config.neon

@@ -436,8 +436,6 @@ services:
 
 	-
 		class: PHPStan\Dependency\ExportedNodeFetcher
-		arguments:
-			phpParser: @phpParserDecorator
 
 	-
 		class: PHPStan\Dependency\ExportedNodeResolver
@@ -545,8 +543,6 @@ services:
 
 	-
 		class: PHPStan\Reflection\BetterReflection\SourceLocator\FileNodesFetcher
-		arguments:
-			phpParser: @phpParserDecorator
 
 	-
 		class: PHPStan\Reflection\BetterReflection\SourceLocator\AutoloadSourceLocator

src/Dependency/ExportedNodeFetcher.php

@@ -3,21 +3,21 @@
 namespace PHPStan\Dependency;
 
 use PhpParser\NodeTraverser;
-use PHPStan\File\FileReader;
+use PHPStan\Parser\Parser;
 
 class ExportedNodeFetcher
 {
 
-	private \PhpParser\Parser $phpParser;
+	private Parser $parser;
 
 	private ExportedNodeVisitor $visitor;
 
 	public function __construct(
-		\PhpParser\Parser $phpParser,
+		Parser $parser,
 		ExportedNodeVisitor $visitor
 	)
 	{
-		$this->phpParser = $phpParser;
+		$this->parser = $parser;
 		$this->visitor = $visitor;
 	}
 
@@ -30,12 +30,10 @@ public function fetchNodes(string $fileName): array
 		$nodeTraverser = new NodeTraverser();
 		$nodeTraverser->addVisitor($this->visitor);
 
-		$contents = FileReader::read($fileName);
-
 		try {
 			/** @var \PhpParser\Node[] $ast */
-			$ast = $this->phpParser->parse($contents);
-		} catch (\PhpParser\Error $e) {
+			$ast = $this->parser->parseFile($fileName);
+		} catch (\PHPStan\Parser\ParserErrorsException $e) {
 			return [];
 		}
 		$this->visitor->reset($fileName);

src/Parser/PathRoutingParser.php

@@ -40,7 +40,7 @@ public function parseFile(string $file): array
 
 	public function parseString(string $sourceCode): array
 	{
-		return $this->currentPhpVersionRichParser->parseString($sourceCode);
+		return $this->currentPhpVersionSimpleParser->parseString($sourceCode);
 	}
 
 }

src/Reflection/BetterReflection/SourceLocator/FileNodesFetcher.php

@@ -3,24 +3,24 @@
 namespace PHPStan\Reflection\BetterReflection\SourceLocator;
 
 use PhpParser\NodeTraverser;
-use PhpParser\Parser;
 use PHPStan\File\FileReader;
+use PHPStan\Parser\Parser;
 use Roave\BetterReflection\SourceLocator\Located\LocatedSource;
 
 class FileNodesFetcher
 {
 
 	private \PHPStan\Reflection\BetterReflection\SourceLocator\CachingVisitor $cachingVisitor;
 
-	private Parser $phpParser;
+	private Parser $parser;
 
 	public function __construct(
 		CachingVisitor $cachingVisitor,
-		Parser $phpParser
+		Parser $parser
 	)
 	{
 		$this->cachingVisitor = $cachingVisitor;
-		$this->phpParser = $phpParser;
+		$this->parser = $parser;
 	}
 
 	public function fetchNodes(string $fileName): FetchedNodesResult
@@ -33,8 +33,8 @@ public function fetchNodes(string $fileName): FetchedNodesResult
 
 		try {
 			/** @var \PhpParser\Node[] $ast */
-			$ast = $this->phpParser->parse($contents);
-		} catch (\PhpParser\Error $e) {
+			$ast = $this->parser->parseFile($fileName);
+		} catch (\PHPStan\Parser\ParserErrorsException $e) {
 			return new FetchedNodesResult([], [], [], $locatedSource);
 		}
 		$this->cachingVisitor->reset($fileName);