From aa94847574efcc6f19a7cedf1cc890c6795041f6 Mon Sep 17 00:00:00 2001 From: robert schouten Date: Mon, 7 Nov 2016 19:29:52 +0800 Subject: [PATCH] [minor]dont show doctypes with all disabled roles in permission manager (#2268) --- frappe/core/page/permission_manager/permission_manager.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/frappe/core/page/permission_manager/permission_manager.py b/frappe/core/page/permission_manager/permission_manager.py index 18e7057d4b..b223352e77 100644 --- a/frappe/core/page/permission_manager/permission_manager.py +++ b/frappe/core/page/permission_manager/permission_manager.py @@ -17,7 +17,8 @@ def get_roles_and_doctypes(): "doctypes": [d[0] for d in frappe.db.sql("""select name from `tabDocType` dt where istable=0 and name not in ('DocType') and - exists(select * from `tabDocField` where parent=dt.name)""")], + exists(select * from `tabDocField` where parent=dt.name) and + exists(select * from `tabDocPerm` dp,`tabRole` role where dp.role = role.name and dp.parent=dt.name and not role.disabled)""")], "roles": [d[0] for d in frappe.db.sql("""select name from tabRole where name != 'Administrator' and disabled=0""")] }