Merge pull request #128 from ForgeFlow/13.0-rma_account

[13.0] rma_account

rma/models/res_config_settings.py

@@ -18,3 +18,5 @@ class ResConfigSettings(models.TransientModel):
         readonly=False,
         implied_group="rma.group_rma_groups",
     )
+
+    module_rma_account = fields.Boolean(string="RMA invoicing")

rma/models/rma_order.py

@@ -110,7 +110,7 @@ class RmaOrder(models.Model):
     )
     rma_line_ids = fields.One2many("rma.order.line", "rma_id", string="RMA lines")
     in_shipment_count = fields.Integer(
-        compute="_compute_in_shipment_count", string="# of Invoices"
+        compute="_compute_in_shipment_count", string="# of Shipments"
     )
     out_shipment_count = fields.Integer(
         compute="_compute_out_shipment_count", string="# of Outgoing Shipments"

rma/views/res_config_settings_views.xml

@@ -34,6 +34,18 @@
                                 </div>
                             </div>
                         </div>
+                        <div class="col-12 col-lg-6 o_setting_box">
+                            <div class="o_setting_left_pane">
+                                <field name="module_rma_account"/>
+                            </div>
+                            <div class="o_setting_right_pane">
+                                <label for="module_rma_account"/>
+                                <div class="text-muted">
+                                    Enable RMA invoicing
+                                </div>
+                            </div>
+                        </div>
+
                     </div>
                 </div>
             </xpath>

rma/views/rma_menu.xml

@@ -75,4 +75,25 @@
         sequence="40"
         parent="rma.menu_rma_config"
         action="action_rma_operation_supplier"/>
+
+    <!-- Master data menus-->
+
+    <menuitem id="menu_rma_rma_control" name="Master Data" parent="menu_rma_root" sequence="50"/>
+    <menuitem id="menu_rma_config_warehouse" name="Warehouse"
+        parent="rma.menu_rma_rma_control" sequence="10"/>
+    <menuitem id="menu_rma_config_stock" name="Products" action="stock.product_template_action_product"
+        parent="rma.menu_rma_config_warehouse" sequence="10"/>
+    <menuitem id="rma_product_product_menu" name="Product Variants" action="stock.stock_product_normal_action"
+        parent="rma.menu_rma_config_warehouse" sequence="20" groups="product.group_product_variant"/>
+
+    <menuitem id="menu_rma_config_partners" name="Partners"
+        parent="rma.menu_rma_rma_control" sequence="20"/>
+    <menuitem id="rma_res_partner_menu_customer"
+        name="Customers"
+        parent="rma.menu_rma_config_partners"
+        action="base.action_partner_form"
+        sequence="50"/>
+    <menuitem id="rma_res_partner_menu_suppliers" name="Suppliers"
+        parent="rma.menu_rma_config_partners"
+        action="account.res_partner_action_supplier" sequence="60"/>
 </odoo>

rma_account/README.rst

@@ -0,0 +1,52 @@
+.. image:: https://img.shields.io/badge/licence-LGPL--3-blue.svg
+    :alt: License LGPL-3
+
+===========
+RMA Account
+===========
+
+This module integrates Return Merchandise Authorizations (RMA) with invoices,
+allowing to:
+
+#. Create complete RMA's using existing invoices as a reference.
+#. Create refunds from a RMA.
+
+Usage
+=====
+
+RMA are accessible though Inventory menu. There's four menus, divided by type.
+Users can access to the list of RMA or RMA lines.
+
+Create an RMA:
+
+#. Select a partner. Fill the rma lines by selecting an invoice.
+#. Request approval and approve.
+#. Click on RMA Lines button.
+#. Click on more and select an option: "Receive products", "Create Delivery
+   Order, Create Refund".
+#. Go back to the RMA. Set the RMA to done if not further action is required.
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues
+<https://github.com/ForgeFlow/stock-rma/issues>`_. In case of trouble, please
+check there if your issue has already been reported. If you spotted it first,
+help us smashing it by providing a detailed and welcomed feedback.
+
+Credits
+=======
+
+Contributors
+------------
+
+* Jordi Ballester Alomar <jordi.ballester@ForgeFlow.com>
+* Aaron Henriquez <ahenriquez@ForgeFlow.com>
+* Lois Rilo <lois.rilo@ForgeFlow.com>
+* Bhavesh Odedra <bodedra@opensourceintegrators.com>
+* Akim Juillerat <akim.juillerat@camptocamp.com>
+
+Maintainer
+----------
+
+This module is maintained by ForgeFlow.

rma_account/__init__.py

@@ -0,0 +1,5 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from . import models
+from . import wizards

rma_account/__manifest__.py

@@ -0,0 +1,25 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+{
+    "name": "RMA Account",
+    "version": "13.0.1.0.0",
+    "license": "LGPL-3",
+    "category": "RMA",
+    "summary": "Integrates RMA with Invoice Processing",
+    "author": "ForgeFlow",
+    "website": "https://github.com/ForgeFlow/stock-rma",
+    "depends": ["stock_account", "rma"],
+    "data": [
+        "security/ir.model.access.csv",
+        "data/rma_operation.xml",
+        "views/rma_order_view.xml",
+        "views/rma_operation_view.xml",
+        "views/rma_order_line_view.xml",
+        "views/account_move_view.xml",
+        "views/rma_account_menu.xml",
+        "wizards/rma_add_account_move.xml",
+        "wizards/rma_refund.xml",
+    ],
+    "installable": True,
+}

rma_account/data/rma_operation.xml

@@ -0,0 +1,42 @@
+<?xml version="1.0"?>
+<odoo noupdate="1">
+
+    <record id="rma.rma_operation_customer_replace" model="rma.operation">
+        <field name="refund_policy">no</field>
+    </record>
+
+    <record id="rma.rma_operation_supplier_replace" model="rma.operation">
+        <field name="refund_policy">no</field>
+    </record>
+
+    <record id="rma_operation_customer_refund" model="rma.operation">
+        <field name="name">Refund after receive</field>
+        <field name="code">RF-C</field>
+        <field name="refund_policy">received</field>
+        <field name="receipt_policy">ordered</field>
+        <field name="delivery_policy">no</field>
+        <field name="type">customer</field>
+        <field name="in_route_id" ref="rma.route_rma_customer"/>
+        <field name="out_route_id" ref="rma.route_rma_customer"/>
+    </record>
+
+    <record id="rma_operation_supplier_refund" model="rma.operation">
+        <field name="name">Refund after deliver</field>
+        <field name="code">RF-S</field>
+        <field name="refund_policy">ordered</field>
+        <field name="receipt_policy">no</field>
+        <field name="delivery_policy">ordered</field>
+        <field name="type">supplier</field>
+        <field name="in_route_id" ref="rma.route_rma_supplier"/>
+        <field name="out_route_id" ref="rma.route_rma_supplier"/>
+    </record>
+
+    <record id="rma.rma_operation_ds_replace" model="rma.operation">
+        <field name="refund_policy">no</field>
+    </record>
+
+    <record id="rma.rma_operation_ds_replace_supplier" model="rma.operation">
+        <field name="refund_policy">no</field>
+    </record>
+
+</odoo>

rma_account/models/__init__.py

@@ -0,0 +1,7 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from . import rma_order
+from . import rma_order_line
+from . import rma_operation
+from . import account_move

rma_account/models/account_move.py

@@ -0,0 +1,174 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import api, fields, models
+from odoo.tools.float_utils import float_compare
+
+
+class AccountMove(models.Model):
+    _inherit = "account.move"
+
+    @api.depends("line_ids.rma_line_ids")
+    def _compute_rma_count(self):
+        for inv in self:
+            rmas = self.mapped("line_ids.rma_line_ids")
+            inv.rma_count = len(rmas)
+
+    def _prepare_invoice_line_from_rma_line(self, line):
+        qty = line.qty_to_refund
+        if float_compare(qty, 0.0, precision_rounding=line.uom_id.rounding) <= 0:
+            qty = 0.0
+        # Todo fill taxes from somewhere
+        invoice_line = self.env["account.move.line"]
+        data = {
+            "purchase_line_id": line.id,
+            "name": line.name + ": " + line.name,
+            "product_uom_id": line.uom_id.id,
+            "product_id": line.product_id.id,
+            "account_id": invoice_line.with_context(
+                {"journal_id": self.journal_id.id, "type": "in_invoice"}
+            )._default_account(),
+            "price_unit": line.company_id.currency_id.with_context(
+                date=self.date
+            ).compute(line.price_unit, self.currency_id, round=False),
+            "quantity": qty,
+            "discount": 0.0,
+            "rma_line_ids": [(4, line.id)],
+        }
+        return data
+
+    @api.onchange("add_rma_line_id")
+    def on_change_add_rma_line_id(self):
+        if not self.add_rma_line_id:
+            return {}
+        if not self.partner_id:
+            self.partner_id = self.add_rma_line_id.partner_id.id
+
+        new_line = self.env["account.move.line"]
+        if self.add_rma_line_id not in (self.line_ids.mapped("rma_line_id")):
+            data = self._prepare_invoice_line_from_rma_line(self.add_rma_line_id)
+            new_line = new_line.new(data)
+            new_line._set_additional_fields(self)
+        self.line_ids += new_line
+        self.add_rma_line_id = False
+        return {}
+
+    rma_count = fields.Integer(compute="_compute_rma_count", string="# of RMA")
+
+    add_rma_line_id = fields.Many2one(
+        comodel_name="rma.order.line",
+        string="Add from RMA line",
+        ondelete="set null",
+        help="Create a refund in based on an existing rma_line",
+    )
+
+    def action_view_rma_supplier(self):
+        action = self.env.ref("rma.action_rma_supplier_lines")
+        result = action.read()[0]
+        rma_ids = self.mapped("line_ids.rma_line_ids").ids
+        if rma_ids:
+            # choose the view_mode accordingly
+            if len(rma_ids) > 1:
+                result["domain"] = [("id", "in", rma_ids)]
+            else:
+                res = self.env.ref("rma.view_rma_line_supplier_form", False)
+                result["views"] = [(res and res.id or False, "form")]
+                result["res_id"] = rma_ids[0]
+        return result
+
+    def action_view_rma_customer(self):
+        action = self.env.ref("rma.action_rma_customer_lines")
+        result = action.read()[0]
+        rma_ids = self.mapped("line_ids.rma_line_ids").ids
+        if rma_ids:
+            # choose the view_mode accordingly
+            if len(rma_ids) > 1:
+                result["domain"] = [("id", "in", rma_ids)]
+            else:
+                res = self.env.ref("rma.view_rma_line_form", False)
+                result["views"] = [(res and res.id or False, "form")]
+                result["res_id"] = rma_ids[0]
+        return result
+
+
+class AccountMoveLine(models.Model):
+    _inherit = "account.move.line"
+
+    @api.model
+    def name_search(self, name, args=None, operator="ilike", limit=100):
+        """Allows to search by Invoice number. This has to be done this way,
+        as Odoo adds extra args to name_search on _name_search method that
+        will make impossible to get the desired result."""
+        if not args:
+            args = []
+        lines = self.search([("move_id.name", operator, name)] + args, limit=limit)
+        res = lines.name_get()
+        if limit:
+            limit_rest = limit - len(lines)
+        else:
+            # limit can be 0 or None representing infinite
+            limit_rest = limit
+        if limit_rest or not limit:
+            args += [("id", "not in", lines.ids)]
+            res += super(AccountMoveLine, self).name_search(
+                name, args=args, operator=operator, limit=limit_rest
+            )
+        return res
+
+    def name_get(self):
+        res = []
+        if self.env.context.get("rma"):
+            for inv in self:
+                if inv.move_id.ref:
+                    res.append(
+                        (
+                            inv.id,
+                            "INV:%s | REF:%s | ORIG:%s | PART:%s | QTY:%s"
+                            % (
+                                inv.move_id.name or "",
+                                inv.move_id.invoice_origin or "",
+                                inv.move_id.ref or "",
+                                inv.product_id.name,
+                                inv.quantity,
+                            ),
+                        )
+                    )
+                elif inv.move_id.name:
+                    res.append(
+                        (
+                            inv.id,
+                            "INV:%s | ORIG:%s | PART:%s | QTY:%s"
+                            % (
+                                inv.move_id.name or "",
+                                inv.move_id.invoice_origin or "",
+                                inv.product_id.name,
+                                inv.quantity,
+                            ),
+                        )
+                    )
+                else:
+                    res.append(super(AccountMoveLine, inv).name_get()[0])
+            return res
+        else:
+            return super(AccountMoveLine, self).name_get()
+
+    def _compute_rma_count(self):
+        for invl in self:
+            rma_lines = invl.mapped("rma_line_ids")
+            invl.rma_line_count = len(rma_lines)
+
+    rma_line_count = fields.Integer(compute="_compute_rma_count", string="# of RMA")
+    rma_line_ids = fields.One2many(
+        comodel_name="rma.order.line",
+        inverse_name="account_move_line_id",
+        string="RMA",
+        readonly=True,
+        help="This will contain the RMA lines for the invoice line",
+    )
+
+    rma_line_id = fields.Many2one(
+        comodel_name="rma.order.line",
+        string="RMA line refund",
+        ondelete="set null",
+        help="This will contain the rma line that originated the refund line",
+    )

rma_account/models/rma_operation.py

@@ -0,0 +1,19 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import fields, models
+
+
+class RmaOperation(models.Model):
+    _inherit = "rma.operation"
+
+    refund_policy = fields.Selection(
+        [
+            ("no", "No refund"),
+            ("ordered", "Based on Ordered Quantities"),
+            ("delivered", "Based on Delivered Quantities"),
+            ("received", "Based on Received Quantities"),
+        ],
+        string="Refund Policy",
+        default="no",
+    )

rma_account/models/rma_order.py

@@ -0,0 +1,117 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import api, fields, models
+
+
+class RmaOrder(models.Model):
+    _inherit = "rma.order"
+
+    def _compute_invoice_refund_count(self):
+        for rec in self:
+            invoices = rec.mapped("rma_line_ids.refund_line_ids.move_id")
+            rec.invoice_refund_count = len(invoices)
+
+    def _compute_invoice_count(self):
+        for rec in self:
+            invoices = rec.mapped("rma_line_ids.move_id")
+            rec.invoice_count = len(invoices)
+
+    add_move_id = fields.Many2one(
+        comodel_name="account.move",
+        string="Add Invoice",
+        ondelete="set null",
+        readonly=True,
+    )
+    invoice_refund_count = fields.Integer(
+        compute="_compute_invoice_refund_count", string="# of Refunds"
+    )
+    invoice_count = fields.Integer(
+        compute="_compute_invoice_count", string="# of Invoices"
+    )
+
+    def _prepare_rma_line_from_inv_line(self, line):
+        if self.type == "customer":
+            operation = (
+                self.rma_line_ids.product_id.rma_customer_operation_id
+                or self.rma_line_ids.product_id.categ_id.rma_customer_operation_id
+            )
+        else:
+            operation = (
+                self.rma_line_ids.product_id.rma_supplier_operation_id
+                or self.rma_line_ids.product_id.categ_id.rma_supplier_operation_id
+            )
+        data = {
+            "account_move_line_id": line.id,
+            "product_id": line.product_id.id,
+            "name": line.name,
+            "origin": line.move_id.name,
+            "uom_id": line.product_uom_id.id,
+            "operation_id": operation,
+            "product_qty": line.quantity,
+            "price_unit": line.move_id.currency_id.compute(
+                line.price_unit, line.currency_id, round=False
+            ),
+            "rma_id": self.id,
+        }
+        return data
+
+    @api.onchange("add_move_id")
+    def on_change_invoice(self):
+        if not self.add_move_id:
+            return {}
+        if not self.partner_id:
+            self.partner_id = self.add_move_id.partner_id.id
+        new_lines = self.env["rma.order.line"]
+        for line in self.add_move_id.line_ids:
+            # Load a PO line only once
+            if line in self.rma_line_ids.mapped("account_move_line_id"):
+                continue
+            data = self._prepare_rma_line_from_inv_line(line)
+            new_line = new_lines.new(data)
+            new_lines += new_line
+
+        self.rma_line_ids += new_lines
+        self.date_rma = fields.Datetime.now()
+        self.delivery_address_id = self.add_move_id.partner_id.id
+        self.invoice_address_id = self.add_move_id.partner_id.id
+        self.add_move_id = False
+        return {}
+
+    @api.model
+    def prepare_rma_line(self, origin_rma, rma_id, line):
+        line_values = super(RmaOrder, self).prepare_rma_line(origin_rma, rma_id, line)
+        line_values["invoice_address_id"] = line.invoice_address_id.id
+        return line_values
+
+    @api.model
+    def _prepare_rma_data(self, partner, origin_rma):
+        res = super(RmaOrder, self)._prepare_rma_data(partner, origin_rma)
+        res["invoice_address_id"] = partner.id
+        return res
+
+    def action_view_invoice_refund(self):
+        move_ids = self.mapped("rma_line_ids.move_id").ids
+        form_view_ref = self.env.ref("account.view_move_form", False)
+        tree_view_ref = self.env.ref("account.view_move_tree", False)
+
+        return {
+            "domain": [("id", "in", move_ids)],
+            "name": "Refunds",
+            "res_model": "account.move",
+            "type": "ir.actions.act_window",
+            "views": [(tree_view_ref.id, "tree"), (form_view_ref.id, "form")],
+        }
+
+    def action_view_invoice(self):
+        move_ids = self.mapped("rma_line_ids.move_id").ids
+        form_view_ref = self.env.ref("account.view_move_form", False)
+        tree_view_ref = self.env.ref("account.view_move_tree", False)
+
+        return {
+            "domain": [("id", "in", move_ids)],
+            "name": "Originating Invoice",
+            "res_model": "account.move",
+            "type": "ir.actions.act_window",
+            "views": [(tree_view_ref.id, "tree"), (form_view_ref.id, "form")],
+        }

rma_account/models/rma_order_line.py

@@ -0,0 +1,289 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import _, api, fields, models
+from odoo.exceptions import UserError, ValidationError
+
+
+class RmaOrderLine(models.Model):
+    _inherit = "rma.order.line"
+
+    @api.model
+    def _default_invoice_address(self):
+        partner_id = self.env.context.get("partner_id")
+        if partner_id:
+            return self.env["res.partner"].browse(partner_id)
+        return self.env["res.partner"]
+
+    @api.depends(
+        "refund_line_ids", "refund_line_ids.move_id.state", "refund_policy", "type"
+    )
+    def _compute_qty_refunded(self):
+        for rec in self:
+            rec.qty_refunded = sum(
+                rec.refund_line_ids.filtered(
+                    lambda i: i.move_id.state in ("posted")
+                ).mapped("quantity")
+            )
+
+    @api.depends(
+        "refund_line_ids",
+        "refund_line_ids.move_id.state",
+        "refund_policy",
+        "move_ids",
+        "move_ids.state",
+        "type",
+    )
+    def _compute_qty_to_refund(self):
+        qty = 0.0
+        for res in self:
+            if res.refund_policy == "ordered":
+                qty = res.product_qty - res.qty_refunded
+            elif res.refund_policy == "received":
+                qty = res.qty_received - res.qty_refunded
+            elif res.refund_policy == "delivered":
+                qty = res.qty_delivered - res.qty_refunded
+            res.qty_to_refund = qty
+
+    def _compute_refund_count(self):
+        for rec in self:
+            rec.refund_count = len(rec.refund_line_ids.mapped("move_id"))
+
+    invoice_address_id = fields.Many2one(
+        "res.partner",
+        string="Partner invoice address",
+        default=lambda self: self._default_invoice_address(),
+        readonly=True,
+        states={"draft": [("readonly", False)]},
+        help="Invoice address for current rma order.",
+    )
+    refund_count = fields.Integer(
+        compute="_compute_refund_count", string="# of Refunds", default=0
+    )
+    account_move_line_id = fields.Many2one(
+        comodel_name="account.move.line",
+        string="Originating Invoice Line",
+        ondelete="restrict",
+        index=True,
+        readonly=True,
+        states={"draft": [("readonly", False)]},
+    )
+    refund_line_ids = fields.One2many(
+        comodel_name="account.move.line",
+        inverse_name="rma_line_id",
+        string="Refund Lines",
+        copy=False,
+        index=True,
+        readonly=True,
+    )
+    move_id = fields.Many2one(
+        "account.move",
+        string="Source",
+        related="account_move_line_id.move_id",
+        index=True,
+        readonly=True,
+    )
+    refund_policy = fields.Selection(
+        [
+            ("no", "No refund"),
+            ("ordered", "Based on Ordered Quantities"),
+            ("delivered", "Based on Delivered Quantities"),
+            ("received", "Based on Received Quantities"),
+        ],
+        string="Refund Policy",
+        required=True,
+        default="no",
+        readonly=True,
+        states={"draft": [("readonly", False)]},
+    )
+    qty_to_refund = fields.Float(
+        string="Qty To Refund",
+        copy=False,
+        digits="Product Unit of Measure",
+        readonly=True,
+        compute="_compute_qty_to_refund",
+        store=True,
+    )
+    qty_refunded = fields.Float(
+        string="Qty Refunded",
+        copy=False,
+        digits="Product Unit of Measure",
+        readonly=True,
+        compute="_compute_qty_refunded",
+        store=True,
+    )
+
+    @api.onchange("product_id", "partner_id")
+    def _onchange_product_id(self):
+        """Domain for sale_line_id is computed here to make it dynamic."""
+        res = super(RmaOrderLine, self)._onchange_product_id()
+        if not res.get("domain"):
+            res["domain"] = {}
+        domain = [
+            "&",
+            "&",
+            ("rma_line_id", "=", False),
+            ("exclude_from_invoice_tab", "=", False),
+            "|",
+            ("move_id.partner_id", "=", self.partner_id.id),
+            ("move_id.partner_id", "child_of", self.partner_id.id),
+        ]
+        # if self.product_id:
+        #     domain.insert(2, ("product_id", "=", self.product_id.id))
+        res["domain"]["account_move_line_id"] = domain
+        return res
+
+    def _prepare_rma_line_from_inv_line(self, line):
+        self.ensure_one()
+        if not self.type:
+            self.type = self._get_default_type()
+        if self.type == "customer":
+            operation = (
+                line.product_id.rma_customer_operation_id
+                or line.product_id.categ_id.rma_customer_operation_id
+            )
+        else:
+            operation = (
+                line.product_id.rma_supplier_operation_id
+                or line.product_id.categ_id.rma_supplier_operation_id
+            )
+        if not operation:
+            operation = self.env["rma.operation"].search(
+                [("type", "=", self.type)], limit=1
+            )
+            if not operation:
+                raise ValidationError(_("Please define an operation first"))
+
+        if not operation.in_route_id or not operation.out_route_id:
+            route = self.env["stock.location.route"].search(
+                [("rma_selectable", "=", True)], limit=1
+            )
+            if not route:
+                raise ValidationError(_("Please define an rma route"))
+
+        if not operation.in_warehouse_id or not operation.out_warehouse_id:
+            warehouse = self.env["stock.warehouse"].search(
+                [("company_id", "=", self.company_id.id), ("lot_rma_id", "!=", False)],
+                limit=1,
+            )
+            if not warehouse:
+                raise ValidationError(
+                    _("Please define a warehouse with a" " default rma location")
+                )
+        data = {
+            "product_id": line.product_id.id,
+            "origin": line.move_id.name,
+            "uom_id": line.product_uom_id.id,
+            "operation_id": operation.id,
+            "product_qty": line.quantity,
+            "price_unit": line.move_id.currency_id.compute(
+                line.price_unit, line.currency_id, round=False
+            ),
+            "delivery_address_id": line.move_id.partner_id.id,
+            "invoice_address_id": line.move_id.partner_id.id,
+            "receipt_policy": operation.receipt_policy,
+            "refund_policy": operation.refund_policy,
+            "delivery_policy": operation.delivery_policy,
+            "currency_id": line.currency_id.id,
+            "in_warehouse_id": operation.in_warehouse_id.id or warehouse.id,
+            "out_warehouse_id": operation.out_warehouse_id.id or warehouse.id,
+            "in_route_id": operation.in_route_id.id or route.id,
+            "out_route_id": operation.out_route_id.id or route.id,
+            "location_id": (
+                operation.location_id.id
+                or operation.in_warehouse_id.lot_rma_id.id
+                or warehouse.lot_rma_id.id
+            ),
+        }
+        return data
+
+    @api.onchange("account_move_line_id")
+    def _onchange_account_move_line_id(self):
+        if not self.account_move_line_id:
+            return
+        data = self._prepare_rma_line_from_inv_line(self.account_move_line_id)
+        self.update(data)
+        self._remove_other_data_origin("account_move_line_id")
+
+    @api.constrains("account_move_line_id", "partner_id")
+    def _check_invoice_partner(self):
+        for rec in self:
+            if (
+                rec.account_move_line_id
+                and rec.account_move_line_id.move_id.partner_id != rec.partner_id
+            ):
+                raise ValidationError(
+                    _(
+                        "RMA customer and originating invoice line customer "
+                        "doesn't match."
+                    )
+                )
+
+    def _remove_other_data_origin(self, exception):
+        res = super(RmaOrderLine, self)._remove_other_data_origin(exception)
+        if not exception == "account_move_line_id":
+            self.account_move_line_id = False
+        return res
+
+    @api.onchange("operation_id")
+    def _onchange_operation_id(self):
+        result = super(RmaOrderLine, self)._onchange_operation_id()
+        if self.operation_id:
+            self.refund_policy = self.operation_id.refund_policy or "no"
+        return result
+
+    @api.constrains("account_move_line_id")
+    def _check_duplicated_lines(self):
+        for line in self:
+            matching_inv_lines = self.env["account.move.line"].search(
+                [("id", "=", line.account_move_line_id.id)]
+            )
+            if len(matching_inv_lines) > 1:
+                raise UserError(
+                    _(
+                        "There's an rma for the invoice line %s "
+                        "and invoice %s"
+                        % (line.account_move_line_id, line.account_move_line_id.move_id)
+                    )
+                )
+        return {}
+
+    def action_view_invoice(self):
+        form_view_ref = self.env.ref("account.view_move_form", False)
+        tree_view_ref = self.env.ref("account.view_move_tree", False)
+
+        return {
+            "domain": [("id", "in", [self.account_move_line_id.move_id.id])],
+            "name": "Originating Invoice",
+            "res_model": "account.move",
+            "type": "ir.actions.act_window",
+            "views": [(tree_view_ref.id, "tree"), (form_view_ref.id, "form")],
+        }
+
+    def action_view_refunds(self):
+        move_ids = self.mapped("refund_line_ids.move_id").ids
+        form_view_ref = self.env.ref("account.view_move_form", False)
+        tree_view_ref = self.env.ref("account.view_move_tree", False)
+
+        return {
+            "domain": [("id", "in", move_ids)],
+            "name": "Refunds",
+            "res_model": "account.move",
+            "type": "ir.actions.act_window",
+            "views": [(tree_view_ref.id, "tree"), (form_view_ref.id, "form")],
+        }
+
+    def name_get(self):
+        res = []
+        if self.env.context.get("rma"):
+            for rma in self:
+                res.append(
+                    (
+                        rma.id,
+                        "%s %s qty:%s"
+                        % (rma.name, rma.product_id.name, rma.product_qty),
+                    )
+                )
+            return res
+        else:
+            return super(RmaOrderLine, self).name_get()

rma_account/security/ir.model.access.csv

@@ -0,0 +1,5 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_account_move_customer_user,access_account_move,account.model_account_move,rma.group_rma_customer_user,1,0,0,0
+access_account_move_supplier_user,access_account_move,account.model_account_move,rma.group_rma_supplier_user,1,0,0,0
+access_account_move_line_customer_user,access_account_move_line,account.model_account_move_line,rma.group_rma_customer_user,1,0,0,0
+access_account_move_line_supplier_user,access_account_move_line,account.model_account_move_line,rma.group_rma_supplier_user,1,0,0,0

rma_account/tests/__init__.py

@@ -0,0 +1,4 @@
+# Copyright 2018 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from . import test_rma_account

rma_account/tests/test_rma_account.py

@@ -0,0 +1,226 @@
+# Copyright 2017-18 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import fields
+from odoo.tests import common
+
+
+class TestRmaAccount(common.SingleTransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super(TestRmaAccount, cls).setUpClass()
+
+        cls.rma_obj = cls.env["rma.order"]
+        cls.rma_line_obj = cls.env["rma.order.line"]
+        cls.rma_op_obj = cls.env["rma.operation"]
+        cls.rma_add_invoice_wiz = cls.env["rma_add_account_move"]
+        cls.rma_refund_wiz = cls.env["rma.refund"]
+        cls.acc_obj = cls.env["account.account"]
+        cls.inv_obj = cls.env["account.move"]
+        cls.invl_obj = cls.env["account.move.line"]
+        cls.product_obj = cls.env["product.product"]
+        customer1_obj = cls.env["res.partner"]
+
+        cls.rma_route_cust = cls.env.ref("rma.route_rma_customer")
+        receivable_type = cls.env.ref("account.data_account_type_receivable")
+        cls.cust_refund_op = cls.env.ref("rma_account.rma_operation_customer_refund")
+        cls.sup_refund_op = cls.env.ref("rma_account.rma_operation_supplier_refund")
+        cls.company_id = cls.env.user.company_id
+        # Create partners
+        customer1 = customer1_obj.create({"name": "Customer 1"})
+        supplier1 = customer1_obj.create({"name": "Supplier 1"})
+
+        # Create RMA group and operation:
+        cls.rma_group_customer = cls.rma_obj.create(
+            {"partner_id": customer1.id, "type": "customer"}
+        )
+        cls.rma_group_supplier = cls.rma_obj.create(
+            {"partner_id": supplier1.id, "type": "supplier"}
+        )
+        cls.operation_1 = cls.rma_op_obj.create(
+            {
+                "code": "TEST",
+                "name": "Refund and receive",
+                "type": "customer",
+                "receipt_policy": "ordered",
+                "refund_policy": "ordered",
+                "in_route_id": cls.rma_route_cust.id,
+                "out_route_id": cls.rma_route_cust.id,
+            }
+        )
+
+        # Create products
+        cls.product_1 = cls.product_obj.create(
+            {
+                "name": "Test Product 1",
+                "type": "product",
+                "list_price": 100.0,
+                "rma_customer_operation_id": cls.cust_refund_op.id,
+                "rma_supplier_operation_id": cls.sup_refund_op.id,
+            }
+        )
+        cls.product_2 = cls.product_obj.create(
+            {
+                "name": "Test Product 2",
+                "type": "product",
+                "list_price": 150.0,
+                "rma_customer_operation_id": cls.operation_1.id,
+                "rma_supplier_operation_id": cls.sup_refund_op.id,
+            }
+        )
+        cls.currency_id = cls.company_id.currency_id
+
+        # Create Invoices:
+        cls.customer_account = cls.acc_obj.search(
+            [("user_type_id", "=", receivable_type.id)], limit=1
+        ).id
+
+        cls.invoices = cls.env["account.move"].create(
+            [
+                {
+                    "type": "out_invoice",
+                    "partner_id": customer1.id,
+                    "invoice_date": fields.Date.from_string("2016-01-01"),
+                    "currency_id": cls.currency_id.id,
+                    "invoice_line_ids": [
+                        (
+                            0,
+                            None,
+                            {
+                                "product_id": cls.product_1.id,
+                                "product_uom_id": cls.product_1.uom_id.id,
+                                "quantity": 3,
+                                "price_unit": 1000,
+                            },
+                        ),
+                        (
+                            0,
+                            None,
+                            {
+                                "product_id": cls.product_2.id,
+                                "product_uom_id": cls.product_2.uom_id.id,
+                                "quantity": 2,
+                                "price_unit": 3000,
+                            },
+                        ),
+                    ],
+                },
+                {
+                    "type": "in_invoice",
+                    "partner_id": supplier1.id,
+                    "invoice_date": fields.Date.from_string("2016-01-01"),
+                    "currency_id": cls.currency_id.id,
+                    "invoice_line_ids": [
+                        (
+                            0,
+                            None,
+                            {
+                                "product_id": cls.product_1.id,
+                                "product_uom_id": cls.product_1.uom_id.id,
+                                "quantity": 3,
+                                "price_unit": 1000,
+                            },
+                        ),
+                        (
+                            0,
+                            None,
+                            {
+                                "product_id": cls.product_2.id,
+                                "product_uom_id": cls.product_2.uom_id.id,
+                                "quantity": 2,
+                                "price_unit": 3000,
+                            },
+                        ),
+                    ],
+                },
+            ]
+        )
+        cls.inv_customer = cls.invoices[0]
+        cls.inv_supplier = cls.invoices[1]
+
+    def test_01_add_from_invoice_customer(self):
+        """Test wizard to create RMA from a customer invoice."""
+        add_inv = self.rma_add_invoice_wiz.with_context(
+            {
+                "customer": True,
+                "active_ids": self.rma_group_customer.id,
+                "active_model": "rma.order",
+            }
+        ).create({"line_ids": [(6, 0, self.inv_customer.invoice_line_ids.ids)]})
+        add_inv.add_lines()
+
+        self.assertEqual(len(self.rma_group_customer.rma_line_ids), 2)
+        for t in self.rma_group_supplier.rma_line_ids.mapped("type"):
+            self.assertEqual(t, "customer")
+        rma_1 = self.rma_group_customer.rma_line_ids.filtered(
+            lambda r: r.product_id == self.product_1
+        )
+        self.assertEqual(rma_1.operation_id, self.cust_refund_op)
+        rma_2 = self.rma_group_customer.rma_line_ids.filtered(
+            lambda r: r.product_id == self.product_2
+        )
+        self.assertEqual(rma_2.operation_id, self.operation_1)
+
+    def test_02_add_from_invoice_supplier(self):
+        """Test wizard to create RMA from a vendor bill."""
+        add_inv = self.rma_add_invoice_wiz.with_context(
+            {
+                "supplier": True,
+                "active_ids": self.rma_group_supplier.id,
+                "active_model": "rma.order",
+            }
+        ).create({"line_ids": [(6, 0, self.inv_supplier.line_ids.ids)]})
+        add_inv.add_lines()
+        self.assertEqual(len(self.rma_group_supplier.rma_line_ids), 2)
+        for t in self.rma_group_supplier.rma_line_ids.mapped("type"):
+            self.assertEqual(t, "supplier")
+
+    def test_03_rma_refund_operation(self):
+        """Test RMA quantities using refund operations."""
+        # Received refund_policy:
+        rma_1 = self.rma_group_customer.rma_line_ids.filtered(
+            lambda r: r.product_id == self.product_1
+        )
+        self.assertEqual(rma_1.refund_policy, "received")
+        self.assertEqual(rma_1.qty_to_refund, 0.0)
+        # TODO: receive and check qty_to_refund is 12.0
+        # Ordered refund_policy:
+        rma_2 = self.rma_group_customer.rma_line_ids.filtered(
+            lambda r: r.product_id == self.product_2
+        )
+        rma_2._onchange_operation_id()
+        self.assertEqual(rma_2.refund_policy, "ordered")
+        self.assertEqual(rma_2.qty_to_refund, 2.0)
+
+    def test_04_rma_create_refund(self):
+        """Generate a Refund from a customer RMA."""
+        rma = self.rma_group_customer.rma_line_ids.filtered(
+            lambda r: r.product_id == self.product_2
+        )
+        rma.action_rma_to_approve()
+        rma.action_rma_approve()
+        self.assertEqual(rma.refund_count, 0)
+        self.assertEqual(rma.qty_to_refund, 2.0)
+        self.assertEqual(rma.qty_refunded, 0.0)
+        make_refund = self.rma_refund_wiz.with_context(
+            {"customer": True, "active_ids": rma.ids, "active_model": "rma.order.line"}
+        ).create({"description": "Test refund"})
+        make_refund.invoice_refund()
+        rma.refund_line_ids.move_id.post()
+        rma._compute_refund_count()
+        self.assertEqual(rma.refund_count, 1)
+        self.assertEqual(rma.qty_to_refund, 0.0)
+        self.assertEqual(rma.qty_refunded, 2.0)
+
+    def test_05_fill_rma_from_inv_line(self):
+        """Test filling a RMA (line) from a invoice line."""
+        rma = self.rma_line_obj.new(
+            {
+                "partner_id": self.inv_customer.partner_id.id,
+                "account_move_line_id": self.inv_supplier.line_ids.ids[0],
+            }
+        )
+        self.assertFalse(rma.product_id)
+        rma._onchange_account_move_line_id()
+        self.assertEqual(rma.product_id, self.product_1)
+        self.assertEqual(rma.product_qty, 3.0)

rma_account/views/account_move_view.xml

@@ -0,0 +1,69 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="invoice_form" model="ir.ui.view">
+        <field name="name">account.move.form</field>
+        <field name="model">account.move</field>
+        <field name="inherit_id" ref="account.view_move_form"/>
+        <field name="arch" type="xml">
+            <field name="name" position="before">
+                <div class="oe_button_box" attrs="{'invisible': [('rma_count', '=', 0)]}">
+                    <button type="object"  name="action_view_rma_customer"
+                            class="oe_stat_button"
+                            icon="fa-eject"
+                            groups="rma.group_rma_customer_user,rma.group_rma_supplier_user">
+                        <field name="rma_count" widget="statinfo"
+                               string="RMA"/>
+                    </button>
+                </div>
+            </field>
+        </field>
+    </record>
+
+
+    <record id="view_invoice_line_form" model="ir.ui.view">
+        <field name="name">rma.invoice.line.form</field>
+        <field name="model">account.move.line</field>
+        <field name="inherit_id" ref="account.view_move_line_form"/>
+        <field eval="16" name="priority"/>
+        <field name="arch" type="xml">
+            <data>
+                <field name="name" position="after">
+                    <field name="rma_line_count" invisible="1"/>
+                    <field name="rma_line_id"
+                           string="RMA line originated"/>
+                    <notebook attrs="{'invisible': [('rma_line_count', '=', 0)]}">
+                        <page string="RMA Lines">
+                            <field name="rma_line_ids"/>
+                        </page>
+                    </notebook>
+                </field>
+            </data>
+        </field>
+    </record>
+
+
+    <record id="view_invoice_customer_rma_form" model="ir.ui.view">
+        <field name="name">account.move.customer.rma</field>
+        <field name="model">account.move</field>
+        <field name="inherit_id" ref="account.view_move_form"/>
+        <field name="arch" type="xml">
+            <field name="invoice_vendor_bill_id" position="after">
+                 <field name="add_rma_line_id"
+                        context="{'rma': True}"
+                        domain="[('type', '=', 'customer'),
+                                 ('partner_id', '=', partner_id)]"
+                        attrs="{'readonly': [('state','not in',['draft'])],
+                         'invisible': ['|', ('state', '=', 'paid'),
+                          ('type', '=', 'in_invoice')]}" class="oe_edit_only"
+                        options="{'no_create': True}"/>
+            </field>
+        </field>
+    </record>
+
+    <record id="action_invoice_line" model="ir.actions.act_window">
+        <field name="name">Invoice Line</field>
+        <field name="res_model">account.move.line</field>
+        <field name="view_mode">form</field>
+        <field name="view_id" ref="account.view_move_line_form"/>
+    </record>
+</odoo>

