fixed test cases
This commit is contained in:
parent
0e9cd2e00a
commit
38238b0cfd
@ -609,6 +609,9 @@ def get_items_for_material_requests(doc, ignore_existing_ordered_qty=None):
|
|||||||
so_item_details = frappe._dict()
|
so_item_details = frappe._dict()
|
||||||
for data in po_items:
|
for data in po_items:
|
||||||
planned_qty = data.get('required_qty') or data.get('planned_qty')
|
planned_qty = data.get('required_qty') or data.get('planned_qty')
|
||||||
|
ignore_existing_ordered_qty = data.get('ignore_existing_ordered_qty') or ignore_existing_ordered_qty
|
||||||
|
warehouse = data.get("warehouse") or warehouse
|
||||||
|
|
||||||
item_details = {}
|
item_details = {}
|
||||||
if data.get("bom") or data.get("bom_no"):
|
if data.get("bom") or data.get("bom_no"):
|
||||||
if data.get('required_qty'):
|
if data.get('required_qty'):
|
||||||
|
@ -365,6 +365,8 @@ erpnext.selling.SalesOrderController = erpnext.selling.SellingController.extend(
|
|||||||
fields: [
|
fields: [
|
||||||
{fieldtype:'Read Only', fieldname:'item_code',
|
{fieldtype:'Read Only', fieldname:'item_code',
|
||||||
label: __('Item Code'), in_list_view:1},
|
label: __('Item Code'), in_list_view:1},
|
||||||
|
{fieldtype:'Link', fieldname:'warehouse', options: 'Warehouse',
|
||||||
|
label: __('For Warehouse'), in_list_view:1},
|
||||||
{fieldtype:'Link', fieldname:'bom', options: 'BOM', reqd: 1,
|
{fieldtype:'Link', fieldname:'bom', options: 'BOM', reqd: 1,
|
||||||
label: __('BOM'), in_list_view:1, get_query: function(doc) {
|
label: __('BOM'), in_list_view:1, get_query: function(doc) {
|
||||||
return {filters: {item: doc.item_code}};
|
return {filters: {item: doc.item_code}};
|
||||||
@ -372,8 +374,6 @@ erpnext.selling.SalesOrderController = erpnext.selling.SellingController.extend(
|
|||||||
},
|
},
|
||||||
{fieldtype:'Float', fieldname:'required_qty', reqd: 1,
|
{fieldtype:'Float', fieldname:'required_qty', reqd: 1,
|
||||||
label: __('Qty'), in_list_view:1},
|
label: __('Qty'), in_list_view:1},
|
||||||
{fieldtype:'Link', fieldname:'for_warehouse', options: 'Warehouse',
|
|
||||||
label: __('For Warehouse')}
|
|
||||||
],
|
],
|
||||||
data: r.message,
|
data: r.message,
|
||||||
get_data: function() {
|
get_data: function() {
|
||||||
|
@ -936,7 +936,12 @@ def make_raw_material_request(items, company, sales_order, project=None):
|
|||||||
item["ignore_existing_ordered_qty"] = items.get('ignore_existing_ordered_qty')
|
item["ignore_existing_ordered_qty"] = items.get('ignore_existing_ordered_qty')
|
||||||
item["include_raw_materials_from_sales_order"] = items.get('include_raw_materials_from_sales_order')
|
item["include_raw_materials_from_sales_order"] = items.get('include_raw_materials_from_sales_order')
|
||||||
|
|
||||||
raw_materials = get_items_for_material_requests(items, sales_order, company)
|
items.update({
|
||||||
|
'company': company,
|
||||||
|
'sales_order': sales_order
|
||||||
|
})
|
||||||
|
|
||||||
|
raw_materials = get_items_for_material_requests(items)
|
||||||
if not raw_materials:
|
if not raw_materials:
|
||||||
frappe.msgprint(_("Material Request not created, as quantity for Raw Materials already available."))
|
frappe.msgprint(_("Material Request not created, as quantity for Raw Materials already available."))
|
||||||
return
|
return
|
||||||
|
Loading…
x
Reference in New Issue
Block a user