Skip to content

Commit e30b2f1

Browse files
Sudharsanan11mergify[bot]
authored andcommitted
fix(manufacturing): remove delete query of job card & batch and serial no (#52840)
* fix(manufacturing): remove delete query of batch and serial no * fix(manufacturing): remove delete query of job card * fix: remove delete function call for work order (cherry picked from commit 8b2a971)
1 parent d59d30c commit e30b2f1

2 files changed

Lines changed: 0 additions & 14 deletions

File tree

erpnext/manufacturing/doctype/routing/test_routing.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ def test_sequence_id(self):
5656
self.assertEqual(job_card_doc.total_completed_qty, 10)
5757

5858
wo_doc.cancel()
59-
wo_doc.delete()
6059

6160
def test_update_bom_operation_time(self):
6261
"""Update cost shouldn't update routing times."""

erpnext/manufacturing/doctype/work_order/work_order.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -517,7 +517,6 @@ def on_cancel(self):
517517
self.db_set("status", "Cancelled")
518518

519519
self.on_close_or_cancel()
520-
self.delete_job_card()
521520

522521
def on_close_or_cancel(self):
523522
if self.production_plan and frappe.db.exists(
@@ -531,7 +530,6 @@ def on_close_or_cancel(self):
531530
self.update_planned_qty()
532531
self.update_ordered_qty()
533532
self.update_reserved_qty_for_production()
534-
self.delete_auto_created_batch_and_serial_no()
535533

536534
def create_serial_no_batch_no(self):
537535
if not (self.has_serial_no or self.has_batch_no):
@@ -588,13 +586,6 @@ def create_batch_for_finished_good(self):
588586
)
589587
)
590588

591-
def delete_auto_created_batch_and_serial_no(self):
592-
for row in frappe.get_all("Serial No", filters={"work_order": self.name}):
593-
frappe.delete_doc("Serial No", row.name)
594-
595-
for row in frappe.get_all("Batch", filters={"reference_name": self.name}):
596-
frappe.delete_doc("Batch", row.name)
597-
598589
def make_serial_nos(self, args):
599590
item_details = frappe.get_cached_value(
600591
"Item", self.production_item, ["serial_no_series", "item_name", "description"], as_dict=1
@@ -1027,10 +1018,6 @@ def set_lead_time(self):
10271018
if self.actual_start_date and self.actual_end_date:
10281019
self.lead_time = flt(time_diff_in_hours(self.actual_end_date, self.actual_start_date) * 60)
10291020

1030-
def delete_job_card(self):
1031-
for d in frappe.get_all("Job Card", ["name"], {"work_order": self.name}):
1032-
frappe.delete_doc("Job Card", d.name)
1033-
10341021
def validate_production_item(self):
10351022
if frappe.get_cached_value("Item", self.production_item, "has_variants"):
10361023
frappe.throw(_("Work Order cannot be raised against a Item Template"), ItemHasVariantError)

0 commit comments

Comments
 (0)