rma_account/views/rma_account_menu.xml

@@ -0,0 +1,59 @@
+<?xml version="1.0" encoding="utf-8"?>
+<!-- Copyright 2018 ForgeFlow S.L.
+     License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl-3.0) -->
+<odoo>
+
+    <record id="action_rma_account_customer_lines" model="ir.actions.act_window">
+        <field name="name">Customer RMA</field>
+        <field name="res_model">rma.order.line</field>
+        <field name="domain">[('type','=', 'customer')]</field>
+        <field name="context">{"search_default_to_refund":1}</field>
+        <field name="view_mode">tree,form</field>
+    </record>
+
+    <record id="action_rma_supplier_lines" model="ir.actions.act_window">
+        <field name="name">Supplier RMA</field>
+        <field name="res_model">rma.order.line</field>
+        <field name="domain">[('type','=', 'supplier')]</field>
+        <field name="context">{"search_default_to_refund":1, "supplier":1}</field>
+        <field name="view_mode">tree,form</field>
+        <field name="view_id" ref="rma.view_rma_line_supplier_tree"/>
+    </record>
+
+    <!-- Accountants menu RMA -->
+    <menuitem id="menu_rma_account"
+        name="Accounting"
+        groups="account.group_account_user"
+        sequence="32"
+        parent="rma.menu_rma_root"/>
+
+    <menuitem id="menu_rma_customer_refunds"
+        name="Customer RMA to Refund"
+        sequence="20"
+        parent="rma_account.menu_rma_account"
+        groups="rma.group_rma_customer_user"
+        action="action_rma_account_customer_lines"/>
+
+    <menuitem id="menu_rma_line_supplier_refunds"
+        name="Supplier RMA to Refund"
+        sequence="20"
+        parent="rma_account.menu_rma_account"
+        groups="rma.group_rma_customer_user"
+        action="action_rma_supplier_lines"/>
+
+
+    <menuitem id="menu_rma_line_account_customer"
+        name="Customer RMA to Refund"
+        sequence="20"
+        parent="account.menu_finance_receivables"
+        groups="rma.group_rma_customer_user"
+        action="action_rma_account_customer_lines"/>
+
+    <menuitem id="menu_rma_line_account_supplier"
+        name="Supplier RMA to Refund"
+        sequence="20"
+        parent="account.menu_finance_payables"
+        groups="rma.group_rma_supplier_user"
+        action="action_rma_supplier_lines"/>
+
+</odoo>

