[17.0][MIG] rma_purchase

rma_purchase/README.rst

@@ -0,0 +1,47 @@
+.. image:: https://img.shields.io/badge/licence-LGPL--3-blue.svg
+    :alt: License LGPL-3
+
+============
+RMA Purchase
+============
+
+This modules extend the RMA functionality allowing to use Purchase Orders as
+a RMA source.
+
+Usage
+=====
+
+To add lines to a RMA from PO act as follows:
+
+#. Go to a supplier RMA.
+#. Fill the *Supplier* field.
+#. Click on *Add From Purchase Order*.
+#. Select the Purchase Order.
+#. Click on *Add an item* and select the lines you would like to add to the
+   RMA.
+#. Hit *Confirm*.
+
+Bug Tracker
+===========
+
+Bugs are tracked on `GitHub Issues
+<https://github.com/Eficent/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 <[email protected]>
+* Aaron Henriquez <[email protected]>
+* Lois Rilo <[email protected]>
+* Bhavesh Odedra <[email protected]>
+* Serpent Consulting Services Pvt. Ltd. <[email protected]>
+
+Maintainer
+----------
+
+This module is maintained by Eficent.

rma_purchase/__init__.py

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

rma_purchase/__manifest__.py

@@ -0,0 +1,22 @@
+# Copyright 2017-2022 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+{
+    "name": "RMA Purchase",
+    "version": "17.0.1.0.0",
+    "category": "RMA",
+    "summary": "RMA from PO",
+    "license": "LGPL-3",
+    "author": "ForgeFlow, Odoo Community Association (OCA)",
+    "website": "https://github.com/ForgeFlow",
+    "depends": ["rma_account", "purchase_stock"],
+    "data": [
+        "wizards/rma_order_line_make_purchase_order_view.xml",
+        "security/ir.model.access.csv",
+        "views/rma_operation_view.xml",
+        "views/rma_order_view.xml",
+        "views/rma_order_line_view.xml",
+        "wizards/rma_add_purchase.xml",
+    ],
+    "installable": True,
+    "auto_install": True,
+}

rma_purchase/models/__init__.py

@@ -0,0 +1,8 @@
+from . import rma_order
+from . import rma_order_line
+from . import purchase_order
+from . import purchase_order_line
+from . import rma_operation
+from . import procurement
+from . import account_move
+from . import account_move_line

rma_purchase/models/account_move.py

@@ -0,0 +1,213 @@
+# Copyright 2017-22 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import fields, models
+from odoo.tools.float_utils import float_compare
+
+
+class AccountMove(models.Model):
+    _inherit = "account.move"
+
+    def _prepare_invoice_line_from_rma_line(self, line):
+        data = super()._prepare_invoice_line_from_rma_line(line)
+        if line.purchase_order_line_id:
+            data["purchase_line_id"] = line.purchase_order_line_id.id
+        return data
+
+    def action_post(self):
+        res = super().action_post()
+        for move in self:
+            rma_mls = move.line_ids.filtered(lambda x: x.rma_line_id)
+            if rma_mls:
+                # Try to reconcile the interim accounts for RMA lines
+                rmas = rma_mls.mapped("rma_line_id")
+                for rma in rmas:
+                    product_accounts = (
+                        rma.product_id.product_tmpl_id._get_product_accounts()
+                    )
+                    if rma.type == "customer":
+                        product_interim_account = product_accounts["stock_output"]
+                    else:
+                        product_interim_account = product_accounts["stock_input"]
+                    if product_interim_account.reconcile:
+                        # Get the in and out moves
+                        amls = self.env["account.move.line"].search(
+                            [
+                                ("rma_line_id", "=", rma.id),
+                                ("account_id", "=", product_interim_account.id),
+                                ("parent_state", "=", "posted"),
+                                ("reconciled", "=", False),
+                            ]
+                        )
+                        amls.reconcile()
+        return res
+
+    def _stock_account_prepare_anglo_saxon_in_lines_vals(self):
+        lines_vals_list_rma = []
+        rma_refunds = self.env["account.move"]
+        price_unit_prec = self.env["decimal.precision"].precision_get("Product Price")
+        for move in self:
+            if (
+                move.move_type != "in_refund"
+                or not move.company_id.anglo_saxon_accounting
+            ):
+                continue
+            move = move.with_company(move.company_id)
+            for line in move.invoice_line_ids.filtered(lambda il: il.rma_line_id):
+                # Filter out lines being not eligible for price difference.
+                # Moreover, this function is used for standard cost method only.
+                if (
+                    line.product_id.type != "product"
+                    or line.product_id.valuation != "real_time"
+                ):
+                    continue
+
+                # Retrieve accounts needed to generate the price difference.
+                debit_expense_account = line._get_price_diff_account()
+                if not debit_expense_account:
+                    continue
+                # Retrieve stock valuation moves.
+                valuation_stock_moves = (
+                    self.env["stock.move"].search(
+                        [
+                            ("rma_line_id", "=", line.rma_line_id.id),
+                            ("state", "=", "done"),
+                            ("product_qty", "!=", 0.0),
+                        ]
+                    )
+                    if line.rma_line_id
+                    else self.env["stock.move"]
+                )
+
+                if line.product_id.cost_method != "standard" and line.rma_line_id:
+                    if move.move_type == "in_refund":
+                        valuation_stock_moves = valuation_stock_moves.filtered(
+                            lambda stock_move: stock_move._is_out()
+                        )
+                    else:
+                        valuation_stock_moves = valuation_stock_moves.filtered(
+                            lambda stock_move: stock_move._is_in()
+                        )
+
+                    if not valuation_stock_moves:
+                        continue
+
+                    (
+                        valuation_price_unit_total,
+                        valuation_total_qty,
+                    ) = valuation_stock_moves._get_valuation_price_and_qty(
+                        line, move.currency_id
+                    )
+                    valuation_price_unit = (
+                        valuation_price_unit_total / valuation_total_qty
+                    )
+                    valuation_price_unit = line.product_id.uom_id._compute_price(
+                        valuation_price_unit, line.product_uom_id
+                    )
+                else:
+                    # Valuation_price unit is always expressed in invoice currency,
+                    # so that it can always be computed with the good rate
+                    price_unit = line.product_id.uom_id._compute_price(
+                        line.product_id.standard_price, line.product_uom_id
+                    )
+                    price_unit = (
+                        -price_unit
+                        if line.move_id.move_type == "in_refund"
+                        else price_unit
+                    )
+                    valuation_date = (
+                        valuation_stock_moves
+                        and max(valuation_stock_moves.mapped("date"))
+                        or move.date
+                    )
+                    valuation_price_unit = line.company_currency_id._convert(
+                        price_unit,
+                        move.currency_id,
+                        move.company_id,
+                        valuation_date,
+                        round=False,
+                    )
+
+                price_unit = line._get_gross_unit_price()
+
+                price_unit_val_dif = abs(price_unit) - valuation_price_unit
+                relevant_qty = line.quantity
+                price_subtotal = relevant_qty * price_unit_val_dif
+                # We consider there is a price difference if the subtotal is not zero.
+                # In case a discount has been applied, we can't round the price unit
+                # anymore, and hence we can't compare them.
+                if (
+                    not move.currency_id.is_zero(price_subtotal)
+                    and float_compare(
+                        line["price_unit"],
+                        line.price_unit,
+                        precision_digits=price_unit_prec,
+                    )
+                    == 0
+                ):
+                    # Add price difference account line.
+                    vals = {
+                        "name": line.name[:64],
+                        "move_id": move.id,
+                        "partner_id": line.partner_id.id
+                        or move.commercial_partner_id.id,
+                        "currency_id": line.currency_id.id,
+                        "product_id": line.product_id.id,
+                        "product_uom_id": line.product_uom_id.id,
+                        "quantity": relevant_qty,
+                        "price_unit": price_unit_val_dif,
+                        "price_subtotal": relevant_qty * price_unit_val_dif,
+                        "amount_currency": relevant_qty
+                        * price_unit_val_dif
+                        * line.move_id.direction_sign,
+                        "balance": line.currency_id._convert(
+                            relevant_qty
+                            * price_unit_val_dif
+                            * line.move_id.direction_sign,
+                            line.company_currency_id,
+                            line.company_id,
+                            fields.Date.today(),
+                        ),
+                        "account_id": debit_expense_account.id,
+                        "analytic_distribution": line.analytic_distribution,
+                        "display_type": "cogs",
+                        "rma_line_id": line.rma_line_id.id,
+                    }
+
+                    lines_vals_list_rma.append(vals)
+
+                    # Correct the amount of the current line.
+                    vals = {
+                        "name": line.name[:64],
+                        "move_id": move.id,
+                        "partner_id": line.partner_id.id
+                        or move.commercial_partner_id.id,
+                        "currency_id": line.currency_id.id,
+                        "product_id": line.product_id.id,
+                        "product_uom_id": line.product_uom_id.id,
+                        "quantity": relevant_qty,
+                        "price_unit": -price_unit_val_dif,
+                        "price_subtotal": relevant_qty * -price_unit_val_dif,
+                        "amount_currency": relevant_qty
+                        * -price_unit_val_dif
+                        * line.move_id.direction_sign,
+                        "balance": line.currency_id._convert(
+                            relevant_qty
+                            * -price_unit_val_dif
+                            * line.move_id.direction_sign,
+                            line.company_currency_id,
+                            line.company_id,
+                            fields.Date.today(),
+                        ),
+                        "account_id": line.account_id.id,
+                        "analytic_distribution": line.analytic_distribution,
+                        "display_type": "cogs",
+                        "rma_line_id": line.rma_line_id.id,
+                    }
+                    lines_vals_list_rma.append(vals)
+                    rma_refunds |= move
+        lines_vals_list = super(
+            AccountMove, self - rma_refunds
+        )._stock_account_prepare_anglo_saxon_in_lines_vals()
+        lines_vals_list += lines_vals_list_rma
+        return lines_vals_list

rma_purchase/models/account_move_line.py

@@ -0,0 +1,22 @@
+from odoo import models
+
+
+class AccountMoveLine(models.Model):
+    _inherit = "account.move.line"
+
+    def _get_price_diff_account(self):
+        # force the price difference account to be taken from the price
+        # different properties as they were in the previous Odoo versions
+        self.ensure_one()
+        if self.product_id.cost_method != "standard":
+            # product variable needed, pre-commit trolling with 88 character
+            product = self.product_id
+            debit_pdiff_account = (
+                product.property_account_creditor_price_difference
+                or product.categ_id.property_account_creditor_price_difference_categ
+            )
+            debit_pdiff_account = self.move_id.fiscal_position_id.map_account(
+                debit_pdiff_account
+            )
+            return debit_pdiff_account
+        return super()._get_price_diff_account()

rma_purchase/models/procurement.py

@@ -0,0 +1,15 @@
+# Copyright 2017-2022 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import fields, models
+
+
+class ProcurementGroup(models.Model):
+    _inherit = "procurement.group"
+
+    rma_id = fields.Many2one(
+        comodel_name="rma.order", string="RMA", ondelete="set null"
+    )
+    rma_line_id = fields.Many2one(
+        comodel_name="rma.order.line", string="RMA line", ondelete="set null"
+    )

rma_purchase/models/purchase_order.py

@@ -0,0 +1,30 @@
+# Copyright 2017-2022 ForgeFlow S.L.
+# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html)
+
+from odoo import api, models
+
+
+class PurchaseOrder(models.Model):
+    _inherit = "purchase.order"
+
+    @api.model
+    def new(self, vals, origin=None, ref=None):
+        """Allows to propose a line based on the RMA information."""
+        res = super().new(vals)
+        rma_line_id = self.env.context.get("rma_line_id")
+        if rma_line_id:
+            rma_line = self.env["rma.order.line"].browse(rma_line_id)
+            line = self.env["purchase.order.line"].new(
+                {
+                    "product_id": rma_line.product_id.id,
+                }
+            )
+            line.onchange_product_id()
+            line.update(
+                {
+                    "product_qty": rma_line.qty_to_purchase,
+                    "product_uom": rma_line.uom_id.id,
+                }
+            )
+            res.order_line = line
+        return res