Merge pull request #44999 from frappe/fix_multiple_sco_pr

fix: Added patch and fallback code to prevent future issues similar …
This commit is contained in:
rohitwaghchaure
2025-01-01 16:35:41 +05:30
committed by GitHub
2 changed files with 11 additions and 1 deletions

View File

@@ -103,6 +103,16 @@ class SubcontractingController(StockController):
_("Row {0}: Item {1} must be a subcontracted item.").format(item.idx, item.item_name)
)
if (
self.doctype == "Subcontracting Order" and not item.sc_conversion_factor
): # this condition will only be true if user has recently updated from develop branch
service_item_qty = frappe.get_value(
"Subcontracting Order Service Item",
filters={"purchase_order_item": item.purchase_order_item, "parent": self.name},
fieldname=["qty"],
)
item.sc_conversion_factor = service_item_qty / item.qty
if (
self.doctype not in "Subcontracting Receipt"
and item.qty