rma_account/views/rma_operation_view.xml

@@ -0,0 +1,24 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="rma_operation_tree" model="ir.ui.view">
+        <field name="name">rma.operation.tree</field>
+        <field name="model">rma.operation</field>
+        <field name="inherit_id" ref="rma.rma_operation_tree"/>
+        <field name="arch" type="xml">
+            <field name="name" position="after">
+                <field name="refund_policy"/>
+            </field>
+        </field>
+    </record>
+
+    <record id="rma_operation_form" model="ir.ui.view">
+        <field name="name">rma.operation.form</field>
+        <field name="model">rma.operation</field>
+        <field name="inherit_id" ref="rma.rma_operation_form"/>
+        <field name="arch" type="xml">
+            <field name="receipt_policy" position="before">
+                <field name="refund_policy"/>
+            </field>
+        </field>
+    </record>
+</odoo>

rma_account/views/rma_order_line_view.xml

@@ -0,0 +1,110 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="view_rma_line_supplier_form" model="ir.ui.view">
+        <field name="name">rma.order.line.supplier.form</field>
+        <field name="model">rma.order.line</field>
+        <field name="inherit_id" ref="rma.view_rma_line_supplier_form"/>
+        <field name="arch" type="xml">
+            <button name="action_view_out_shipments" position="after">
+                <button type="object"  name="action_view_invoice"
+                                    class="oe_stat_button"
+                                    icon="fa-pencil-square-o"
+                                    string="Origin Inv">
+                </button>
+                <button type="object"  name="action_view_refunds"
+                            class="oe_stat_button"
+                            icon="fa-pencil-square-o"
+                            groups="account.group_account_user">
+                        <field name="refund_count" widget="statinfo"
+                               string="Refunds"/>
+                    </button>
+            </button>
+            <group name="main_info" position="inside">
+                <field name="account_move_line_id"
+                       options="{'no_create': True}"
+                       context="{'rma': True}"
+                       domain="['|',
+                                ('move_id.partner_id', '=', partner_id),
+                                ('move_id.partner_id', 'child_of', partner_id)]"/>
+            </group>
+            <field name="operation_id" position="after">
+                <field name="refund_policy"/>
+            </field>
+            <page name="stock" position="after">
+                <page name="refunds" string="Refunds">
+                    <field name="refund_line_ids" nolabel="1"/>
+                </page>
+            </page>
+            <field name="delivery_address_id" position="after">
+                <field name="invoice_address_id"
+                       groups='rma.group_rma_delivery_invoice_address'/>
+            </field>
+            <group name="deliver" position="after">
+                <group name="refund">
+                    <field name="qty_to_refund"/>
+                    <field name="qty_refunded"/>
+                </group>
+            </group>
+        </field>
+    </record>
+
+    <record id="view_rma_line_form" model="ir.ui.view">
+        <field name="name">rma.order.line.form</field>
+        <field name="model">rma.order.line</field>
+        <field name="inherit_id" ref="rma.view_rma_line_form"/>
+        <field name="arch" type="xml">
+            <button name="action_view_out_shipments" position="after">
+                <button type="object"  name="action_view_invoice"
+                                    class="oe_stat_button"
+                                    icon="fa-pencil-square-o"
+                                    string="Origin Inv">
+                </button>
+                 <button type="object"  name="action_view_refunds"
+                            class="oe_stat_button"
+                            icon="fa-pencil-square-o"
+                            groups="account.group_account_user">
+                        <field name="refund_count" widget="statinfo"
+                               string="Refunds"/>
+                    </button>
+            </button>
+            <group name="main_info" position="inside">
+                <field name="account_move_line_id"
+                       options="{'no_create': True}"
+                       context="{'rma': True}"
+                       domain="['|',
+                                ('move_id.partner_id', '=', partner_id),
+                                ('move_id.partner_id', 'child_of', partner_id)]"/>
+            </group>
+            <field name="operation_id" position="after">
+                <field name="refund_policy"/>
+            </field>
+            <page name="stock" position="after">
+                <page name="refunds" string="Refunds">
+                    <field name="refund_line_ids" nolabel="1"/>
+                </page>
+            </page>
+            <field name="delivery_address_id" position="after">
+                <field name="invoice_address_id"
+                       groups='rma.group_rma_delivery_invoice_address'/>
+            </field>
+            <group name="supplier_rma" position="after">
+                <group name="refund">
+                    <field name="qty_to_refund"/>
+                    <field name="qty_refunded"/>
+                </group>
+            </group>
+        </field>
+    </record>
+
+    <record id="view_rma_rma_line_filter" model="ir.ui.view">
+        <field name="name">rma.order.line.select</field>
+        <field name="model">rma.order.line</field>
+        <field name="inherit_id" ref="rma.view_rma_rma_line_filter"/>
+        <field name="arch" type="xml">
+            <field name="product_id" position="after">
+                <filter name="to_refund" string="To Refund" domain="[('qty_to_refund', '>', 0)]"
+context="{'group_by':'partner_id'}"/>
+            </field>
+        </field>
+    </record>
+</odoo>

