Skip to content

Commit f7f10a5

Browse files
authored
Merge pull request #52789 from frappe/mergify/bp/version-15/pr-52763
fix: better permissions on make payment request (backport #52763)
2 parents a66854d + f06e20d commit f7f10a5

1 file changed

Lines changed: 4 additions & 1 deletion

File tree

erpnext/accounts/doctype/payment_request/payment_request.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -548,6 +548,9 @@ def make_payment_request(**args):
548548
if args.dn and not isinstance(args.dn, str):
549549
frappe.throw(_("Invalid parameter. 'dn' should be of type str"))
550550

551+
frappe.has_permission("Payment Request", "create", throw=True)
552+
frappe.has_permission(args.dt, "read", args.dn, throw=True)
553+
551554
ref_doc = args.ref_doc or frappe.get_doc(args.dt, args.dn)
552555
if not args.get("company"):
553556
args.company = ref_doc.company
@@ -822,7 +825,7 @@ def get_print_format_list(ref_doctype):
822825
return {"print_format": print_format_list}
823826

824827

825-
@frappe.whitelist(allow_guest=True)
828+
@frappe.whitelist()
826829
def resend_payment_email(docname):
827830
return frappe.get_doc("Payment Request", docname).send_email()
828831

0 commit comments

Comments
 (0)