Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[7.1-stable] Merge pull request #2689 from robinboening/fix_switching_to_default_language #2746

Merged
merged 1 commit into from
Feb 25, 2024

Conversation

alchemycms-bot
Copy link
Collaborator

Backport

This will backport the following commits from main to 7.1-stable:

Questions ?

Please refer to the Backport tool documentation

Closes #2632

From this commit on the language id won't be stored in the session any more
for frontend requests.

Upon every page visit the language id got stored in the session. When switching from a secondary language to the default language no language param is passed to the controller. The order how the language is found prioritized the session and the default was just a fallback. Coming from a secondary language and therefore it's id in the session, it would never load the default language, but the secondary.

Storing the language in the session is still done in the admin section.

(cherry picked from commit f63157e)
@tvdeyen tvdeyen merged commit 0e069c3 into 7.1-stable Feb 25, 2024
55 checks passed
@tvdeyen tvdeyen deleted the backport/7.1-stable/pr-2689 branch February 25, 2024 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants