[IMP] repair_stock_consumption_step: support partial consumption

Author: nicolas-delbovier-acsone

repair_stock_consumption_step/README.rst

@@ -1,3 +1,7 @@
+.. image:: https://odoo-community.org/readme-banner-image
+   :target: https://odoo-community.org/get-involved?utm_source=readme
+   :alt: Odoo Community Association
+
 =============================
 Repair Stock Consumption Step
 =============================
@@ -13,7 +17,7 @@ Repair Stock Consumption Step
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
     :target: https://odoo-community.org/page/development-status
     :alt: Beta
-.. |badge2| image:: https://img.shields.io/badge/licence-AGPL--3-blue.png
+.. |badge2| image:: https://img.shields.io/badge/license-AGPL--3-blue.png
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Frepair-lightgray.png?logo=github

repair_stock_consumption_step/__init__.py

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

repair_stock_consumption_step/__manifest__.py

@@ -12,6 +12,11 @@
     "maintainers": ["sbejaoui"],
     "depends": ["repair", "repair_warehouse"],
     "excludes": ["repair_stock_move"],
-    "data": ["views/repair_order.xml", "views/stock_warehouse.xml"],
+    "data": [
+        "security/ir.model.access.csv",
+        "wizards/repair_consumption_partial_wizard.xml",
+        "views/repair_order.xml",
+        "views/stock_warehouse.xml",
+    ],
     "demo": [],
 }

repair_stock_consumption_step/models/repair_order.py

@@ -2,10 +2,10 @@
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
 from odoo import Command, fields, models
+from odoo.tools import float_is_zero
 
 
 class RepairOrder(models.Model):
-
     _inherit = "repair.order"
 
     consumption_picking_id = fields.Many2one(
@@ -84,3 +84,48 @@ def _action_consumption_done(self):
             if not rec.invoice_id and rec.invoice_method == "after_repair":
                 state = "2binvoiced"
             rec.state = state
+
+    def _update_parts(self, return_consumption_moves):
+        """Update Repair Order operations based on unconsumed quantities.
+
+        This method reduces the demand (product_uom_qty) on the repair order
+        lines by matching them against moves that were cancelled/returned
+        during the partial consumption process. It uses a 'pop' logic across
+        multiple lines of the same product until the returned quantity
+        is fully accounted for.
+
+        Complexity Note:
+        The pop logic here is needed because there might be more than one repair.line
+        per product.
+        """
+        self.ensure_one()
+        operations_to_delete = self.env["repair.line"]
+        for return_move in return_consumption_moves:
+            remaining_qty = return_move.product_uom_qty
+            operations = self.operations.filtered(
+                lambda o: o.product_id == return_move.product_id
+            )
+            while (
+                not float_is_zero(
+                    remaining_qty, precision_rounding=return_move.product_uom.rounding
+                )
+                and operations
+            ):
+                # TODO: find a better solution to extract the "best matching" operation
+                # instead of simply taking the first
+                operation = operations[0]
+                qty_before_update = operation.product_uom_qty
+                operation.product_uom_qty -= min(
+                    operation.product_uom_qty, remaining_qty
+                )
+                if float_is_zero(
+                    operation.product_uom_qty,
+                    precision_rounding=operation.product_uom.rounding,
+                ):
+                    operations_to_delete |= operation
+
+                remaining_qty -= qty_before_update - operation.product_uom_qty
+                operations -= operation
+
+        if operations_to_delete:
+            operations_to_delete.unlink()

repair_stock_consumption_step/models/stock_picking.py

@@ -1,13 +1,17 @@
 # Copyright 2025 ACSONE SA/NV
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import models
+from odoo import _, fields, models
+from odoo.exceptions import ValidationError
 
 
 class StockPicking(models.Model):
-
     _inherit = "stock.picking"
 
+    consumption_repair_order_id = fields.One2many(
+        "repair.order", "consumption_picking_id"
+    )
+
     def _action_done(self):
         res = super()._action_done()
         done_pickings = self.filtered(lambda p: p.state == "done")
@@ -19,3 +23,62 @@ def _action_done(self):
         )
         repair_orders._action_consumption_done()
         return res
