Merge pull request #1123 from hydephp/refactor-formatLink-helper

emmadesilva · web-flow · commit 4b62ebb0eb0d · 2023-02-26T20:36:52.000+01:00
Update Hyde::formatLink helper to trim all index.html suffixes when using pretty URLs
diff --git a/packages/framework/src/Foundation/Kernel/Hyperlinks.php b/packages/framework/src/Foundation/Kernel/Hyperlinks.php
@@ -7,7 +7,6 @@
 use Hyde\Foundation\HydeKernel;
 use Hyde\Framework\Exceptions\BaseUrlNotSetException;
 use Hyde\Framework\Exceptions\FileNotFoundException;
-use Hyde\Pages\DocumentationPage;
 use Illuminate\Support\Str;
 
 /**
@@ -39,8 +38,8 @@ public function formatLink(string $destination): string
                     return '/';
                 }
 
-                if ($destination === DocumentationPage::outputDirectory().'/index.html') {
-                    return DocumentationPage::outputDirectory().'/';
+                if (str_ends_with($destination, 'index.html')) {
+                    return substr($destination, 0, -10);
                 }
 
                 return substr($destination, 0, -5);
diff --git a/packages/framework/tests/Unit/Foundation/HyperlinkFormatHtmlPathTest.php b/packages/framework/tests/Unit/Foundation/HyperlinkFormatHtmlPathTest.php
@@ -98,4 +98,16 @@ public function test_helper_does_not_rewrite_documentation_page_index_when_not_u
         self::mockConfig(['hyde.pretty_urls' => false]);
         $this->assertEquals('docs/index.html', Hyde::formatLink('docs/index.html'));
     }
+
+    public function test_helpers_rewrites_arbitrary_nested_index_pages_when_using_pretty_urls()
+    {
+        self::mockConfig(['hyde.pretty_urls' => true]);
+        $this->assertEquals('foo/bar/', Hyde::formatLink('foo/bar/index.html'));
+    }
+
+    public function test_helpers_does_not_rewrite_arbitrary_nested_index_pages_when_not_using_pretty_urls()
+    {
+        self::mockConfig(['hyde.pretty_urls' => false]);
+        $this->assertEquals('foo/bar/index.html', Hyde::formatLink('foo/bar/index.html'));
+    }
 }
