diff --git a/rma/wizards/rma_make_picking.py b/rma/wizards/rma_make_picking.py index 637caea9b..bf0a3209c 100644 --- a/rma/wizards/rma_make_picking.py +++ b/rma/wizards/rma_make_picking.py @@ -137,6 +137,10 @@ def _get_procurement_data(self, item, group, qty, picking_type): } return procurement_data + @api.model + def _get_product(self, item): + return item.line_id.product_id + @api.model def _create_procurement(self, item, picking_type): errors = [] @@ -149,7 +153,7 @@ def _create_procurement(self, item, picking_type): else: qty = item.qty_to_deliver values = self._get_procurement_data(item, group, qty, picking_type) - product = item.line_id.product_id + product = self._get_product(item) if float_compare(qty, 0, product.uom_id.rounding) != 1: raise ValidationError( _( @@ -162,16 +166,15 @@ def _create_procurement(self, item, picking_type): procurements = [] try: procurement = group.Procurement( - item.line_id.product_id, + product, qty, - item.line_id.product_id.product_tmpl_id.uom_id, + product.product_tmpl_id.uom_id, values.get("location_id"), values.get("origin"), values.get("origin"), self.env.company, values, ) - procurements.append(procurement) # Trigger a route check with a mutable in the context that can be # cleared after the first rule selection diff --git a/rma_product_exchange/README.rst b/rma_product_exchange/README.rst new file mode 100644 index 000000000..79c63e777 --- /dev/null +++ b/rma_product_exchange/README.rst @@ -0,0 +1,34 @@ +.. image:: https://img.shields.io/badge/licence-LGPL--3-blue.svg + :alt: License LGPL-3 + +==================== +RMA Product Exchange +==================== + +This module allows you to: + +#. to exchange one product for another in an RMA. + +Usage +===== + +Bug Tracker +=========== + +Bugs are tracked on `GitHub 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 +------------ + +* Chafique DELLI + +Maintainer +---------- + +This module is maintained by Eficent diff --git a/rma_product_exchange/__init__.py b/rma_product_exchange/__init__.py new file mode 100644 index 000000000..aee8895e7 --- /dev/null +++ b/rma_product_exchange/__init__.py @@ -0,0 +1,2 @@ +from . import models +from . import wizards diff --git a/rma_product_exchange/__manifest__.py b/rma_product_exchange/__manifest__.py new file mode 100644 index 000000000..1b7f8da52 --- /dev/null +++ b/rma_product_exchange/__manifest__.py @@ -0,0 +1,21 @@ +# Copyright 2024 Akretion +# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html) + +{ + "name": "RMA Product Exchange", + "version": "16.0.1.0.0", + "license": "LGPL-3", + "category": "RMA", + "summary": "Manage RMA for product exchange", + "author": "Akretion, ForgeFlow", + "website": "https://github.com/ForgeFlow/stock-rma", + "depends": [ + "rma", + ], + "data": [ + "views/rma_order_view.xml", + "views/rma_operation_view.xml", + "views/rma_order_line_view.xml", + ], + "installable": True, +} diff --git a/rma_product_exchange/i18n/fr.po b/rma_product_exchange/i18n/fr.po new file mode 100644 index 000000000..226161935 --- /dev/null +++ b/rma_product_exchange/i18n/fr.po @@ -0,0 +1,72 @@ +# Translation of Odoo Server. +# This file contains the translation of the following modules: +# * rma_product_exchange +# +msgid "" +msgstr "" +"Project-Id-Version: Odoo Server 16.0\n" +"Report-Msgid-Bugs-To: \n" +"POT-Creation-Date: 2024-09-16 20:17+0000\n" +"PO-Revision-Date: 2024-09-16 20:17+0000\n" +"Last-Translator: \n" +"Language-Team: \n" +"MIME-Version: 1.0\n" +"Content-Type: text/plain; charset=UTF-8\n" +"Content-Transfer-Encoding: \n" +"Plural-Forms: \n" + +#. module: rma_product_exchange +#: model:ir.model.fields,help:rma_product_exchange.field_rma_operation__product_exchange +#: model:ir.model.fields,help:rma_product_exchange.field_rma_order_line__product_exchange +msgid "Check if you wish to authorize product exchange." +msgstr "Cochez si vous souhaitez autoriser l'échange de produit." + +#. module: rma_product_exchange +#: model:ir.model.fields,field_description:rma_product_exchange.field_rma_order_line__new_product_id +msgid "New Product" +msgstr "Nouveau Produit" + +#. module: rma_product_exchange +#: model:ir.model,name:rma_product_exchange.model_procurement_group +msgid "Procurement Group" +msgstr "Groupe d'approvisionnement" + +#. module: rma_product_exchange +#: model:ir.model.fields,field_description:rma_product_exchange.field_rma_operation__product_exchange +#: model:ir.model.fields,field_description:rma_product_exchange.field_rma_order_line__product_exchange +msgid "Product Exchange" +msgstr "Echange de produit" + +#. module: rma_product_exchange +#: model:ir.model,name:rma_product_exchange.model_rma_order_line +msgid "RMA" +msgstr "" + +#. module: rma_product_exchange +#: model:ir.model,name:rma_product_exchange.model_rma_operation +msgid "RMA Operation" +msgstr "Opération RMA" + +#. module: rma_product_exchange +#: model:ir.model.fields,help:rma_product_exchange.field_rma_order_line__new_product_id +msgid "" +"The new product selected will be shipped instead of the product initially " +"ordered." +msgstr "" +"Le nouveau produit sélectionné sera expédié à la place du produit initialement " +"commandé" +#. module: rma_product_exchange +#. odoo-python +#: code:addons/rma_product_exchange/models/rma_order_line.py:0 +#, python-format +msgid "" +"The selected replacement product must have the same unit of measurement as " +"the initial product !" +msgstr "" +"Le produit de remplacement sélectionné doit avoir la même unité de mesure que " +"le produit initial !" + +#. module: rma_product_exchange +#: model:ir.model,name:rma_product_exchange.model_rma_make_picking_wizard +msgid "Wizard to create Pickings from rma" +msgstr "Assistant pour créer des transferts à partir de rma" diff --git a/rma_product_exchange/models/__init__.py b/rma_product_exchange/models/__init__.py new file mode 100644 index 000000000..b1fae22f5 --- /dev/null +++ b/rma_product_exchange/models/__init__.py @@ -0,0 +1,2 @@ +from . import rma_order_line +from . import rma_operation diff --git a/rma_product_exchange/models/rma_operation.py b/rma_product_exchange/models/rma_operation.py new file mode 100644 index 000000000..398a8968f --- /dev/null +++ b/rma_product_exchange/models/rma_operation.py @@ -0,0 +1,11 @@ +# Copyright 2024 Akretion +# 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" + + product_exchange = fields.Boolean( + help="Check if you wish to authorize product exchange.", default=False + ) diff --git a/rma_product_exchange/models/rma_order_line.py b/rma_product_exchange/models/rma_order_line.py new file mode 100644 index 000000000..32a0d2a60 --- /dev/null +++ b/rma_product_exchange/models/rma_order_line.py @@ -0,0 +1,30 @@ +# Copyright 2024 Akretion +# 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 RmaOrderLine(models.Model): + _inherit = "rma.order.line" + + new_product_id = fields.Many2one( + comodel_name="product.product", + tracking=True, + help="The new product selected will be shipped " + "instead of the product initially ordered.", + ) + product_exchange = fields.Boolean(related="operation_id.product_exchange") + + @api.constrains("new_product_id") + def _check_new_product_id(self): + for record in self: + if ( + record.new_product_id + and record.new_product_id.uom_id != record.product_id.uom_id + ): + raise ValidationError( + _( + "The selected replacement product must " + "have the same unit of measurement as the initial product !" + ) + ) diff --git a/rma_product_exchange/tests/__init__.py b/rma_product_exchange/tests/__init__.py new file mode 100644 index 000000000..5e38e638b --- /dev/null +++ b/rma_product_exchange/tests/__init__.py @@ -0,0 +1 @@ +from . import test_rma_product_exchange diff --git a/rma_product_exchange/tests/test_rma_product_exchange.py b/rma_product_exchange/tests/test_rma_product_exchange.py new file mode 100644 index 000000000..d123c09e4 --- /dev/null +++ b/rma_product_exchange/tests/test_rma_product_exchange.py @@ -0,0 +1,28 @@ +# Copyright 2024 Akretion +# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html) + +from odoo.addons.rma.tests.test_rma import TestRma + + +class TestRmaProductExchange(TestRma): + @classmethod + def setUpClass(cls): + super().setUpClass() + + def test_customer_rma_with_product_exchange(self): + self.rma_cust_replace_op_id.product_exchange = True + self.rma_customer_id.rma_line_ids.delivery_policy = "ordered" + self.rma_customer_id.rma_line_ids[0].new_product_id = self.product_id + self.rma_customer_id.rma_line_ids.action_rma_to_approve() + wizard = self.rma_make_picking.with_context( + **{ + "active_ids": self.rma_customer_id.rma_line_ids.ids, + "active_model": "rma.order.line", + "picking_type": "outgoing", + "active_id": 1, + } + ).create({}) + wizard._create_picking() + res = self.rma_customer_id.action_view_out_shipments() + picking = self.env["stock.picking"].browse(res["res_id"]) + self.assertEqual(picking.move_ids[0].product_id, self.product_id) diff --git a/rma_product_exchange/views/rma_operation_view.xml b/rma_product_exchange/views/rma_operation_view.xml new file mode 100644 index 000000000..13dbf0476 --- /dev/null +++ b/rma_product_exchange/views/rma_operation_view.xml @@ -0,0 +1,27 @@ + + + + + rma.operation + + + + + + + + + + rma.operation + + + + + + + + + diff --git a/rma_product_exchange/views/rma_order_line_view.xml b/rma_product_exchange/views/rma_order_line_view.xml new file mode 100644 index 000000000..159910d27 --- /dev/null +++ b/rma_product_exchange/views/rma_order_line_view.xml @@ -0,0 +1,21 @@ + + + + + rma.order.line + + + + + + + + + + diff --git a/rma_product_exchange/views/rma_order_view.xml b/rma_product_exchange/views/rma_order_view.xml new file mode 100644 index 000000000..a5a14b51c --- /dev/null +++ b/rma_product_exchange/views/rma_order_view.xml @@ -0,0 +1,19 @@ + + + + rma.order + + + + + + + + + diff --git a/rma_product_exchange/wizards/__init__.py b/rma_product_exchange/wizards/__init__.py new file mode 100644 index 000000000..8a3b735f4 --- /dev/null +++ b/rma_product_exchange/wizards/__init__.py @@ -0,0 +1 @@ +from . import rma_make_picking diff --git a/rma_product_exchange/wizards/rma_make_picking.py b/rma_product_exchange/wizards/rma_make_picking.py new file mode 100644 index 000000000..ebc36a2c6 --- /dev/null +++ b/rma_product_exchange/wizards/rma_make_picking.py @@ -0,0 +1,36 @@ +# Copyright (C) 2024 Akretion +# License LGPL-3.0 or later (https://www.gnu.org/licenses/lgpl.html) + +from odoo import api, models + + +class RmaMakePicking(models.TransientModel): + _inherit = "rma_make_picking.wizard" + + @api.returns("rma.order.line") + def _prepare_item(self, line): + values = super()._prepare_item(line) + if ( + self.env.context.get("picking_type") == "outgoing" + and line.product_exchange + and line.new_product_id + ): + values["product_id"] = line.new_product_id.id + return values + + @api.model + def _create_procurement(self, item, picking_type): + if self.env.context.get("picking_type") == "outgoing": + self = self.with_context(rma_item=item) + return super()._create_procurement(item, picking_type) + + @api.model + def _get_product(self, item): + product = super()._get_product(item) + if ( + self.env.context.get("picking_type") == "outgoing" + and item.line_id.product_exchange + and item.line_id.new_product_id + ): + product = item.line_id.new_product_id + return product diff --git a/setup/rma_product_exchange/odoo/addons/rma_product_exchange b/setup/rma_product_exchange/odoo/addons/rma_product_exchange new file mode 120000 index 000000000..c47713064 --- /dev/null +++ b/setup/rma_product_exchange/odoo/addons/rma_product_exchange @@ -0,0 +1 @@ +../../../../rma_product_exchange \ No newline at end of file diff --git a/setup/rma_product_exchange/setup.py b/setup/rma_product_exchange/setup.py new file mode 100644 index 000000000..28c57bb64 --- /dev/null +++ b/setup/rma_product_exchange/setup.py @@ -0,0 +1,6 @@ +import setuptools + +setuptools.setup( + setup_requires=['setuptools-odoo'], + odoo_addon=True, +)