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

fix: Item Price using batch selector

Browse Source
This commit is contained in:
Deepesh Garg
2020-12-28 20:59:58 +05:30
parent 2466c3214a
commit a679d43722
4 changed files with 11 additions and 17 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -195,10 +195,6 @@ erpnext.buying.BuyingController = erpnext.TransactionController.extend({
this._super(doc, cdt, cdn); this._super(doc, cdt, cdn);
}, },
batch_no: function(doc, cdt, cdn) {
this._super(doc, cdt, cdn);
},
received_qty: function(doc, cdt, cdn) { received_qty: function(doc, cdt, cdn) {
this.calculate_accepted_qty(doc, cdt, cdn) this.calculate_accepted_qty(doc, cdt, cdn)
}, },

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

@@ -1106,11 +1106,6 @@ erpnext.TransactionController = erpnext.taxes_and_totals.extend({
} }
}, },
batch_no: function(doc, cdt, cdn) {
let item = frappe.get_doc(cdt, cdn);
this.apply_price_list(item, true);
},
toggle_conversion_factor: function(item) { toggle_conversion_factor: function(item) {
// toggle read only property for conversion factor field if the uom and stock uom are same // toggle read only property for conversion factor field if the uom and stock uom are same
if(this.frm.get_field('items').grid.fields_map.conversion_factor) { if(this.frm.get_field('items').grid.fields_map.conversion_factor) {
@@ -1527,7 +1522,9 @@ erpnext.TransactionController = erpnext.taxes_and_totals.extend({
return; return;
} }
if (me.in_apply_price_list == true) return; let for_batch = (item && item.batch_no) ? true : false;
if (me.in_apply_price_list == true && (!for_batch)) return;
me.in_apply_price_list = true; me.in_apply_price_list = true;
return this.frm.call({ return this.frm.call({

3
erpnext/public/js/utils/serial_no_batch_selector.js
View File

@@ -220,6 +220,7 @@ erpnext.SerialNoBatchSelector = Class.extend({
if (i !== 0 && !this.batch_exists(batch_no)) { if (i !== 0 && !this.batch_exists(batch_no)) {
row = this.frm.add_child("items", { ...this.item }); row = this.frm.add_child("items", { ...this.item });
frappe.model.set_value(row.doctype, row.name, 'batch_no', batch_no);
} else { } else {
row = this.frm.doc.items.find(i => i.batch_no === batch_no); row = this.frm.doc.items.find(i => i.batch_no === batch_no);
} }
@@ -336,6 +337,8 @@ erpnext.SerialNoBatchSelector = Class.extend({
}, },
change: function () { change: function () {
const batch_no = this.get_value(); const batch_no = this.get_value();
let val = this.get_value();
if (!batch_no) { if (!batch_no) {
this.grid_row.on_grid_fields_dict this.grid_row.on_grid_fields_dict
.available_qty.set_value(0); .available_qty.set_value(0);

6
erpnext/selling/sales_common.js
View File

@@ -326,6 +326,8 @@ erpnext.selling.SellingController = erpnext.TransactionController.extend({
}); });
} }
}) })
me.apply_price_list(item, true);
}, },
set_dynamic_labels: function() { set_dynamic_labels: function() {
@@ -399,10 +401,6 @@ erpnext.selling.SellingController = erpnext.TransactionController.extend({
} }
}, },
batch_no: function(doc, cdt, cdn) {
this._super(doc, cdt, cdn);
},
qty: function(doc, cdt, cdn) { qty: function(doc, cdt, cdn) {
this._super(doc, cdt, cdn); this._super(doc, cdt, cdn);