Merge pull request #4332 from nabinhait/pi_fix
[fix] Check Purchase Receipt mandatory only for stock items
This commit is contained in:
commit
b73f3da02c
@ -176,9 +176,10 @@ class PurchaseInvoice(BuyingController):
|
|||||||
throw(_("Purchse Order number required for Item {0}").format(d.item_code))
|
throw(_("Purchse Order number required for Item {0}").format(d.item_code))
|
||||||
|
|
||||||
def pr_required(self):
|
def pr_required(self):
|
||||||
|
stock_items = self.get_stock_items()
|
||||||
if frappe.db.get_value("Buying Settings", None, "pr_required") == 'Yes':
|
if frappe.db.get_value("Buying Settings", None, "pr_required") == 'Yes':
|
||||||
for d in self.get('items'):
|
for d in self.get('items'):
|
||||||
if not d.purchase_receipt:
|
if not d.purchase_receipt and d.item_code in stock_items:
|
||||||
throw(_("Purchase Receipt number required for Item {0}").format(d.item_code))
|
throw(_("Purchase Receipt number required for Item {0}").format(d.item_code))
|
||||||
|
|
||||||
def validate_write_off_account(self):
|
def validate_write_off_account(self):
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user