fix(view): prevent $var from being nulled after passing to component (#1160)

Co-authored-by: brendt <[email protected]>

Author: erikaraujo

src/Tempest/View/src/Elements/PhpDataElement.php

@@ -26,14 +26,16 @@ public function getWrappingElement(): Element
 
     public function compile(): string
     {
-        $variableName = str($this->name)->ltrim(':')->camel()->toString();
+        $localVariableName = str($this->name)->ltrim(':')->camel()->toString();
         $isExpression = str_starts_with($this->name, ':');
         $value = $this->value ?? '';
 
         // We'll declare the variable in PHP right before the actual element
         $variableDeclaration = sprintf(
-            '$%s ??= %s ?? null;',
-            $variableName,
+            '$_%sIsLocal = isset($%s) === false; $%s ??= %s ?? null;',
+            $localVariableName,
+            $localVariableName,
+            $localVariableName,
             $isExpression
                 ? ($value ?: 'null')
                 : var_export($value, true), // @mago-expect best-practices/no-debug-symbols
@@ -42,8 +44,9 @@ public function compile(): string
         // And we'll remove it right after the element, this way we've created a "local scope"
         // where the variable is only available to that specific element.
         $variableRemoval = sprintf(
-            'unset($%s);',
-            $variableName,
+            'if ($_%sIsLocal) { unset($%s); }',
+            $localVariableName,
+            $localVariableName,
         );
 
         // Support for boolean attributes. When an expression attribute has a falsy value, it won't be rendered at all.

tests/Integration/View/TempestViewRendererDataPassingTest.php

@@ -371,4 +371,23 @@ public function test_boolean_attributes_in_view_component(): void
 
         $this->assertStringContainsString(' href="hi"', $html);
     }
+
+    public function test_global_variables_are_kept(): void
+    {
+        $this->registerViewComponent('x-test', <<<'HTML'
+        <div>{{ $item }}</div>
+        HTML);
+
+        $html = $this->render(<<<'HTML'
+        <x-test :item="$item"></x-test>
+        <x-test :item="$item"></x-test>
+        <x-test :item="$item"></x-test>
+        HTML, item: 'foo');
+
+        $this->assertSnippetsMatch(<<<'HTML'
+        <div>foo</div>
+        <div>foo</div>
+        <div>foo</div>
+        HTML, $html);
+    }
 }

tests/Integration/View/ViewComponentTest.php

@@ -652,20 +652,6 @@ public function test_merge_class(): void
         HTML, $html);
     }
 
-    public function test_attribute_precedence(): void
-    {
-        $this->markTestSkipped('TODO');
-
-        // Order should be: upperB > upperA > innerB > innerA
-        //        $this->registerViewComponent('x-test', <<<'HTML'
-        //        <div data-foo="innerA" :data-foo="'innerB'"></div>
-        //        HTML);
-        //        $html = $this->render(<<<'HTML'
-        //        <x-test data-foo="upperA" :data-foo="'upperB'"></x-test>
-        //        HTML);
-        //        $this->assertStringEqualsStringIgnoringLineEndings('<div data-foo="upperB"></div>', $html);
-    }
-
     public function test_does_not_duplicate_br(): void
     {
         $this->registerViewComponent('x-html-base', <<<'HTML'