From f784d5d0263c855430cbd09a70e3a8cbf044461e Mon Sep 17 00:00:00 2001 From: Mangesh-Khairnar Date: Fri, 4 Oct 2019 21:21:03 +0530 Subject: [PATCH] fix: return default user permission as the leading element --- frappe/core/doctype/user_permission/user_permission.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/frappe/core/doctype/user_permission/user_permission.py b/frappe/core/doctype/user_permission/user_permission.py index 49e7fec278..7be915cbec 100644 --- a/frappe/core/doctype/user_permission/user_permission.py +++ b/frappe/core/doctype/user_permission/user_permission.py @@ -143,7 +143,11 @@ def get_applicable_for_doctype_list(doctype, txt, searchfield, start, page_len, return return_list def get_permitted_documents(doctype): - return [d.get('doc') for d in get_user_permissions().get(doctype, []) \ + ''' Returns permitted documents from the given doctype for the session user ''' + # sort permissions in a way to make the first permission in the list to be default + user_perm_list = sorted(get_user_permissions().get(doctype, []), key=lambda x: x.get('is_default'), reverse=True) + + return [d.get('doc') for d in user_perm_list \ if d.get('doc')] @frappe.whitelist()