PropertyHookNameVisitor is no longer needed, PHP-parser comes with propertyName attribute

Author: ondrejmirtes

conf/config.neon

@@ -320,11 +320,6 @@ services:
 		tags:
 			- phpstan.parser.richParserNodeVisitor
 
-	-
-		class: PHPStan\Parser\PropertyHookNameVisitor
-		tags:
-			- phpstan.parser.richParserNodeVisitor
-
 	-
 		class: PHPStan\Node\Printer\ExprPrinter
 

src/Analyser/NodeScopeResolver.php

@@ -126,7 +126,6 @@
 use PHPStan\Parser\ImmediatelyInvokedClosureVisitor;
 use PHPStan\Parser\LineAttributesVisitor;
 use PHPStan\Parser\Parser;
-use PHPStan\Parser\PropertyHookNameVisitor;
 use PHPStan\Php\PhpVersion;
 use PHPStan\PhpDoc\PhpDocInheritanceResolver;
 use PHPStan\PhpDoc\ResolvedPhpDocBlock;
@@ -6436,7 +6435,7 @@ public function getPhpDocs(Scope $scope, Node\FunctionLike|Node\Stmt\Property $n
 		} elseif ($node instanceof Node\Stmt\Function_) {
 			$functionName = trim($scope->getNamespace() . '\\' . $node->name->name, '\\');
 		} elseif ($node instanceof Node\PropertyHook) {
-			$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+			$propertyName = $node->getAttribute('propertyName');
 			if ($propertyName !== null) {
 				$functionName = sprintf('$%s::%s', $propertyName, $node->name->toString());
 			}

src/Parser/CleaningVisitor.php

@@ -37,7 +37,7 @@ public function enterNode(Node $node): ?Node
 		}
 
 		if ($node instanceof Node\PropertyHook && is_array($node->body)) {
-			$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+			$propertyName = $node->getAttribute('propertyName');
 			if ($propertyName !== null) {
 				$node->body = $this->keepVariadicsAndYields($node->body, $propertyName);
 				return $node;

src/Parser/PropertyHookNameVisitor.php

@@ -17,7 +17,6 @@ public function __construct(
 		private NameResolver $nameResolver,
 		private VariadicMethodsVisitor $variadicMethodsVisitor,
 		private VariadicFunctionsVisitor $variadicFunctionsVisitor,
-		private PropertyHookNameVisitor $propertyHookNameVisitor,
 	)
 	{
 	}
@@ -53,7 +52,6 @@ public function parseString(string $sourceCode): array
 		$nodeTraverser->addVisitor($this->nameResolver);
 		$nodeTraverser->addVisitor($this->variadicMethodsVisitor);
 		$nodeTraverser->addVisitor($this->variadicFunctionsVisitor);
-		$nodeTraverser->addVisitor($this->propertyHookNameVisitor);
 
 		/** @var array<Node\Stmt> */
 		return $nodeTraverser->traverse($nodes);

src/Parser/SimpleParser.php

@@ -9,7 +9,6 @@
 use PHPStan\BetterReflection\Reflection\Adapter\ReflectionFunction;
 use PHPStan\BetterReflection\Reflection\Adapter\ReflectionParameter;
 use PHPStan\BetterReflection\Reflection\ReflectionConstant;
-use PHPStan\Parser\PropertyHookNameVisitor;
 use PHPStan\Reflection\Php\PhpMethodFromParserNodeReflection;
 use PHPStan\ShouldNotHappenException;
 use function array_slice;
@@ -144,15 +143,15 @@ public static function fromStubParameter(
 		} elseif ($function instanceof ClassMethod) {
 			$functionName = $function->name->toString();
 		} elseif ($function instanceof PropertyHook) {
-			$propertyName = $function->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+			$propertyName = $function->getAttribute('propertyName');
 			$functionName = sprintf('$%s::%s', $propertyName, $function->name->toString());
 		}
 
 		$methodName = null;
 		if ($function instanceof ClassMethod && $className !== null) {
 			$methodName = sprintf('%s::%s', $className, $function->name->toString());
 		} elseif ($function instanceof PropertyHook) {
-			$propertyName = $function->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+			$propertyName = $function->getAttribute('propertyName');
 			$methodName = sprintf('%s::$%s::%s', $className, $propertyName, $function->name->toString());
 		} elseif ($function instanceof Function_ && $function->namespacedName !== null) {
 			$methodName = $function->namespacedName->toString();

src/Reflection/InitializerExprContext.php

@@ -9,7 +9,6 @@
 use PHPStan\Broker\AnonymousClassNameHelper;
 use PHPStan\File\FileHelper;
 use PHPStan\Parser\Parser;
-use PHPStan\Parser\PropertyHookNameVisitor;
 use PHPStan\PhpDoc\PhpDocNodeResolver;
 use PHPStan\PhpDoc\PhpDocStringResolver;
 use PHPStan\PhpDoc\ResolvedPhpDocBlock;
@@ -281,7 +280,7 @@ function (Node $node) use ($fileName, $lookForTrait, &$traitFound, $traitMethodA
 				} elseif ($node instanceof Node\Stmt\Function_) {
 					$functionStack[] = ltrim(sprintf('%s\\%s', $namespace, $node->name->name), '\\');
 				} elseif ($node instanceof Node\PropertyHook) {
-					$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+					$propertyName = $node->getAttribute('propertyName');
 					if ($propertyName !== null) {
 						$functionStack[] = sprintf('$%s::%s', $propertyName, $node->name->toString());
 					}
@@ -299,7 +298,7 @@ function (Node $node) use ($fileName, $lookForTrait, &$traitFound, $traitMethodA
 
 					return null;
 				} elseif ($node instanceof Node\PropertyHook) {
-					$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+					$propertyName = $node->getAttribute('propertyName');
 					if ($propertyName !== null) {
 						$docComment = GetLastDocComment::forNode($node);
 						if ($docComment !== null) {
@@ -394,7 +393,7 @@ static function (Node $node) use (&$namespace, &$functionStack, &$classStack): v
 
 					array_pop($functionStack);
 				} elseif ($node instanceof Node\PropertyHook) {
-					$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+					$propertyName = $node->getAttribute('propertyName');
 					if ($propertyName !== null) {
 						if (count($functionStack) === 0) {
 							throw new ShouldNotHappenException();
@@ -503,7 +502,7 @@ function (Node $node) use ($fileName, $lookForTrait, $phpDocNodeMap, &$traitFoun
 				} elseif ($node instanceof Node\Stmt\Function_) {
 					$functionStack[] = ltrim(sprintf('%s\\%s', $namespace, $node->name->name), '\\');
 				} elseif ($node instanceof Node\PropertyHook) {
-					$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+					$propertyName = $node->getAttribute('propertyName');
 					if ($propertyName !== null) {
 						$functionStack[] = sprintf('$%s::%s', $propertyName, $node->name->toString());
 					}
@@ -742,7 +741,7 @@ static function (Node $node, $callbackResult) use (&$namespace, &$functionStack,
 
 					array_pop($functionStack);
 				} elseif ($node instanceof Node\PropertyHook) {
-					$propertyName = $node->getAttribute(PropertyHookNameVisitor::ATTRIBUTE_NAME);
+					$propertyName = $node->getAttribute('propertyName');
 					if ($propertyName !== null) {
 						if (count($functionStack) === 0) {
 							throw new ShouldNotHappenException();

src/Type/FileTypeMapper.php

@@ -75,7 +75,6 @@ public function testParse(
 				new NameResolver(),
 				new VariadicMethodsVisitor(),
 				new VariadicFunctionsVisitor(),
-				new PropertyHookNameVisitor(),
 			),
 			new PhpVersion($phpVersionId),
 		);