From 85f227b1d1c7df8dc2c903809bd66becc410c473 Mon Sep 17 00:00:00 2001
From: nni-odoo <nni@odoo.com>
Date: Tue, 14 Dec 2021 04:12:08 +0000
Subject: [PATCH] [FIX] sale_mrp: unit price calculation BoM in different UoM
Fix for task 2620026.
_get_bom_component_qty function is only referred to once in the whole odoo app,
which is on addons/sale_mrp/models/account_move.py#22.
The result returned from this function is used as a multiplier to quantity
to convert into the expected UoM. However, the quantity being referred to on
_stock_account_get_anglo_saxon_price_unit (qty_invoiced, qty_to_invoice), are
already converted into the SO Line's product's default UoM. In this function,
it however tries to convert 1 from SO Line's UoM to BoM's UOM.
With this, if Product UoM=x, SO Line UoM=y, BOM UoM=x, in the function _stock_account_get_anglo_saxon_price_unit,
the quantities will be multiplied by the same factor twice, resulting to the incorrect computation of price_unit to follow.
Part-of: odoo/odoo#81355
---
addons/sale_mrp/models/sale_mrp.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/addons/sale_mrp/models/sale_mrp.py b/addons/sale_mrp/models/sale_mrp.py
index c1af5b254759..cddc331f896a 100644
--- a/addons/sale_mrp/models/sale_mrp.py
+++ b/addons/sale_mrp/models/sale_mrp.py
@@ -67,7 +67,7 @@ class SaleOrderLine(models.Model):
order_line.qty_delivered = 0.0
def _get_bom_component_qty(self, bom):
- bom_quantity = self.product_uom._compute_quantity(1, bom.product_uom_id)
+ bom_quantity = self.product_id.uom_id._compute_quantity(1, bom.product_uom_id)
boms, lines = bom.explode(self.product_id, bom_quantity)
components = {}
for line, line_data in lines:
--
GitLab