[IMP] create refund from rma group

rma_account/models/rma_order.py

@@ -17,6 +17,11 @@ def _compute_invoice_count(self):
             invoices = rec.mapped("rma_line_ids.move_id")
             rec.invoice_count = len(invoices)
 
+    @api.depends("rma_line_ids", "rma_line_ids.qty_to_refund")
+    def _compute_qty_to_refund(self):
+        for rec in self:
+            rec.qty_to_refund = sum(rec.rma_line_ids.mapped("qty_to_refund"))
+
     add_move_id = fields.Many2one(
         comodel_name="account.move",
         string="Add Invoice",
@@ -29,6 +34,10 @@ def _compute_invoice_count(self):
     invoice_count = fields.Integer(
         compute="_compute_invoice_count", string="# of Invoices"
     )
+    qty_to_refund = fields.Float(
+        digits="Product Unit of Measure",
+        compute="_compute_qty_to_refund",
+    )
 
     def _prepare_rma_line_from_inv_line(self, line):
         if self.type == "customer":

rma_account/views/rma_order_view.xml

@@ -32,6 +32,9 @@
             <xpath expr="//field[@name='rma_line_ids']/tree" position="inside">
                 <field name="refund_policy" invisible="True" />
             </xpath>
+            <xpath expr="//field[@name='qty_to_deliver']" position="after">
+                <field name="qty_to_refund" invisible="1" />
+            </xpath>
         </field>
     </record>
 

rma_account/wizards/rma_add_account_move.py

@@ -11,7 +11,7 @@ class RmaAddAccountMove(models.TransientModel):
 
     @api.model
     def default_get(self, fields_list):
-        res = super(RmaAddAccountMove, self).default_get(fields_list)
+        res = super().default_get(fields_list)
         rma_obj = self.env["rma.order"]
         rma_id = self.env.context["active_ids"] or []
         active_model = self.env.context["active_model"]

rma_account/wizards/rma_refund.py

@@ -12,7 +12,14 @@ class RmaRefund(models.TransientModel):
     @api.model
     def _get_reason(self):
         active_ids = self.env.context.get("active_ids", False)
-        return self.env["rma.order.line"].browse(active_ids[0]).rma_id.name or ""
+        active_model = self.env.context.get("active_model", False)
+        reason = ""
+        if active_model and active_ids:
+            if active_model == "rma.order":
+                reason = self.env[active_model].browse(active_ids[0]).name or ""
+            else:
+                reason = self.env[active_model].browse(active_ids[0]).rma_id.name or ""
+        return reason
 
     @api.returns("rma.order.line")
     def _prepare_item(self, line):
@@ -37,16 +44,24 @@ def default_get(self, fields_list):
         supplier.
         """
         context = self._context.copy()
-        res = super(RmaRefund, self).default_get(fields_list)
+        res = super().default_get(fields_list)
         rma_line_obj = self.env["rma.order.line"]
-        rma_line_ids = self.env.context["active_ids"] or []
-        active_model = self.env.context["active_model"]
-        if not rma_line_ids:
+        rma_obj = self.env["rma.order"]
+        active_ids = context.get("active_ids") or []
+        active_model = context.get("active_model")
+        if not active_ids:
             return res
-        assert active_model == "rma.order.line", "Bad context propagation"
+        assert active_model in [
+            "rma.order.line",
+            "rma.order",
+        ], "Bad context propagation"
 
         items = []
-        lines = rma_line_obj.browse(rma_line_ids)
+        if active_model == "rma.order":
+            rma = rma_obj.browse(active_ids)
+            lines = rma.rma_line_ids
+        else:
+            lines = rma_line_obj.browse(active_ids)
         if len(lines.mapped("partner_id")) > 1:
             raise ValidationError(
                 _(
@@ -90,8 +105,16 @@ def compute_refund(self):
             return new_refund
 
     def invoice_refund(self):
-        rma_line_ids = self.env["rma.order.line"].browse(self.env.context["active_ids"])
-        for line in rma_line_ids:
+        rma_line_obj = self.env["rma.order.line"]
+        rma_obj = self.env["rma.order"]
+        active_ids = self.env.context.get("active_ids") or []
+        active_model = self.env.context.get("active_model")
+        if active_model == "rma.order":
+            rma = rma_obj.browse(active_ids)
+            lines = rma.rma_line_ids
+        else:
+            lines = rma_line_obj.browse(active_ids)
+        for line in lines:
             if line.state != "approved":
                 raise ValidationError(_("RMA %s is not approved") % line.name)
         new_invoice = self.compute_refund()

rma_account/wizards/rma_refund.xml

@@ -72,4 +72,26 @@
         </field>
     </record>
 
+    <record id="view_rma_button_refund_form" model="ir.ui.view">
+        <field name="model">rma.order</field>
+        <field name="inherit_id" ref="rma.view_rma_form" />
+        <field name="arch" type="xml">
+            <xpath expr="//header" position="inside">
+                <button
+                    name="%(action_rma_refund)d"
+                    string="Create Refund"
+                    class="oe_highlight"
+                    attrs="{'invisible':['|', '|', ('qty_to_refund', '=', 0),  ('qty_to_refund', '&lt;', 0), ('state', '!=', 'approved')]}"
+                    type="action"
+                />
+                <button
+                    name="%(action_rma_refund)d"
+                    string="Create Refund"
+                    attrs="{'invisible':['|', ('qty_to_refund', '>', 0), ('state', '!=', 'approved')]}"
+                    type="action"
+                />
+            </xpath>
+        </field>
+    </record>
+
 </odoo>