From d547bce39d1ce59db64e52819efbff58d80a4e30 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C3=A9my=20Voet=20=28ryv=29?= <ryv@odoo.com>
Date: Fri, 17 Sep 2021 08:01:27 +0000
Subject: [PATCH] [REV] stock: group_by WH when multiple WH
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

This reverts commit 70ce14e798c3e2c715f17fc697c53d645c0014e6
because :
- It is ugly by default with dropship activated.
- This behavior can be done easily with a favorite search (for
poeple which wanted this feature).
- Technical debt

closes odoo/odoo#77092

Signed-off-by: Arnold Moyaux <amoyaux@users.noreply.github.com>
Signed-off-by: Rémy Voet <ryv-odoo@users.noreply.github.com>
---
 addons/stock/models/stock_picking.py       | 10 ----------
 addons/stock/views/stock_picking_views.xml | 12 +-----------
 2 files changed, 1 insertion(+), 21 deletions(-)

diff --git a/addons/stock/models/stock_picking.py b/addons/stock/models/stock_picking.py
index 7d6a640aa45..bdf42794e05 100644
--- a/addons/stock/models/stock_picking.py
+++ b/addons/stock/models/stock_picking.py
@@ -206,16 +206,6 @@ class PickingType(models.Model):
         if self.show_operations and self.code != 'incoming':
             self.show_reserved = True
 
-    @api.model
-    def action_view_picking_type(self):
-        action = self.env["ir.actions.actions"]._for_xml_id("stock.stock_picking_type_action")
-        if self.user_has_groups('stock.group_stock_multi_locations'):
-            context = literal_eval(action['context'])
-            context['search_default_groupby_warehouse_id'] = True
-            action['context'] = context
-
-        return action
-
     def _get_action(self, action_xmlid):
         action = self.env["ir.actions.actions"]._for_xml_id(action_xmlid)
         if self:
diff --git a/addons/stock/views/stock_picking_views.xml b/addons/stock/views/stock_picking_views.xml
index ab9e2cb9c8b..e84fbc67860 100644
--- a/addons/stock/views/stock_picking_views.xml
+++ b/addons/stock/views/stock_picking_views.xml
@@ -97,16 +97,6 @@
             </field>
         </record>
 
-        <record id="stock_picking_type_menu_action" model="ir.actions.server">
-            <field name="name">Overview</field>
-            <field name="model_id" ref="stock.model_stock_picking_type"/>
-            <field name="groups_id" eval="[(4, ref('stock.group_stock_user'))]"/>
-            <field name="state">code</field>
-            <field name="code">
-                action = model.action_view_picking_type()
-            </field>
-        </record>
-
         <record id="action_picking_type_list" model="ir.actions.act_window">
             <field name="name">Operations Types</field>
             <field name="res_model">stock.picking.type</field>
@@ -123,7 +113,7 @@
         </record>
 
         <menuitem
-            action="stock_picking_type_menu_action"
+            action="stock_picking_type_action"
             id="stock_picking_type_menu"
             parent="menu_stock_root" sequence="0"
             name="Overview"/>