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

[REF] mrp_multi_level: location management

Browse Source 
Small refactoring adding a _get_locations method on product.mrp.area
which by defaults delegates the computation to the related mrp.area.

This enables extending a few things related to locations at the
product.mrp.area level.

Change the way `_get_locations()` work: don't return the list of child
locations, only the top-most locations, and then use the `child_of`
operator in the code that looks for locations.
This commit is contained in:
Alexandre Fayolle
2023-12-04 11:59:32 +01:00
committed by JasminSForgeFlow
parent b0b9f1f008
commit 9d288ee154
3 changed files with 20 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
mrp_multi_level/models/mrp_area.py
View File

@@ -39,6 +39,4 @@ class MrpArea(models.Model):
def _get_locations(self):
self.ensure_one()
return self.env["stock.location"].search(
[("id", "child_of", self.location_id.id)]
)
return self.location_id

24
mrp_multi_level/models/product_mrp_area.py
View File

@@ -176,16 +176,16 @@ class ProductMRPArea(models.Model):
def _compute_qty_available(self):
for rec in self:
rec.qty_available = rec.product_id.with_context(
location=rec.mrp_area_id.location_id.id
location=rec._get_locations().ids
).qty_available
def _compute_supply_method(self):
group_obj = self.env["procurement.group"]
for rec in self:
proc_loc = rec.location_proc_id or rec.mrp_area_id.location_id
proc_loc = rec.location_proc_id or rec.location_id
values = {
"warehouse_id": rec.mrp_area_id.warehouse_id,
"company_id": rec.mrp_area_id.company_id,
"company_id": rec.company_id,
}
rule = group_obj._get_rule(rec.product_id, proc_loc, values)
if not rule:
@@ -259,24 +259,26 @@ class ProductMRPArea(models.Model):
def _in_stock_moves_domain(self):
self.ensure_one()
locations = self.mrp_area_id._get_locations()
locations = self._get_locations()
return [
("product_id", "=", self.product_id.id),
("state", "not in", ["done", "cancel"]),
("product_qty", ">", 0.00),
("location_id", "not in", locations.ids),
("location_dest_id", "in", locations.ids),
"!",
("location_id", "child_of", locations.ids),
("location_dest_id", "child_of", locations.ids),
]
def _out_stock_moves_domain(self):
self.ensure_one()
locations = self.mrp_area_id._get_locations()
locations = self._get_locations()
return [
("product_id", "=", self.product_id.id),
("state", "not in", ["done", "cancel"]),
("product_qty", ">", 0.00),
("location_id", "in", locations.ids),
("location_dest_id", "not in", locations.ids),
("location_id", "child_of", locations.ids),
"!",
("location_dest_id", "child_of", locations.ids),
]
def action_view_stock_moves(self, domain):
@@ -295,3 +297,7 @@ class ProductMRPArea(models.Model):
def _to_be_exploded(self):
self.ensure_one()
return self.supply_method in ["manufacture", "phantom"]
def _get_locations(self):
self.ensure_one()
return self.mrp_area_id._get_locations()