fix: Load doc before save
This commit is contained in:
parent
77b82894ff
commit
319c912f91
@ -20,13 +20,12 @@ class BOMUpdateTool(Document):
|
||||
for bom in bom_list:
|
||||
try:
|
||||
bom_obj = frappe.get_doc("BOM", bom)
|
||||
bom_obj.get_doc_before_save()
|
||||
bom_obj.load_doc_before_save()
|
||||
updated_bom = bom_obj.update_cost_and_exploded_items(updated_bom)
|
||||
bom_obj.calculate_cost()
|
||||
bom_obj.update_parent_cost()
|
||||
bom_obj.db_update()
|
||||
if (getattr(bom_obj.meta, 'track_changes', False)
|
||||
and bom_obj._doc_before_save and not bom_obj.flags.ignore_version):
|
||||
if (getattr(bom_obj.meta, 'track_changes', False) and not bom_obj.flags.ignore_version):
|
||||
bom_obj.save_version()
|
||||
|
||||
frappe.db.commit()
|
||||
@ -37,7 +36,7 @@ class BOMUpdateTool(Document):
|
||||
def validate_bom(self):
|
||||
if cstr(self.current_bom) == cstr(self.new_bom):
|
||||
frappe.throw(_("Current BOM and New BOM can not be same"))
|
||||
|
||||
|
||||
if frappe.db.get_value("BOM", self.current_bom, "item") \
|
||||
!= frappe.db.get_value("BOM", self.new_bom, "item"):
|
||||
frappe.throw(_("The selected BOMs are not for the same item"))
|
||||
|
@ -90,8 +90,6 @@ class Item(WebsiteGenerator):
|
||||
self.set_opening_stock()
|
||||
|
||||
def validate(self):
|
||||
self.get_doc_before_save()
|
||||
|
||||
super(Item, self).validate()
|
||||
|
||||
if not self.item_name:
|
||||
|
@ -729,7 +729,7 @@ class TestStockEntry(unittest.TestCase):
|
||||
make_stock_entry(item_code="_Test Item 2", target="_Test Warehouse - _TC", qty=50, basic_rate=20)
|
||||
|
||||
item_quantity = {
|
||||
'_Test Item': 10.0,
|
||||
'_Test Item': 2.0,
|
||||
'_Test Item 2': 12.0,
|
||||
'_Test Serialized Item With Series': 6.0
|
||||
}
|
||||
|
Loading…
x
Reference in New Issue
Block a user