vassili/seitime-frappe
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

refactor(mapper): use doc.check_permission (#28316)

Browse source
This commit is contained in:
Raffael Meyer 2024-10-31 19:52:47 +01:00 • committed by GitHub
parent c01ea099d4
commit 73a04e9a5d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 5 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

15
frappe/model/mapper.py
View file

@ -90,12 +90,8 @@ def get_mapped_doc(
else:
ret_doc = target_doc
if (
not apply_strict_user_permissions
and not ignore_permissions
and not target_doc.has_permission("create")
):
target_doc.raise_no_permission_to("create")
if not apply_strict_user_permissions and not ignore_permissions:
target_doc.check_permission("create")
if cached:
source_doc = frappe.get_cached_doc(from_doctype, from_docname)
@ -103,8 +99,7 @@ def get_mapped_doc(
source_doc = frappe.get_doc(from_doctype, from_docname)
if not ignore_permissions:
if not source_doc.has_permission("read"):
source_doc.raise_no_permission_to("read")
source_doc.check_permission("read")
ret_doc.run_method("before_mapping", source_doc, table_maps)
@ -178,8 +173,8 @@ def get_mapped_doc(
ret_doc.run_method("after_mapping", source_doc)
ret_doc.set_onload("load_after_mapping", True)
if apply_strict_user_permissions and not ignore_permissions and not ret_doc.has_permission("create"):
ret_doc.raise_no_permission_to("create")
if apply_strict_user_permissions and not ignore_permissions:
ret_doc.check_permission("create")
return ret_doc