mirror of
https://github.com/frappe/erpnext.git
synced 2026-05-13 18:21:22 +00:00
refactor(supplier): Using query builder for get_total_accepted_items (#54874)
Use the query builder for get_total_accepted_items to assure compatibility with PostgreSQL.
This commit is contained in:
@@ -388,28 +388,21 @@ def get_total_accepted_amount(scorecard):
|
||||
|
||||
def get_total_accepted_items(scorecard):
|
||||
"""Gets the total number of rejected items in the period (based on Purchase Receipts)"""
|
||||
supplier = frappe.get_doc("Supplier", scorecard.supplier)
|
||||
pr = frappe.qb.DocType("Purchase Receipt")
|
||||
pr_item = frappe.qb.DocType("Purchase Receipt Item")
|
||||
|
||||
# Look up all PO Items with delivery dates between our dates
|
||||
data = frappe.db.sql(
|
||||
"""
|
||||
SELECT
|
||||
SUM(pr_item.qty)
|
||||
FROM
|
||||
`tabPurchase Receipt Item` pr_item,
|
||||
`tabPurchase Receipt` pr
|
||||
WHERE
|
||||
pr.supplier = %(supplier)s
|
||||
AND pr.posting_date BETWEEN %(start_date)s AND %(end_date)s
|
||||
AND pr_item.docstatus = 1
|
||||
AND pr_item.parent = pr.name""",
|
||||
{"supplier": supplier.name, "start_date": scorecard.start_date, "end_date": scorecard.end_date},
|
||||
as_dict=0,
|
||||
)[0][0]
|
||||
query = (
|
||||
frappe.qb.from_(pr)
|
||||
.join(pr_item)
|
||||
.on(pr_item.parent == pr.name)
|
||||
.select(frappe.qb.fn.Sum(pr_item.qty))
|
||||
.where(pr.supplier == scorecard.supplier)
|
||||
.where(pr.posting_date[scorecard.start_date : scorecard.end_date])
|
||||
.where(pr_item.docstatus == 1)
|
||||
)
|
||||
|
||||
if not data:
|
||||
data = 0
|
||||
return data
|
||||
result = query.run()
|
||||
return frappe.utils.flt(result[0][0]) if result else 0.0
|
||||
|
||||
|
||||
def get_total_shipments(scorecard):
|
||||
|
||||
Reference in New Issue
Block a user