[hotfix] dont Patch Log, Module Def in Permission Manager (#4070)

frappe/core/page/permission_manager/permission_manager.py

@@ -3,14 +3,16 @@
 
 from __future__ import unicode_literals
 import frappe
+from frappe import _
 import frappe.defaults
 from frappe.modules.import_file import get_file_path, read_doc_from_file
 from frappe.translate import send_translations
-from frappe.permissions import (reset_perms, get_linked_doctypes, get_all_perms,
-	setup_custom_perms, add_permission, update_permission_property)
 from frappe.core.doctype.doctype.doctype import (clear_permissions_cache,
 	validate_permissions_for_doctype)
-from frappe import _
+from frappe.permissions import (reset_perms, get_linked_doctypes, get_all_perms,
+	setup_custom_perms, add_permission, update_permission_property)
+
+not_allowed_in_permission_manager = ["DocType", "Patch Log", "Module Def"]
 
 @frappe.whitelist()
 def get_roles_and_doctypes():
@@ -21,7 +23,7 @@ def get_roles_and_doctypes():
 
 	doctypes = frappe.get_all("DocType", filters={
 		"istable": 0,
-		"name": ("not in", "DocType"),
+		"name": ("not in", ",".join(not_allowed_in_permission_manager)),
 	}, or_filters={
 		"ifnull(restrict_to_domain, '')": "",
 		"restrict_to_domain": ("in", active_domains)
@@ -93,7 +95,6 @@ def reset(doctype):
 	reset_perms(doctype)
 	clear_permissions_cache(doctype)
 
-
 @frappe.whitelist()
 def get_users_with_role(role):
 	frappe.only_for("System Manager")