rma_account/views/rma_order_view.xml

@@ -0,0 +1,64 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="view_rma_form" model="ir.ui.view">
+        <field name="name">rma.order.form - rma_account</field>
+        <field name="model">rma.order</field>
+        <field name="inherit_id" ref="rma.view_rma_form"/>
+        <field name="arch" type="xml">
+            <button name="action_view_out_shipments"
+                    position="after">
+                <button type="object"  name="action_view_invoice_refund"
+                        class="oe_stat_button"
+                        icon="fa-pencil-square-o"
+                        groups="account.group_account_user">
+                    <field name="invoice_refund_count" widget="statinfo"
+                           string="Refunds"/>
+                </button>
+                <button type="object"  name="action_view_invoice"
+                        class="oe_stat_button"
+                        icon="fa-pencil-square-o"
+                        groups="account.group_account_user">
+                    <field name="invoice_count" widget="statinfo"
+                           string="Origin Inv"/>
+                </button>
+            </button>
+            <!--
+            <xpath expr="//field[@name='rma_line_ids']/tree/field[@name='price_unit']"
+                   position="after">
+                <field name="account_move_line_id" invisible="True"/>
+            </xpath>
+            -->
+        </field>
+    </record>
+
+    <record id="view_rma_supplier_form" model="ir.ui.view">
+        <field name="name">rma.order.supplier.form</field>
+        <field name="model">rma.order</field>
+        <field name="inherit_id" ref="rma.view_rma_supplier_form"/>
+        <field name="arch" type="xml">
+             <button name="action_view_out_shipments"
+                    position="after">
+                 <button type="object"  name="action_view_invoice_refund"
+                         class="oe_stat_button"
+                         icon="fa-pencil-square-o"
+                         groups="account.group_account_user">
+                     <field name="invoice_refund_count" widget="statinfo"
+                            string="Refunds"/>
+                 </button>
+                 <button type="object"  name="action_view_invoice"
+                         class="oe_stat_button"
+                         icon="fa-pencil-square-o"
+                         groups="account.group_account_user">
+                     <field name="invoice_count" widget="statinfo"
+                            string="Origin Inv"/>
+                 </button>
+            </button>
+            <!--
+            <xpath expr="//field[@name='rma_line_ids']/tree"
+                   position="inside">
+                <field name="account_move_line_id" invisible="True"/>
+            </xpath>
+            -->
+        </field>
+    </record>
+</odoo>

