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

refactor(supplier): Using query builder for get_rfq_response_days (#54880)

Browse Source 
Use the query builder for get_rfq_response_days to assure compatibility with PostgreSQL.
This commit is contained in:
Loïc Oberle
2026-05-12 18:59:31 +02:00
committed by GitHub
parent 5560f6c270
commit 1b9eaed4d2
1 changed files with 25 additions and 27 deletions
Download Patch File Download Diff File Expand all files Collapse all files

52
erpnext/buying/doctype/supplier_scorecard_variable/supplier_scorecard_variable.py
View File

@@ -595,31 +595,29 @@ def get_sq_total_items(scorecard):
def get_rfq_response_days(scorecard): def get_rfq_response_days(scorecard):
"""Gets the total number of days it has taken a supplier to respond to rfqs in the period""" """Gets the total number of days it has taken a supplier to respond to rfqs in the period"""
supplier = frappe.get_doc("Supplier", scorecard.supplier) rfq = frappe.qb.DocType("Request for Quotation")
total_sq_days = frappe.db.sql( rfq_item = frappe.qb.DocType("Request for Quotation Item")
""" rfq_sup = frappe.qb.DocType("Request for Quotation Supplier")
SELECT sq = frappe.qb.DocType("Supplier Quotation")
SUM(DATEDIFF(sq.transaction_date, rfq.transaction_date)) sq_item = frappe.qb.DocType("Supplier Quotation Item")
FROM
`tabRequest for Quotation Item` rfq_item,
`tabSupplier Quotation Item` sq_item,
`tabSupplier Quotation` sq,
`tabRequest for Quotation Supplier` rfq_sup,
`tabRequest for Quotation` rfq
WHERE
rfq_sup.supplier = %(supplier)s
AND rfq.transaction_date BETWEEN %(start_date)s AND %(end_date)s
AND sq_item.request_for_quotation_item = rfq_item.name
AND sq_item.docstatus = 1
AND sq.supplier = %(supplier)s
AND sq_item.parent = sq.name
AND rfq_item.docstatus = 1
AND rfq_item.parent = rfq.name
AND rfq_sup.parent = rfq.name""",
{"supplier": supplier.name, "start_date": scorecard.start_date, "end_date": scorecard.end_date},
as_dict=0,
)[0][0]
if not total_sq_days:
total_sq_days = 0
return total_sq_days query = (
frappe.qb.from_(rfq)
.join(rfq_item)
.on(rfq_item.parent == rfq.name)
.join(rfq_sup)
.on(rfq_sup.parent == rfq.name)
.join(sq_item)
.on(sq_item.request_for_quotation_item == rfq_item.name)
.join(sq)
.on(sq_item.parent == sq.name)
.select(frappe.qb.fn.Sum(frappe.qb.fn.Datediff(sq.transaction_date, rfq.transaction_date)))
.where(rfq_sup.supplier == scorecard.supplier)
.where(sq.supplier == scorecard.supplier)
.where(rfq.transaction_date[scorecard.start_date : scorecard.end_date])
.where(rfq_item.docstatus == 1)
.where(sq_item.docstatus == 1)
)
result = query.run()
return frappe.utils.cint(result[0][0]) if result else 0