From 9df946e96c07eb52a330d0f3285301f42817def5 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Wed, 21 Dec 2016 13:02:07 +0530 Subject: [PATCH] [Fix] Warehouse is mandatory for packing materials on the sales invoice --- erpnext/accounts/doctype/sales_invoice/sales_invoice.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/erpnext/accounts/doctype/sales_invoice/sales_invoice.py b/erpnext/accounts/doctype/sales_invoice/sales_invoice.py index 5dce71f9c7..db6d594580 100644 --- a/erpnext/accounts/doctype/sales_invoice/sales_invoice.py +++ b/erpnext/accounts/doctype/sales_invoice/sales_invoice.py @@ -403,9 +403,9 @@ class SalesInvoice(SellingController): def validate_warehouse(self): super(SalesInvoice, self).validate_warehouse() - for d in self.get('items'): + for d in self.get_item_list(): if not d.warehouse and frappe.db.get_value("Item", d.item_code, "is_stock_item"): - frappe.throw(_("Warehouse required at Row No {0}").format(d.idx)) + frappe.throw(_("Warehouse required for stock Item {0}").format(d.item_code)) def validate_delivery_note(self): for d in self.get("items"):