mirror of
https://github.com/frappe/erpnext.git
synced 2026-06-05 05:09:11 +00:00
Merge pull request #55587 from frappe/mergify/bp/version-15-hotfix/pr-55150
fix(accounts): include asset items in purchase receipt validation (backport #55150)
This commit is contained in:
@@ -634,15 +634,16 @@ class PurchaseInvoice(BuyingController):
|
|||||||
throw(msg, title=_("Mandatory Purchase Order"))
|
throw(msg, title=_("Mandatory Purchase Order"))
|
||||||
|
|
||||||
def pr_required(self):
|
def pr_required(self):
|
||||||
stock_items = self.get_stock_items()
|
if frappe.db.get_single_value("Buying Settings", "pr_required") == "Yes":
|
||||||
if frappe.db.get_value("Buying Settings", None, "pr_required") == "Yes":
|
stock_and_asset_items = self.get_stock_items()
|
||||||
|
stock_and_asset_items.extend(self.get_asset_items())
|
||||||
if frappe.get_value(
|
if frappe.get_value(
|
||||||
"Supplier", self.supplier, "allow_purchase_invoice_creation_without_purchase_receipt"
|
"Supplier", self.supplier, "allow_purchase_invoice_creation_without_purchase_receipt"
|
||||||
):
|
):
|
||||||
return
|
return
|
||||||
|
|
||||||
for d in self.get("items"):
|
for d in self.get("items"):
|
||||||
if not d.purchase_receipt and d.item_code in stock_items:
|
if not d.purchase_receipt and d.item_code in stock_and_asset_items:
|
||||||
msg = _("Purchase Receipt Required for item {}").format(frappe.bold(d.item_code))
|
msg = _("Purchase Receipt Required for item {}").format(frappe.bold(d.item_code))
|
||||||
msg += "<br><br>"
|
msg += "<br><br>"
|
||||||
msg += _(
|
msg += _(
|
||||||
|
|||||||
Reference in New Issue
Block a user