[Translation] Fix constant domain resolution in PhpAstExtractor

Author: VincentLanglet

Extractor/PhpAstExtractor.php

@@ -46,6 +46,11 @@ public function extract(iterable|string $resource, MessageCatalogue $catalogue):
     {
         foreach ($this->extractFiles($resource) as $file) {
             $traverser = new NodeTraverser();
+
+            // This is needed to resolve namespaces in class methods/constants.
+            $nameResolver = new NodeVisitor\NameResolver();
+            $traverser->addVisitor($nameResolver);
+
             /** @var AbstractVisitor&NodeVisitor $visitor */
             foreach ($this->visitors as $visitor) {
                 $visitor->initialize($catalogue, $file, $this->prefix);

Extractor/Visitor/AbstractVisitor.php

@@ -119,6 +119,17 @@ private function getStringValue(Node $node): ?string
             return $node->expr->value;
         }
 
+        if ($node instanceof Node\Expr\ClassConstFetch) {
+            try {
+                $reflection = new \ReflectionClass($node->class->toString());
+                $constant = $reflection->getReflectionConstant($node->name->toString());
+                if (false !== $constant && \is_string($constant->getValue())) {
+                    return $constant->getValue();
+                }
+            } catch (\ReflectionException) {
+            }
+        }
+
         return null;
     }
 }

Extractor/Visitor/ConstraintVisitor.php

@@ -32,6 +32,11 @@ public function beforeTraverse(array $nodes): ?Node
     }
 
     public function enterNode(Node $node): ?Node
+    {
+        return null;
+    }
+
+    public function leaveNode(Node $node): ?Node
     {
         if (!$node instanceof Node\Expr\New_ && !$node instanceof Node\Attribute) {
             return null;
@@ -100,11 +105,6 @@ public function enterNode(Node $node): ?Node
         return null;
     }
 
-    public function leaveNode(Node $node): ?Node
-    {
-        return null;
-    }
-
     public function afterTraverse(array $nodes): ?Node
     {
         return null;

Extractor/Visitor/TransMethodVisitor.php

@@ -25,6 +25,11 @@ public function beforeTraverse(array $nodes): ?Node
     }
 
     public function enterNode(Node $node): ?Node
+    {
+        return null;
+    }
+
+    public function leaveNode(Node $node): ?Node
     {
         if (!$node instanceof Node\Expr\MethodCall && !$node instanceof Node\Expr\FuncCall) {
             return null;
@@ -53,11 +58,6 @@ public function enterNode(Node $node): ?Node
         return null;
     }
 
-    public function leaveNode(Node $node): ?Node
-    {
-        return null;
-    }
-
     public function afterTraverse(array $nodes): ?Node
     {
         return null;

Extractor/Visitor/TranslatableMessageVisitor.php

@@ -25,6 +25,11 @@ public function beforeTraverse(array $nodes): ?Node
     }
 
     public function enterNode(Node $node): ?Node
+    {
+        return null;
+    }
+
+    public function leaveNode(Node $node): ?Node
     {
         if (!$node instanceof Node\Expr\New_) {
             return null;
@@ -53,11 +58,6 @@ public function enterNode(Node $node): ?Node
         return null;
     }
 
-    public function leaveNode(Node $node): ?Node
-    {
-        return null;
-    }
-
     public function afterTraverse(array $nodes): ?Node
     {
         return null;

Tests/Extractor/PhpAstExtractorTest.php

@@ -20,6 +20,8 @@
 
 final class PhpAstExtractorTest extends TestCase
 {
+    public const OTHER_DOMAIN = 'not_messages';
+
     /**
      * @dataProvider resourcesProvider
      */
@@ -124,6 +126,7 @@ public function testExtraction(iterable|string $resource)
                 'variable-assignation-inlined-with-named-arguments-in-trans-method' => 'prefixvariable-assignation-inlined-with-named-arguments-in-trans-method',
                 'mix-named-arguments-without-parameters' => 'prefixmix-named-arguments-without-parameters',
                 'mix-named-arguments-disordered' => 'prefixmix-named-arguments-disordered',
+                'const-domain' => 'prefixconst-domain',
             ],
             'validators' => [
                 'message-in-constraint-attribute' => 'prefixmessage-in-constraint-attribute',

Tests/Fixtures/extractor-ast/translation.html.php

@@ -62,3 +62,8 @@
 <?php echo $view['translator']->trans('mix-named-arguments-disordered', domain: 'not_messages', parameters: []); ?>
 
 <?php echo $view['translator']->trans(...); // should not fail ?>
+
+<?php
+use Symfony\Component\Translation\Tests\Extractor\PhpAstExtractorTest;
+echo $view['translator']->trans('const-domain', [], PhpAstExtractorTest::OTHER_DOMAIN);
+?>