add parenthesis

Author: staabm

src/Rules/Functions/UselessFunctionReturnValueRule.php

@@ -64,7 +64,7 @@ public function processNode(Node $funcCall, Scope $scope): array
 		if (count($reorderedArgs) === 1 || (count($reorderedArgs) >= 2 && $scope->getType($reorderedArgs[1]->value)->isFalse()->yes())) {
 			return [RuleErrorBuilder::message(
 				sprintf(
-					'Return value of function %s is always true and the result is printed instead of being returned. Pass in true as parameter #%d $%s to return the output instead.',
+					'Return value of function %s() is always true and the result is printed instead of being returned. Pass in true as parameter #%d $%s to return the output instead.',
 					$functionReflection->getName(),
 					2,
 					$parametersAcceptor->getParameters()[1]->getName(),

tests/PHPStan/Levels/data/unreachable-0.json

@@ -1,12 +1,12 @@
 [
     {
-        "message": "Return value of function print_r is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.",
+        "message": "Return value of function print_r() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.",
         "line": 38,
         "ignorable": true
     },
     {
-        "message": "Return value of function print_r is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.",
+        "message": "Return value of function print_r() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.",
         "line": 89,
         "ignorable": true
     }
-]
+]

tests/PHPStan/Rules/Functions/UselessFunctionReturnValueRuleTest.php

@@ -23,15 +23,15 @@ public function testUselessReturnValue(): void
 	{
 		$this->analyse([__DIR__ . '/data/useless-fn-return.php'], [
 			[
-				'Return value of function print_r is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
+				'Return value of function print_r() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
 				47,
 			],
 			[
-				'Return value of function var_export is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
+				'Return value of function var_export() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
 				56,
 			],
 			[
-				'Return value of function print_r is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
+				'Return value of function print_r() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
 				64,
 			],
 		]);
@@ -45,7 +45,7 @@ public function testUselessReturnValuePhp8(): void
 
 		$this->analyse([__DIR__ . '/data/useless-fn-return-php8.php'], [
 			[
-				'Return value of function print_r is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
+				'Return value of function print_r() is always true and the result is printed instead of being returned. Pass in true as parameter #2 $return to return the output instead.',
 				18,
 			],
 		]);