Shiloh/brotherton-erpnext
1
1
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Default warehouse is mandatory only for stock item in POS. Fixed #6893

Browse Source
This commit is contained in:
Nabin Hait 2016-11-21 18:29:16 +05:30
parent 8d161ef2e1
commit 977eff911f
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
erpnext/accounts/doctype/sales_invoice/pos.py
View File

@ -122,7 +122,8 @@ def get_items_list(pos_profile):
return frappe.db.sql(""" return frappe.db.sql("""
select select
name, item_code, item_name, description, item_group, expense_account, has_batch_no, name, item_code, item_name, description, item_group, expense_account, has_batch_no,
has_serial_no, expense_account, selling_cost_center, stock_uom, image, default_warehouse has_serial_no, expense_account, selling_cost_center, stock_uom, image,
default_warehouse, is_stock_item
from from
tabItem tabItem
where where

2
erpnext/accounts/page/pos/pos.js
View File

@ -1076,7 +1076,7 @@ erpnext.pos.PointOfSale = erpnext.taxes_and_totals.extend({
}, },
validate_warehouse: function(){ validate_warehouse: function(){
if(!this.items[0].default_warehouse && !this.pos_profile_data['warehouse']){ if(this.items[0].is_stock_item && !this.items[0].default_warehouse && !this.pos_profile_data['warehouse']){
frappe.throw(__("Default warehouse is required for selected item")) frappe.throw(__("Default warehouse is required for selected item"))
} }
}, },