From a2fadd9347b8f5474ec4fc12321206ff25beacb8 Mon Sep 17 00:00:00 2001 From: Smit Vora Date: Fri, 28 Nov 2025 13:33:24 +0530 Subject: [PATCH] fix: use `ValueWrapper` consistently --- erpnext/manufacturing/doctype/bom_creator/bom_creator.py | 3 +-- erpnext/patches/v14_0/single_to_multi_dunning.py | 4 ++-- 2 files changed, 3 insertions(+), 4 deletions(-) diff --git a/erpnext/manufacturing/doctype/bom_creator/bom_creator.py b/erpnext/manufacturing/doctype/bom_creator/bom_creator.py index bb62c7a84da..18599b2c20c 100644 --- a/erpnext/manufacturing/doctype/bom_creator/bom_creator.py +++ b/erpnext/manufacturing/doctype/bom_creator/bom_creator.py @@ -7,6 +7,7 @@ import frappe from frappe import _ from frappe.model.document import Document from frappe.utils import cint, flt, sbool +from pypika.terms import ValueWrapper from erpnext.manufacturing.doctype.bom.bom import get_bom_item_rate @@ -360,8 +361,6 @@ class BOMCreator(Document): @frappe.whitelist() def get_children(doctype=None, parent=None, **kwargs): - from pypika.terms import ValueWrapper - if isinstance(kwargs, str): kwargs = frappe.parse_json(kwargs) diff --git a/erpnext/patches/v14_0/single_to_multi_dunning.py b/erpnext/patches/v14_0/single_to_multi_dunning.py index 3ddba8c76e6..47d13a0e383 100644 --- a/erpnext/patches/v14_0/single_to_multi_dunning.py +++ b/erpnext/patches/v14_0/single_to_multi_dunning.py @@ -1,5 +1,5 @@ import frappe -from pypika.terms import LiteralValue +from pypika.terms import ValueWrapper from erpnext.accounts.general_ledger import make_reverse_gl_entries @@ -40,7 +40,7 @@ def execute(): "payment_amount", # at the time of creating this dunning, the full amount was outstanding "payment_amount as outstanding", - LiteralValue(0).as_("paid_amount"), + ValueWrapper(0).as_("paid_amount"), "discounted_amount", ], )