From 448c9b71c12871e5ee5c062678074b1bff7b63f1 Mon Sep 17 00:00:00 2001 From: Nabin Hait Date: Mon, 18 Aug 2014 15:38:53 +0530 Subject: [PATCH] minot fix in material request --- .../material_request/material_request.py | 21 +++++++++++-------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/erpnext/stock/doctype/material_request/material_request.py b/erpnext/stock/doctype/material_request/material_request.py index 89121e322c..10b5114f40 100644 --- a/erpnext/stock/doctype/material_request/material_request.py +++ b/erpnext/stock/doctype/material_request/material_request.py @@ -287,15 +287,18 @@ def make_purchase_order_based_on_supplier(source_name, target_doc=None): def get_material_requests_based_on_supplier(supplier): supplier_items = [d[0] for d in frappe.db.get_values("Item", {"default_supplier": supplier})] - material_requests = frappe.db.sql_list("""select distinct mr.name - from `tabMaterial Request` mr, `tabMaterial Request Item` mr_item - where mr.name = mr_item.parent - and mr_item.item_code in (%s) - and mr.material_request_type = 'Purchase' - and ifnull(mr.per_ordered, 0) < 99.99 - and mr.docstatus = 1 - and mr.status != 'Stopped'""" % ', '.join(['%s']*len(supplier_items)), - tuple(supplier_items)) + if supplier_items: + material_requests = frappe.db.sql_list("""select distinct mr.name + from `tabMaterial Request` mr, `tabMaterial Request Item` mr_item + where mr.name = mr_item.parent + and mr_item.item_code in (%s) + and mr.material_request_type = 'Purchase' + and ifnull(mr.per_ordered, 0) < 99.99 + and mr.docstatus = 1 + and mr.status != 'Stopped'""" % ', '.join(['%s']*len(supplier_items)), + tuple(supplier_items)) + else: + material_requests = [] return material_requests, supplier_items @frappe.whitelist()