From da682ae44c9636680ea81e23b7d5ece7407176ee Mon Sep 17 00:00:00 2001 From: Ankush Menat Date: Mon, 8 Apr 2024 12:59:23 +0530 Subject: [PATCH] fix: type error in workflow (#25847) if state is null, we can't get roles for it. --- frappe/public/js/frappe/model/workflow.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/frappe/public/js/frappe/model/workflow.js b/frappe/public/js/frappe/model/workflow.js index 359b9ba310..d7a37b0f82 100644 --- a/frappe/public/js/frappe/model/workflow.js +++ b/frappe/public/js/frappe/model/workflow.js @@ -59,10 +59,9 @@ frappe.workflow = { var state = doc[state_fieldname] || frappe.workflow.get_default_state(doctype, doc.docstatus); + if (!state) return false; - let allow_edit_roles = state - ? frappe.workflow.get_document_state_roles(doctype, state) - : null; + let allow_edit_roles = frappe.workflow.get_document_state_roles(doctype, state); let has_common_role = frappe.user_roles.some((role) => allow_edit_roles.includes(role) );