geckon01/Twig
1
0
Fork 0
mirror of https://github.com/twigphp/Twig.git synced 2026-08-30 12:06:56 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use proper node names when they are integers

Browse Source
This commit is contained in:
Fabien Potencier
2024-08-26 09:59:54 +02:00
parent 4696eeba1d
commit cde9e7dcd4
11 changed files with 17 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files
extra/cache-extra/TokenParser/CacheTokenParser.php
+2 -2
View File
@@ -39,13 +39,13 @@ class CacheTokenParser extends AbstractTokenParser
if (1 !== \count($args)) {
throw new SyntaxError(\sprintf('The "ttl" modifier takes exactly one argument (%d given).', \count($args)), $stream->getCurrent()->getLine(), $stream->getSourceContext());
}
$ttl = $args->getNode('0');
$ttl = $args->getNode(0);
break;
case 'tags':
if (1 !== \count($args)) {
throw new SyntaxError(\sprintf('The "tags" modifier takes exactly one argument (%d given).', \count($args)), $stream->getCurrent()->getLine(), $stream->getSourceContext());
}
$tags = $args->getNode('0');
$tags = $args->getNode(0);
break;
default:
throw new SyntaxError(\sprintf('Unknown "%s" configuration.', $k), $stream->getCurrent()->getLine(), $stream->getSourceContext());
src/Node/Expression/Filter/DefaultFilter.php
+1 -1
View File
@@ -40,7 +40,7 @@ class DefaultFilter extends FilterExpression
if ('default' === $name && ($node instanceof NameExpression || $node instanceof GetAttrExpression)) {
$test = new DefinedTest(clone $node, new TwigTest('defined'), new Node(), $node->getTemplateLine());
$false = \count($arguments) ? $arguments->getNode('0') : new ConstantExpression('', $node->getTemplateLine());
$false = \count($arguments) ? $arguments->getNode(0) : new ConstantExpression('', $node->getTemplateLine());
$node = new ConditionalExpression($test, $default, $false, $node->getTemplateLine());
} else {
src/Node/Expression/FunctionNode/EnumCasesFunction.php
+1 -1
View File
@@ -15,7 +15,7 @@ class EnumCasesFunction extends FunctionExpression
if ($arguments->hasNode('enum')) {
$firstArgument = $arguments->getNode('enum');
} elseif ($arguments->hasNode('0')) {
$firstArgument = $arguments->getNode('0');
$firstArgument = $arguments->getNode(0);
} else {
$firstArgument = null;
}
src/Node/Expression/Test/ConstantTest.php
+2 -2
View File
@@ -36,13 +36,13 @@ class ConstantTest extends TestExpression
if ($this->getNode('arguments')->hasNode('1')) {
$compiler
->raw('get_class(')
->subcompile($this->getNode('arguments')->getNode('1'))
->subcompile($this->getNode('arguments')->getNode(1))
->raw(')."::".')
;
}
$compiler
->subcompile($this->getNode('arguments')->getNode('0'))
->subcompile($this->getNode('arguments')->getNode(0))
->raw('))')
;
}
src/Node/Expression/Test/DivisiblebyTest.php
+1 -1
View File
@@ -29,7 +29,7 @@ class DivisiblebyTest extends TestExpression
->raw('(0 == ')
->subcompile($this->getNode('node'))
->raw(' % ')
->subcompile($this->getNode('arguments')->getNode('0'))
->subcompile($this->getNode('arguments')->getNode(0))
->raw(')')
;
}
src/Node/Expression/Test/SameasTest.php
+1 -1
View File
@@ -27,7 +27,7 @@ class SameasTest extends TestExpression
->raw('(')
->subcompile($this->getNode('node'))
->raw(' === ')
->subcompile($this->getNode('arguments')->getNode('0'))
->subcompile($this->getNode('arguments')->getNode(0))
->raw(')')
;
}
src/Node/ModuleNode.php
+1 -1
View File
@@ -392,7 +392,7 @@ final class ModuleNode extends Node
$traitable = !$this->hasNode('parent') && 0 === \count($this->getNode('macros'));
if ($traitable) {
if ($this->getNode('body') instanceof BodyNode) {
$nodes = $this->getNode('body')->getNode('0');
$nodes = $this->getNode('body')->getNode(0);
} else {
$nodes = $this->getNode('body');
}
src/TokenParser/ForTokenParser.php
+3 -3
View File
@@ -49,12 +49,12 @@ final class ForTokenParser extends AbstractTokenParser
$stream->expect(Token::BLOCK_END_TYPE);
if (\count($targets) > 1) {
$keyTarget = $targets->getNode('0');
$keyTarget = $targets->getNode(0);
$keyTarget = new AssignNameExpression($keyTarget->getAttribute('name'), $keyTarget->getTemplateLine());
$valueTarget = $targets->getNode('1');
$valueTarget = $targets->getNode(1);
} else {
$keyTarget = new AssignNameExpression('_key', $lineno);
$valueTarget = $targets->getNode('0');
$valueTarget = $targets->getNode(0);
}
$valueTarget = new AssignNameExpression($valueTarget->getAttribute('name'), $valueTarget->getTemplateLine());
tests/ExpressionParserTest.php
+2 -2
View File
@@ -71,7 +71,7 @@ class ExpressionParserTest extends TestCase
$parser = new Parser($env);
$expected->setSourceContext($source);
$this->assertEquals($expected, $parser->parse($stream)->getNode('body')->getNode('0')->getNode('expr'));
$this->assertEquals($expected, $parser->parse($stream)->getNode('body')->getNode(0)->getNode('expr'));
}
/**
@@ -224,7 +224,7 @@ class ExpressionParserTest extends TestCase
$parser = new Parser($env);
$expected->setSourceContext($source);
$this->assertEquals($expected, $parser->parse($stream)->getNode('body')->getNode('0')->getNode('expr'));
$this->assertEquals($expected, $parser->parse($stream)->getNode('body')->getNode(0)->getNode('expr'));
}
public function getTestsForString()
tests/Node/Expression/ArrayTest.php
+1 -1
View File
@@ -22,7 +22,7 @@ class ArrayTest extends NodeTestCase
$elements = [new ConstantExpression('foo', 1), $foo = new ConstantExpression('bar', 1)];
$node = new ArrayExpression($elements, 1);
$this->assertEquals($foo, $node->getNode('1'));
$this->assertEquals($foo, $node->getNode(1));
}
public function getTests()
tests/NodeVisitor/OptimizerTest.php
+2 -2
View File
@@ -37,7 +37,7 @@ class OptimizerTest extends TestCase
$stream = $env->parse($env->tokenize(new Source('{{ block("foo") }}', 'index')));
$node = $stream->getNode('body')->getNode('0');
$node = $stream->getNode('body')->getNode(0);
$this->assertInstanceOf(BlockReferenceExpression::class, $node);
$this->assertTrue($node->getAttribute('output'));
@@ -49,7 +49,7 @@ class OptimizerTest extends TestCase
$stream = $env->parse($env->tokenize(new Source('{% extends "foo" %}{% block content %}{{ parent() }}{% endblock %}', 'index')));
$node = $stream->getNode('blocks')->getNode('content')->getNode('0')->getNode('body');
$node = $stream->getNode('blocks')->getNode('content')->getNode(0)->getNode('body');
$this->assertInstanceOf(ParentExpression::class, $node);
$this->assertTrue($node->getAttribute('output'));