Merge pull request #38439 from s-aga-r/FIX-38433

fix: use `docstatus` instead of `status`
2023-11-30 12:09:43 +05:30 · 2023-11-30 12:09:43 +05:30 · 7cb0b1b7c3
commit 7cb0b1b7c3
parent 591f4ebdca 1423b38d50
1 changed files with 4 additions and 4 deletions
--- a/erpnext/buying/doctype/purchase_order/purchase_order.py
+++ b/erpnext/buying/doctype/purchase_order/purchase_order.py
@ -770,8 +770,8 @@ def get_mapped_subcontracting_order(source_name, target_doc=None):

@frappe.whitelist()
 def is_subcontracting_order_created(po_name) -> bool:
-	count = frappe.db.count(
-		"Subcontracting Order", {"purchase_order": po_name, "status": ["not in", ["Draft", "Cancelled"]]}
+	return (
+		True
+		if frappe.db.exists("Subcontracting Order", {"purchase_order": po_name, "docstatus": ["=", 1]})
+		else False
 	)
-
-	return True if count else False