refactor(supplier_scorecard): Replace sql with orm (#55170)

This commit is contained in:
Loic Oberle
2026-05-22 12:37:12 +02:00
committed by GitHub
parent e75de4d337
commit ab99c9a54e

View File

@@ -207,25 +207,16 @@ def make_all_scorecards(docname: str):
while (start_date < todays) and (end_date <= todays): while (start_date < todays) and (end_date <= todays):
# check to make sure there is no scorecard period already created # check to make sure there is no scorecard period already created
scorecards = frappe.db.sql( scorecards = frappe.get_all(
""" "Supplier Scorecard Period",
SELECT fields=["name"],
scp.name filters={
FROM "scorecard": docname,
`tabSupplier Scorecard Period` scp "docstatus": 1,
WHERE "start_date": ["<", end_date],
scp.scorecard = %(sc)s "end_date": [">", start_date],
AND scp.docstatus = 1 },
AND ( order_by="end_date desc",
(scp.start_date > %(end_date)s
AND scp.end_date < %(start_date)s)
OR
(scp.start_date < %(end_date)s
AND scp.end_date > %(start_date)s))
ORDER BY
scp.end_date DESC""",
{"sc": docname, "start_date": start_date, "end_date": end_date},
as_dict=1,
) )
if len(scorecards) == 0: if len(scorecards) == 0:
period_card = make_supplier_scorecard(docname, None) period_card = make_supplier_scorecard(docname, None)