Fix: unable to render conversionsApiDataLayer directive without arguments

Author: jordyvanderhaegen

src/ConversionsApiServiceProvider.php

@@ -26,7 +26,11 @@ public function register()
     protected function registerBladeDirectives(): void
     {
         Blade::directive('conversionsApiDataLayer', function (?string $dataLayerVariableName = null) {
-            return "<?php echo view('conversions-api::data-layer', ['dataLayerVariableName' => {$dataLayerVariableName}]); ?>";
+            if (! $dataLayerVariableName) {
+                return "<?php echo view('conversions-api::data-layer'); ?>";
+            }
+
+            return "<?php echo view('conversions-api::data-layer', ['dataLayerVariableName' => $dataLayerVariableName]); ?>";
         });
         Blade::directive('conversionsApiFacebookPixelScript', function () {
             return "<?php echo view('conversions-api::facebook-pixel-script'); ?>";

tests/ConversionsApiViewTest.php

@@ -77,4 +77,13 @@ public function it_can_render_the_data_layer_directive()
             Blade::compileString("@conversionsApiDataLayer('customDataLayerVariableName')"),
         );
     }
+
+    /** @test */
+    public function it_can_render_the_data_layer_directive_when_no_arguments_are_provided()
+    {
+        $this->assertStringContainsString(
+            "<?php echo view('conversions-api::data-layer'); ?>",
+            Blade::compileString("@conversionsApiDataLayer"),
+        );
+    }
 }