[IMP] delivery_driver_stock_picking_batch: Show only name in drivers

- Assign the driver's user in batch if batch isn't wave.
- Show only name in drivers

MT-5232 @moduon

Author: EmilioPascual

delivery_driver_stock_picking_batch/models/stock_picking_batch.py

@@ -15,8 +15,19 @@ class StockPickingBatch(models.Model):
         store=True,
         domain="[('is_company', '=', False)]",
     )
+    user_id = fields.Many2one(
+        compute="_compute_user_id",
+        store=True,
+    )
 
     @api.depends("picking_ids.driver_id")
     def _compute_driver_ids(self):
         for batch in self:
             batch.driver_ids = batch.picking_ids.mapped("driver_id")
+
+    @api.depends("driver_ids")
+    def _compute_user_id(self):
+        for batch in self.filtered(lambda b: not b.is_wave):
+            users = batch.mapped("driver_ids.user_ids")
+            if users:
+                batch.user_id = users[0]

delivery_driver_stock_picking_batch/tests/test_delivery_driver_stock_picking_batch.py

@@ -2,6 +2,7 @@
 # License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl-3.0)
 
 
+from odoo.tests import new_test_user
 from odoo.tests.common import TransactionCase
 
 
@@ -11,7 +12,10 @@ def setUpClass(cls):
         super().setUpClass()
         cls.partner_test = cls.env["res.partner"].create({"name": "My Test Customer"})
         cls.driver_test_1 = cls.env["res.partner"].create({"name": "My Test Driver 1"})
-        cls.driver_test_2 = cls.env["res.partner"].create({"name": "My Test Driver 2"})
+        cls.driver_user = new_test_user(
+            cls.env, login="internal_user", groups="base.group_user", name="Driver User"
+        )
+        cls.driver_test_2 = cls.driver_user.partner_id
         cls.product_test = cls.env["product.product"].create(
             {"name": "A product to deliver"}
         )
@@ -106,8 +110,11 @@ def test_delivery_driver_stock_picking_batch(self):
             }
         )
         self.assertEqual(batch.driver_ids, self.driver_test_1 | self.driver_test_2)
+        self.assertEqual(batch.user_id, self.driver_user)
         batch.write({"picking_ids": [(3, picking_2.id)]})
         self.assertEqual(batch.driver_ids, self.driver_test_1)
         batch.write({"picking_ids": [(4, picking_2.id)]})
+        self.assertEqual(batch.user_id, self.driver_user)
         picking_2.write({"carrier_id": self.delivery_test_1.id})
         self.assertEqual(batch.driver_ids, self.driver_test_1)
+        self.assertEqual(batch.user_id, self.driver_user)

delivery_driver_stock_picking_batch/views/stock_picking_batch.xml

@@ -8,7 +8,12 @@
         <field name="inherit_id" ref="stock_picking_batch.view_picking_tree_batch" />
         <field name="arch" type="xml">
             <xpath expr="//field[@name='scheduled_date']" position="after">
-                <field name="driver_id" optional="hide" />
+                <field
+                    name="driver_id"
+                    optional="hide"
+                    context="{'show_driver': True}"
+                    options='{"always_reload": True}'
+                />
             </xpath>
         </field>
     </record>
@@ -18,7 +23,12 @@
         <field name="inherit_id" ref="stock_picking_batch.stock_picking_batch_form" />
         <field name="arch" type="xml">
             <xpath expr="//field[@name='scheduled_date']" position="after">
-                <field name="driver_ids" widget="many2many_tags" />
+                <field
+                    name="driver_ids"
+                    widget="many2many_tags"
+                    context="{'show_driver': True}"
+                    options='{"always_reload": True}'
+                />
             </xpath>
         </field>
     </record>
@@ -28,7 +38,13 @@
         <field name="inherit_id" ref="stock_picking_batch.stock_picking_batch_tree" />
         <field name="arch" type="xml">
             <xpath expr="//field[@name='scheduled_date']" position="after">
-                <field name="driver_ids" widget="many2many_tags" optional="hide" />
+                <field
+                    name="driver_ids"
+                    widget="many2many_tags"
+                    optional="hide"
+                    context="{'show_driver': True}"
+                    options='{"always_reload": True}'
+                />
             </xpath>
         </field>
     </record>