[IMP] stock_buffer_route: Do not consider other locations Buy routes

Considering all buy routes should not be the default behaviour. If this is needed should be in a customized module.

stock_buffer_route/README.rst

@@ -2,12 +2,12 @@
 Stock Buffer Route
 ==================
 
-..
+.. 
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:012eb267c5f3b2dc1aee2e761f61510e4a51b010f0c23ac588a92bf7c3d50344
+   !! source digest: sha256:71fbdc11c56d2d40834b454605a88fb709845b8483b6d4962c4afba9a2099d8e
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png

stock_buffer_route/models/__init__.py

@@ -1 +1,3 @@
+from . import mrp_production
+from . import purchase_order
 from . import stock_buffer

stock_buffer_route/models/mrp_production.py

@@ -0,0 +1,42 @@
+# Copyright 2023 ForgeFlow S.L. (https://www.forgeflow.com)
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
+
+from odoo import models
+
+
+class MrpProduction(models.Model):
+    _inherit = "mrp.production"
+
+    def _get_domain_buffer_link_alternative(self, warehouse_level=False):
+        self.ensure_one()
+        if not warehouse_level:
+            locations = self.env["stock.location"].search(
+                [("id", "child_of", [self.location_dest_id.id])]
+            )
+            return [
+                ("product_id", "=", self.product_id.id),
+                ("company_id", "=", self.company_id.id),
+                ("item_type_alternative", "=", "manufactured"),
+                ("location_id", "in", locations.ids),
+            ]
+        else:
+            return [
+                ("product_id", "=", self.product_id.id),
+                ("company_id", "=", self.company_id.id),
+                ("item_type_alternative", "=", "manufactured"),
+                ("warehouse_id", "=", self.picking_type_id.warehouse_id.id),
+            ]
+
+    def _find_buffer_link(self):
+        res = super()._find_buffer_link()
+        buffer_model = self.env["stock.buffer"]
+        for rec in self.filtered(lambda r: not r.buffer_id):
+            domain = rec._get_domain_buffer_link_alternative()
+            buffer = buffer_model.search(domain, limit=1)
+            if not buffer:
+                domain = rec._get_domain_buffer_link_alternative(warehouse_level=True)
+                buffer = buffer_model.search(domain, limit=1)
+            rec.buffer_id = buffer
+            if buffer:
+                rec._calc_execution_priority()
+        return res

stock_buffer_route/models/purchase_order.py

@@ -0,0 +1,38 @@
+# Copyright 2023 ForgeFlow S.L. (https://www.forgeflow.com)
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html).
+
+from odoo import models
+
+
+class PurchaseOrderLine(models.Model):
+    _inherit = "purchase.order.line"
+
+    def _get_domain_buffer_link_alternative(self):
+        self.ensure_one()
+        if not self.product_id:
+            # Return impossible domain -> no buffer.
+            return [(0, "=", 1)]
+        return [
+            ("product_id", "=", self.product_id.id),
+            ("company_id", "=", self.order_id.company_id.id),
+            ("item_type_alternative", "=", "purchased"),
+            ("warehouse_id", "=", self.order_id.picking_type_id.warehouse_id.id),
+        ]
+
+    def _find_buffer_link(self):
+        res = super()._find_buffer_link()
+        buffer_model = self.env["stock.buffer"]
+        move_model = self.env["stock.move"]
+        for rec in self.filtered(lambda r: not r.buffer_ids):
+            mto_move = move_model.search(
+                [("created_purchase_line_id", "=", rec.id)], limit=1
+            )
+            if mto_move:
+                # MTO lines are not accounted in MTS stock buffers.
+                continue
+            domain = rec._get_domain_buffer_link_alternative()
+            buffer = buffer_model.search(domain, limit=1)
+            if buffer:
+                rec.buffer_ids = buffer
+                rec._calc_execution_priority()
+        return res

stock_buffer_route/models/stock_buffer.py

@@ -21,6 +21,7 @@ class StockBuffer(models.Model):
         "stock.route",
         string="Allowed routes",
         compute="_compute_route_ids",
