Fix deprecation in Twig 3.12

netgen-layouts · Sep 6, 2024 · 6dab9e7 · 6dab9e7
1 parent fbf49e7
commit 6dab9e7
Show file tree

Hide file tree

Showing 6 changed files with 29 additions and 4 deletions.
diff --git a/bundles/LayoutsBundle/Templating/Twig/Node/DefaultContext.php b/bundles/LayoutsBundle/Templating/Twig/Node/DefaultContext.php
@@ -6,6 +6,7 @@
 
 use Twig\Attribute\YieldReady;
 use Twig\Compiler;
+use Twig\Environment;
 use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Node;
 
@@ -14,7 +15,9 @@ final class DefaultContext extends Node
 {
     public function __construct(AbstractExpression $expr, int $line = 0, ?string $tag = null)
     {
-        parent::__construct(['expr' => $expr], [], $line, $tag);
+        Environment::MAJOR_VERSION === 3 && Environment::MINOR_VERSION < 12 ?
+            parent::__construct(['expr' => $expr], [], $line, $tag) :
+            parent::__construct(['expr' => $expr], [], $line);
     }
 
     public function compile(Compiler $compiler): void

diff --git a/bundles/LayoutsBundle/Templating/Twig/Node/RenderZone.php b/bundles/LayoutsBundle/Templating/Twig/Node/RenderZone.php
@@ -26,7 +26,9 @@ public function __construct(AbstractExpression $zone, ?AbstractExpression $conte
             $nodes['context'] = $context;
         }
 
-        parent::__construct($nodes, [], $line, $tag);
+        Environment::MAJOR_VERSION === 3 && Environment::MINOR_VERSION < 12 ?
+            parent::__construct($nodes, [], $line, $tag) :
+            parent::__construct($nodes, [], $line);
     }
 
     public function compile(Compiler $compiler): void

diff --git a/phpstan.neon b/phpstan.neon
@@ -8,6 +8,8 @@ parameters:
     dynamicConstantNames:
         - Symfony\Component\HttpKernel\Kernel::VERSION_ID
         - Twig\Environment::VERSION_ID
+        - Twig\Environment::MAJOR_VERSION
+        - Twig\Environment::MINOR_VERSION
 
     ignoreErrors:
         # Doctrine DBAL

diff --git a/phpstan.tests.neon b/phpstan.tests.neon
@@ -53,6 +53,8 @@ parameters:
             message: '#Undefined variable: \$this#'
             path: tests/lib/Transfer/Output/Visitor/Integration
 
+        - "#Call to function method_exists\\(\\) with .* and 'setNodeTag' will always evaluate to true.#"
+
         -
             message: "#Offset 'db' does not exist on array#"
             path: tests/lib/Persistence/Doctrine/DatabaseTrait.php
diff --git a/tests/bundles/LayoutsBundle/Templating/Twig/TokenParser/DefaultContextTest.php b/tests/bundles/LayoutsBundle/Templating/Twig/TokenParser/DefaultContextTest.php
@@ -15,6 +15,8 @@
 use Twig\Parser;
 use Twig\Source;
 
+use function method_exists;
+
 final class DefaultContextTest extends TestCase
 {
     private Environment $environment;
@@ -42,10 +44,14 @@ protected function setUp(): void
      *
      * @dataProvider compileDataProvider
      */
-    public function testCompile(string $source, DefaultContextNode $node): void
+    public function testCompile(string $source, DefaultContextNode $node, string $tag): void
     {
         $stream = $this->environment->tokenize(new Source($source, ''));
 
+        if (method_exists($node, 'setNodeTag')) {
+            $node->setNodeTag($tag);
+        }
+
         self::assertSame((string) $node, (string) $this->parser->parse($stream)->getNode('body')->getNode('0'));
     }
 
@@ -75,6 +81,7 @@ public static function compileDataProvider(): iterable
                     1,
                     'nglayouts_default_context',
                 ),
+                'nglayouts_default_context',
             ],
             [
                 '{% nglayouts_default_context "foo" %}',
@@ -83,6 +90,7 @@ public static function compileDataProvider(): iterable
                     1,
                     'nglayouts_default_context',
                 ),
+                'nglayouts_default_context',
             ],
         ];
     }

diff --git a/tests/bundles/LayoutsBundle/Templating/Twig/TokenParser/RenderZoneTest.php b/tests/bundles/LayoutsBundle/Templating/Twig/TokenParser/RenderZoneTest.php
@@ -15,6 +15,8 @@
 use Twig\Parser;
 use Twig\Source;
 
+use function method_exists;
+
 final class RenderZoneTest extends TestCase
 {
     private Environment $environment;
@@ -42,10 +44,14 @@ protected function setUp(): void
      *
      * @dataProvider compileDataProvider
      */
-    public function testCompile(string $source, RenderZoneNode $node): void
+    public function testCompile(string $source, RenderZoneNode $node, string $tag): void
     {
         $stream = $this->environment->tokenize(new Source($source, ''));
 
+        if (method_exists($node, 'setNodeTag')) {
+            $node->setNodeTag($tag);
+        }
+
         self::assertSame((string) $node, (string) $this->parser->parse($stream)->getNode('body')->getNode('0'));
     }
 
@@ -76,6 +82,7 @@ public static function compileDataProvider(): iterable
                     1,
                     'nglayouts_render_zone',
                 ),
+                'nglayouts_render_zone',
             ],
             [
                 '{% nglayouts_render_zone zone context="json" %}',
@@ -85,6 +92,7 @@ public static function compileDataProvider(): iterable
                     1,
                     'nglayouts_render_zone',
                 ),
+                'nglayouts_render_zone',
             ],
         ];
     }