From 99139242029a7aa29cdce3e107c602b63253126c Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Fri, 20 Dec 2024 16:57:59 +0100
Subject: [PATCH] stock_usability: avoid double menu entry "Locations" under
 Inventory > Analysis (one native on quants and a second one on stock.location
 added by stock_usability)

---
 stock_usability/views/stock_location.xml | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/stock_usability/views/stock_location.xml b/stock_usability/views/stock_location.xml
index f35d6b1..47dcd5e 100644
--- a/stock_usability/views/stock_location.xml
+++ b/stock_usability/views/stock_location.xml
@@ -66,12 +66,18 @@
 <!-- By default, the menu entry for stock location is only under
 Configuration > Warehouse management
 But, the view of stock location is very useful to be able to list
-of the items present on a particular stock location => so every user
-should be able to access it. So I add a menu entry under Inventory Control. -->
+the items present on a particular stock location => so every user
+should be able to access it. So I add a menu entry under Inventory Control.
+The problem is that, in addons/stock/views/stock_quant_views.xml,
+there is a menu entry XMLID stock.menu_valuation pointing to stock.quant
+with name="Locations"... so we end up having 2 different menu entries "Locations"
+pointing to different models.
+Until I find a proper solution to that, I decided to remove the menu entry
 
 <menuitem id="stock_location_menu" action="stock.action_location_form"
     parent="stock.menu_warehouse_report"
     groups="stock.group_stock_multi_locations"
     sequence="160"/>
+-->
 
 </odoo>