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

refactor: parse native JSON request args in utilities/bulk_transaction.py

Browse Source 
Use frappe.parse_json instead of json.loads so the whitelisted endpoints
accept native JSON types (list/dict/bool) in addition to JSON strings.
This commit is contained in:
Mihir Kandoi
2026-06-24 20:37:35 +05:30
parent 7cbebc0545
commit 7e8965c6be
1 changed files with 2 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
erpnext/utilities/bulk_transaction.py
View File

@@ -13,13 +13,9 @@ def transaction_processing(
frappe.has_permission(from_doctype, "read", throw=True) frappe.has_permission(from_doctype, "read", throw=True)
frappe.has_permission(to_doctype, "create", throw=True) frappe.has_permission(to_doctype, "create", throw=True)
if isinstance(data, str): deserialized_data = frappe.parse_json(data)
deserialized_data = json.loads(data)
else:
deserialized_data = data
if isinstance(args, str): args = frappe._dict(frappe.parse_json(args))
args = frappe._dict(json.loads(args))
skipped_records = [d for d in deserialized_data if d.get("status") in ("On Hold", "Closed")] skipped_records = [d for d in deserialized_data if d.get("status") in ("On Hold", "Closed")]