rma_account/wizards/__init__.py

@@ -0,0 +1,6 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from . import rma_refund
+from . import rma_add_account_move
+from . import rma_order_line_make_supplier_rma

rma_account/wizards/rma_add_account_move.py

@@ -0,0 +1,131 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import _, api, fields, models
+from odoo.exceptions import ValidationError
+
+
+class RmaAddAccountMove(models.TransientModel):
+    _name = "rma_add_account_move"
+    _description = "Wizard to add rma lines"
+
+    @api.model
+    def default_get(self, fields_list):
+        res = super(RmaAddAccountMove, self).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"]
+        if not rma_id:
+            return res
+        assert active_model == "rma.order", "Bad context propagation"
+        rma = rma_obj.browse(rma_id)
+        res["rma_id"] = rma.id
+        res["partner_id"] = rma.partner_id.id
+        res["line_ids"] = False
+        return res
+
+    rma_id = fields.Many2one(
+        "rma.order", string="RMA Order", readonly=True, ondelete="cascade"
+    )
+    partner_id = fields.Many2one(
+        comodel_name="res.partner", string="Partner", readonly=True
+    )
+    line_ids = fields.Many2many(
+        "account.move.line",
+        "rma_add_account_move_add_line_rel",
+        "account_move_line_id",
+        "rma_add_move_id",
+        string="Invoice Lines",
+    )
+
+    def _prepare_rma_line_from_inv_line(self, line):
+        if self.env.context.get("customer"):
+            operation = (
+                line.product_id.rma_customer_operation_id
+                or line.product_id.categ_id.rma_customer_operation_id
+            )
+        else:
+            operation = (
+                line.product_id.rma_supplier_operation_id
+                or line.product_id.categ_id.rma_supplier_operation_id
+            )
+        if not operation:
+            operation = self.env["rma.operation"].search(
+                [("type", "=", self.rma_id.type)], limit=1
+            )
+            if not operation:
+                raise ValidationError(_("Please define an operation first"))
+        if not operation.in_route_id or not operation.out_route_id:
+            route = self.env["stock.location.route"].search(
+                [("rma_selectable", "=", True)], limit=1
+            )
+            if not route:
+                raise ValidationError(_("Please define an rma route"))
+
+        if not operation.in_warehouse_id or not operation.out_warehouse_id:
+            warehouse = self.env["stock.warehouse"].search(
+                [
+                    ("company_id", "=", self.rma_id.company_id.id),
+                    ("lot_rma_id", "!=", False),
+                ],
+                limit=1,
+            )
+            if not warehouse:
+                raise ValidationError(
+                    _("Please define a warehouse with a" " default rma location")
+                )
+        data = {
+            "partner_id": self.partner_id.id,
+            "account_move_line_id": line.id,
+            "product_id": line.product_id.id,
+            "origin": line.move_id.name,
+            "uom_id": line.product_uom_id.id,
+            "operation_id": operation.id,
+            "product_qty": line.quantity,
+            "price_unit": line.move_id.currency_id.compute(
+                line.price_unit, line.currency_id, round=False
+            ),
+            "delivery_address_id": line.move_id.partner_id.id,
+            "invoice_address_id": line.move_id.partner_id.id,
+            "rma_id": self.rma_id.id,
+            "receipt_policy": operation.receipt_policy,
+            "refund_policy": operation.refund_policy,
+            "delivery_policy": operation.delivery_policy,
+            "in_warehouse_id": operation.in_warehouse_id.id or warehouse.id,
+            "out_warehouse_id": operation.out_warehouse_id.id or warehouse.id,
+            "in_route_id": operation.in_route_id.id or route.id,
+            "out_route_id": operation.out_route_id.id or route.id,
+            "location_id": (
+                operation.location_id.id
+                or operation.in_warehouse_id.lot_rma_id.id
+                or warehouse.lot_rma_id.id
+            ),
+        }
+        return data
+
+    @api.model
+    def _get_rma_data(self):
+        data = {"date_rma": fields.Datetime.now()}
+        return data
+
+    @api.model
+    def _get_existing_invoice_lines(self):
+        existing_invoice_lines = []
+        for rma_line in self.rma_id.rma_line_ids:
+            existing_invoice_lines.append(rma_line.account_move_line_id)
+        return existing_invoice_lines
+
+    def add_lines(self):
+        rma_line_obj = self.env["rma.order.line"]
+        existing_invoice_lines = self._get_existing_invoice_lines()
+        for line in self.line_ids.filtered(
+            lambda aml: aml.exclude_from_invoice_tab is False
+        ):
+            # Load a PO line only once
+            if line not in existing_invoice_lines:
+                data = self._prepare_rma_line_from_inv_line(line)
+                rma_line_obj.create(data)
+        rma = self.rma_id
+        data_rma = self._get_rma_data()
+        rma.write(data_rma)
+        return {"type": "ir.actions.act_window_close"}

