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

Bug liste déroulante de sélection de la langue #592

Open
marcantoinedupre opened this issue Jan 8, 2025 · 3 comments
Open

Bug liste déroulante de sélection de la langue #592

marcantoinedupre opened this issue Jan 8, 2025 · 3 comments
Labels

Comments

@marcantoinedupre
Copy link

Dans le bandeau supérieur des pages de l'atlas la liste déroulante qui permet de choisir la langue du site est cassée. Au clic sur un autre langage la page est rafraîchie mais la langue reste la même (sur la page et dans le menu déroulant).

Capture d’écran du 2025-01-08 10-44-41

Il me semble que le problème vient de la fonction pull_lang_code à atlas/atlasRoutes.py qui vérifie les valeurs de la signature des endpoints (path params) alors que le code de la langue est envoyée avec un query param. La version suivante semble corriger le problème mais je n'ai pas fait le tour du reste du code qui gère les langues.

@main.url_value_preprocessor
def pull_lang_code(endpoint, values):
    lang_code = request.args.get("lang_code", None)
    if lang_code:
        g.lang_code = lang_code
    else:
        # If no language code has been set, get the best language from the browser settings
        g.lang_code = request.accept_languages.best_match(current_app.config["LANGUAGES"])

Qu'en penses-tu @amandine-sahl ?

@marcantoinedupre
Copy link
Author

Bug reproductible sur la branche develop actuelle (version 1.6.2).

@camillemonchicourt
Copy link
Member

Oui OK le multi-langue a encore été peu implémenté et utilisé donc c'est possible qu'il soit cassé.

@camillemonchicourt
Copy link
Member

L'implémentation du multilingue est détaillée dans #175

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants