Shiloh/brotherton-erpnext
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

fix: use serial batch fields not enabled for new stock entry

Browse Source 
(cherry picked from commit dc9115a5865c86721863cfbda55bb00ac00cb756)
This commit is contained in:
Rohit Waghchaure 2024-02-16 15:41:30 +05:30 committed by Mergify
parent bb77f85d89
commit 40d4e3261e
2 changed files with 35 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -234,7 +234,7 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
} }
set_fields_onload_for_line_item() { set_fields_onload_for_line_item() {
if (this.frm.is_new && this.frm.doc?.items) { if (this.frm.is_new() && this.frm.doc?.items) {
this.frm.doc.items.forEach(item => { this.frm.doc.items.forEach(item => {
if (item.docstatus === 0 if (item.docstatus === 0
&& frappe.meta.has_field(item.doctype, "use_serial_batch_fields") && frappe.meta.has_field(item.doctype, "use_serial_batch_fields")

34
erpnext/stock/doctype/stock_entry/stock_entry.js
View File

@ -936,6 +936,7 @@ erpnext.stock.StockEntry = class StockEntry extends erpnext.stock.StockControlle
this.toggle_related_fields(this.frm.doc); this.toggle_related_fields(this.frm.doc);
this.toggle_enable_bom(); this.toggle_enable_bom();
this.show_stock_ledger(); this.show_stock_ledger();
this.set_fields_onload_for_line_item();
erpnext.utils.view_serial_batch_nos(this.frm); erpnext.utils.view_serial_batch_nos(this.frm);
if (this.frm.doc.docstatus===1 && erpnext.is_perpetual_inventory_enabled(this.frm.doc.company)) { if (this.frm.doc.docstatus===1 && erpnext.is_perpetual_inventory_enabled(this.frm.doc.company)) {
this.show_general_ledger(); this.show_general_ledger();
@ -944,6 +945,35 @@ erpnext.stock.StockEntry = class StockEntry extends erpnext.stock.StockControlle
erpnext.utils.add_item(this.frm); erpnext.utils.add_item(this.frm);
} }
serial_no(doc, cdt, cdn) {
var item = frappe.get_doc(cdt, cdn);
if (item?.serial_no) {
// Replace all occurences of comma with line feed
item.serial_no = item.serial_no.replace(/,/g, '\n');
item.conversion_factor = item.conversion_factor || 1;
let valid_serial_nos = [];
let serialnos = item.serial_no.split("\n");
for (var i = 0; i < serialnos.length; i++) {
if (serialnos[i] != "") {
valid_serial_nos.push(serialnos[i]);
}
}
frappe.model.set_value(item.doctype, item.name,
"qty", valid_serial_nos.length / item.conversion_factor);
}
}
set_fields_onload_for_line_item() {
if (this.frm.is_new() && this.frm.doc?.items
&& cint(frappe.user_defaults?.use_serial_batch_fields) === 1) {
this.frm.doc.items.forEach(item => {
frappe.model.set_value(item.doctype, item.name, "use_serial_batch_fields", 1);
})
}
}
scan_barcode() { scan_barcode() {
frappe.flags.dialog_set = false; frappe.flags.dialog_set = false;
const barcode_scanner = new erpnext.utils.BarcodeScanner({frm:this.frm}); const barcode_scanner = new erpnext.utils.BarcodeScanner({frm:this.frm});
@ -1074,6 +1104,10 @@ erpnext.stock.StockEntry = class StockEntry extends erpnext.stock.StockControlle
if(!row.s_warehouse) row.s_warehouse = this.frm.doc.from_warehouse; if(!row.s_warehouse) row.s_warehouse = this.frm.doc.from_warehouse;
if(!row.t_warehouse) row.t_warehouse = this.frm.doc.to_warehouse; if(!row.t_warehouse) row.t_warehouse = this.frm.doc.to_warehouse;
if (cint(frappe.user_defaults?.use_serial_batch_fields)) {
frappe.model.set_value(row.doctype, row.name, "use_serial_batch_fields", 1);
}
} }
from_warehouse(doc) { from_warehouse(doc) {