Skip to content

Commit 115a7bb

Browse files
MatTheCatchalasr
authored andcommitted
[HttpKernel] Use Accept-Language header even if there are no enabled locales
1 parent 99ea8d0 commit 115a7bb

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

EventListener/LocaleListener.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,10 @@ private function setLocale(Request $request)
6868
{
6969
if ($locale = $request->attributes->get('_locale')) {
7070
$request->setLocale($locale);
71-
} elseif ($this->useAcceptLanguageHeader && $this->enabledLocales && ($preferredLanguage = $request->getPreferredLanguage($this->enabledLocales))) {
72-
$request->setLocale($preferredLanguage);
71+
} elseif ($this->useAcceptLanguageHeader) {
72+
if ($preferredLanguage = $request->getPreferredLanguage($this->enabledLocales)) {
73+
$request->setLocale($preferredLanguage);
74+
}
7375
$request->attributes->set('_vary_by_language', true);
7476
}
7577
}

Tests/EventListener/LocaleListenerTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -179,7 +179,7 @@ public function testRequestNoLocaleFromAcceptLanguageHeader()
179179

180180
$listener->setDefaultLocale($event);
181181
$listener->onKernelRequest($event);
182-
$this->assertEquals('de', $request->getLocale());
182+
$this->assertEquals('fr_FR', $request->getLocale());
183183
}
184184

185185
public function testRequestAttributeLocaleNotOverridenFromAcceptLanguageHeader()

0 commit comments

Comments
 (0)