fix: add GROUP BY for dn_detail and convert SQL query to QB

(cherry picked from commit fd9167f2af)

# Conflicts:
#	erpnext/stock/doctype/delivery_note/delivery_note.py
This commit is contained in:
l0gesh29
2025-10-13 11:22:29 +05:30
committed by Mergify
parent b426b8c07f
commit 9aa9b181e5

View File

@@ -10,6 +10,8 @@ from frappe.contacts.doctype.address.address import get_company_address
from frappe.desk.notifications import clear_doctype_notifications
from frappe.model.mapper import get_mapped_doc
from frappe.model.utils import get_fetch_values
from frappe.query_builder import DocType
from frappe.query_builder.functions import Abs, Sum
from frappe.utils import cint, flt
from erpnext.accounts.party import get_due_date
@@ -790,23 +792,27 @@ def get_list_context(context=None):
def get_invoiced_qty_map(delivery_note):
"""returns a map: {dn_detail: invoiced_qty}"""
invoiced_qty_map = {}
sii = DocType("Sales Invoice Item")
for dn_detail, qty in frappe.db.sql(
"""select dn_detail, qty from `tabSales Invoice Item`
where delivery_note=%s and docstatus=1""",
delivery_note,
):
if not invoiced_qty_map.get(dn_detail):
invoiced_qty_map[dn_detail] = 0
invoiced_qty_map[dn_detail] += qty
invoiced_qty_map = frappe._dict(
(
frappe.qb.from_(sii)
.select(sii.dn_detail, Sum(sii.qty).as_("qty"))
.where((sii.delivery_note == delivery_note) & (sii.docstatus == 1))
.groupby(sii.dn_detail)
).run()
)
return invoiced_qty_map
def get_returned_qty_map(delivery_note):
"""returns a map: {so_detail: returned_qty}"""
dn = DocType("Delivery Note")
dni = DocType("Delivery Note Item")
returned_qty_map = frappe._dict(
<<<<<<< HEAD
frappe.db.sql(
"""select dn_item.dn_detail, sum(abs(dn_item.qty)) as qty
from `tabDelivery Note Item` dn_item, `tabDelivery Note` dn
@@ -819,6 +825,21 @@ def get_returned_qty_map(delivery_note):
""",
delivery_note,
)
=======
(
frappe.qb.from_(dni)
.join(dn)
.on(dn.name == dni.parent)
.select(dni.dn_detail, Sum(Abs(dni.qty)).as_("qty"))
.where(
(dn.docstatus == 1)
& (dn.is_return == 1)
& (dn.return_against == delivery_note)
& (dni.qty <= 0)
)
.groupby(dni.dn_detail)
).run()
>>>>>>> fd9167f2af (fix: add GROUP BY for dn_detail and convert SQL query to QB)
)
return returned_qty_map