From af806f7bcc9994e9233f809c507c75873886b9fe Mon Sep 17 00:00:00 2001
From: Alexis de Lattre <alexis.delattre@akretion.com>
Date: Fri, 21 Jan 2022 11:24:59 +0100
Subject: [PATCH] stock_usability: usability improvements on inventory and
 picking form view

---
 stock_usability/stock.py       | 13 +++++++++++++
 stock_usability/stock_view.xml | 19 ++++++++++++++++++-
 2 files changed, 31 insertions(+), 1 deletion(-)

diff --git a/stock_usability/stock.py b/stock_usability/stock.py
index c0c67dd..fdf858c 100644
--- a/stock_usability/stock.py
+++ b/stock_usability/stock.py
@@ -146,3 +146,16 @@ class StockQuant(models.Model):
         action = self.action_view_stock_moves()
         action['context'] = {'search_default_todo': True}
         return action
+
+
+class StockInventoryLine(models.Model):
+    _inherit = 'stock.inventory.line'
+
+    state = fields.Selection(store=True)
+    partner_id = fields.Many2one(states={'done': [('readonly', True)]})
+    product_id = fields.Many2one(states={'done': [('readonly', True)]})
+    product_uom_id = fields.Many2one(states={'done': [('readonly', True)]})
+    product_qty = fields.Float(states={'done': [('readonly', True)]})
+    location_id = fields.Many2one(states={'done': [('readonly', True)]})
+    package_id = fields.Many2one(states={'done': [('readonly', True)]})
+    prod_lot_id = fields.Many2one(states={'done': [('readonly', True)]})
diff --git a/stock_usability/stock_view.xml b/stock_usability/stock_view.xml
index f408b3e..8e52d9c 100644
--- a/stock_usability/stock_view.xml
+++ b/stock_usability/stock_view.xml
@@ -23,6 +23,11 @@
         <button name="action_cancel" type="object" position="attributes">
             <attribute name="confirm">Are you sure you want to cancel this picking?</attribute>
         </button>
+        <!-- picking_type_id updates location_id and location_dest_id, we it's
+             better to put it BEFORE in the view, not AFTER ! -->
+        <field name="location_id" position="before">
+            <field name="picking_type_id" position="move"/>
+        </field>
         <!-- STOCK MOVE -->
         <!-- This sum is useful to check the 'number of items' to transfer... -->
         <xpath expr="//field[@name='move_ids_without_package']/tree/field[@name='product_uom_qty']" position="attributes">
@@ -350,7 +355,9 @@ should be able to access it. So I add a menu entry under Inventory Control. -->
         <button name="action_reset_product_qty" type="object" position="attributes">
             <attribute name="confirm">Are you sure you want to reset all quantities to 0 ?</attribute>
         </button>
-
+        <button name="action_inventory_line_tree" position="attributes">
+            <attribute name="states">confirm,done</attribute>
+        </button>
     </field>
 </record>
 
@@ -370,6 +377,16 @@ should be able to access it. So I add a menu entry under Inventory Control. -->
     </field>
 </record>
 
+<record id="stock_inventory_line_tree2" model="ir.ui.view">
+    <field name="model">stock.inventory.line</field>
+    <field name="inherit_id" ref="stock.stock_inventory_line_tree2"/>
+    <field name="arch" type="xml">
+        <field name="prod_lot_id" position="attributes">
+            <attribute name="attrs">{'readonly': ['|', ('product_tracking', '=', 'none'), ('state', '=', 'done')]}</attribute>
+        </field>
+    </field>
+</record>
+
 <record id="view_stock_quant_tree" model="ir.ui.view">
     <field name="name">stock.usability.quant.tree</field>
     <field name="model">stock.quant</field>