From 69a08ccd439cdadce2f2728f155b62e6afb1dce6 Mon Sep 17 00:00:00 2001 From: Raffael Meyer <14891507+barredterra@users.noreply.github.com> Date: Tue, 14 Feb 2023 08:12:55 +0100 Subject: [PATCH] feat: actually redirect to login (#20018) --- frappe/public/js/frappe/desk.js | 58 +++------------------------------ 1 file changed, 4 insertions(+), 54 deletions(-) diff --git a/frappe/public/js/frappe/desk.js b/frappe/public/js/frappe/desk.js index 43263d5632..4a81e8620b 100644 --- a/frappe/public/js/frappe/desk.js +++ b/frappe/public/js/frappe/desk.js @@ -430,62 +430,12 @@ frappe.Application = class Application { }); } handle_session_expired() { - if (!frappe.app.session_expired_dialog) { - var dialog = new frappe.ui.Dialog({ - title: __("Session Expired"), - keep_open: true, - fields: [ - { - fieldtype: "Password", - fieldname: "password", - label: __("Please Enter Your Password to Continue"), - }, - ], - onhide: () => { - if (!dialog.logged_in) { - frappe.app.redirect_to_login(); - } - }, - }); - dialog.get_field("password").disable_password_checks(); - dialog.set_primary_action(__("Login"), () => { - dialog.set_message(__("Authenticating...")); - frappe.call({ - method: "login", - args: { - usr: frappe.session.user, - pwd: dialog.get_values().password, - }, - callback: (r) => { - if (r.message === "Logged In") { - dialog.logged_in = true; - - // revert backdrop - $(".modal-backdrop").css({ - opacity: "", - "background-color": "#334143", - }); - } - dialog.hide(); - }, - statusCode: () => { - dialog.hide(); - }, - }); - }); - frappe.app.session_expired_dialog = dialog; - } - if (!frappe.app.session_expired_dialog.display) { - frappe.app.session_expired_dialog.show(); - // add backdrop - $(".modal-backdrop").css({ - opacity: 1, - "background-color": "#4B4C9D", - }); - } + frappe.app.redirect_to_login(); } redirect_to_login() { - window.location.href = "/"; + window.location.href = `/login?redirect-to=${encodeURIComponent( + window.location.pathname + window.location.search + )}`; } set_favicon() { var link = $('link[type="image/x-icon"]').remove().attr("href");