+
+    def _check_no_partial_qties_on_repaired_product(self):
+        """
+        Ensure the consumption pickings can not partially process the qties
+        for the repaired product. (Partial quantities are only allowed on
+        the spare parts).
+        """
+        invalid_picks = self.env["stock.picking"]
+        for pick in self.filtered("consumption_repair_order_id"):
+            repair = pick.consumption_repair_order_id
+            repaired_product_move = pick.move_ids.filtered(
+                lambda m: m.product_id == repair.product_id
+            )
+            if (
+                repaired_product_move
+                and repaired_product_move.product_uom_qty
+                != repaired_product_move.quantity_done
+            ):
+                invalid_picks |= pick
+
+        if invalid_picks:
+            raise ValidationError(
+                _(
+                    "Invalid partial quantities on repaired product."
+                    "You can only partially process spare parts.\n"
+                    "\n"
+                    "Picking(s):\n- %s",
+                    "\n- ".join(invalid_picks.mapped("name")),
+                )
+            )
+
+    def _pre_action_done_hook(self):
+        partially_processed_pickings = self._check_backorder()
+        if partial_consumption_picks := partially_processed_pickings.filtered(
+            "consumption_repair_order_id"
+        ):
+            partial_consumption_picks._check_no_partial_qties_on_repaired_product()
+            return partial_consumption_picks._action_repair_consumption_partial_wizard(
+                default_pick_ids=partial_consumption_picks.ids
+            )
+        return super()._pre_action_done_hook()
+
+    def _action_repair_consumption_partial_wizard(self, default_pick_ids):
+        view = self.env.ref(
+            "repair_stock_consumption_step.repair_consumption_backorder_wizard_form_view"
+        )
+        return {
+            "name": _("Process Unused Spare Parts?"),
+            "type": "ir.actions.act_window",
+            "view_mode": "form",
+            "res_model": "repair.consumption.partial.wizard",
+            "views": [(view.id, "form")],
+            "view_id": view.id,
+            "target": "new",
+            "context": dict(
+                self.env.context,
+                default_pick_ids=[(4, _id) for _id in default_pick_ids],
+            ),
+        }

repair_stock_consumption_step/security/ir.model.access.csv

@@ -0,0 +1,2 @@
+id,name,model_id:id,group_id:id,perm_read,perm_write,perm_create,perm_unlink
+access_repair_consumption_partial_wizard_user,repair.consumption.partial.wizard,model_repair_consumption_partial_wizard,stock.group_stock_user,1,1,1,1

repair_stock_consumption_step/static/description/index.html

@@ -3,7 +3,7 @@
 <head>
 <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
 <meta name="generator" content="Docutils: https://docutils.sourceforge.io/" />
