diff --git a/frappe/desk/doctype/kanban_board/kanban_board.py b/frappe/desk/doctype/kanban_board/kanban_board.py index c9eb48564d..e3257e25be 100644 --- a/frappe/desk/doctype/kanban_board/kanban_board.py +++ b/frappe/desk/doctype/kanban_board/kanban_board.py @@ -88,13 +88,15 @@ def update_order(board_name, order): """Save the order of cards in columns""" board = frappe.get_doc("Kanban Board", board_name) doctype = board.reference_doctype + updated_cards = [] - frappe.has_permission(doctype, "write", throw=True) + if not frappe.has_permission(doctype, "write"): + # Return board data from db + return board, updated_cards fieldname = board.field_name order_dict = json.loads(order) - updated_cards = [] for col_name, cards in order_dict.items(): for card in cards: column = frappe.get_value(doctype, {"name": card}, fieldname) diff --git a/frappe/public/js/frappe/views/kanban/kanban_board.bundle.js b/frappe/public/js/frappe/views/kanban/kanban_board.bundle.js index 703f49f205..1810101820 100644 --- a/frappe/public/js/frappe/views/kanban/kanban_board.bundle.js +++ b/frappe/public/js/frappe/views/kanban/kanban_board.bundle.js @@ -327,13 +327,7 @@ frappe.provide("frappe.views"); return state.empty_state; }, show_empty_state); - if (frappe.model.can_write(store.state.doctype)) { - // Check for reference doctype access before trying to modify it's value - store.dispatch("update_order"); - } else { - // Render columns without state change - make_columns(); - } + store.dispatch("update_order"); } function prepare() {