vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

Merge pull request #21763 from resilient-tech/fix-preferred-language-setter

Browse source 
fix: no API call required to set cookie in browser
This commit is contained in:
mergify[bot] 2023-07-22 12:47:08 +00:00 • committed by GitHub
commit a86125ac41
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
frappe/translate.py
View file

@ -1320,11 +1320,6 @@ def get_all_languages(with_language_name: bool = False) -> list:
return frappe.cache.get_value("languages", get_language_codes)
@frappe.whitelist(allow_guest=True)
def set_preferred_language_cookie(preferred_language: str):
frappe.local.cookie_manager.set_cookie("preferred_language", preferred_language)
def get_preferred_language_cookie():
return frappe.request.cookies.get("preferred_language")

11
frappe/website/js/website.js
View file

@ -394,14 +394,9 @@ $.extend(frappe, {
language_switcher.val(language);
document.documentElement.lang = language;
language_switcher.change(() => {
let lang = language_switcher.val();
frappe
.call("frappe.translate.set_preferred_language_cookie", {
preferred_language: lang,
})
.then(() => {
window.location.reload();
});
const lang = language_switcher.val();
document.cookie = `preferred_language=${lang}`;
window.location.reload();
});
});
}