From a9f969e9427fc74fa7773a41542d5340416083a5 Mon Sep 17 00:00:00 2001
From: R-Jayaraman <jayaramanoffl@gmail.com>
Date: Mon, 3 Aug 2026 16:41:43 +0530
Subject: [PATCH] fix(opportunity): add validation for positive item quantities

(cherry picked from commit c47cc374411c24c6fb62ca5e8114441e20f78414)
---
 erpnext/crm/doctype/opportunity/opportunity.py | 10 ++++++++++
 1 file changed, 10 insertions(+)

diff --git a/erpnext/crm/doctype/opportunity/opportunity.py b/erpnext/crm/doctype/opportunity/opportunity.py
index c76b76b856d..ea42af616d9 100644
--- a/erpnext/crm/doctype/opportunity/opportunity.py
+++ b/erpnext/crm/doctype/opportunity/opportunity.py
@@ -134,6 +134,7 @@ class Opportunity(TransactionBase, CRMNote):
 		self.validate_uom_is_integer("uom", "qty")
 		self.validate_cust_name()
 		self.map_fields()
+		self.validate_qty()
 		self.set_exchange_rate()
 
 		if not self.title:
@@ -144,6 +145,15 @@ class Opportunity(TransactionBase, CRMNote):
 	def on_update(self):
 		self.update_prospect()
 
+	def validate_qty(self):
+		for item in self.items:
+			if item.qty <= 0:
+				frappe.throw(
+					_("Row #{0}: Quantity must be greater than 0 for Item {1}").format(
+						item.idx, item.item_code
+					)
+				)
+
 	def map_fields(self):
 		for field in self.meta.get_valid_columns():
 			if not self.get(field) and frappe.db.field_exists(self.opportunity_from, field):