nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-09-17 02:26:33 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(accounts): add permission checks on invoice_discounting.get_invoices (backport #58975) (#58989)

Browse Source 
Co-authored-by: Diptanil Saha <diptanil@frappe.io>
This commit is contained in:
mergify[bot]
2026-09-11 04:42:54 +00:00
committed by GitHub
parent 85a152946e
commit 62b9f31297
3 changed files with 11 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
erpnext/accounts/doctype/invoice_discounting/invoice_discounting.js
View File

@@ -136,6 +136,7 @@ frappe.ui.form.on("Invoice Discounting", {
], ],
primary_action: function () { primary_action: function () {
var data = d.get_values(); var data = d.get_values();
data.company = frm.doc.company;
frappe.call({ frappe.call({
method: "erpnext.accounts.doctype.invoice_discounting.invoice_discounting.get_invoices", method: "erpnext.accounts.doctype.invoice_discounting.invoice_discounting.get_invoices",

6
erpnext/accounts/doctype/invoice_discounting/invoice_discounting.json
View File

@@ -168,7 +168,7 @@
} }
], ],
"is_submittable": 1, "is_submittable": 1,
"modified": "2019-05-30 19:08:21.199759", "modified": "2026-09-09 17:04:59.512294",
"modified_by": "Administrator", "modified_by": "Administrator",
"module": "Accounts", "module": "Accounts",
"name": "Invoice Discounting", "name": "Invoice Discounting",
@@ -185,7 +185,7 @@
"print": 1, "print": 1,
"read": 1, "read": 1,
"report": 1, "report": 1,
"role": "System Manager", "role": "Accounts Manager",
"share": 1, "share": 1,
"submit": 1, "submit": 1,
"write": 1 "write": 1
@@ -194,4 +194,4 @@
"sort_field": "modified", "sort_field": "modified",
"sort_order": "DESC", "sort_order": "DESC",
"track_changes": 1 "track_changes": 1
} }

7
erpnext/accounts/doctype/invoice_discounting/invoice_discounting.py
View File

@@ -319,6 +319,13 @@ class InvoiceDiscounting(AccountsController):
@frappe.whitelist() @frappe.whitelist()
def get_invoices(filters): def get_invoices(filters):
filters = frappe._dict(json.loads(filters)) filters = frappe._dict(json.loads(filters))
if not filters.get("company"):
frappe.throw(_("Please set company on the Document before requesting for invoices."))
frappe.has_permission("Company", doc=filters.get("company"), throw=True)
frappe.has_permission("Invoice Discounting", throw=True)
cond = [] cond = []
if filters.customer: if filters.customer:
cond.append("customer=%(customer)s") cond.append("customer=%(customer)s")