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

perf: optimize code for subcontracting

Browse Source
This commit is contained in:
Mihir Kandoi
2025-07-15 11:55:03 +05:30
parent 896b21e78b
commit bc6f69ad54
3 changed files with 13 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
erpnext/controllers/subcontracting_controller.py
View File

@@ -959,7 +959,7 @@ class SubcontractingController(StockController):
)
sco_doc.update_ordered_qty_for_subcontracting(sco_item_rows)
sco_doc.update_reserved_qty_for_subcontracting()
sco_doc.update_reserved_qty_for_subcontracting(sco_item_rows)
def make_sl_entries_for_supplier_warehouse(self, sl_entries):
if hasattr(self, "supplied_items"):
@@ -1052,7 +1052,7 @@ class SubcontractingController(StockController):
return supplied_items_cost
def set_subcontracting_order_status(self):
def set_subcontracting_order_status(self, update_bin=True):
if self.doctype == "Subcontracting Order":
self.update_status()
elif self.doctype == "Subcontracting Receipt":
@@ -1061,7 +1061,7 @@ class SubcontractingController(StockController):
if self.subcontract_orders:
for sco in set(self.subcontract_orders):
sco_doc = frappe.get_doc("Subcontracting Order", sco)
sco_doc.update_status()
sco_doc.update_status(update_bin=update_bin)
def calculate_additional_costs(self):
self.total_additional_costs = sum(flt(item.amount) for item in self.get("additional_costs"))

12
erpnext/subcontracting/doctype/subcontracting_order/subcontracting_order.py
View File

@@ -236,8 +236,11 @@ class SubcontractingOrder(SubcontractingController):
return flt(query[0][0]) if query else 0
def update_reserved_qty_for_subcontracting(self):
def update_reserved_qty_for_subcontracting(self, sco_item_rows=None):
for item in self.supplied_items:
if sco_item_rows and item.reference_name not in sco_item_rows:
continue
if item.rm_item_code:
stock_bin = get_bin(item.rm_item_code, item.reserve_warehouse)
stock_bin.update_reserved_qty_for_sub_contracting()
@@ -299,7 +302,7 @@ class SubcontractingOrder(SubcontractingController):
self.set_missing_values()
def update_status(self, status=None, update_modified=True):
def update_status(self, status=None, update_modified=True, update_bin=True):
if self.status == "Closed" and self.status != status:
check_on_hold_or_closed_status("Purchase Order", self.purchase_order)
@@ -329,8 +332,9 @@ class SubcontractingOrder(SubcontractingController):
self.db_set("status", status, update_modified=update_modified)
self.update_requested_qty()
self.update_ordered_qty_for_subcontracting()
self.update_reserved_qty_for_subcontracting()
if update_bin:
self.update_ordered_qty_for_subcontracting()
self.update_reserved_qty_for_subcontracting()
def update_subcontracted_quantity_in_po(self, cancel=False):
for service_item in self.service_items:

4
erpnext/subcontracting/doctype/subcontracting_receipt/subcontracting_receipt.py
View File

@@ -153,7 +153,7 @@ class SubcontractingReceipt(SubcontractingController):
self.validate_available_qty_for_consumption()
self.update_status_updater_args()
self.update_prevdoc_status()
self.set_subcontracting_order_status()
self.set_subcontracting_order_status(update_bin=False)
self.set_consumed_qty_in_subcontract_order()
for table_name in ["items", "supplied_items"]:
@@ -181,7 +181,7 @@ class SubcontractingReceipt(SubcontractingController):
self.update_status_updater_args()
self.update_prevdoc_status()
self.set_consumed_qty_in_subcontract_order()
self.set_subcontracting_order_status()
self.set_subcontracting_order_status(update_bin=False)
self.update_stock_ledger()
self.make_gl_entries_on_cancel()
self.repost_future_sle_and_gle()