Revert old fix

Author: VincentLanglet

src/Rules/FunctionCallParametersCheck.php

@@ -89,12 +89,6 @@ public function check(
 		string $namedArgumentMessage,
 	): array
 	{
-		if ($funcCall instanceof Node\Expr\MethodCall || $funcCall instanceof Node\Expr\StaticCall || $funcCall instanceof Node\Expr\FuncCall) {
-			$funcCallLine = $funcCall->name->getStartLine();
-		} else {
-			$funcCallLine = $funcCall->getStartLine();
-		}
-
 		$functionParametersMinCount = 0;
 		$functionParametersMaxCount = 0;
 		foreach ($parametersAcceptor->getParameters() as $parameter) {
@@ -231,7 +225,6 @@ public function check(
 		if ($hasNamedArguments && !$scope->getPhpVersion()->supportsNamedArguments()->yes() && !(bool) $funcCall->getAttribute('isAttribute', false)) {
 			$errors[] = RuleErrorBuilder::message('Named arguments are supported only on PHP 8.0 and later.')
 				->identifier('argument.namedNotSupported')
-				->line($funcCallLine)
 				->nonIgnorable()
 				->build();
 		}
@@ -256,7 +249,6 @@ public function check(
 						$functionParametersMinCount,
 					))
 						->identifier('arguments.count')
-						->line($funcCallLine)
 						->build();
 				} elseif ($functionParametersMaxCount === -1 && $invokedParametersCount < $functionParametersMinCount) {
 					$errors[] = RuleErrorBuilder::message(sprintf(
@@ -265,7 +257,6 @@ public function check(
 						$functionParametersMinCount,
 					))
 						->identifier('arguments.count')
-						->line($funcCallLine)
 						->build();
 				} elseif ($functionParametersMaxCount !== -1) {
 					$errors[] = RuleErrorBuilder::message(sprintf(
@@ -275,7 +266,6 @@ public function check(
 						$functionParametersMaxCount,
 					))
 						->identifier('arguments.count')
-						->line($funcCallLine)
 						->build();
 				}
 			}
@@ -288,11 +278,10 @@ public function check(
 		) {
 			$errors[] = RuleErrorBuilder::message($voidReturnTypeUsed)
 				->identifier(sprintf('%s.void', $nodeType))
-				->line($funcCallLine)
 				->build();
 		}
 
-		[$addedErrors, $argumentsWithParameters] = $this->processArguments($parametersAcceptor, $funcCallLine, $isBuiltin, $arguments, $hasNamedArguments, $missingParameterMessage, $unknownParameterMessage);
+		[$addedErrors, $argumentsWithParameters] = $this->processArguments($parametersAcceptor, $isBuiltin, $arguments, $hasNamedArguments, $missingParameterMessage, $unknownParameterMessage);
 		foreach ($addedErrors as $error) {
 			$errors[] = $error;
 		}
@@ -534,7 +523,6 @@ static function (Type $type, callable $traverse) use (&$returnTemplateTypes): Ty
 
 					$errors[] = RuleErrorBuilder::message(sprintf($unresolvableTemplateTypeMessage, $name))
 						->identifier('argument.templateType')
-						->line($funcCallLine)
 						->tip('See: https://phpstan.org/blog/solving-phpstan-error-unable-to-resolve-template-type')
 						->build();
 				}
@@ -546,7 +534,6 @@ static function (Type $type, callable $traverse) use (&$returnTemplateTypes): Ty
 			) {
 				$errors[] = RuleErrorBuilder::message($unresolvableReturnTypeMessage)
 					->identifier(sprintf('%s.unresolvableReturnType', $nodeType))
-					->line($funcCallLine)
 					->build();
 			}
 		}
@@ -560,7 +547,6 @@ static function (Type $type, callable $traverse) use (&$returnTemplateTypes): Ty
 	 */
 	private function processArguments(
 		ParametersAcceptor $parametersAcceptor,
-		int $line,
 		bool $isBuiltin,
 		array $arguments,
 		bool $hasNamedArguments,
@@ -675,7 +661,6 @@ private function processArguments(
 
 				$errors[] = RuleErrorBuilder::message(sprintf($missingParameterMessage, sprintf('%s (%s)', $parameter->getName(), $parameter->getType()->describe(VerbosityLevel::typeOnly()))))
 					->identifier('argument.missing')
-					->line($line)
 					->build();
 			}
 		}