Move to PHPStan level 5

Author: fabpot

phpstan.neon.dist

@@ -2,7 +2,7 @@ includes:
     - phpstan-baseline.neon
 
 parameters:
-    level: 4
+    level: 5
     paths:
         - src
     excludePaths:

src/Environment.php

@@ -495,7 +495,7 @@ public function resolveTemplate(string|TemplateWrapper|array $names): TemplateWr
             return $this->load($name);
         }
 
-        throw new LoaderError(\sprintf('Unable to find one of the following templates: "%s".', implode('", "', $names)));
+        throw new LoaderError(\sprintf('Unable to find one of the following templates: "%s".', implode('", "', array_map(fn ($name) => $name instanceof TemplateWrapper ? $name->getTemplateName() : $name, $names))));
     }
 
     /**

src/ExpressionParser/Infix/ArgumentsTrait.php

@@ -12,6 +12,7 @@
 namespace Twig\ExpressionParser\Infix;
 
 use Twig\Error\SyntaxError;
+use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Expression\ArrayExpression;
 use Twig\Node\Expression\Binary\SetBinary;
 use Twig\Node\Expression\Unary\SpreadUnary;
@@ -27,6 +28,7 @@ private function parseCallableArguments(Parser $parser, int $line, bool $parseOp
     {
         $arguments = new ArrayExpression([], $line);
         foreach ($this->parseNamedArguments($parser, $parseOpenParenthesis) as $k => $n) {
+            /** @var AbstractExpression $n */
             $arguments->addElement($n, new LocalVariable($k, $line));
         }
 

src/ExpressionParser/Infix/FunctionExpressionParser.php

@@ -20,6 +20,7 @@
 use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Expression\MacroReferenceExpression;
 use Twig\Node\Expression\Variable\ContextVariable;
+use Twig\Node\Expression\Variable\TemplateVariable;
 use Twig\Parser;
 use Twig\Token;
 
@@ -40,7 +41,10 @@ public function parse(Parser $parser, AbstractExpression $expr, Token $token): A
         $name = $expr->getAttribute('name');
 
         if (null !== $alias = $parser->getImportedSymbol('function', $name)) {
-            return new MacroReferenceExpression($alias['node']->getNode('var'), $alias['name'], $this->parseCallableArguments($parser, $line, false), $line);
+            /** @var TemplateVariable $var */
+            $var = $alias['node']->getNode('var');
+
+            return new MacroReferenceExpression($var, $alias['name'], $this->parseCallableArguments($parser, $line, false), $line);
         }
 
         $args = $this->parseNamedArguments($parser, false);

src/ExpressionParser/Infix/IsExpressionParser.php

@@ -19,6 +19,7 @@
 use Twig\Node\Expression\ArrayExpression;
 use Twig\Node\Expression\MacroReferenceExpression;
 use Twig\Node\Expression\Variable\ContextVariable;
+use Twig\Node\Expression\Variable\TemplateVariable;
 use Twig\Node\Nodes;
 use Twig\Parser;
 use Twig\Token;
@@ -43,7 +44,9 @@ public function parse(Parser $parser, AbstractExpression $expr, Token $token): A
         }
 
         if ('defined' === $test->getName() && $expr instanceof ContextVariable && null !== $alias = $parser->getImportedSymbol('function', $expr->getAttribute('name'))) {
-            $expr = new MacroReferenceExpression($alias['node']->getNode('var'), $alias['name'], new ArrayExpression([], $expr->getTemplateLine()), $expr->getTemplateLine());
+            /** @var TemplateVariable $var */
+            $var = $alias['node']->getNode('var');
+            $expr = new MacroReferenceExpression($var, $alias['name'], new ArrayExpression([], $expr->getTemplateLine()), $expr->getTemplateLine());
         }
 
         return new ($test->getNodeClass())($expr, $test, $arguments, $stream->getCurrent()->getLine());

src/Extension/CoreExtension.php

@@ -2041,7 +2041,10 @@ public static function parseAttributeFunction(Parser $parser, Node $fakeNode, $a
         $dep->triggerDeprecation($src->getPath() ?: $src->getName(), $line);
         */
 
-        return new GetAttrExpression($args[0], $args[1], $args[2] ?? null, Template::ANY_CALL, $line);
+        /** @var ArrayExpression|ContextVariable|null $arguments */
+        $arguments = $args[2] ?? null;
+
+        return new GetAttrExpression($args[0], $args[1], $arguments, Template::ANY_CALL, $line);
     }
 
     /**

src/Node/Expression/Binary/MatchesBinary.php

@@ -13,13 +13,13 @@
 
 use Twig\Compiler;
 use Twig\Error\SyntaxError;
+use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Expression\ConstantExpression;
 use Twig\Node\Expression\ReturnBoolInterface;
-use Twig\Node\Node;
 
 class MatchesBinary extends AbstractBinary implements ReturnBoolInterface
 {
-    public function __construct(Node $left, Node $right, int $lineno)
+    public function __construct(AbstractExpression $left, AbstractExpression $right, int $lineno)
     {
         if ($right instanceof ConstantExpression) {
             $regexp = $right->getAttribute('value');

src/Node/Expression/Filter/DefaultFilter.php

@@ -42,6 +42,7 @@ public function __construct(AbstractExpression $node, TwigFilter $filter, Node $
 
         if ('default' === $name && ($node instanceof ContextVariable || $node instanceof GetAttrExpression)) {
             $test = new DefinedTest(clone $node, new TwigTest('defined'), new EmptyNode(), $node->getTemplateLine());
+            /** @var AbstractExpression $false */
             $false = \count($arguments) ? $arguments->getNode(0) : new ConstantExpression('', $node->getTemplateLine());
 
             $node = new ConditionalTernary($test, $default, $false, $node->getTemplateLine());

src/Node/Expression/ListExpression.php

@@ -17,7 +17,7 @@
 class ListExpression extends AbstractExpression
 {
     /**
-     * @param array<ContextVariable> $items
+     * @param array<AbstractExpression> $items
      */
     public function __construct(array $items, int $lineno)
     {

src/Node/IfNode.php

@@ -14,6 +14,7 @@
 
 use Twig\Attribute\YieldReady;
 use Twig\Compiler;
+use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Expression\ReturnPrimitiveTypeInterface;
 use Twig\Node\Expression\Test\TrueTest;
 use Twig\TwigTest;
@@ -31,6 +32,7 @@ public function __construct(Node $tests, ?Node $else, int $lineno)
         for ($i = 0, $count = \count($tests); $i < $count; $i += 2) {
             $test = $tests->getNode((string) $i);
             if (!$test instanceof ReturnPrimitiveTypeInterface) {
+                /** @var AbstractExpression $test */
                 $tests->setNode($i, new TrueTest($test, new TwigTest('true'), null, $test->getTemplateLine()));
             }
         }