From: Fabien Potencier Date: Mon, 7 Jun 2010 11:49:29 +0000 (+0200) Subject: changed order of Twig_Node_Expression_GetAttr constructor arguments X-Git-Url: http://git.silmor.de/gitweb/?a=commitdiff_plain;h=dc37698ce0d4d7fe647c0f9076e843068e34129c;p=konrad%2Ftwig.git changed order of Twig_Node_Expression_GetAttr constructor arguments --- diff --git a/lib/Twig/ExpressionParser.php b/lib/Twig/ExpressionParser.php index a86d7c1..af6eb84 100644 --- a/lib/Twig/ExpressionParser.php +++ b/lib/Twig/ExpressionParser.php @@ -391,7 +391,7 @@ class Twig_ExpressionParser $this->parser->getStream()->expect(Twig_Token::OPERATOR_TYPE, ']'); } - return new Twig_Node_Expression_GetAttr($node, $arg, $arguments, $lineno, $type); + return new Twig_Node_Expression_GetAttr($node, $arg, $arguments, $type, $lineno); } public function parseFilterExpression($node) diff --git a/lib/Twig/Node/Expression/GetAttr.php b/lib/Twig/Node/Expression/GetAttr.php index 583c890..cd905c0 100644 --- a/lib/Twig/Node/Expression/GetAttr.php +++ b/lib/Twig/Node/Expression/GetAttr.php @@ -15,7 +15,7 @@ class Twig_Node_Expression_GetAttr extends Twig_Node_Expression const TYPE_ARRAY = 'array'; const TYPE_METHOD = 'method'; - public function __construct(Twig_Node_Expression $node, Twig_Node_Expression $attribute, Twig_NodeInterface $arguments, $lineno, $type = self::TYPE_ANY) + public function __construct(Twig_Node_Expression $node, Twig_Node_Expression $attribute, Twig_NodeInterface $arguments, $type, $lineno) { parent::__construct(array('node' => $node, 'attribute' => $attribute, 'arguments' => $arguments), array('type' => $type), $lineno); } diff --git a/test/Twig/Tests/Node/Expression/GetAttrTest.php b/test/Twig/Tests/Node/Expression/GetAttrTest.php index 4694ae2..28eb30b 100644 --- a/test/Twig/Tests/Node/Expression/GetAttrTest.php +++ b/test/Twig/Tests/Node/Expression/GetAttrTest.php @@ -24,7 +24,7 @@ class Twig_Tests_Node_Expression_GetAttrTest extends Twig_Tests_Node_TestCase new Twig_Node_Expression_Name('foo', 0), new Twig_Node_Expression_Constant('bar', 0), )); - $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, 0, Twig_Node_Expression_GetAttr::TYPE_ARRAY); + $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, Twig_Node_Expression_GetAttr::TYPE_ARRAY, 0); $this->assertEquals($expr, $node->node); $this->assertEquals($attr, $node->attribute); @@ -48,10 +48,10 @@ class Twig_Tests_Node_Expression_GetAttrTest extends Twig_Tests_Node_TestCase $expr = new Twig_Node_Expression_Name('foo', 0); $attr = new Twig_Node_Expression_Constant('bar', 0); $args = new Twig_Node(); - $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, 0); + $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, Twig_Node_Expression_GetAttr::TYPE_ANY, 0); $tests[] = array($node, '$this->getAttribute($this->getContext($context, \'foo\'), "bar", array(), "any")'); - $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, 0, Twig_Node_Expression_GetAttr::TYPE_ARRAY); + $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, Twig_Node_Expression_GetAttr::TYPE_ARRAY, 0); $tests[] = array($node, '$this->getAttribute($this->getContext($context, \'foo\'), "bar", array(), "array")'); @@ -59,7 +59,7 @@ class Twig_Tests_Node_Expression_GetAttrTest extends Twig_Tests_Node_TestCase new Twig_Node_Expression_Name('foo', 0), new Twig_Node_Expression_Constant('bar', 0), )); - $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, 0, Twig_Node_Expression_GetAttr::TYPE_METHOD); + $node = new Twig_Node_Expression_GetAttr($expr, $attr, $args, Twig_Node_Expression_GetAttr::TYPE_METHOD, 0); $tests[] = array($node, '$this->getAttribute($this->getContext($context, \'foo\'), "bar", array($this->getContext($context, \'foo\'), "bar", ), "method")'); return $tests;