mirror of
https://github.com/frappe/erpnext.git
synced 2026-10-29 22:57:42 +00:00
fix(manufacturing): validate BOM in update_cost
This commit is contained in:
@@ -500,6 +500,18 @@ class BOM(WebsiteGenerator):
|
||||
|
||||
@frappe.whitelist()
|
||||
def update_cost(self, update_parent=True, from_child_bom=False, update_hour_rate=True, save=True):
|
||||
if cint(save) and not self.is_new():
|
||||
self.load_from_db()
|
||||
self.check_permission("write")
|
||||
|
||||
self._update_cost(
|
||||
update_parent=cint(update_parent) and cint(save),
|
||||
from_child_bom=from_child_bom,
|
||||
update_hour_rate=update_hour_rate,
|
||||
save=cint(save),
|
||||
)
|
||||
|
||||
def _update_cost(self, update_parent=True, from_child_bom=False, update_hour_rate=True, save=True):
|
||||
if self.docstatus == 2:
|
||||
return
|
||||
|
||||
@@ -523,7 +535,7 @@ class BOM(WebsiteGenerator):
|
||||
)
|
||||
|
||||
for bom in parent_boms:
|
||||
frappe.get_doc("BOM", bom).update_cost(from_child_bom=True)
|
||||
frappe.get_doc("BOM", bom)._update_cost(from_child_bom=True)
|
||||
|
||||
if not from_child_bom:
|
||||
msg = "Cost Updated"
|
||||
|
||||
Reference in New Issue
Block a user