From dfd91f9ba44debcf9d5016f6398545bd11230c46 Mon Sep 17 00:00:00 2001
From: Christopher Ormaza <chris.ormaza@forgeflow.com>
Date: Fri, 28 Jan 2022 11:23:47 -0500
Subject: [PATCH] [IMP] rma, rma_sale: added select all, messages for UX

---
 rma/wizards/rma_add_stock_move.py       | 16 +++++++++++
 rma/wizards/rma_add_stock_move_view.xml | 37 +++++++++++++++++--------
 2 files changed, 42 insertions(+), 11 deletions(-)

diff --git a/rma/wizards/rma_add_stock_move.py b/rma/wizards/rma_add_stock_move.py
index a9425373..ce932a7b 100644
--- a/rma/wizards/rma_add_stock_move.py
+++ b/rma/wizards/rma_add_stock_move.py
@@ -58,6 +58,22 @@ class RmaAddStockMove(models.TransientModel):
         comodel_name="stock.production.lot", string="Lots/Serials selected"
     )
 
+    def select_all(self):
+        self.ensure_one()
+        self.write(
+            {
+                "lot_ids": [(6, 0, self.lot_domain_ids.ids)],
+            }
+        )
+        return {
+            "type": "ir.actions.act_window",
+            "name": _("Add from Stock Move"),
+            "view_mode": "form",
+            "res_model": self._name,
+            "res_id": self.id,
+            "target": "new",
+        }
+
     def _prepare_rma_line_from_stock_move(self, sm, lot=False):
         if self.env.context.get("customer"):
             operation = (
diff --git a/rma/wizards/rma_add_stock_move_view.xml b/rma/wizards/rma_add_stock_move_view.xml
index cd35c0d3..9c4e02a9 100644
--- a/rma/wizards/rma_add_stock_move_view.xml
+++ b/rma/wizards/rma_add_stock_move_view.xml
@@ -34,18 +34,33 @@
                 </field>
                 <field name="show_lot_filter" invisible="1" />
                 <field name="lot_domain_ids" widget="many2many_tags" invisible="1" />
-                <label
-                    for="lot_ids"
+                <div
+                    class="oe_grey"
                     attrs="{'invisible': [('show_lot_filter', '=', False)]}"
-                    string="Create rma with lot/serials selected"
-                />
-                <field
-                    name="lot_ids"
-                    widget="many2many_tags"
-                    domain="[('id', 'in', lot_domain_ids)]"
-                    attrs="{'invisible': [('show_lot_filter', '=', False)], 'required': [('show_lot_filter', '=', True)]}"
-                    options="{'no_create': True}"
-                />
+                >
+                    The creation of the RMA Lines will be separated according to the lots or serials selected
+                </div>
+                <div class="o_row">
+                    <label
+                        for="lot_ids"
+                        attrs="{'invisible': [('show_lot_filter', '=', False)]}"
+                        string="Selected Lot/Serial Numbers"
+                    />
+                    <field
+                        name="lot_ids"
+                        widget="many2many_tags"
+                        domain="[('id', 'in', lot_domain_ids)]"
+                        attrs="{'invisible': [('show_lot_filter', '=', False)]}"
+                        options="{'no_create': True}"
+                    />
+                    <button
+                        name="select_all"
+                        type="object"
+                        string="Select all"
+                        class="oe_inline"
+                        attrs="{'invisible': [('show_lot_filter', '=', False)]}"
+                    />
+                </div>
                 <footer>
                     <button
                         string="Confirm"