Merge pull request #25 from Laravel-Lang/1.x

Fixed determination of localization by the heading
Laravel-Lang · Feb 16, 2025 · c1d40ff · c1d40ff
2 parents c364cae + cfba8a3
commit c1d40ff
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/src/Middlewares/LocalizationByHeader.php b/src/Middlewares/LocalizationByHeader.php
@@ -16,9 +16,11 @@ protected function detect(Request $request): bool|float|int|string|null
             return null;
         }
 
-        return Locales::isInstalled(
-            $this->strBefore($value, [',', ';'])
-        );
+        if (! Locales::isInstalled($this->strBefore($value, [',', ';']))) {
+            return null;
+        }
+
+        return $value;
     }
 
     protected function strBefore(string $value, array $search): string

diff --git a/tests/Feature/Middlewares/HeaderTest.php b/tests/Feature/Middlewares/HeaderTest.php
@@ -52,7 +52,7 @@
         ->assertSuccessful()
         ->assertJsonPath($foo, LocaleValue::TranslationFrench);
 
-    assertEventDispatched();
+    assertEventNotDispatched();
 })->with('uninstalled-locales');
 
 test('unknown locale', function (int|string $locale) {
@@ -64,5 +64,5 @@
         ->assertSuccessful()
         ->assertJsonPath($foo, LocaleValue::TranslationFrench);
 
-    assertEventDispatched();
+    assertEventNotDispatched();
 })->with('unknown-locales');