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

refactor: dont hardcode child table "items"

Browse Source
This commit is contained in:
Ankush Menat
2022-03-27 19:37:23 +05:30
committed by Ankush Menat
parent 893139f963
commit c34847e801
3 changed files with 16 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -345,8 +345,7 @@ erpnext.TransactionController = class TransactionController extends erpnext.taxe
} }
scan_barcode() { scan_barcode() {
let me = this; const barcode_scanner = new erpnext.utils.BarcodeScanner({frm:this.frm});
const barcode_scanner = new erpnext.stock.BarcodeScanner({frm:this.frm});
barcode_scanner.process_scan(); barcode_scanner.process_scan();
} }

21
erpnext/public/js/utils/barcode_scanner.js
View File

@@ -1,10 +1,14 @@
erpnext.stock.BarcodeScanner = class BarcodeScanner { erpnext.utils.BarcodeScanner = class BarcodeScanner {
constructor(opts) { constructor(opts) {
$.extend(this, opts); $.extend(this, opts);
// field from which to capture input of scanned data // field from which to capture input of scanned data
this.scan_field_name = opts.scan_field_name || "scan_barcode"; this.scan_field_name = opts.scan_field_name || "scan_barcode";
this.scan_barcode_field = this.frm.fields_dict[this.scan_field_name]; this.scan_barcode_field = this.frm.fields_dict[this.scan_field_name];
this.items_table_name = opts.items_table_name || "items";
this.items_table = this.frm.doc[this.items_table_name];
this.scan_api = opts.scan_api || "erpnext.selling.page.point_of_sale.point_of_sale.search_for_serial_or_batch_or_barcode_number"; this.scan_api = opts.scan_api || "erpnext.selling.page.point_of_sale.point_of_sale.search_for_serial_or_batch_or_barcode_number";
} }
@@ -38,7 +42,7 @@ erpnext.stock.BarcodeScanner = class BarcodeScanner {
modify_table_after_scan(data) { modify_table_after_scan(data) {
let cur_grid = this.frm.fields_dict.items.grid; let cur_grid = this.frm.fields_dict[this.items_table_name].grid;
let row_to_modify = null; let row_to_modify = null;
// Check if batch is scanned and table has batch no field // Check if batch is scanned and table has batch no field
@@ -53,8 +57,9 @@ erpnext.stock.BarcodeScanner = class BarcodeScanner {
if (!row_to_modify) { if (!row_to_modify) {
// add new row if new item/batch is scanned // add new row if new item/batch is scanned
row_to_modify = frappe.model.add_child(this.frm.doc, cur_grid.doctype, 'items'); row_to_modify = frappe.model.add_child(this.frm.doc, cur_grid.doctype, this.items_table_name);
this.frm.script_manager.trigger("items_add", row_to_modify.doctype, row_to_modify.name); // trigger any row add triggers defined on child table.
this.frm.script_manager.trigger(`${this.items_table_name}_add`, row_to_modify.doctype, row_to_modify.name);
} }
if (this.is_duplicate_serial_no(row_to_modify, data.serial_no)) { if (this.is_duplicate_serial_no(row_to_modify, data.serial_no)) {
@@ -120,14 +125,14 @@ erpnext.stock.BarcodeScanner = class BarcodeScanner {
get_batch_row_to_modify(batch_no) { get_batch_row_to_modify(batch_no) {
// get row if batch already exists in table // get row if batch already exists in table
const existing_batch_row = this.frm.doc.items.find(d => d.batch_no === batch_no); const existing_batch_row = this.items_table.find(d => d.batch_no === batch_no);
return existing_batch_row || null; return existing_batch_row || null;
} }
get_row_to_modify_on_scan(row_to_modify, data) { get_row_to_modify_on_scan(row_to_modify, data) {
// get an existing item row to increment or blank row to modify // get an existing item row to increment or blank row to modify
const existing_item_row = this.frm.doc.items.find(d => d.item_code === data.item_code); const existing_item_row = this.items_table.find(d => d.item_code === data.item_code);
const blank_item_row = this.frm.doc.items.find(d => !d.item_code); const blank_item_row = this.items_table.find(d => !d.item_code);
if (existing_item_row) { if (existing_item_row) {
row_to_modify = existing_item_row; row_to_modify = existing_item_row;
@@ -140,6 +145,6 @@ erpnext.stock.BarcodeScanner = class BarcodeScanner {
clean_up() { clean_up() {
this.scan_barcode_field.set_value(""); this.scan_barcode_field.set_value("");
refresh_field("items"); refresh_field(this.items_table_name);
} }
}; };

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

@@ -845,8 +845,8 @@ erpnext.stock.StockEntry = class StockEntry extends erpnext.stock.StockControlle
} }
scan_barcode() { scan_barcode() {
let transaction_controller= new erpnext.TransactionController({frm:this.frm}); const barcode_scanner = new erpnext.utils.BarcodeScanner({frm:this.frm});
transaction_controller.scan_barcode(); barcode_scanner.process_scan();
} }
on_submit() { on_submit() {