nsinnovations/erpnext
2
0
Fork 0
mirror of https://github.com/frappe/erpnext.git synced 2026-05-23 23:19:20 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix: incorrect job card timer issue

Browse Source
This commit is contained in:
Rohit Waghchaure
2020-09-01 14:02:34 +05:30
parent d0aafece32
commit 86a190652e
3 changed files with 18 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

12
erpnext/manufacturing/doctype/job_card/job_card.js
View File

@@ -34,7 +34,7 @@ frappe.ui.form.on('Job Card', {
frm.trigger("toggle_operation_number"); frm.trigger("toggle_operation_number");
if (frm.doc.docstatus == 0 && (frm.doc.for_quantity > frm.doc.total_completed_qty || !frm.doc.for_quantity) if (frm.doc.docstatus == 0 && (frm.doc.for_quantity > frm.doc.total_completed_qty || !frm.doc.for_quantity)
&& (!frm.doc.items || !frm.doc.items.length || frm.doc.for_quantity == frm.doc.transferred_qty)) { && (frm.doc.items || !frm.doc.items.length || frm.doc.for_quantity == frm.doc.transferred_qty)) {
frm.trigger("prepare_timer_buttons"); frm.trigger("prepare_timer_buttons");
} }
}, },
@@ -94,9 +94,9 @@ frappe.ui.form.on('Job Card', {
fieldname: 'employee'}, d => { fieldname: 'employee'}, d => {
if (d.employee) { if (d.employee) {
frm.set_value("employee", d.employee); frm.set_value("employee", d.employee);
} else {
frm.events.start_job(frm);
} }
frm.events.start_job(frm);
}, __("Enter Value"), __("Start")); }, __("Enter Value"), __("Start"));
} else { } else {
frm.events.start_job(frm); frm.events.start_job(frm);
@@ -141,9 +141,7 @@ frappe.ui.form.on('Job Card', {
frm.set_value('current_time' , 0); frm.set_value('current_time' , 0);
} }
frm.save("Save", () => {}, "", () => { frm.save();
frm.doc.time_logs.pop(-1);
});
}, },
complete_job: function(frm, completed_time, completed_qty) { complete_job: function(frm, completed_time, completed_qty) {
@@ -175,6 +173,8 @@ frappe.ui.form.on('Job Card', {
employee: function(frm) { employee: function(frm) {
if (frm.doc.job_started && !frm.doc.current_time) { if (frm.doc.job_started && !frm.doc.current_time) {
frm.trigger("reset_timer"); frm.trigger("reset_timer");
} else {
frm.events.start_job(frm);
} }
}, },

12
erpnext/manufacturing/doctype/job_card/test_job_card.py
View File

@@ -31,6 +31,9 @@ class TestJobCard(unittest.TestCase):
doc.operation_id = "Test Data" doc.operation_id = "Test Data"
self.assertRaises(OperationMismatchError, doc.save) self.assertRaises(OperationMismatchError, doc.save)
for d in job_cards:
frappe.delete_doc("Job Card", d.name)
def test_job_card_with_different_work_station(self): def test_job_card_with_different_work_station(self):
data = frappe.get_cached_value('BOM', data = frappe.get_cached_value('BOM',
{'docstatus': 1, 'with_operations': 1, 'company': '_Test Company'}, ['name', 'item']) {'docstatus': 1, 'with_operations': 1, 'company': '_Test Company'}, ['name', 'item'])
@@ -40,9 +43,11 @@ class TestJobCard(unittest.TestCase):
work_order = make_wo_order_test_record(item=bom_item, qty=1, bom_no=bom) work_order = make_wo_order_test_record(item=bom_item, qty=1, bom_no=bom)
job_card = frappe.get_all('Job Card', job_cards = frappe.get_all('Job Card',
filters = {'work_order': work_order.name}, filters = {'work_order': work_order.name},
fields = ["operation_id", "workstation", "name", "for_quantity"])[0] fields = ["operation_id", "workstation", "name", "for_quantity"])
job_card = job_cards[0]
if job_card: if job_card:
workstation = frappe.db.get_value("Workstation", workstation = frappe.db.get_value("Workstation",
@@ -65,3 +70,6 @@ class TestJobCard(unittest.TestCase):
self.assertEqual(completed_qty, job_card.for_quantity) self.assertEqual(completed_qty, job_card.for_quantity)
doc.cancel() doc.cancel()
for d in job_cards:
frappe.delete_doc("Job Card", d.name)