+        store=True,
     )
     route_id = fields.Many2one(
         "stock.route",
@@ -94,31 +95,37 @@ def _compute_dlt_alternative(self):
         "item_type",
         "product_id.seller_ids",
         "product_id.seller_ids.company_id",
-        "product_id.seller_ids.name",
+        "product_id.seller_ids.partner_id",
         "product_id.seller_ids.product_id",
         "product_id.seller_ids.sequence",
         "product_id.seller_ids.min_qty",
         "product_id.seller_ids.price",
-        "item_type_alternative",
     )
     def _compute_main_supplier(self):
         res = super()._compute_main_supplier()
         for rec in self:
             if rec.item_type_alternative == "purchased":
                 suppliers = rec._get_product_sellers()
-                rec.main_supplier_id = suppliers[0].name if suppliers else False
+                rec.main_supplier_id = (
+                    suppliers[0].partner_id.id if suppliers else False
+                )
         return res
 
-    def _calc_incoming_dlt_qty(self):
-        res = super()._calc_incoming_dlt_qty()
-        for rec in self:
-            if rec.item_type_alternative == "purchased":
-                cut_date = rec._get_incoming_supply_date_limit()
-                pols = rec.purchase_line_ids.filtered(
-                    lambda l: l.date_planned > fields.Datetime.to_datetime(cut_date)
+    def _get_rfq_dlt_qty(self, outside_dlt=False):
+        res = super()._get_rfq_dlt_qty(outside_dlt)
+        if self.item_type_alternative == "purchased":
+            cut_date = self._get_incoming_supply_date_limit()
+            if not outside_dlt:
+                pols = self.purchase_line_ids.filtered(
+                    lambda l: l.date_planned <= fields.Datetime.to_datetime(cut_date)
                     and l.state in ("draft", "sent")
                 )
-                rec.rfq_outside_dlt_qty = sum(pols.mapped("product_qty"))
+            else:
+                pols = self.purchase_line_ids.filtered(
+                    lambda l: l.date_planned > fields.Datetime.to_datetime(cut_date)
+                    and l.order_id.state in ("draft", "sent")
+                )
+            res += sum(pols.mapped("product_qty"))
         return res
 
     def _get_date_planned(self, force_lt=None):
@@ -145,7 +152,6 @@ def _get_location_routes_of_parents(self, routes, parents):
                 )
                 & parents
             )
-            or any(rule.action == "buy" for rule in route.rule_ids)
         )
 
     def get_parents(self):
@@ -200,11 +206,11 @@ def write(self, vals):
             self._calc_distributed_source_location()
         return res
 
-    def action_view_supply(self, outside_dlt=False, view_rfq=False):
-        res = super().action_view_supply(outside_dlt, view_rfq)
+    def action_view_supply(self, outside_dlt=False):
+        res = super().action_view_supply(outside_dlt)
         # If route is set it means that there is at least two alternatively ways to
         # procure the buffer. Therefore, we will show Stock Pickings.
-        if self.route_id and not view_rfq:
+        if self.route_id:
             moves = self._search_stock_moves_incoming(outside_dlt)
             picks = moves.mapped("picking_id")
             res = self.env["ir.actions.actions"]._for_xml_id(

stock_buffer_route/static/description/index.html

@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="utf-8"?>
 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
 <head>
@@ -366,7 +367,7 @@ <h1 class="title">Stock Buffer Route</h1>
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:012eb267c5f3b2dc1aee2e761f61510e4a51b010f0c23ac588a92bf7c3d50344
+!! source digest: sha256:71fbdc11c56d2d40834b454605a88fb709845b8483b6d4962c4afba9a2099d8e
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/lgpl-3.0-standalone.html"><img alt="License: LGPL-3" src="https://img.shields.io/badge/licence-LGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/ddmrp/tree/16.0/stock_buffer_route"><img alt="OCA/ddmrp" src="https://img.shields.io/badge/github-OCA%2Fddmrp-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/ddmrp-16-0/ddmrp-16-0-stock_buffer_route"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/ddmrp&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module allows to restrict a specific route to be used in the

stock_buffer_route/wizards/make_procurement_buffer.py

@@ -30,11 +30,11 @@ class MakeProcurementBufferItem(models.TransientModel):
     _inherit = "make.procurement.buffer.item"
 
     route_ids = fields.Many2many(
-        "stock.location.route",
+        "stock.route",
         string="Allowed routes",
     )
     route_id = fields.Many2one(
-        "stock.location.route",
+        "stock.route",
         domain="[('id', 'in', route_ids)]",
     )
     item_type_alternative = fields.Selection(