refactor: Consolidate duplicate zero-quantity transaction Items checks.
diff --git a/erpnext/controllers/accounts_controller.py b/erpnext/controllers/accounts_controller.py
index 1b8f66c..09bcea9 100644
--- a/erpnext/controllers/accounts_controller.py
+++ b/erpnext/controllers/accounts_controller.py
@@ -915,13 +915,15 @@
return flt(args.get(field, 0) / self.get("conversion_rate", 1))
def validate_qty_is_not_zero(self):
- if self.doctype == "Purchase Receipt":
- return
-
for item in self.items:
+ if self.doctype == "Purchase Receipt" and item.rejected_qty:
+ continue
+
if not flt(item.qty):
frappe.throw(
- msg=_("Row #{0}: Item quantity cannot be zero").format(item.idx),
+ msg=_("Row #{0}: Quantity for Item {1} cannot be zero.").format(
+ item.idx, frappe.bold(item.item_code)
+ ),
title=_("Invalid Quantity"),
exc=InvalidQtyError,
)
@@ -3019,7 +3021,7 @@
def validate_quantity(child_item, new_data):
if not flt(new_data.get("qty")):
frappe.throw(
- _("Row # {0}: Quantity for Item {1} cannot be zero").format(
+ _("Row #{0}: Quantity for Item {1} cannot be zero.").format(
new_data.get("idx"), frappe.bold(new_data.get("item_code"))
),
title=_("Invalid Qty"),
diff --git a/erpnext/controllers/selling_controller.py b/erpnext/controllers/selling_controller.py
index d6e3ee2..3bc054f 100644
--- a/erpnext/controllers/selling_controller.py
+++ b/erpnext/controllers/selling_controller.py
@@ -7,7 +7,7 @@
from frappe.utils import cint, flt, get_link_to_form, nowtime
from erpnext.accounts.party import render_address
-from erpnext.controllers.accounts_controller import InvalidQtyError, get_taxes_and_charges
+from erpnext.controllers.accounts_controller import get_taxes_and_charges
from erpnext.controllers.sales_and_purchase_return import get_rate_for_return
from erpnext.controllers.stock_controller import StockController
from erpnext.stock.doctype.item.item import set_item_default
@@ -295,10 +295,6 @@
def get_item_list(self):
il = []
for d in self.get("items"):
- if d.qty is None:
- message = _("Row {0}: Qty is mandatory").format(d.idx)
- frappe.throw(message, InvalidQtyError)
-
if self.has_product_bundle(d.item_code):
for p in self.get("packed_items"):
if p.parent_detail_docname == d.name and p.parent_item == d.item_code: