audrius/manufacture
1
0
Fork 0
mirror of https://github.com/OCA/manufacture.git synced 2025-01-28 16:37:15 +02:00
Code Issues Packages Projects Releases Wiki Activity

[FIX] mrp_multi_level: fix main supplier computation

Browse Source 
* api.depends

supply_method is not stored and therefore not reliable in the
api.depends, including procure location in the api.depends cover
changes in the supply method due to procure location change.

* remove usages of filtered

it was iterating the recordset more than 2 times when it could be
done in a single interation.
This commit is contained in:
Lois Rilo
2025-01-10 13:52:38 +01:00
parent 3f3fdbaf6e
commit 4abbbaee15
1 changed files with 9 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

14
mrp_multi_level/models/product_mrp_area.py
View File

@@ -220,11 +220,18 @@ class ProductMRPArea(models.Model):
rec.supply_bom_id = False
@api.depends(
"mrp_area_id", "supply_method", "product_id.route_ids", "product_id.seller_ids"
"mrp_area_id",
"product_id.route_ids",
"product_id.seller_ids",
"location_proc_id",
)
def _compute_main_supplier(self):
"""Simplified and similar to procurement.rule logic."""
for rec in self.filtered(lambda r: r.supply_method == "buy"):
for rec in self:
if rec.supply_method != "buy":
rec.main_supplierinfo_id = False
rec.main_supplier_id = False
continue
suppliers = rec.product_id.seller_ids.filtered(
lambda r, rec=rec: (not r.product_id or r.product_id == rec.product_id)
and (not r.company_id or r.company_id == rec.company_id)
@@ -235,9 +242,6 @@ class ProductMRPArea(models.Model):
continue
rec.main_supplierinfo_id = suppliers[0]
rec.main_supplier_id = suppliers[0].partner_id
for rec in self.filtered(lambda r: r.supply_method != "buy"):
rec.main_supplierinfo_id = False
rec.main_supplier_id = False
def _adjust_qty_to_order(self, qty_to_order):
self.ensure_one()