Fix potential assert failures - use getRawArgs() instead

phpstan · Sep 29, 2022 · 6cd3d5d · 6cd3d5d
1 parent 37c643c
commit 6cd3d5d
Show file tree

Hide file tree

Showing 8 changed files with 23 additions and 6 deletions.
diff --git a/src/Parser/ArrayFilterArgVisitor.php b/src/Parser/ArrayFilterArgVisitor.php
@@ -15,7 +15,7 @@ public function enterNode(Node $node): ?Node
 		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name) {
 			$functionName = $node->name->toLowerString();
 			if ($functionName === 'array_filter') {
-				$args = $node->getArgs();
+				$args = $node->getRawArgs();
 				if (isset($args[0])) {
 					$args[0]->setAttribute(self::ATTRIBUTE_NAME, true);
 				}

diff --git a/src/Parser/ArrayMapArgVisitor.php b/src/Parser/ArrayMapArgVisitor.php
@@ -14,7 +14,7 @@ class ArrayMapArgVisitor extends NodeVisitorAbstract
 
 	public function enterNode(Node $node): ?Node
 	{
-		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name) {
+		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name && !$node->isFirstClassCallable()) {
 			$functionName = $node->name->toLowerString();
 			if ($functionName === 'array_map') {
 				$args = $node->getArgs();

diff --git a/src/Parser/ArrayWalkArgVisitor.php b/src/Parser/ArrayWalkArgVisitor.php
@@ -15,7 +15,7 @@ public function enterNode(Node $node): ?Node
 		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name) {
 			$functionName = $node->name->toLowerString();
 			if ($functionName === 'array_walk') {
-				$args = $node->getArgs();
+				$args = $node->getRawArgs();
 				if (isset($args[0])) {
 					$args[0]->setAttribute(self::ATTRIBUTE_NAME, true);
 				}

diff --git a/src/Parser/ArrowFunctionArgVisitor.php b/src/Parser/ArrowFunctionArgVisitor.php
@@ -14,7 +14,7 @@ class ArrowFunctionArgVisitor extends NodeVisitorAbstract
 	public function enterNode(Node $node): ?Node
 	{
 		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Expr\ArrowFunction) {
-			$args = $node->getArgs();
+			$args = $node->getRawArgs();
 
 			if (count($args) > 0) {
 				$node->name->setAttribute(self::ATTRIBUTE_NAME, $args);

diff --git a/src/Parser/ClosureArgVisitor.php b/src/Parser/ClosureArgVisitor.php
@@ -14,7 +14,7 @@ class ClosureArgVisitor extends NodeVisitorAbstract
 	public function enterNode(Node $node): ?Node
 	{
 		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Expr\Closure) {
-			$args = $node->getArgs();
+			$args = $node->getRawArgs();
 
 			if (count($args) > 0) {
 				$node->name->setAttribute(self::ATTRIBUTE_NAME, $args);

diff --git a/src/Parser/CurlSetOptArgVisitor.php b/src/Parser/CurlSetOptArgVisitor.php
@@ -15,7 +15,7 @@ public function enterNode(Node $node): ?Node
 		if ($node instanceof Node\Expr\FuncCall && $node->name instanceof Node\Name) {
 			$functionName = $node->name->toLowerString();
 			if ($functionName === 'curl_setopt') {
-				$args = $node->getArgs();
+				$args = $node->getRawArgs();
 				if (isset($args[0])) {
 					$args[0]->setAttribute(self::ATTRIBUTE_NAME, true);
 				}

diff --git a/tests/PHPStan/Analyser/AnalyserIntegrationTest.php b/tests/PHPStan/Analyser/AnalyserIntegrationTest.php
@@ -982,6 +982,12 @@ public function testBug7963Two(): void
 		$this->assertNoErrors($errors);
 	}
 
+	public function testBug8078(): void
+	{
+		$errors = $this->runAnalyse(__DIR__ . '/data/bug-8078.php');
+		$this->assertNoErrors($errors);
+	}
+
 	/**
 	 * @param string[]|null $allAnalysedFiles
 	 * @return Error[]

diff --git a/tests/PHPStan/Analyser/data/bug-8078.php b/tests/PHPStan/Analyser/data/bug-8078.php
@@ -0,0 +1,11 @@
+<?php // lint >= 8.1
+
+namespace Bug8078;
+
+class HelloWorld
+{
+	public function test(): void
+	{
+		$closure = (static fn (): string => 'evaluated Closure')(...);
+	}
+}