From 15ecda86a7272dcc92ddc8a9b4485a8be0c2d86a Mon Sep 17 00:00:00 2001
From: Caen De Silva <caen@desilva.se>
Date: Wed, 24 Jul 2024 17:58:44 +0200
Subject: [PATCH] Update Markdown link support to work without single quotes

---
 .../Processing/DynamicMarkdownLinkProcessor.php | 12 ++++++------
 .../DynamicMarkdownLinkProcessorTest.php        | 17 +++++++++++++++++
 2 files changed, 23 insertions(+), 6 deletions(-)

diff --git a/packages/framework/src/Markdown/Processing/DynamicMarkdownLinkProcessor.php b/packages/framework/src/Markdown/Processing/DynamicMarkdownLinkProcessor.php
index 52ae7a605eb..bf00a9a8c1a 100644
--- a/packages/framework/src/Markdown/Processing/DynamicMarkdownLinkProcessor.php
+++ b/packages/framework/src/Markdown/Processing/DynamicMarkdownLinkProcessor.php
@@ -22,14 +22,14 @@ public static function postprocess(string $html): string
     protected static function patterns(): array
     {
         return [
-            '/<a href="hyde::route\(\'([^\']+)\'\)"/' => function (array $matches): string {
-                return '<a href="'.Hyde::route($matches[1]).'"';
+            '/<a href="hyde::route\(([\'"]?)([^\'"]+)\1\)"/' => function (array $matches): string {
+                return '<a href="'.Hyde::route($matches[2]).'"';
             },
-            '/<a href="hyde::relativeLink\(\'([^\']+)\'\)"/' => function (array $matches): string {
-                return '<a href="'.Hyde::relativeLink($matches[1]).'"';
+            '/<a href="hyde::relativeLink\(([\'"]?)([^\'"]+)\1\)"/' => function (array $matches): string {
+                return '<a href="'.Hyde::relativeLink($matches[2]).'"';
             },
-            '/<img src="hyde::asset\(\'([^\']+)\'\)"/' => function (array $matches): string {
-                return '<img src="'.Hyde::asset($matches[1]).'"';
+            '/<img src="hyde::asset\(([\'"]?)([^\'"]+)\1\)"/' => function (array $matches): string {
+                return '<img src="'.Hyde::asset($matches[2]).'"';
             },
         ];
     }
diff --git a/packages/framework/tests/Feature/Services/Markdown/DynamicMarkdownLinkProcessorTest.php b/packages/framework/tests/Feature/Services/Markdown/DynamicMarkdownLinkProcessorTest.php
index 48026b79b39..8191fdb7c45 100644
--- a/packages/framework/tests/Feature/Services/Markdown/DynamicMarkdownLinkProcessorTest.php
+++ b/packages/framework/tests/Feature/Services/Markdown/DynamicMarkdownLinkProcessorTest.php
@@ -36,6 +36,9 @@ public function testRouteReplacement()
         $input = '<p><a href="hyde::route(\'home\')">Home</a></p>';
         $expected = '<p><a href="home.html">Home</a></p>';
         $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($input));
+
+        $inputUnquoted = '<p><a href="hyde::route(home)">Home</a></p>';
+        $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($inputUnquoted));
     }
 
     public function testRelativeLinkReplacement()
@@ -43,6 +46,9 @@ public function testRelativeLinkReplacement()
         $input = '<p><a href="hyde::relativeLink(\'about\')">About</a></p>';
         $expected = '<p><a href="about">About</a></p>';
         $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($input));
+
+        $inputUnquoted = '<p><a href="hyde::relativeLink(about)">About</a></p>';
+        $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($inputUnquoted));
     }
 
     public function testAssetReplacement()
@@ -50,6 +56,9 @@ public function testAssetReplacement()
         $input = '<p><img src="hyde::asset(\'image.jpg\')" alt="Image" /></p>';
         $expected = '<p><img src="media/image.jpg" alt="Image" /></p>';
         $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($input));
+
+        $inputUnquoted = '<p><img src="hyde::asset(image.jpg)" alt="Image" /></p>';
+        $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($inputUnquoted));
     }
 
     public function testMultipleReplacements()
@@ -67,6 +76,14 @@ public function testMultipleReplacements()
         HTML;
 
         $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($input));
+
+        $inputUnquoted = <<<'MARKDOWN'
+        <a href="hyde::route(home)">Home</a>
+        <a href="hyde::relativeLink(about)">About</a>
+        <img src="hyde::asset(logo.png)" alt="Logo" />
+        MARKDOWN;
+
+        $this->assertSame($expected, DynamicMarkdownLinkProcessor::postprocess($inputUnquoted));
     }
 
     public function testNoReplacements()