schuetz3/schuetz3-erpnext
3
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore: Commonify autofilling warehouses in child tables

Browse Source
This commit is contained in:
marination
2020-04-23 00:18:21 +05:30
parent eaa956b994
commit 8a30cb2565
6 changed files with 29 additions and 56 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
erpnext/public/js/controllers/buying.js
View File

@@ -253,6 +253,13 @@ erpnext.buying.BuyingController = erpnext.TransactionController.extend({
}
},
rejected_warehouse: function(doc, cdt) {
// trigger autofill_warehouse only if parent rejected_warehouse field is triggered
if (["Purchase Invoice", "Purchase Receipt"].includes(cdt)) {
this.autofill_warehouse(doc.items, "rejected_warehouse", doc.rejected_warehouse);
}
},
category: function(doc, cdt, cdn) {
// should be the category field of tax table
if(cdt != doc.doctype) {

15
erpnext/public/js/controllers/transaction.js
View File

@@ -1889,21 +1889,16 @@ erpnext.TransactionController = erpnext.taxes_and_totals.extend({
},
set_reserve_warehouse: function() {
this.autofill_warehouse("reserve_warehouse");
this.autofill_warehouse(this.frm.doc.supplied_items, "reserve_warehouse", this.frm.doc.set_reserve_warehouse);
},
set_warehouse: function() {
this.autofill_warehouse("warehouse");
this.autofill_warehouse(this.frm.doc.items, "warehouse", this.frm.doc.set_warehouse);
},
autofill_warehouse : function (warehouse_field) {
// set warehouse in all child table rows
var me = this;
let warehouse = (warehouse_field === "warehouse") ? me.frm.doc.set_warehouse : me.frm.doc.set_reserve_warehouse;
let child_table = (warehouse_field === "warehouse") ? me.frm.doc.items : me.frm.doc.supplied_items;
let doctype = (warehouse_field === "warehouse") ? (me.frm.doctype + " Item") : (me.frm.doctype + " Item Supplied");
if(warehouse) {
autofill_warehouse : function (child_table, warehouse_field, warehouse) {
let doctype = child_table[0].doctype;
if (warehouse) {
$.each(child_table || [], function(i, item) {
frappe.model.set_value(doctype, item.name, warehouse_field, warehouse);
});