diff --git a/erpnext/buying/doctype/purchase_order/purchase_order.py b/erpnext/buying/doctype/purchase_order/purchase_order.py index 0a28177ba74..637a86846f9 100644 --- a/erpnext/buying/doctype/purchase_order/purchase_order.py +++ b/erpnext/buying/doctype/purchase_order/purchase_order.py @@ -310,12 +310,13 @@ class PurchaseOrder(BuyingController): itemwise_qty.setdefault(d.item_code, 0) itemwise_qty[d.item_code] += flt(d.stock_qty) + precision = self.items[0].precision("stock_qty") for item_code, qty in itemwise_qty.items(): - if flt(qty) < flt(itemwise_min_order_qty.get(item_code)): + if flt(qty, precision) < flt(itemwise_min_order_qty.get(item_code), precision): frappe.throw( _( "Item {0}: Ordered qty {1} cannot be less than minimum order qty {2} (defined in Item)." - ).format(item_code, qty, itemwise_min_order_qty.get(item_code)) + ).format(item_code, flt(qty, precision), itemwise_min_order_qty.get(item_code)) ) def get_schedule_dates(self): diff --git a/erpnext/buying/doctype/purchase_order/test_purchase_order.py b/erpnext/buying/doctype/purchase_order/test_purchase_order.py index d36dedb200a..05c6542939d 100644 --- a/erpnext/buying/doctype/purchase_order/test_purchase_order.py +++ b/erpnext/buying/doctype/purchase_order/test_purchase_order.py @@ -707,6 +707,23 @@ class TestPurchaseOrder(ERPNextTestSuite): po = create_purchase_order(qty=3.4, do_not_save=True) self.assertRaises(UOMMustBeIntegerError, po.insert) + def test_min_order_qty_with_uom_conversion_dust(self): + item_doc = make_item(properties={"min_order_qty": 2000, "stock_uom": "Kg"}) + item_doc.append("uoms", {"uom": "Litre", "conversion_factor": 0.6}) + item_doc.save() + item = item_doc.name + + precision = frappe.get_precision("Purchase Order Item", "stock_qty") + po = create_purchase_order(item_code=item, qty=flt(2000 / 0.6, precision), do_not_save=1) + po.items[0].uom = "Litre" + po.items[0].conversion_factor = 0.6 + po.insert() + + below_minimum = create_purchase_order(item_code=item, qty=3000, do_not_save=1) + below_minimum.items[0].uom = "Litre" + below_minimum.items[0].conversion_factor = 0.6 + self.assertRaises(frappe.ValidationError, below_minimum.insert) + def test_ordered_qty_for_closing_po(self): bin = frappe.get_all( "Bin",