[Translation] Fix extraction when dealing with VariadicPlaceholder parameters

Kocal · Kocal · commit 2030c265adb5 · 2022-12-03T16:04:18.000+01:00
diff --git a/src/Symfony/Component/Translation/Extractor/Visitor/AbstractVisitor.php b/src/Symfony/Component/Translation/Extractor/Visitor/AbstractVisitor.php
@@ -42,32 +42,31 @@ protected function addMessageToCatalogue(string $message, ?string $domain, int $
 
     protected function getStringArguments(Node\Expr\CallLike|Node\Attribute|Node\Expr\New_ $node, int|string $index, bool $indexIsRegex = false): array
     {
-        $args = $node instanceof Node\Expr\CallLike ? $node->getArgs() : $node->args;
-
         if (\is_string($index)) {
             return $this->getStringNamedArguments($node, $index, $indexIsRegex);
         }
 
+        $args = $node instanceof Node\Expr\CallLike ? $node->getRawArgs() : $node->args;
+
         if (\count($args) < $index) {
             return [];
         }
 
-        /** @var Node\Arg $arg */
-        $arg = $args[$index];
-        if (!$result = $this->getStringValue($arg->value)) {
-            return [];
+        if (($arg = $args[$index]) instanceof Node\Arg) {
+            if ($result = $this->getStringValue($arg->value)) {
+                return [$result];
+            }
         }
 
-        return [$result];
+        return [];
     }
 
     protected function hasNodeNamedArguments(Node\Expr\CallLike|Node\Attribute|Node\Expr\New_ $node): bool
     {
-        $args = $node instanceof Node\Expr\CallLike ? $node->getArgs() : $node->args;
+        $args = $node instanceof Node\Expr\CallLike ? $node->getRawArgs() : $node->args;
 
-        /** @var Node\Arg $arg */
         foreach ($args as $arg) {
-            if (null !== $arg->name) {
+            if ($arg instanceof Node\Arg && null !== $arg->name) {
                 return true;
             }
         }

Original file line number	Diff line number	Diff line change
`@@ -42,32 +42,31 @@ protected function addMessageToCatalogue(string $message, ?string $domain, int $`
`42`	`42`
`43`	`43`	`protected function getStringArguments(Node\Expr\CallLike\|Node\Attribute\|Node\Expr\New_ $node, int\|string $index, bool $indexIsRegex = false): array`
`44`	`44`	`{`
`45`		`- $args = $node instanceof Node\Expr\CallLike ? $node->getArgs() : $node->args;`
`46`		`-`
`47`	`45`	`if (\is_string($index)) {`
`48`	`46`	`return $this->getStringNamedArguments($node, $index, $indexIsRegex);`
`49`	`47`	`}`
`50`	`48`
	`49`	`+ $args = $node instanceof Node\Expr\CallLike ? $node->getRawArgs() : $node->args;`
	`50`	`+`
`51`	`51`	`if (\count($args) < $index) {`
`52`	`52`	`return [];`
`53`	`53`	`}`
`54`	`54`
`55`		`- /** @var Node\Arg $arg */`
`56`		`- $arg = $args[$index];`
`57`		`- if (!$result = $this->getStringValue($arg->value)) {`
`58`		`- return [];`
	`55`	`+ if (($arg = $args[$index]) instanceof Node\Arg) {`
	`56`	`+ if ($result = $this->getStringValue($arg->value)) {`
	`57`	`+ return [$result];`
	`58`	`+ }`
`59`	`59`	`}`
`60`	`60`
`61`		`- return [$result];`
	`61`	`+ return [];`
`62`	`62`	`}`
`63`	`63`
`64`	`64`	`protected function hasNodeNamedArguments(Node\Expr\CallLike\|Node\Attribute\|Node\Expr\New_ $node): bool`
`65`	`65`	`{`
`66`		`- $args = $node instanceof Node\Expr\CallLike ? $node->getArgs() : $node->args;`
	`66`	`+ $args = $node instanceof Node\Expr\CallLike ? $node->getRawArgs() : $node->args;`
`67`	`67`
`68`		`- /** @var Node\Arg $arg */`
`69`	`68`	`foreach ($args as $arg) {`
`70`		`- if (null !== $arg->name) {`
	`69`	`+ if ($arg instanceof Node\Arg && null !== $arg->name) {`
`71`	`70`	`return true;`
`72`	`71`	`}`
`73`	`72`	`}`