Browse Source

chore: remove old code where User Permissions were set in `tabDefaultValue`

(cherry picked from commit 63e760e3ad)
version-14
Sagar Vora 2 years ago
committed by Mergify
parent
commit
dc8cd73666
3 changed files with 1 additions and 13 deletions
  1. +1
    -2
      frappe/defaults.py
  2. +0
    -3
      frappe/model/delete_doc.py
  3. +0
    -8
      frappe/model/rename_doc.py

+ 1
- 2
frappe/defaults.py View File

@@ -6,8 +6,7 @@ from frappe.cache_manager import clear_defaults_cache, common_default_keys
from frappe.desk.notifications import clear_notifications
from frappe.query_builder import DocType

# Note: DefaultValue records are identified by parenttype
# __default, __global or 'User Permission'
# Note: DefaultValue records are identified by parent (e.g. __default, __global)


def set_user_default(key, value, user=None, parenttype=None):


+ 0
- 3
frappe/model/delete_doc.py View File

@@ -167,9 +167,6 @@ def delete_doc(
except ImportError:
pass

# delete user_permissions
frappe.defaults.clear_default(parenttype="User Permission", key=doctype, value=name)


def add_to_deleted_document(doc):
"""Add this document to Deleted Document table. Called after delete"""


+ 0
- 8
frappe/model/rename_doc.py View File

@@ -197,14 +197,6 @@ def rename_doc(
if not merge:
rename_password(doctype, old, new)

# update user_permissions
DefaultValue = frappe.qb.DocType("DefaultValue")
frappe.qb.update(DefaultValue).set(DefaultValue.defvalue, new).where(
(DefaultValue.parenttype == "User Permission")
& (DefaultValue.defkey == doctype)
& (DefaultValue.defvalue == old)
).run()

if merge:
new_doc.add_comment("Edit", _("merged {0} into {1}").format(frappe.bold(old), frappe.bold(new)))
else:


Loading…
Cancel
Save