rma_account/wizards/rma_add_account_move.xml

@@ -0,0 +1,132 @@
+<?xml version="1.0"?>
+
+<odoo>
+
+    <record id="view_rma_add_account_move" model="ir.ui.view">
+        <field name="name">rma.add.invoice</field>
+        <field name="model">rma_add_account_move</field>
+        <field name="arch" type="xml">
+            <form string="Select Invoice from customer">
+                <group>
+                    <field name="partner_id"
+                           domain="[('customer','=',True)]"
+                           string="Customer"/>
+                </group>
+                <separator string="Select Invoices lines to add"/>
+                <field name="line_ids"
+                       domain="[('move_id.partner_id', '=', partner_id),
+                       ('move_id.type', '=', 'out_invoice')]">
+                    <tree string="Invoice Lines">
+                        <field name="move_id"/>
+                        <field name="name"/>
+                        <field name="account_id" groups="account.group_account_user"/>
+                        <field name="quantity"/>
+                        <field name="product_id"/>
+                        <field name="product_uom_id"/>
+                        <field name="price_unit"/>
+                        <field name="discount" groups="base.group_no_one"/>
+                        <field name="price_subtotal"/>
+                        <field name="currency_id" invisible="1"/>
+                    </tree>
+                </field>
+                <footer>
+                    <button
+                            string="Confirm"
+                            name="add_lines" type="object"
+                            class="oe_highlight"/>
+                    or
+                    <button name="action_cancel"
+                            string="Cancel" class="oe_link" special="cancel"/>
+                </footer>
+            </form>
+        </field>
+    </record>
+
+    <record id="view_rma_add_account_move_supplier" model="ir.ui.view">
+        <field name="name">rma.add.invoice.supplier</field>
+        <field name="model">rma_add_account_move</field>
+        <field name="arch" type="xml">
+            <form string="Select Invoice from supplier">
+                <group>
+                    <field name="partner_id"
+                           domain="[('supplier','=',True)]"
+                           string="Supplier"/>
+                </group>
+                <separator string="Select Invoices lines to add"/>
+                <field name="line_ids"
+                       domain="[('move_id.partner_id', '=', partner_id),
+                       ('move_id.type', '=', 'in_invoice')]">
+                    <tree string="Invoice Lines">
+                        <field name="move_id"/>
+                        <field name="name"/>
+                        <field name="account_id" groups="account.group_account_user"/>
+                        <field name="quantity"/>
+                        <field name="product_id"/>
+                        <field name="product_uom_id"/>
+                        <field name="price_unit"/>
+                        <field name="discount" groups="base.group_no_one"/>
+                        <field name="price_subtotal"/>
+                        <field name="currency_id" invisible="1"/>
+                    </tree>
+                </field>
+                <footer>
+                    <button
+                            string="Confirm"
+                            name="add_lines" type="object"
+                            class="oe_highlight"/>
+                    or
+                    <button name="action_cancel"
+                            string="Cancel" class="oe_link" special="cancel"/>
+                </footer>
+            </form>
+        </field>
+    </record>
+
+    <record id="action_rma_add_account_move" model="ir.actions.act_window">
+        <field name="name">Add Invoice</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">rma_add_account_move</field>
+        <field name="view_mode">form</field>
+        <field name="target">new</field>
+        <field name="view_id" ref="view_rma_add_account_move"/>
+        <field name="groups_id" eval="[(4, ref('rma.group_rma_customer_user'))]"/>
+    </record>
+
+    <record id="action_rma_add_account_move_supplier" model="ir.actions.act_window">
+        <field name="name">Add Invoice</field>
+        <field name="type">ir.actions.act_window</field>
+        <field name="res_model">rma_add_account_move</field>
+        <field name="binding_model_id" ref="rma.model_rma_order_line"/>
+        <field name="view_mode">form</field>
+        <field name="target">new</field>
+        <field name="view_id" ref="view_rma_add_account_move_supplier"/>
+        <field name="groups_id" eval="[(4, ref('rma.group_rma_supplier_user'))]"/>
+    </record>
+
+    <record id="view_rma_add_account_move_form" model="ir.ui.view">
+        <field name="name">rma.order.form - invoice wizard</field>
+        <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_add_account_move)d"
+                        string="Add From Invoice"
+                        type="action"/>
+            </xpath>
+        </field>
+    </record>
+
+    <record id="view_rma_add_account_move_button_form" model="ir.ui.view">
+        <field name="name">rma.order.supplier.form - invoice wizard</field>
+        <field name="model">rma.order</field>
+        <field name="inherit_id" ref="rma.view_rma_supplier_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//header" position="inside">
+                <button name="%(action_rma_add_account_move_supplier)d"
+                        string="Add From Invoice"
+                        type="action"/>
+            </xpath>
+        </field>
+    </record>
+
+</odoo>

