fix: return from accepted and rejected warehouse at a same time not w…

…orking

erpnext/controllers/sales_and_purchase_return.py

@@ -936,13 +936,17 @@ def get_serial_and_batch_bundle(field, doctype, reference_ids, is_rejected=False
 		if is_rejected:
 			fields.extend(["rejected_serial_and_batch_bundle", "return_qty_from_rejected_warehouse"])
 
+		del filters["rejected_serial_and_batch_bundle"]
 		data = frappe.get_all(
 			doctype,
 			fields=fields,
 			filters=filters,
 		)
 
 		for d in data:
+			if not d.get("serial_and_batch_bundle") and not d.get("rejected_serial_and_batch_bundle"):
+				continue
+
 			if is_rejected:
 				if d.get("return_qty_from_rejected_warehouse"):
 					_bundle_ids.append(d.get("serial_and_batch_bundle"))
@@ -1027,7 +1031,7 @@ def get_available_batch_qty(parent_doc, batch_no, warehouse):
 	)
 
 
-def make_serial_batch_bundle_for_return(data, child_doc, parent_doc, warehouse_field=None):
+def make_serial_batch_bundle_for_return(data, child_doc, parent_doc, warehouse_field=None, qty_field=None):
 	from erpnext.stock.serial_batch_bundle import SerialBatchCreation
 
 	type_of_transaction = "Outward"
@@ -1037,11 +1041,21 @@ def make_serial_batch_bundle_for_return(data, child_doc, parent_doc, warehouse_f
 	if not warehouse_field:
 		warehouse_field = "warehouse"
 
+	if not qty_field:
+		qty_field = "qty"
+
 	warehouse = child_doc.get(warehouse_field)
 	if parent_doc.get("is_internal_customer"):
 		warehouse = child_doc.get("target_warehouse")
 		type_of_transaction = "Outward"
 
+	if not child_doc.get(qty_field):
+		frappe.throw(
+			_("For the {0}, the quantity is required to make the return entry").format(
+				frappe.bold(child_doc.item_code)
+			)
+		)
+
 	cls_obj = SerialBatchCreation(
 		{
 			"type_of_transaction": type_of_transaction,
@@ -1054,7 +1068,7 @@ def make_serial_batch_bundle_for_return(data, child_doc, parent_doc, warehouse_f
 			"voucher_type": parent_doc.doctype,
 			"voucher_no": parent_doc.name,
 			"voucher_detail_no": child_doc.name,
-			"qty": child_doc.qty,
+			"qty": child_doc.get(qty_field),
 			"company": parent_doc.company,
 			"do_not_submit": True,
 		}

erpnext/controllers/stock_controller.py

@@ -255,10 +255,17 @@ def make_bundle_for_rejected_qty(self, table_name=None):
 					qty_field = "qty"
 					warehouse_field = "warehouse"
 
+				if not data.get("qty"):
+					frappe.throw(
+						_("For the {0}, no stock is available for the return in the warehouse {1}.").format(
+							frappe.bold(row.item_code), row.get(warehouse_field)
+						)
+					)
+
 				data = filter_serial_batches(
 					self, data, row, warehouse_field=warehouse_field, qty_field=qty_field
 				)
-				bundle = make_serial_batch_bundle_for_return(data, row, self, warehouse_field)
+				bundle = make_serial_batch_bundle_for_return(data, row, self, warehouse_field, qty_field)
 				if row.get("return_qty_from_rejected_warehouse"):
 					row.db_set(
 						{