[IMP] rma_reason: add rma reason to stock return wizard

Author: sbejaoui

rma_reason/__init__.py

@@ -1 +1,2 @@
 from . import models
+from . import wizards

rma_reason/__manifest__.py

@@ -17,6 +17,7 @@
         "views/rma_reason.xml",
         "views/res_config_settings.xml",
         "views/rma_portal_templates.xml",
+        "wizards/stock_return_picking.xml",
     ],
     "demo": [
         "demo/rma_reason.xml",

rma_reason/wizards/__init__.py

@@ -0,0 +1,2 @@
+from . import stock_return_picking
+from . import stock_return_picking_line

rma_reason/wizards/stock_return_picking.py

@@ -0,0 +1,13 @@
+# Copyright 2024 ACSONE SA/NV
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import fields, models
+
+
+class StockReturnPicking(models.TransientModel):
+
+    _inherit = "stock.return.picking"
+
+    rma_reason_id = fields.Many2one(
+        comodel_name="rma.reason", readonly=False, string="RMA Reason"
+    )

rma_reason/wizards/stock_return_picking.xml

@@ -0,0 +1,34 @@
+<?xml version="1.0" encoding="utf-8" ?>
+<!-- Copyright 2024 ACSONE SA/NV
+     License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl). -->
+<odoo>
+
+    <record model="ir.ui.view" id="stock_return_picking_form_view">
+        <field name="model">stock.return.picking</field>
+        <field name="inherit_id" ref="stock.view_stock_return_picking_form" />
+        <field name="arch" type="xml">
+            <xpath
+                expr="//field[@name='product_return_moves']//tree//field[@name='rma_operation_id']"
+                position="before"
+            >
+                <field
+                    name="rma_reason_id"
+                    attrs="{'column_invisible': [('parent.create_rma', '=', False)], 'required': [('is_rma_reason_required', '=', True)]}"
+                />
+                <field name="is_rma_reason_required" invisible="1" />
+            </xpath>
+            <xpath
+                expr="//group[@name='group_rma']//field[@name='rma_operation_id']"
+                position="before"
+            >
+                <field
+                    name="rma_reason_id"
+                    attrs="{'invisible': [('create_rma', '=', False)]}"
+                />
+            </xpath>
+        </field>
+    </record>
+
+
+
+</odoo>

rma_reason/wizards/stock_return_picking_line.py

@@ -0,0 +1,32 @@
+# Copyright 2024 ACSONE SA/NV
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import api, fields, models
+
+
+class StockReturnPickingLine(models.TransientModel):
+
+    _inherit = "stock.return.picking.line"
+
+    rma_reason_id = fields.Many2one(
+        comodel_name="rma.reason",
+        compute="_compute_rma_reason_id",
+        store=True,
+        readonly=False,
+        string="RMA Reason",
+    )
+    is_rma_reason_required = fields.Boolean(
+        related="wizard_id.company_id.is_rma_reason_required"
+    )
+
+    @api.depends("wizard_id.rma_reason_id")
+    def _compute_rma_reason_id(self):
+        for rec in self:
+            if rec.wizard_id.rma_reason_id:
+                rec.rma_reason_id = rec.wizard_id.rma_reason_id
+
+    def _prepare_rma_vals(self):
+        self.ensure_one()
+        vals = super()._prepare_rma_vals()
+        vals["reason_id"] = self.rma_reason_id.id
+        return vals

rma_sale_reason/views/rma_portal_templates.xml

@@ -8,7 +8,7 @@
         name="My RMA"
         inherit_id="rma_sale.sale_rma_request_form"
     >
-        <xpath expr="//t[@t-set='operations']" position="after">
+        <xpath expr="//t[@t-set='operations']" position="before">
 
         <t t-set="reasons" t-value="sale_order.env['rma.reason'].search([])" />
 

rma_sale_reason/wizards/sale_order_rma_wizard.xml

@@ -14,7 +14,7 @@
             </xpath>
             <xpath
                 expr="//field[@name='line_ids']//field[@name='operation_id']"
-                position="after"
+                position="before"
             >
                 <field name="is_rma_reason_required" invisible="True" />
                 <field