minor changes
This commit is contained in:
parent
a20157af17
commit
14bd43d5eb
@ -198,7 +198,6 @@ class AccountsController(TransactionBase):
|
|||||||
|
|
||||||
if self.get("is_subcontracted"):
|
if self.get("is_subcontracted"):
|
||||||
args["is_subcontracted"] = self.is_subcontracted
|
args["is_subcontracted"] = self.is_subcontracted
|
||||||
|
|
||||||
ret = get_item_details(args)
|
ret = get_item_details(args)
|
||||||
|
|
||||||
for fieldname, value in ret.items():
|
for fieldname, value in ret.items():
|
||||||
|
@ -304,7 +304,6 @@ erpnext.TransactionController = erpnext.taxes_and_totals.extend({
|
|||||||
this.frm.doc.doctype === 'Delivery Note') {
|
this.frm.doc.doctype === 'Delivery Note') {
|
||||||
show_batch_dialog = 1;
|
show_batch_dialog = 1;
|
||||||
}
|
}
|
||||||
|
|
||||||
// clear barcode if setting item (else barcode will take priority)
|
// clear barcode if setting item (else barcode will take priority)
|
||||||
if(!from_barcode) {
|
if(!from_barcode) {
|
||||||
item.barcode = null;
|
item.barcode = null;
|
||||||
|
@ -69,7 +69,7 @@ $.extend(erpnext, {
|
|||||||
"get_query": function () {
|
"get_query": function () {
|
||||||
return {
|
return {
|
||||||
filters: {
|
filters: {
|
||||||
item_code:grid_row.doc.item_code ,
|
item_code:grid_row.doc.item_code,
|
||||||
warehouse:cur_frm.doc.is_return ? null : grid_row.doc.warehouse
|
warehouse:cur_frm.doc.is_return ? null : grid_row.doc.warehouse
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
@ -76,7 +76,6 @@ def get_item_details(args):
|
|||||||
args[key] = value
|
args[key] = value
|
||||||
|
|
||||||
out.update(get_pricing_rule_for_item(args))
|
out.update(get_pricing_rule_for_item(args))
|
||||||
|
|
||||||
if (args.get("doctype") == "Delivery Note" or
|
if (args.get("doctype") == "Delivery Note" or
|
||||||
(args.get("doctype") == "Sales Invoice" and args.get('update_stock'))) \
|
(args.get("doctype") == "Sales Invoice" and args.get('update_stock'))) \
|
||||||
and out.warehouse and out.stock_qty > 0:
|
and out.warehouse and out.stock_qty > 0:
|
||||||
@ -87,7 +86,11 @@ def get_item_details(args):
|
|||||||
if actual_batch_qty:
|
if actual_batch_qty:
|
||||||
out.update(actual_batch_qty)
|
out.update(actual_batch_qty)
|
||||||
|
|
||||||
if out.has_serial_no:
|
if out.has_serial_no and args.get('batch_no'):
|
||||||
|
out.batch_no = args.get('batch_no')
|
||||||
|
out.serial_no = get_serial_no(out, args.serial_no)
|
||||||
|
|
||||||
|
elif out.has_serial_no:
|
||||||
out.serial_no = get_serial_no(out, args.serial_no)
|
out.serial_no = get_serial_no(out, args.serial_no)
|
||||||
|
|
||||||
if args.transaction_date and item.lead_time_days:
|
if args.transaction_date and item.lead_time_days:
|
||||||
@ -468,7 +471,7 @@ def get_serial_nos_by_fifo(args):
|
|||||||
def get_serial_no_batchwise(args):
|
def get_serial_no_batchwise(args):
|
||||||
if frappe.db.get_single_value("Stock Settings", "automatically_set_serial_nos_based_on_fifo"):
|
if frappe.db.get_single_value("Stock Settings", "automatically_set_serial_nos_based_on_fifo"):
|
||||||
return "\n".join(frappe.db.sql_list("""select name from `tabSerial No`
|
return "\n".join(frappe.db.sql_list("""select name from `tabSerial No`
|
||||||
where item_code=%(item_code)s and warehouse=%(warehouse)s and batch_no=%(batch_no)s
|
where item_code=%(item_code)s and warehouse=%(warehouse)s and (batch_no=%(batch_no)s or batch_no is NULL)
|
||||||
order by timestamp(purchase_date, purchase_time) asc limit %(qty)s""", {
|
order by timestamp(purchase_date, purchase_time) asc limit %(qty)s""", {
|
||||||
"item_code": args.item_code,
|
"item_code": args.item_code,
|
||||||
"warehouse": args.warehouse,
|
"warehouse": args.warehouse,
|
||||||
@ -688,14 +691,13 @@ def get_serial_no(args, serial_nos=None):
|
|||||||
|
|
||||||
if args.get('warehouse') and args.get('stock_qty') and args.get('item_code'):
|
if args.get('warehouse') and args.get('stock_qty') and args.get('item_code'):
|
||||||
has_serial_no = frappe.get_value('Item', {'item_code': args.item_code}, "has_serial_no")
|
has_serial_no = frappe.get_value('Item', {'item_code': args.item_code}, "has_serial_no")
|
||||||
|
|
||||||
if args.get('batch_no') and has_serial_no == 1:
|
if args.get('batch_no') and has_serial_no == 1:
|
||||||
return get_serial_no_batchwise(args)
|
return get_serial_no_batchwise(args)
|
||||||
# elif (args.get('has_batch_no') == 1) and has_serial_no == 1 and not args.get(batch_no):
|
# elif (args.get('has_batch_no') == 1) and has_serial_no == 1 and not args.get('batch_no'):
|
||||||
# args.batch_no = get_batch_no(args.item_code, args.warehouse, args.qty)
|
# args.batch_no = get_batch_no(args.item_code, args.warehouse, args.qty)
|
||||||
# actual_batch_qty = get_batch_qty(out.batch_no, out.warehouse, out.item_code)
|
# actual_batch_qty = get_batch_qty(args.batch_no, args.warehouse, args.item_code)
|
||||||
# if actual_batch_qty:
|
# if actual_batch_qty:
|
||||||
# out.update(actual_batch_qty)
|
# args.update(actual_batch_qty)
|
||||||
# return get_serial_no_batchwise(args)
|
# return get_serial_no_batchwise(args)
|
||||||
elif has_serial_no == 1:
|
elif has_serial_no == 1:
|
||||||
args = json.dumps({"item_code": args.get('item_code'),"warehouse": args.get('warehouse'),"stock_qty": args.get('stock_qty')})
|
args = json.dumps({"item_code": args.get('item_code'),"warehouse": args.get('warehouse'),"stock_qty": args.get('stock_qty')})
|
||||||
|
Loading…
x
Reference in New Issue
Block a user