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

fix: no API call required to set cookie in browser

Browse source
This commit is contained in:
Sagar Vora 2023-07-22 17:54:41 +05:30
parent 319da422f9
commit d2becc7f1d
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) 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(): def get_preferred_language_cookie():
return frappe.request.cookies.get("preferred_language") 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); language_switcher.val(language);
document.documentElement.lang = language; document.documentElement.lang = language;
language_switcher.change(() => { language_switcher.change(() => {
let lang = language_switcher.val(); const lang = language_switcher.val();
frappe document.cookie = `preferred_language=${lang}`;
.call("frappe.translate.set_preferred_language_cookie", { window.location.reload();
preferred_language: lang,
})
.then(() => {
window.location.reload();
});
}); });
}); });
} }