Merge pull request #581 from flightphp/fix/bug-with-multiple-view-renders

Fix/bug with multiple view renders
8 months ago · 39ac87c54b
parent c8a1c88899 a68fc32204
commit 39ac87c54b
5 changed files with 138 additions and 4 deletions
--- a/flight/template/View.php
+++ b/flight/template/View.php
@ -20,6 +20,8 @@ class View
    /** File extension. */
    public string $extension = '.php';

+    public bool $preserveVars = true;
+
    /**
     * View variables.
     *
@ -114,11 +116,15 @@ class View
            throw new \Exception("Template file not found: {$normalized_path}.");
        }

-        if (\is_array($data)) {
+        \extract($this->vars);
+
+        if (\is_array($data) === true) {
+            \extract($data);
+
+            if ($this->preserveVars === true) {
                $this->vars = \array_merge($this->vars, $data);
            }
-
-        \extract($this->vars);
+        }

        include $this->template;
    }
--- a/tests/FlightTest.php
+++ b/tests/FlightTest.php
@ -22,6 +22,7 @@ class FlightTest extends TestCase
        $_REQUEST = [];
        Flight::init();
        Flight::setEngine(new Engine());
+        Flight::set('flight.views.path', __DIR__ . '/views');
    }

    protected function tearDown(): void
@ -354,4 +355,43 @@ class FlightTest extends TestCase

        $this->expectOutputString('Thisisaroutewithhtml');
    }
+
+    /** @dataProvider \tests\ViewTest::renderDataProvider */
+    public function testDoesNotPreserveVarsWhenFlagIsDisabled(
+        string $output,
+        array $renderParams,
+        string $regexp
+    ): void
+    {
+        Flight::view()->preserveVars = false;
+
+        $this->expectOutputString($output);
+        Flight::render(...$renderParams);
+
+        set_error_handler(function (int $code, string $message) use ($regexp): void {
+            $this->assertMatchesRegularExpression($regexp, $message);
+        });
+
+        Flight::render($renderParams[0]);
+
+        restore_error_handler();
+    }
+
+    public function testKeepThePreviousStateOfOneViewComponentByDefault(): void
+    {
+        $this->expectOutputString(<<<html
+        <div>Hi</div>
+        <div>Hi</div>
+
+        <input type="number" />
+
+        <input type="number" />
+
+        html);
+
+        Flight::render('myComponent', ['prop' => 'Hi']);
+        Flight::render('myComponent');
+        Flight::render('input', ['type' => 'number']);
+        Flight::render('input');
+    }
 }
--- a/tests/ViewTest.php
+++ b/tests/ViewTest.php
@ -152,4 +152,84 @@ class ViewTest extends TestCase
            $viewMock::normalizePath('C:/xampp/htdocs/libs/Flight\core\index.php', '°')
        );
    }
+
+    /** @dataProvider renderDataProvider */
+    public function testDoesNotPreserveVarsWhenFlagIsDisabled(
+        string $output,
+        array $renderParams,
+        string $regexp
+    ): void {
+        $this->view->preserveVars = false;
+
+        $this->expectOutputString($output);
+        $this->view->render(...$renderParams);
+
+        set_error_handler(function (int $code, string $message) use ($regexp): void {
+            $this->assertMatchesRegularExpression($regexp, $message);
+        });
+
+        $this->view->render($renderParams[0]);
+
+        restore_error_handler();
+    }
+
+    public function testKeepThePreviousStateOfOneViewComponentByDefault(): void
+    {
+        $this->expectOutputString(<<<html
+        <div>Hi</div>
+        <div>Hi</div>
+
+        <input type="number" />
+
+        <input type="number" />
+
+        html);
+
+        $this->view->render('myComponent', ['prop' => 'Hi']);
+        $this->view->render('myComponent');
+        $this->view->render('input', ['type' => 'number']);
+        $this->view->render('input');
+    }
+
+    public function testKeepThePreviousStateOfDataSettedBySetMethod(): void
+    {
+        $this->view->preserveVars = false;
+
+        $this->view->set('prop', 'bar');
+
+        $this->expectOutputString(<<<html
+        <div>qux</div>
+        <div>bar</div>
+
+        html);
+
+        $this->view->render('myComponent', ['prop' => 'qux']);
+        $this->view->render('myComponent');
+    }
+
+    public static function renderDataProvider(): array
+    {
+        return [
+            [
+                <<<html
+                <div>Hi</div>
+                <div></div>
+
+                html,
+                ['myComponent', ['prop' => 'Hi']],
+                '/^Undefined variable:? \$?prop$/'
+            ],
+            [
+                <<<html
+
+                <input type="number" />
+
+                <input type="text" />
+
+                html,
+                ['input', ['type' => 'number']],
+                '/^.*$/'
+            ],
+        ];
+    }
 }
--- a/tests/views/input.php
+++ b/tests/views/input.php
@ -0,0 +1,7 @@
+<?php
+
+$type ??= 'text';
+
+?>
+
+<input type="<?= $type ?>" />
--- a/tests/views/myComponent.php
+++ b/tests/views/myComponent.php
@ -0,0 +1 @@
+<div><?= $prop ?></div>