Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 57028ec

Browse filesBrowse files
bug #58703 [TwigBridge] Use INTERNAL_VAR_NAME instead of getVarName (pan93412)
This PR was squashed before being merged into the 7.2 branch. Discussion ---------- [TwigBridge] Use INTERNAL_VAR_NAME instead of getVarName | Q | A | ------------- | --- | Branch? | 7.2 | Bug fix? | yes | New feature? | no | Deprecations? | no | Issues | Fix #58702 | License | MIT Reapply ab116bb. This PR is tested and should be worked. ![CleanShot 2024-10-29 at 10 55 17@2x](https://github.com/user-attachments/assets/0639d8b1-a471-4ea0-ae52-b9fd815fd4f8) Commits ------- eea0dd3 [TwigBridge] Use INTERNAL_VAR_NAME instead of getVarName
2 parents 91588b3 + eea0dd3 commit 57028ec
Copy full SHA for 57028ec

File tree

1 file changed

+2
-3
lines changed
Filter options

1 file changed

+2
-3
lines changed

‎src/Symfony/Bridge/Twig/NodeVisitor/TranslationDefaultDomainNodeVisitor.php

Copy file name to clipboardExpand all lines: src/Symfony/Bridge/Twig/NodeVisitor/TranslationDefaultDomainNodeVisitor.php
+2-3Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,9 +55,8 @@ public function enterNode(Node $node, Environment $env): Node
5555
return $node;
5656
}
5757

58-
$var = $this->getVarName();
59-
$name = class_exists(AssignContextVariable::class) ? new AssignContextVariable($var, $node->getTemplateLine()) : new AssignNameExpression($var, $node->getTemplateLine());
60-
$this->scope->set('domain', class_exists(ContextVariable::class) ? new ContextVariable($var, $node->getTemplateLine()) : new NameExpression($var, $node->getTemplateLine()));
58+
$name = class_exists(AssignContextVariable::class) ? new AssignContextVariable(self::INTERNAL_VAR_NAME, $node->getTemplateLine()) : new AssignNameExpression(self::INTERNAL_VAR_NAME, $node->getTemplateLine());
59+
$this->scope->set('domain', class_exists(ContextVariable::class) ? new ContextVariable(self::INTERNAL_VAR_NAME, $node->getTemplateLine()) : new NameExpression(self::INTERNAL_VAR_NAME, $node->getTemplateLine()));
6160

6261
if (class_exists(Nodes::class)) {
6362
return new SetNode(false, new Nodes([$name]), new Nodes([$node->getNode('expr')]), $node->getTemplateLine());

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.