Yield ready update #22 (#23)

* yield ready update #22 * update twig * remove twig 2 * update php requirement * update component render * migrate configuration * migrate configuration * update phpunit
giorgiopogliani · Sep 12, 2024 · 69c39d2 · 69c39d2
1 parent d31cd31
commit 69c39d2
Show file tree

Hide file tree

Showing 5 changed files with 28 additions and 41 deletions.
diff --git a/.github/workflows/run-tests.yml b/.github/workflows/run-tests.yml
@@ -9,7 +9,7 @@ jobs:
       fail-fast: true
       matrix:
         os: [ubuntu-latest]
-        php: [8.0, 8.1, 8.2]
+        php: [8.1, 8.2, 8.3]
         dependency-version: [prefer-lowest, prefer-stable]
 
     name: P${{ matrix.php }} - ${{ matrix.dependency-version }} - ${{ matrix.os }}

diff --git a/composer.json b/composer.json
@@ -16,15 +16,14 @@
         }
     ],
     "require": {
-        "php": "^7.4|^8.0",
-        "twig/twig": "^2.12.5 || >=3.0 <3.9"
+        "php": ">= 8.1 < 8.4",
+        "twig/twig": ">=3.9"
     },
     "require-dev": {
         "friendsofphp/php-cs-fixer": "^3.0",
-        "pestphp/pest": "^1.0",
-        "phpunit/phpunit": "^9.3",
-        "symfony/var-dumper": "^5.2",
-        "vimeo/psalm": "^3.11"
+        "pestphp/pest": "^2.0",
+        "phpunit/phpunit": "^10.1",
+        "symfony/var-dumper": "^5.2"
     },
     "autoload": {
         "psr-4": {

diff --git a/phpunit.xml.dist b/phpunit.xml.dist
@@ -1,31 +1,13 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<phpunit backupGlobals="false"
-         backupStaticAttributes="false"
-         bootstrap="vendor/autoload.php"
-         colors="true"
-         convertErrorsToExceptions="true"
-         convertNoticesToExceptions="true"
-         convertWarningsToExceptions="true"
-         processIsolation="false"
-         stopOnFailure="false"
-         verbose="true"
->
-    <testsuites>
-        <testsuite name="Digital Test Suite">
-            <directory>tests</directory>
-        </testsuite>
-    </testsuites>
-    <coverage>
-        <include>
-            <directory suffix=".php">./src</directory>
-        </include>
-        <report>
-            <html outputDirectory="build/coverage"/>
-            <text outputFile="build/coverage.txt"/>
-            <clover outputFile="build/logs/clover.xml"/>
-        </report>
-    </coverage>
-    <logging>
-        <junit outputFile="build/report.junit.xml"/>
-    </logging>
+<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" backupGlobals="false" bootstrap="vendor/autoload.php" colors="true" processIsolation="false" stopOnFailure="false" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" cacheDirectory=".phpunit.cache" backupStaticProperties="false">
+  <testsuites>
+    <testsuite name="Digital Test Suite">
+      <directory>tests</directory>
+    </testsuite>
+  </testsuites>
+  <source>
+    <include>
+      <directory suffix=".php">src</directory>
+    </include>
+  </source>
 </phpunit>
diff --git a/src/Node/ComponentNode.php b/src/Node/ComponentNode.php
@@ -6,13 +6,15 @@
 use Performing\TwigComponents\Configuration;
 use Performing\TwigComponents\View\ComponentAttributeBag;
 use Performing\TwigComponents\View\ComponentSlot;
+use Twig\Attribute\YieldReady;
 use Twig\Compiler;
 use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Expression\ConstantExpression;
 
 use Twig\Node\IncludeNode;
 use Twig\Node\Node;
 
+#[YieldReady]
 final class ComponentNode extends IncludeNode
 {
     const DYNAMIC_COMPONENT_NAME = 'dynamic-component';
@@ -43,16 +45,17 @@ public function compile(Compiler $compiler): void
             ->write('$slotsStack = $slotsStack ?? [];' . PHP_EOL)
             ->write('$slotsStack[] = $slots ?? [];' . PHP_EOL)
             ->write('$slots = [];' . PHP_EOL)
-            ->write("ob_start();"  . PHP_EOL)
+            ->write('$slot =  implode("", iterator_to_array((function () use (&$slots, &$context) {'  . PHP_EOL)
             ->subcompile($this->getNode('slot'))
-            ->write('$slot = ob_get_clean();' . PHP_EOL)
-            ->write(sprintf('$%s->display(', $template));
+            ->write("})() ?? new \EmptyIterator()));"  . PHP_EOL)
+            ->write(sprintf('yield $%s->render(', $template));
 
         $this->addTemplateArguments($compiler);
 
         $compiler
             ->raw(");\n")
             ->write('$slots = array_pop($slotsStack);' . PHP_EOL)
+            ->write("yield '';")
             ->write("}\n");
     }
 

diff --git a/src/Node/SlotNode.php b/src/Node/SlotNode.php
@@ -3,12 +3,14 @@
 namespace Performing\TwigComponents\Node;
 
 use Performing\TwigComponents\View\ComponentSlot;
+use Twig\Attribute\YieldReady;
 use Twig\Compiler;
 use Twig\Node\Expression\AbstractExpression;
 use Twig\Node\Node;
 
 use Twig\Node\NodeOutputInterface;
 
+#[YieldReady]
 final class SlotNode extends Node implements NodeOutputInterface
 {
     public function __construct($name, $body, ?AbstractExpression $variables, int $lineno = 0)
@@ -25,9 +27,10 @@ public function compile(Compiler $compiler): void
         $name = $this->getAttribute('name');
 
         $compiler
-            ->write('ob_start();')
+            ->write('$body = (function () use (&$slots, &$context) {')
             ->subcompile($this->getNode('body'))
-            ->write('$body = ob_get_clean();' . PHP_EOL)
+            ->write('})() ?? new \EmptyIterator();' . PHP_EOL)
+            ->write('$body = implode("", iterator_to_array($body));' . PHP_EOL)
             ->write("\$slots['$name'] = new " . ComponentSlot::class . "(\$body, ");
 
         if ($this->hasNode('variables')) {