Merge pull request #8 from LibreSign/fix/empty-path-return-empty

fix: empty path return empty

Author: vitormattos

src/helpers.php

@@ -51,13 +51,16 @@ function translate_path($page, ?string $target_locale = null): string
 
     $current_locale = current_path_locale($page);
 
-    $partial_path = match (true) {
+    $partial_path = (string) match (true) {
         $current_locale === packageDefaultLocale($page) => $page->getPath(),
         default => substr($page->getPath(), strlen($current_locale) + 1),
     };
+    if ($partial_path === '/') {
+        $partial_path = '';
+    }
 
     return match (true) {
-        $target_locale === packageDefaultLocale($page) => $partial_path ?: '/',
+        $target_locale === packageDefaultLocale($page) => $partial_path,
         default => "/{$target_locale}".($partial_path === '/' ? '' : $partial_path),
     };
 }
@@ -89,7 +92,7 @@ function locale_path($page, string $partial_path, ?string $target_locale = null)
     }
 
     return match (true) {
-        $target_locale === packageDefaultLocale($page) => $partial_path ?: '/',
+        $target_locale === packageDefaultLocale($page) => $partial_path,
         default => "/{$target_locale}".($partial_path === '/' ? '' : $partial_path),
     };
 }

tests/unit/LocalePathTest.php

@@ -16,13 +16,13 @@ public static function providerLocalePath(): array
     {
         return [
             // Handling empty paths
-            ['', null, '/'],
+            ['', null, ''],
             ['', 'ar', '/ar'],
 
             // Root path handling
-            ['/', null, '/'],
+            ['/', null, ''],
             ['/', 'ar', '/ar'],
-            ['/', packageDefaultLocale(), '/'],
+            ['/', packageDefaultLocale(), ''],
 
             // Basic path translation
             ['blog', null, '/blog'],
@@ -36,7 +36,7 @@ public static function providerLocalePath(): array
 
             // Paths when `packageDefaultLocale()` should not add a locale prefix
             ['blog', packageDefaultLocale(), '/blog'],
-            ['/', packageDefaultLocale(), '/'],
+            ['/blog', packageDefaultLocale(), '/blog'],
 
             // Special characters handling
             ['café', 'fr', '/fr/café'],

tests/unit/TranslatePathTest.php

@@ -61,8 +61,8 @@ public static function providerTranslatePath(): array
 
             // Testing `/` as a root path (edge case)
             ['/', 'ar', '/ar'],
-            ['/', null, '/'],
-            ['/', packageDefaultLocale(), '/'],
+            ['/', null, ''],
+            ['/', packageDefaultLocale(), ''],
 
             // Testing paths with unrecognized prefixes (should keep them)
             ['/custom-path/blog', 'fr-CA', '/fr-CA/custom-path/blog'],
@@ -74,7 +74,7 @@ public static function providerTranslatePath(): array
 
             // Handling an empty path (should default to `/`)
             ['', 'ar', '/ar'],
-            ['', null, '/'],
+            ['', null, ''],
 
             // Deeply nested paths
             ['/es/a/b/c/d/e', 'ar', '/ar/a/b/c/d/e'],
@@ -91,7 +91,7 @@ public static function providerTranslatePath(): array
 
             // Handling trailing slashes
             ['/es/', 'ar', '/ar'],
-            ['/fr-CA/', null, '/'],
+            ['/fr-CA/', null, ''],
             ['/blog/', 'ar', '/ar/blog'],
 
         /**