-<title>Repair Stock Consumption Step</title>
+<title>README.rst</title>
 <style type="text/css">
 
 /*
@@ -360,16 +360,21 @@
 </style>
 </head>
 <body>
-<div class="document" id="repair-stock-consumption-step">
-<h1 class="title">Repair Stock Consumption Step</h1>
+<div class="document">
 
+
+<a class="reference external image-reference" href="https://odoo-community.org/get-involved?utm_source=readme">
+<img alt="Odoo Community Association" src="https://odoo-community.org/readme-banner-image" />
+</a>
+<div class="section" id="repair-stock-consumption-step">
+<h1>Repair Stock Consumption Step</h1>
 <!-- !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:93fc83002290646de5793f5425fca96699172ab4f9487b4196a3e8481bbaead6
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/repair/tree/16.0/repair_stock_consumption_step"><img alt="OCA/repair" src="https://img.shields.io/badge/github-OCA%2Frepair-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/repair-16-0/repair-16-0-repair_stock_consumption_step"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/repair&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/repair/tree/16.0/repair_stock_consumption_step"><img alt="OCA/repair" src="https://img.shields.io/badge/github-OCA%2Frepair-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/repair-16-0/repair-16-0-repair_stock_consumption_step"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external image-reference" href="https://runboat.odoo-community.org/builds?repo=OCA/repair&amp;target_branch=16.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
 <p>This module introduces an optional intermediate step:</p>
 <ul class="simple">
 <li>When enabled at warehouse level, repair consumption moves are grouped
@@ -395,13 +400,13 @@ <h1 class="title">Repair Stock Consumption Step</h1>
 </ul>
 </div>
 <div class="section" id="use-cases-context">
-<h1><a class="toc-backref" href="#toc-entry-1">Use Cases / Context</a></h1>
+<h2><a class="toc-backref" href="#toc-entry-1">Use Cases / Context</a></h2>
 <p>In the base repair module, consumption moves for spare parts are created
 and immediately validated when the repair order is marked as done. This
 prevents user interaction such as choosing lot/serial numbers.</p>
 </div>
 <div class="section" id="configuration">
-<h1><a class="toc-backref" href="#toc-entry-2">Configuration</a></h1>
+<h2><a class="toc-backref" href="#toc-entry-2">Configuration</a></h2>
 <ol class="arabic">
 <li><p class="first">Go to <strong>Inventory / Configuration / Warehouses</strong>.</p>
 </li>
@@ -420,7 +425,7 @@ <h1><a class="toc-backref" href="#toc-entry-2">Configuration</a></h1>
 </ol>
 </div>
 <div class="section" id="usage">
-<h1><a class="toc-backref" href="#toc-entry-3">Usage</a></h1>
+<h2><a class="toc-backref" href="#toc-entry-3">Usage</a></h2>
 <ol class="arabic simple">
 <li>Create a repair order with spare part lines.</li>
 <li>Confirm the repair order.</li>
@@ -448,29 +453,29 @@ <h1><a class="toc-backref" href="#toc-entry-3">Usage</a></h1>
 </ol>
 </div>
 <div class="section" id="bug-tracker">
-<h1><a class="toc-backref" href="#toc-entry-4">Bug Tracker</a></h1>
+<h2><a class="toc-backref" href="#toc-entry-4">Bug Tracker</a></h2>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/repair/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us to smash it by providing a detailed and welcomed
 <a class="reference external" href="https://github.com/OCA/repair/issues/new?body=module:%20repair_stock_consumption_step%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
 <p>Do not contact contributors directly about support or help with technical issues.</p>
 </div>
 <div class="section" id="credits">
-<h1><a class="toc-backref" href="#toc-entry-5">Credits</a></h1>
+<h2><a class="toc-backref" href="#toc-entry-5">Credits</a></h2>
 <div class="section" id="authors">
-<h2><a class="toc-backref" href="#toc-entry-6">Authors</a></h2>
+<h3><a class="toc-backref" href="#toc-entry-6">Authors</a></h3>
 <ul class="simple">
 <li>ACSONE SA/NV</li>
 </ul>
 </div>
 <div class="section" id="contributors">
-<h2><a class="toc-backref" href="#toc-entry-7">Contributors</a></h2>
+<h3><a class="toc-backref" href="#toc-entry-7">Contributors</a></h3>
 <ul class="simple">
 <li>Souheil Bejaoui <a class="reference external" href="mailto:souheil.bejaoui&#64;acsone.eu">souheil.bejaoui&#64;acsone.eu</a></li>
 </ul>
 </div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#toc-entry-8">Maintainers</a></h2>
+<h3><a class="toc-backref" href="#toc-entry-8">Maintainers</a></h3>
 <p>This module is maintained by the OCA.</p>
 <a class="reference external image-reference" href="https://odoo-community.org">
 <img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
@@ -485,5 +490,6 @@ <h2><a class="toc-backref" href="#toc-entry-8">Maintainers</a></h2>
 </div>
 </div>
 </div>
+</div>
 </body>
 </html>

repair_stock_consumption_step/tests/__init__.py

@@ -1 +1,3 @@
 from . import test_repair_stock_consumption_step
+from . import test_partial_consumption
+from . import common

repair_stock_consumption_step/tests/common.py

@@ -0,0 +1,78 @@
+# Copyright 2026 ACSONE SA/NV
+# License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
+
+from odoo import Command
+from odoo.tests.common import TransactionCase
+
+
+class Common(TransactionCase):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
+        cls.partner = cls.env["res.partner"].create({"name": "Test Partner"})
+        cls.product = cls.env["product.product"].create(
+            {"name": "Product to repair", "type": "product"}
+        )
+        cls.product_c = cls.env["product.product"].create(
+            {"name": "product to consume", "type": "product"}
+        )
+
+        cls.warehouse = cls.env["stock.warehouse"].create(
+            {
+                "name": "WH",
+                "code": "wh_test",
+                "repair_consumption_step": True,
+            }
+        )
+        cls.repair_loc = cls.warehouse.lot_stock_id
+        cls.production_location = cls.env["stock.location"].search(
+            [("usage", "=", "production")], limit=1
+        )
+        cls.consumption_type = cls.env["stock.picking.type"].create(
+            {
+                "name": "Consumption",
+                "warehouse_id": cls.warehouse.id,
+                "code": "internal",
+                "sequence_code": "PREP",
+                "default_location_src_id": cls.repair_loc.id,
+                "default_location_dest_id": cls.production_location.id,
+            }
+        )
+        cls.warehouse.repair_consumption_picking_type_id = cls.consumption_type
+
+        cls.repair = cls.env["repair.order"].create(
+            {
+                "partner_id": cls.partner.id,
+                "product_id": cls.product.id,
+                "location_id": cls.repair_loc.id,
+                "operations": [
+                    Command.create(
+                        {
+                            "name": "replace product",
+                            "type": "add",
+                            "price_unit": 100,
+                            "product_id": cls.product_c.id,
+                            "product_uom_qty": 2.0,
+                            "location_id": cls.repair_loc.id,
+                            "lot_id": cls.env["stock.lot"]
+                            .create(
+                                {"name": "Test Lot", "product_id": cls.product_c.id}
+                            )
+                            .id,
+                        }
+                    )
+                ],
+            }
+        )
+        cls.env["stock.quant"]._update_available_quantity(
+            cls.product, cls.repair_loc, 1.0
+        )
+        cls.env["stock.quant"]._update_available_quantity(
+            cls.product_c, cls.repair_loc, 10.0
+        )
+
+    @classmethod
+    def _do_picking(cls, picking):
+        for move in picking.move_ids:
+            move.quantity_done = move.product_qty
+        picking._action_done()