From b35a0918d07329d2a2affeb7d44e68693f6d9712 Mon Sep 17 00:00:00 2001
From: Adrien Widart <awt@odoo.com>
Date: Tue, 5 Jul 2022 13:44:40 +0000
Subject: [PATCH] [FIX] stock: display Reserve button only if relevant

3-steps delivery. A user confirms a SO with a storable product, it
generates 3 pickings (pick, pack, ship). On the Forecasted Report of P,
there is a line for that SO. A button is available (Reserve) but if the
user clicks on that button, it won't do anything (even if there are some
P in stock).

The Reserve button is linked to the ship SM, so trying to assign it is
useless, we first need to process the pick/pack steps.

In such situation, displaying the button is confusing.

OPW-2784998

closes odoo/odoo#95578

X-original-commit: 71769d536118da8e35dd18672b5d6c74a2180310
Signed-off-by: William Henrotin (whe) <whe@odoo.com>
---
 addons/stock/report/report_stock_forecasted.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/addons/stock/report/report_stock_forecasted.xml b/addons/stock/report/report_stock_forecasted.xml
index 17a990223d1..802804c9d66 100644
--- a/addons/stock/report/report_stock_forecasted.xml
+++ b/addons/stock/report/report_stock_forecasted.xml
@@ -121,7 +121,7 @@
                                 </t>
                                 <t t-elif="line['replenishment_filled']">
                                     <t t-if="line['document_out']">Inventory On Hand
-                                        <button t-if="line['move_out'] and line['move_out'].picking_id"
+                                        <button t-if="line['move_out'] and line['move_out'].state in ('confirmed', 'partially_available') and line['move_out'].picking_id"
                                             class="btn btn-sm btn-primary o_report_replenish_reserve"
                                             t-attf-model="stock.picking"
                                             t-att-model-id="line['move_out'].picking_id.id"