rma_account/wizards/rma_order_line_make_supplier_rma.py

@@ -0,0 +1,16 @@
+# Copyright 2016 ForgeFlow S.L.
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl-3.0).
+
+from odoo import api, models
+
+
+class RmaLineMakeSupplierRma(models.TransientModel):
+    _inherit = "rma.order.line.make.supplier.rma"
+
+    @api.model
+    def _prepare_supplier_rma_line(self, rma, item):
+        res = super(RmaLineMakeSupplierRma, self)._prepare_supplier_rma_line(rma, item)
+        if res["operation_id"]:
+            operation = self.env["rma.operation"].browse(res["operation_id"])
+            res["refund_policy"] = operation.refund_policy
+        return res

rma_account/wizards/rma_refund.py

@@ -0,0 +1,219 @@
+# Copyright 2017 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import _, api, fields, models
+from odoo.exceptions import ValidationError
+
+
+class RmaRefund(models.TransientModel):
+    _name = "rma.refund"
+    _description = "Wizard for RMA Refund"
+
+    @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 ""
+
+    @api.returns("rma.order.line")
+    def _prepare_item(self, line):
+        values = {
+            "product_id": line.product_id.id,
+            "product": line.product_id.id,
+            "name": line.name,
+            "product_qty": line.product_qty,
+            "uom_id": line.uom_id.id,
+            "qty_to_refund": line.qty_to_refund,
+            "refund_policy": line.refund_policy,
+            "invoice_address_id": line.invoice_address_id.id,
+            "line_id": line.id,
+            "rma_id": line.rma_id.id,
+        }
+        return values
+
+    @api.model
+    def default_get(self, fields_list):
+        """Default values for wizard, if there is more than one supplier on
+        lines the supplier field is empty otherwise is the unique line
+        supplier.
+        """
+        context = self._context.copy()
+        res = super(RmaRefund, self).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:
+            return res
+        assert active_model == "rma.order.line", "Bad context propagation"
+
+        items = []
+        lines = rma_line_obj.browse(rma_line_ids)
+        if len(lines.mapped("partner_id")) > 1:
+            raise ValidationError(
+                _(
+                    "Only RMAs from the same partner can be processed at "
+                    "the same time."
+                )
+            )
+        for line in lines:
+            items.append([0, 0, self._prepare_item(line)])
+        res["item_ids"] = items
+        context.update({"items_ids": items})
+        return res
+
+    date_invoice = fields.Date(
+        string="Refund Date", default=fields.Date.context_today, required=True
+    )
+    date = fields.Date(
+        string="Accounting Date", default=fields.Date.context_today, required=True
+    )
+    description = fields.Char(
+        string="Reason", required=True, default=lambda self: self._get_reason()
+    )
+    item_ids = fields.One2many(
+        comodel_name="rma.refund.item", inverse_name="wiz_id", string="Items"
+    )
+
+    def compute_refund(self):
+        for wizard in self:
+            first = self.item_ids[0]
+            values = self._prepare_refund(wizard, first.line_id)
+            new_refund = self.env["account.move"].create(values)
+            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:
+            if line.refund_policy == "no":
+                raise ValidationError(
+                    _("The operation is not refund for at least one line")
+                )
+            if line.state != "approved":
+                raise ValidationError(_("RMA %s is not approved") % line.name)
+        new_invoice = self.compute_refund()
+        action = (
+            "action_move_out_refund_type"
+            if (new_invoice.type in ["out_refund", "out_invoice"])
+            else "action_move_in_refund_type"
+        )
+        result = self.env.ref("account.%s" % action).read()[0]
+        form_view = self.env.ref("account.move_supplier_form", False)
+        result["views"] = [(form_view and form_view.id or False, "form")]
+        result["res_id"] = new_invoice.id
+        return result
+
+    @api.model
+    def prepare_refund_line(self, item):
+        accounts = item.product_id.product_tmpl_id._get_product_accounts()
+        if item.line_id.type == "customer":
+            account = accounts["stock_output"]
+        else:
+            account = accounts["stock_input"]
+        if not account:
+            raise ValidationError(_("Accounts are not configured for this product."))
+
+        values = {
+            "name": item.line_id.name or item.rma_id.name,
+            "account_id": account.id,
+            "price_unit": item.line_id.price_unit,
+            "product_uom_id": item.line_id.uom_id.id,
+            "product_id": item.product_id.id,
+            "rma_line_id": item.line_id.id,
+            "quantity": item.qty_to_refund,
+        }
+        return values
+
+    @api.model
+    def _prepare_refund(self, wizard, rma_line):
+        # origin_invoices = self._get_invoice(rma_line)
+        if rma_line.type == "customer":
+            journal = self.env["account.journal"].search(
+                [("type", "=", "sale")], limit=1
+            )
+        else:
+            journal = self.env["account.journal"].search(
+                [("type", "=", "purchase")], limit=1
+            )
+        values = {
+            "name": rma_line.rma_id.name or rma_line.name,
+            "invoice_payment_ref": rma_line.rma_id.name or rma_line.name,
+            "invoice_origin": rma_line.rma_id.name or rma_line.name,
+            "ref": False,
+            "type": "in_refund" if rma_line.type == "supplier" else "out_refund",
+            "journal_id": journal.id,
+            "fiscal_position_id": rma_line.partner_id.property_account_position_id.id,
+            "state": "draft",
+            "currency_id": rma_line.currency_id.id,
+            "date": wizard.date,
+            "invoice_date": wizard.date_invoice,
+            "partner_id": rma_line.invoice_address_id.id or rma_line.partner_id.id,
+            "invoice_line_ids": [
+                (0, None, self.prepare_refund_line(item)) for item in self.item_ids
+            ],
+        }
+        if self.env.registry.models.get("crm.team", False):
+            team_ids = self.env["crm.team"].search(
+                [
+                    "|",
+                    ("user_id", "=", self.env.uid),
+                    ("member_ids", "=", self.env.uid),
+                ],
+                limit=1,
+            )
+            team_id = team_ids[0] if team_ids else False
+            if team_id:
+                values["team_id"] = team_id.id
+        if rma_line.type == "customer":
+            values["type"] = "out_refund"
+        else:
+            values["type"] = "in_refund"
+        return values
+
+    @api.constrains("item_ids")
+    def check_unique_invoice_address_id(self):
+        addresses = self.item_ids.mapped("invoice_address_id")
+        if len(addresses) > 1:
+            raise ValidationError(
+                _("The invoice address must be the same for all the lines.")
+            )
+        return True
+
+
+class RmaRefundItem(models.TransientModel):
+    _name = "rma.refund.item"
+    _description = "RMA Lines to refund"
+
+    wiz_id = fields.Many2one(comodel_name="rma.refund", string="Wizard", required=True)
+    line_id = fields.Many2one(
+        "rma.order.line",
+        string="RMA order Line",
+        required=True,
+        readonly=True,
+        ondelete="cascade",
+    )
+    rma_id = fields.Many2one(
+        "rma.order", related="line_id.rma_id", string="RMA", readonly=True
+    )
+    product_id = fields.Many2one("product.product", string="Product (Technical)")
+    product = fields.Many2one("product.product", string="Product", readonly=True)
+    name = fields.Char(string="Description", required=True)
+    product_qty = fields.Float(
+        string="Quantity Ordered",
+        copy=False,
+        digits="Product Unit of Measure",
+        readonly=True,
+    )
+    invoice_address_id = fields.Many2one(
+        comodel_name="res.partner", string="Invoice Address"
+    )
+    qty_to_refund = fields.Float(
+        string="Quantity To Refund", digits="Product Unit of Measure"
+    )
+    uom_id = fields.Many2one("uom.uom", string="Unit of Measure", readonly=True)
+    refund_policy = fields.Selection(
+        selection=[
+            ("no", "Not required"),
+            ("ordered", "Based on Ordered Quantities"),
+            ("received", "Based on Received Quantities"),
+        ],
+        string="Refund Policy",
+    )

