Merge pull request #9 from LibreSign/fix/always-return-bar

chore: always return bar

Author: vitormattos

src/helpers.php

@@ -59,10 +59,12 @@ function translate_path($page, ?string $target_locale = null): string
         $partial_path = '';
     }
 
-    return match (true) {
+    $match = match (true) {
         $target_locale === packageDefaultLocale($page) => $partial_path,
         default => "/{$target_locale}".($partial_path === '/' ? '' : $partial_path),
     };
+
+    return ! empty($match) ? $match : '/';
 }
 
 /**
@@ -91,10 +93,12 @@ function locale_path($page, string $partial_path, ?string $target_locale = null)
         $partial_path = '';
     }
 
-    return match (true) {
+    $match = match (true) {
         $target_locale === packageDefaultLocale($page) => $partial_path,
         default => "/{$target_locale}".($partial_path === '/' ? '' : $partial_path),
     };
+
+    return ! empty($match) ? $match : '/';
 }
 
 /**

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'],

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'],
 
         /**