use reproducible variable names in the default domain node visitor

xabbuh · xabbuh · commit 45183e082e46 · 2024-10-29T11:34:51.000+01:00
diff --git a/src/Symfony/Bridge/Twig/NodeVisitor/TranslationDefaultDomainNodeVisitor.php b/src/Symfony/Bridge/Twig/NodeVisitor/TranslationDefaultDomainNodeVisitor.php
@@ -20,8 +20,7 @@
 use Twig\Node\Expression\ConstantExpression;
 use Twig\Node\Expression\FilterExpression;
 use Twig\Node\Expression\NameExpression;
-use Twig\Node\Expression\Variable\AssignContextVariable;
-use Twig\Node\Expression\Variable\ContextVariable;
+use Twig\Node\Expression\Variable\LocalVariable;
 use Twig\Node\ModuleNode;
 use Twig\Node\Node;
 use Twig\Node\Nodes;
@@ -33,8 +32,6 @@
  */
 final class TranslationDefaultDomainNodeVisitor implements NodeVisitorInterface
 {
-    private const INTERNAL_VAR_NAME = '__internal_trans_default_domain';
-
     private Scope $scope;
 
     public function __construct()
@@ -55,13 +52,22 @@ public function enterNode(Node $node, Environment $env): Node
                 return $node;
             }
 
-            $name = class_exists(AssignContextVariable::class) ? new AssignContextVariable(self::INTERNAL_VAR_NAME, $node->getTemplateLine()) : new AssignNameExpression(self::INTERNAL_VAR_NAME, $node->getTemplateLine());
-            $this->scope->set('domain', class_exists(ContextVariable::class) ? new ContextVariable(self::INTERNAL_VAR_NAME, $node->getTemplateLine()) : new NameExpression(self::INTERNAL_VAR_NAME, $node->getTemplateLine()));
-
             if (class_exists(Nodes::class)) {
+                $name = new LocalVariable(null, $node->getTemplateLine());
+                $this->scope->set('domain', $name);
+
                 return new SetNode(false, new Nodes([$name]), new Nodes([$node->getNode('expr')]), $node->getTemplateLine());
             }
 
+            $var = '__internal_trans_default_domain';
+
+            if (null !== $templateName = $node->getTemplateName()) {
+                $var .= hash('xxh128', $templateName);
+            }
+
+            $name = new AssignNameExpression($var, $node->getTemplateLine());
+            $this->scope->set('domain', new NameExpression($var, $node->getTemplateLine()));
+
             return new SetNode(false, new Node([$name]), new Node([$node->getNode('expr')]), $node->getTemplateLine());
         }
 
