Merge pull request #34957 from frappe/mergify/bp/version-13-hotfix/pr-34953

fix: `PermissionError` in Work Order (backport #34953)
This commit is contained in:
Sagar Sharma
2023-04-20 16:01:36 +05:30
committed by GitHub

View File

@@ -592,20 +592,18 @@ erpnext.work_order = {
// all materials transferred for manufacturing, make this primary // all materials transferred for manufacturing, make this primary
finish_btn.addClass('btn-primary'); finish_btn.addClass('btn-primary');
} }
} else { } else if (frm.doc.__onload && frm.doc.__onload.overproduction_percentage) {
frappe.db.get_doc("Manufacturing Settings").then((doc) => { let allowance_percentage = frm.doc.__onload.overproduction_percentage;
let allowance_percentage = doc.overproduction_percentage_for_work_order;
if (allowance_percentage > 0) { if (allowance_percentage > 0) {
let allowed_qty = frm.doc.qty + ((allowance_percentage / 100) * frm.doc.qty); let allowed_qty = frm.doc.qty + ((allowance_percentage / 100) * frm.doc.qty);
if ((flt(doc.produced_qty) < allowed_qty)) { if ((flt(doc.produced_qty) < allowed_qty)) {
frm.add_custom_button(__('Finish'), function() { frm.add_custom_button(__('Finish'), function() {
erpnext.work_order.make_se(frm, 'Manufacture'); erpnext.work_order.make_se(frm, 'Manufacture');
}); });
}
} }
}); }
} }
} }
} else { } else {