rma_account/wizards/rma_refund.xml

@@ -0,0 +1,72 @@
+<?xml version="1.0"?>
+<odoo>
+    <record id="view_rma_refund" model="ir.ui.view">
+        <field name="name">rma.refund.form</field>
+        <field name="model">rma.refund</field>
+        <field name="arch" type="xml">
+            <form string="Credit Note">
+                <group>
+                     <group>
+                         <field name="description"/>
+                     </group>
+                     <group>
+                         <field name="date_invoice"/>
+                         <field name="date"/>
+                     </group>
+                </group>
+                <field name="item_ids">
+                    <tree string="RMA Lines" editable="bottom">
+                        <field name="rma_id" invisible="1"/>
+                        <field name="product_id" invisible="1"/>
+                        <field name="product"/>
+                        <field name="name"/>
+                        <field name="product_qty"/>
+                        <field name="uom_id" groups="uom.group_uom"/>
+                        <field name="qty_to_refund" readonly="0"/>
+                    </tree>
+                </field>
+                <footer>
+                    <button string='Create Refund' name="invoice_refund" type="object" class="btn-primary"/>
+                    <button string="Cancel" class="btn-default" special="cancel"/>
+                </footer>
+           </form>
+        </field>
+    </record>
+
+    <record id="action_rma_refund" model="ir.actions.act_window">
+        <field name="name">Create Refund</field>
+        <field name="res_model">rma.refund</field>
+        <field name="view_mode">tree,form</field>
+        <field name="view_id" ref="view_rma_refund"/>
+        <field name="binding_model_id" ref="rma.model_rma_order_line"/>
+        <field name="groups_id"
+               eval="[(4, ref('account.group_account_invoice'))]"/>
+        <field name="target">new</field>
+    </record>
+
+    <record id="view_rma_line_button_refund_form" model="ir.ui.view">
+        <field name="name">rma.order.line.form</field>
+        <field name="model">rma.order.line</field>
+        <field name="inherit_id" ref="rma.view_rma_line_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//header" position="inside">
+                <button name="%(action_rma_refund)d" states="approved"
+                        string="Create Refund" class="oe_highlight"
+                        type="action"/>
+            </xpath>
+        </field>
+    </record>
+
+    <record id="view_rma_line_supplier_button_refund_form" model="ir.ui.view">
+        <field name="name">rma.order.line.supplier.form</field>
+        <field name="model">rma.order.line</field>
+        <field name="inherit_id" ref="rma.view_rma_line_supplier_button_form"/>
+        <field name="arch" type="xml">
+            <xpath expr="//header" position="inside">
+                <button name="%(action_rma_refund)d" states="approved"
+                        string="Create Refund" class="oe_highlight"
+                        type="action"/>
+            </xpath>
+        </field>
+    </record>
+</odoo>