PyPI - odoo-addon-stock-restrict-lot - Versions diffs - 17.0.1.2.0__py3-none-any.whl → 18.0.1.0.0.2__py3-none-any.whl - Mend

odoo-addon-stock-restrict-lot 17.0.1.2.0py3-none-any.whl → 18.0.1.0.0.2py3-none-any.whl

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Potentially problematic release.

This version of odoo-addon-stock-restrict-lot might be problematic. Click here for more details.

Files changed (17) hide show

odoo/addons/stock_restrict_lot/README.rst CHANGED Viewed

@@ -7,7 +7,7 @@ Stock Restrict Lot
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:00ff8e2abcdd536138a0bf51463e00f076ac3fe94947998cda2b6792ef723d5b
+   !! source digest: sha256:dce023da3482003599a49ae27980bca635a30f0b86afd9ee1bcbeb8d3ae775d9
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
@@ -17,21 +17,22 @@ Stock Restrict Lot
     :target: http://www.gnu.org/licenses/lgpl-3.0-standalone.html
     :alt: License: LGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--workflow-lightgray.png?logo=github
-    :target: https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot
+    :target: https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot
     :alt: OCA/stock-logistics-workflow
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/stock-logistics-workflow-17-0/stock-logistics-workflow-17-0-stock_restrict_lot
+    :target: https://translation.odoo-community.org/projects/stock-logistics-workflow-18-0/stock-logistics-workflow-18-0-stock_restrict_lot
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-workflow&target_branch=17.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-workflow&target_branch=18.0
     :alt: Try me on Runboat
 |badge1| |badge2| |badge3| |badge4| |badge5|
-This module add a field to restrict a stock move to a specific lot. It
+This module adds a field to restrict a stock move to a specific lot. It
 propagates it between chained moves. A move with a restrict lot will
 only be able to reserve or transfer products with the specified lot.
-This module is a based for other modules, it has not effect on its own.
+This module serves as a basis for other modules, it has not effect on
+its own.
 **Table of contents**
@@ -44,7 +45,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/stock-logistics-workflow/issues>`_.
 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
-`feedback <https://github.com/OCA/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 Do not contact contributors directly about support or help with technical issues.
@@ -61,6 +62,7 @@ Contributors
 - Florian da Costa <florian.dacosta@akretion.com>
 - Ruchir Shukla <ruchir@bizzappdev.com>
+- Stefan Rijnhart <stefan@opener.amsterdam>
 Maintainers
 -----------
@@ -83,6 +85,6 @@ Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 |maintainer-florian-dacosta|
-This module is part of the `OCA/stock-logistics-workflow <https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot>`_ project on GitHub.
+This module is part of the `OCA/stock-logistics-workflow <https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot>`_ project on GitHub.
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

odoo/addons/stock_restrict_lot/__manifest__.py CHANGED Viewed

@@ -1,7 +1,9 @@
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 {
     "name": "Stock Restrict Lot",
     "summary": "Base module that add back the concept of restrict lot on stock move",
-    "version": "17.0.1.2.0",
+    "version": "18.0.1.0.0",
     "category": "Warehouse Management",
     "website": "https://github.com/OCA/stock-logistics-workflow",
     "author": "Akretion, Odoo Community Association (OCA)",

odoo/addons/stock_restrict_lot/i18n/stock_restrict_lot.pot CHANGED Viewed

@@ -4,7 +4,7 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: Odoo Server 17.0\n"
+"Project-Id-Version: Odoo Server 18.0\n"
 "Report-Msgid-Bugs-To: \n"
 "Last-Translator: \n"
 "Language-Team: \n"
@@ -16,7 +16,6 @@ msgstr ""
 #. module: stock_restrict_lot
 #. odoo-python
 #: code:addons/stock_restrict_lot/models/stock_move.py:0
-#, python-format
 msgid ""
 "Inconsistencies between reserved quant and lot restriction on stock move"
 msgstr ""
@@ -50,7 +49,6 @@ msgstr ""
 #. module: stock_restrict_lot
 #. odoo-python
 #: code:addons/stock_restrict_lot/models/stock_move.py:0
-#, python-format
 msgid ""
 "The lot(s) %(move_line_lot)s being moved is inconsistent with the "
 "restriction on lot %(move_restrict_lot)s set on the move"
@@ -60,3 +58,11 @@ msgstr ""
 #: model:ir.model,name:stock_restrict_lot.model_stock_picking
 msgid "Transfer"
 msgstr ""
+#. module: stock_restrict_lot
+#. odoo-python
+#: code:addons/stock_restrict_lot/models/stock_move.py:0
+msgid ""
+"You can't modify the Lot/Serial number because at least one move in the "
+"chain has already been done with another Lot/Serial number."
+msgstr ""

odoo/addons/stock_restrict_lot/models/stock_move.py CHANGED Viewed

@@ -1,11 +1,14 @@
-from odoo import _, api, exceptions, fields, models
-from odoo.exceptions import UserError
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+from odoo import api, exceptions, fields, models
+from odoo.exceptions import UserError, ValidationError
+from odoo.tools.misc import OrderedSet
 class StockMove(models.Model):
     _inherit = "stock.move"
-    # seems better to not copy this field except when a move is splitted, because a move
+    # seems better to not copy this field except when a move is split, because a move
     # can be copied in multiple different occasions and could even be copied with a
     # different product...
     restrict_lot_id = fields.Many2one(
@@ -34,7 +37,7 @@ class StockMove(models.Model):
                 and vals["lot_id"] != self.restrict_lot_id.id
             ):
                 raise exceptions.UserError(
-                    _(
+                    self.env._(
                         "Inconsistencies between reserved quant and lot restriction on "
                         "stock move"
                     )
@@ -67,7 +70,6 @@ class StockMove(models.Model):
         self,
         need,
         location_id,
-        quant_ids=None,
         lot_id=None,
         package_id=None,
         owner_id=None,
@@ -79,7 +81,6 @@ class StockMove(models.Model):
         return super()._update_reserved_quantity(
             need,
             location_id,
-            quant_ids=quant_ids,
             lot_id=lot_id,
             package_id=package_id,
             owner_id=owner_id,
@@ -108,7 +109,7 @@ class StockMove(models.Model):
             move_line_lot = move.mapped("move_line_ids.lot_id")
             if move.restrict_lot_id != move_line_lot:
                 raise UserError(
-                    _(
+                    self.env._(
                         "The lot(s) %(move_line_lot)s being moved is "
                         "inconsistent with the restriction on "
                         "lot %(move_restrict_lot)s set on the move",
@@ -116,3 +117,44 @@ class StockMove(models.Model):
                         move_restrict_lot=move.restrict_lot_id.display_name,
                     )
                 )
+    # Same as _rollup_move_origs but also for "done" moves.
+    def _rollup_not_cancelled_move_origs(self, seen=False):
+        if not seen:
+            seen = OrderedSet()
+        unseen = OrderedSet(self.ids) - seen
+        if not unseen:
+            return seen
+        seen.update(unseen)
+        self.filtered(lambda m: m.id in unseen).move_orig_ids.filtered(
+            lambda sm: sm.state != "cancel"
+        )._rollup_not_cancelled_move_origs(seen)
+        return seen
+    def write(self, vals):
+        if "restrict_lot_id" not in vals:
+            return super().write(vals)
+        else:
+            restrict_lot_id = vals.pop("restrict_lot_id")
+            restrict_lot = self.env["stock.lot"].browse(restrict_lot_id)
+            chained_move_ids = self._rollup_move_dests()
+            chained_move_ids = self._rollup_not_cancelled_move_origs(
+                chained_move_ids - self.ids
+            )
+            chained_moves = self.env["stock.move"].browse(chained_move_ids)
+            if any(
+                [
+                    sm.state == "done" and sm.lot_ids and sm.lot_ids != restrict_lot
+                    for sm in chained_moves
+                ]
+            ):
+                raise ValidationError(
+                    self.env._(
+                        "You can't modify the Lot/Serial number "
+                        "because at least one move in the chain has "
+                        "already been done with another Lot/Serial number."
+                    )
+                )
+            for move in chained_moves:
+                super(StockMove, move).write({"restrict_lot_id": restrict_lot_id})
+        return super().write(vals)

odoo/addons/stock_restrict_lot/models/stock_picking.py CHANGED Viewed

@@ -1,4 +1,4 @@
-# License LGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
+# License AGPL-3 - See http://www.gnu.org/licenses/agpl-3.0.html
 from odoo import fields, models

odoo/addons/stock_restrict_lot/models/stock_rule.py CHANGED Viewed

@@ -1,3 +1,5 @@
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 from odoo import models
@@ -8,3 +10,8 @@ class StockRule(models.Model):
         fields = super()._get_custom_move_fields()
         fields += ["restrict_lot_id"]
         return fields
+    def _push_prepare_move_copy_values(self, move_to_copy, new_date):
+        values = super()._push_prepare_move_copy_values(move_to_copy, new_date)
+        values["restrict_lot_id"] = move_to_copy.restrict_lot_id.id
+        return values

odoo/addons/stock_restrict_lot/readme/CONTRIBUTORS.md CHANGED Viewed

@@ -1,2 +1,3 @@
 - Florian da Costa \<<florian.dacosta@akretion.com>\>
 - Ruchir Shukla \<<ruchir@bizzappdev.com>\>
+- Stefan Rijnhart \<<stefan@opener.amsterdam>\>

odoo/addons/stock_restrict_lot/readme/DESCRIPTION.md CHANGED Viewed

@@ -1,4 +1,4 @@
-This module add a field to restrict a stock move to a specific lot. It
+This module adds a field to restrict a stock move to a specific lot. It
 propagates it between chained moves. A move with a restrict lot will
 only be able to reserve or transfer products with the specified lot.
-This module is a based for other modules, it has not effect on its own.
+This module serves as a basis for other modules, it has not effect on its own.

odoo/addons/stock_restrict_lot/static/description/index.html CHANGED Viewed

@@ -367,13 +367,14 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:00ff8e2abcdd536138a0bf51463e00f076ac3fe94947998cda2b6792ef723d5b
+!! source digest: sha256:dce023da3482003599a49ae27980bca635a30f0b86afd9ee1bcbeb8d3ae775d9
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<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/lgpl-3.0-standalone.html"><img alt="License: LGPL-3" src="https://img.shields.io/badge/licence-LGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot"><img alt="OCA/stock-logistics-workflow" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--workflow-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-workflow-17-0/stock-logistics-workflow-17-0-stock_restrict_lot"><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/stock-logistics-workflow&amp;target_branch=17.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
-<p>This module add a field to restrict a stock move to a specific lot. It
+<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/lgpl-3.0-standalone.html"><img alt="License: LGPL-3" src="https://img.shields.io/badge/licence-LGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot"><img alt="OCA/stock-logistics-workflow" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--workflow-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-workflow-18-0/stock-logistics-workflow-18-0-stock_restrict_lot"><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/stock-logistics-workflow&amp;target_branch=18.0"><img alt="Try me on Runboat" src="https://img.shields.io/badge/runboat-Try%20me-875A7B.png" /></a></p>
+<p>This module adds a field to restrict a stock move to a specific lot. It
 propagates it between chained moves. A move with a restrict lot will
 only be able to reserve or transfer products with the specified lot.
-This module is a based for other modules, it has not effect on its own.</p>
+This module serves as a basis for other modules, it has not effect on
+its own.</p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
 <ul class="simple">
@@ -391,7 +392,7 @@ This module is a based for other modules, it has not effect on its own.</p>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/stock-logistics-workflow/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/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**">feedback</a>.</p>
+<a class="reference external" href="https://github.com/OCA/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2018.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">
@@ -407,6 +408,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <ul class="simple">
 <li>Florian da Costa &lt;<a class="reference external" href="mailto:florian.dacosta&#64;akretion.com">florian.dacosta&#64;akretion.com</a>&gt;</li>
 <li>Ruchir Shukla &lt;<a class="reference external" href="mailto:ruchir&#64;bizzappdev.com">ruchir&#64;bizzappdev.com</a>&gt;</li>
+<li>Stefan Rijnhart &lt;<a class="reference external" href="mailto:stefan&#64;opener.amsterdam">stefan&#64;opener.amsterdam</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="maintainers">
@@ -420,7 +422,7 @@ mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/florian-dacosta"><img alt="florian-dacosta" src="https://github.com/florian-dacosta.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot">OCA/stock-logistics-workflow</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot">OCA/stock-logistics-workflow</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

odoo/addons/stock_restrict_lot/tests/test_restrict_lot.py CHANGED Viewed

@@ -1,4 +1,6 @@
-from odoo.exceptions import UserError
+# License LGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
+from odoo.exceptions import UserError, ValidationError
 from odoo.tests.common import TransactionCase
@@ -8,10 +10,23 @@ class TestRestrictLot(TransactionCase):
         super().setUpClass()
         cls.customer_loc = cls.env.ref("stock.stock_location_customers")
         cls.output_loc = cls.env.ref("stock.stock_location_output")
-        cls.product = cls.env.ref("product.product_product_16")
+        cls.product = cls.env.ref("product.product_product_16").copy()
         cls.product.write({"tracking": "lot"})
         cls.warehouse = cls.env.ref("stock.warehouse0")
         cls.warehouse.write({"delivery_steps": "pick_ship"})
+        cls.stock_loc = cls.warehouse.lot_stock_id
+        # Set up a pull route from an external warehouse
+        cls.warehouse2 = cls.env["stock.warehouse"].create(
+            {"name": __name__, "code": "_WH2"}
+        )
+        cls.warehouse.resupply_wh_ids = cls.warehouse2
+        route = cls.env["stock.route"].search(
+            [("supplied_wh_id", "=", cls.warehouse.id)]
+        )
+        assert route
+        cls.product.route_ids = route
         cls.lot = cls.env["stock.lot"].create(
             {
                 "name": "lot1",
@@ -20,12 +35,56 @@ class TestRestrictLot(TransactionCase):
             }
         )
+    def _create_move_with_lot(self):
+        move = self.env["stock.move"].create(
+            {
+                "product_id": self.product.id,
+                "location_id": self.stock_loc.id,
+                "location_dest_id": self.output_loc.id,
+                "product_uom_qty": 1,
+                "product_uom": self.product.uom_id.id,
+                "name": "test",
+                "procure_method": "make_to_order",
+                "warehouse_id": self.warehouse.id,
+                "route_ids": [(6, 0, self.warehouse.delivery_route_id.ids)],
+                "restrict_lot_id": self.lot.id,
+            }
+        )
+        move._action_confirm()
+        move._action_assign()
+        new_lot = self.env["stock.lot"].create(
+            {
+                "name": "lot2",
+                "product_id": self.product.id,
+                "company_id": self.warehouse.company_id.id,
+            }
+        )
+        return move, new_lot
+    def _create_move_dest(self):
+        return self.env["stock.move"].create(
+            {
+                "product_id": self.product.id,
+                "location_id": self.customer_loc.id,
+                "product_uom_qty": 1,
+                "product_uom": self.product.uom_id.id,
+                "picking_type_id": self.warehouse.out_type_id.id,
+                "location_dest_id": self.output_loc.id,
+                "name": "test",
+                "procure_method": "make_to_order",
+                "warehouse_id": self.warehouse.id,
+                "route_ids": [(6, 0, self.warehouse.delivery_route_id.ids)],
+                "state": "waiting",
+                "restrict_lot_id": self.lot.id,
+            }
+        )
     def test_00_move_restrict_lot_propagation(self):
         move = self.env["stock.move"].create(
             {
                 "product_id": self.product.id,
-                "location_id": self.output_loc.id,
-                "location_dest_id": self.customer_loc.id,
+                "location_id": self.stock_loc.id,
+                "location_dest_id": self.output_loc.id,
                 "product_uom_qty": 1,
                 "product_uom": self.product.uom_id.id,
                 "name": "test",
@@ -43,8 +102,8 @@ class TestRestrictLot(TransactionCase):
         move = self.env["stock.move"].create(
             {
                 "product_id": self.product.id,
-                "location_id": self.output_loc.id,
-                "location_dest_id": self.customer_loc.id,
+                "location_id": self.stock_loc.id,
+                "location_dest_id": self.output_loc.id,
                 "product_uom_qty": 2,
                 "product_uom": self.product.uom_id.id,
                 "name": "test",
@@ -114,19 +173,13 @@ class TestRestrictLot(TransactionCase):
                 "group_stock_multi_locations": True,
             }
         )
-        warehouse = self.env["stock.warehouse"].search(
-            [("company_id", "=", self.env.company.id)], limit=1
-        )
-        warehouse.delivery_steps = "pick_ship"
-        self.product.categ_id.route_ids |= self.env["stock.route"].search(
-            [("name", "ilike", "deliver in 2")]
-        )
+        self.product.categ_id.route_ids |= self.warehouse.delivery_route_id
         location_1 = self.env["stock.location"].create(
-            {"name": "loc1", "location_id": warehouse.lot_stock_id.id}
+            {"name": "loc1", "location_id": self.warehouse.lot_stock_id.id}
         )
         location_2 = self.env["stock.location"].create(
-            {"name": "loc2", "location_id": warehouse.lot_stock_id.id}
+            {"name": "loc2", "location_id": self.warehouse.lot_stock_id.id}
         )
         # create goods in stock
@@ -185,9 +238,9 @@ class TestRestrictLot(TransactionCase):
             self.assertEqual(concern_move.product_uom_qty, expect_qty)
         def assert_move_line_per_lot_and_location(
-            moves, expect_lot, expect_from_location, expect_reserved_qty
+            move_lines, expect_lot, expect_from_location, expect_reserved_qty
         ):
-            concern_move_line = moves.filtered(
+            concern_move_line = move_lines.filtered(
                 lambda mov: mov.lot_id == expect_lot
                 and mov.location_id == expect_from_location
             )
@@ -196,23 +249,11 @@ class TestRestrictLot(TransactionCase):
                 concern_move_line.quantity_product_uom, expect_reserved_qty
             )
-        pickings = self.env["stock.picking"].search(
-            [("group_id", "=", procurement_group.id)]
-        )
-        self.assertEqual(len(pickings), 2)
-        delivery = pickings.filtered(
-            lambda pick: pick.picking_type_id.code == "outgoing"
-        )
-        pick = pickings.filtered(lambda pick: pick.picking_type_id.code == "internal")
-        self.assertEqual(delivery.state, "waiting")
-        self.assertEqual(len(delivery.move_ids_without_package), 2)
-        assert_move_qty_per_lot(delivery.move_ids_without_package, self.lot, 15)
-        assert_move_qty_per_lot(delivery.move_ids_without_package, lot2, 30)
-        pick.action_assign()
+        pick = procurement_group.stock_move_ids.picking_id
+        self.assertEqual(len(pick), 1)
+        self.assertEqual(pick.picking_type_id.code, "internal")
         self.assertEqual(pick.state, "assigned")
-        self.assertEqual(len(pick.move_ids_without_package), 2)
+        self.assertEqual(len(pick.move_ids), 2)
         assert_move_qty_per_lot(pick.move_ids_without_package, self.lot, 15)
         assert_move_qty_per_lot(pick.move_ids_without_package, lot2, 30)
         assert_move_line_per_lot_and_location(
@@ -227,17 +268,26 @@ class TestRestrictLot(TransactionCase):
         assert_move_line_per_lot_and_location(
             pick.move_line_ids_without_package, lot2, location_2, 25
         )
+        pick.button_validate()
+        self.assertEqual(pick.state, "done")
+        delivery = procurement_group.stock_move_ids.picking_id - pick
+        self.assertEqual(delivery.picking_type_id.code, "outgoing")
+        self.assertEqual(delivery.state, "assigned")
+        assert_move_qty_per_lot(delivery.move_ids_without_package, self.lot, 15)
+        assert_move_qty_per_lot(delivery.move_ids_without_package, lot2, 30)
     def test_compute_quantites(self):
         move = self.env["stock.move"].create(
             {
                 "product_id": self.product.id,
-                "location_id": self.output_loc.id,
+                "location_id": self.stock_loc.id,
                 "location_dest_id": self.customer_loc.id,
                 "product_uom_qty": 1,
                 "product_uom": self.product.uom_id.id,
                 "name": "test",
-                "procure_method": "make_to_order",
+                "procure_method": "make_to_stock",
                 "warehouse_id": self.warehouse.id,
                 "route_ids": [(6, 0, self.warehouse.delivery_route_id.ids)],
                 "restrict_lot_id": self.lot.id,
@@ -252,6 +302,7 @@ class TestRestrictLot(TransactionCase):
         product = move.product_id
         self.assertEqual(product.outgoing_qty, 2)
         product.invalidate_recordset()
         product = product.with_context(lot_id=self.lot.id)
         self.assertEqual(product.outgoing_qty, 1)
@@ -301,3 +352,41 @@ class TestRestrictLot(TransactionCase):
         self.assertRaises(UserError, move._action_done)
         move_line.lot_id = self.lot
         move._action_done()
+    def test_restrict_lot_propagation_dest_moves(self):
+        move, new_lot = self._create_move_with_lot()
+        move_dest = self._create_move_dest()
+        move.move_dest_ids = [(4, move_dest.id)]
+        self.assertEqual(move_dest.restrict_lot_id, self.lot)
+        move.restrict_lot_id = new_lot.id
+        self.assertEqual(move_dest.restrict_lot_id, new_lot)
+    def test_restrict_lot_propagation_origin_moves(self):
+        move, new_lot = self._create_move_with_lot()
+        orig_move = move.move_orig_ids
+        self.assertEqual(orig_move.restrict_lot_id, self.lot)
+        move.restrict_lot_id = new_lot.id
+        self.assertEqual(orig_move.restrict_lot_id, new_lot)
+    def test_restrict_lot_propagation_origin_and_dest_moves(self):
+        move, new_lot = self._create_move_with_lot()
+        move_dest = self._create_move_dest()
+        move.move_dest_ids = [(4, move_dest.id)]
+        orig_move = move.move_orig_ids
+        self.assertEqual(move_dest.restrict_lot_id, self.lot)
+        self.assertEqual(orig_move.restrict_lot_id, self.lot)
+        move.restrict_lot_id = new_lot.id
+        self.assertEqual(orig_move.restrict_lot_id, new_lot)
+        self.assertEqual(move_dest.restrict_lot_id, new_lot)
+    def test_restrict_lot_no_propagation_error(self):
+        move, new_lot = self._create_move_with_lot()
+        orig_move = move.move_orig_ids
+        orig_move._action_assign()
+        orig_move.quantity = orig_move.product_uom_qty
+        orig_move.picked = True
+        orig_move._action_done()
+        self.assertEqual(orig_move.state, "done")
+        with self.assertRaises(ValidationError) as m:
+            move.restrict_lot_id = new_lot.id
+        self.assertIn("You can't modify the Lot/Serial number", m.exception.args[0])

odoo/addons/stock_restrict_lot/views/stock_move_views.xml CHANGED Viewed

@@ -1,34 +1,32 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-        <record id="stock_move_pick_tree_view" model="ir.ui.view">
-            <field name="model">stock.move</field>
-            <field name="inherit_id" ref="stock.view_picking_move_tree" />
-            <field name="arch" type="xml">
-                <field name="product_id" position="after">
-                    <field
+    <record id="stock_move_pick_tree_view" model="ir.ui.view">
+        <field name="model">stock.move</field>
+        <field name="inherit_id" ref="stock.view_picking_move_tree" />
+        <field name="arch" type="xml">
+            <field name="product_id" position="after">
+                <field
                     name="restrict_lot_id"
                     readonly="1"
                     optional="show"
                     groups="stock.group_production_lot"
                 />
-                </field>
             </field>
-        </record>
+        </field>
+    </record>
-        <record id="stock_move_tree_view" model="ir.ui.view">
-            <field name="model">stock.move</field>
-            <field name="inherit_id" ref="stock.view_move_tree" />
-            <field name="arch" type="xml">
-                <field name="product_id" position="after">
-                    <field
+    <record id="stock_move_tree_view" model="ir.ui.view">
+        <field name="model">stock.move</field>
+        <field name="inherit_id" ref="stock.view_move_tree" />
+        <field name="arch" type="xml">
+            <field name="product_id" position="after">
+                <field
                     name="restrict_lot_id"
                     readonly="1"
                     optional="show"
                     groups="stock.group_production_lot"
                 />
-                </field>
             </field>
-        </record>
+        </field>
+    </record>
 </odoo>

odoo/addons/stock_restrict_lot/views/stock_picking.xml CHANGED Viewed

@@ -1,36 +1,34 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <odoo>
-        <record id="custom_main_stock_picking_form_view" model="ir.ui.view">
-            <field name="model">stock.picking</field>
-            <field name="inherit_id" ref="stock.view_picking_form" />
-            <field name="arch" type="xml">
-                <xpath
-                expr="//field[@name='move_ids_without_package']/tree/field[@name='product_id']"
+    <record id="custom_main_stock_picking_form_view" model="ir.ui.view">
+        <field name="model">stock.picking</field>
+        <field name="inherit_id" ref="stock.view_picking_form" />
+        <field name="arch" type="xml">
+            <xpath
+                expr="//field[@name='move_ids_without_package']/list/field[@name='product_id']"
                 position="after"
             >
-                    <field
+                <field
                     name="restrict_lot_id"
                     readonly="1"
                     optional="show"
                     groups="stock.group_production_lot"
                 />
-                </xpath>
-            </field>
-        </record>
+            </xpath>
+        </field>
+    </record>
-        <record id="custom_stock_picking_search_view" model="ir.ui.view">
-            <field name="model">stock.picking</field>
-            <field name="inherit_id" ref="stock.view_picking_internal_search" />
-            <field name="arch" type="xml">
-                <field name="product_id" position="after">
-                    <field
+    <record id="custom_stock_picking_search_view" model="ir.ui.view">
+        <field name="model">stock.picking</field>
+        <field name="inherit_id" ref="stock.view_picking_internal_search" />
+        <field name="arch" type="xml">
+            <field name="product_id" position="after">
+                <field
                     name="restrict_lot_id"
                     string="Lot"
                     groups="stock.group_production_lot"
                 />
-                </field>
             </field>
-        </record>
+        </field>
+    </record>
 </odoo>

{odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info → odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info}/METADATA RENAMED Viewed

@@ -1,8 +1,8 @@
 Metadata-Version: 2.1
 Name: odoo-addon-stock_restrict_lot
-Version: 17.0.1.2.0
+Version: 18.0.1.0.0.2
 Requires-Python: >=3.10
-Requires-Dist: odoo>=17.0a,<17.1dev
+Requires-Dist: odoo==18.0.*
 Summary: Base module that add back the concept of restrict lot on stock move
 Home-page: https://github.com/OCA/stock-logistics-workflow
 License: LGPL-3
@@ -10,7 +10,7 @@ Author: Akretion, Odoo Community Association (OCA)
 Author-email: support@odoo-community.org
 Classifier: Programming Language :: Python
 Classifier: Framework :: Odoo
-Classifier: Framework :: Odoo :: 17.0
+Classifier: Framework :: Odoo :: 18.0
 Classifier: License :: OSI Approved :: GNU Lesser General Public License v3 (LGPLv3)
 ==================
@@ -22,7 +22,7 @@ Stock Restrict Lot
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:00ff8e2abcdd536138a0bf51463e00f076ac3fe94947998cda2b6792ef723d5b
+   !! source digest: sha256:dce023da3482003599a49ae27980bca635a30f0b86afd9ee1bcbeb8d3ae775d9
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
@@ -32,21 +32,22 @@ Stock Restrict Lot
     :target: http://www.gnu.org/licenses/lgpl-3.0-standalone.html
     :alt: License: LGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--workflow-lightgray.png?logo=github
-    :target: https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot
+    :target: https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot
     :alt: OCA/stock-logistics-workflow
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/stock-logistics-workflow-17-0/stock-logistics-workflow-17-0-stock_restrict_lot
+    :target: https://translation.odoo-community.org/projects/stock-logistics-workflow-18-0/stock-logistics-workflow-18-0-stock_restrict_lot
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runboat-Try%20me-875A7B.png
-    :target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-workflow&target_branch=17.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-workflow&target_branch=18.0
     :alt: Try me on Runboat
 |badge1| |badge2| |badge3| |badge4| |badge5|
-This module add a field to restrict a stock move to a specific lot. It
+This module adds a field to restrict a stock move to a specific lot. It
 propagates it between chained moves. A move with a restrict lot will
 only be able to reserve or transfer products with the specified lot.
-This module is a based for other modules, it has not effect on its own.
+This module serves as a basis for other modules, it has not effect on
+its own.
 **Table of contents**
@@ -59,7 +60,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/stock-logistics-workflow/issues>`_.
 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
-`feedback <https://github.com/OCA/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2017.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/stock-logistics-workflow/issues/new?body=module:%20stock_restrict_lot%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
 Do not contact contributors directly about support or help with technical issues.
@@ -76,6 +77,7 @@ Contributors
 - Florian da Costa <florian.dacosta@akretion.com>
 - Ruchir Shukla <ruchir@bizzappdev.com>
+- Stefan Rijnhart <stefan@opener.amsterdam>
 Maintainers
 -----------
@@ -98,6 +100,6 @@ Current `maintainer <https://odoo-community.org/page/maintainer-role>`__:
 |maintainer-florian-dacosta|
-This module is part of the `OCA/stock-logistics-workflow <https://github.com/OCA/stock-logistics-workflow/tree/17.0/stock_restrict_lot>`_ project on GitHub.
+This module is part of the `OCA/stock-logistics-workflow <https://github.com/OCA/stock-logistics-workflow/tree/18.0/stock_restrict_lot>`_ project on GitHub.
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info/RECORD ADDED Viewed

@@ -0,0 +1,24 @@
+odoo/addons/stock_restrict_lot/README.rst,sha256=L2SAcstft-Wa3ddB34vTd_FMymPvLd-4613euD878Io,3546
+odoo/addons/stock_restrict_lot/__init__.py,sha256=X9EJGOE2GtZbS0G82PtSXmWSZ_R8jEM0rlJTDliQjp4,21
+odoo/addons/stock_restrict_lot/__manifest__.py,sha256=6oKtIpnmhsXQrb7b20kBYGwETT876fKB06zpShsVghA,576
+odoo/addons/stock_restrict_lot/i18n/es.po,sha256=UKHL4vKheMhf0x2mZ_KwUmtjbwiCXnt-UJMeM82FwUs,2068
+odoo/addons/stock_restrict_lot/i18n/hr.po,sha256=YTZunKeyiqSlJhW2cMSi2dQ8jzrrFXk1gDCyPvl847Y,2111
+odoo/addons/stock_restrict_lot/i18n/it.po,sha256=p1GuTRBBNwL-2A5I00vhVf_9vekVVFpMBTOSishnong,2214
+odoo/addons/stock_restrict_lot/i18n/stock_restrict_lot.pot,sha256=nvOiQgDpfOhBpgOFZVbVfgjs12txTi-2BjnxUph6s_k,1954
+odoo/addons/stock_restrict_lot/models/__init__.py,sha256=3nhRO6ipFBMbcrlWXMEMR62djB3_7q2TxcUM42xYMGg,108
+odoo/addons/stock_restrict_lot/models/product_product.py,sha256=sv-ZLxXqmf6UX4MII1lX7s1BsRgyiKOD_wDpo993KmU,888
+odoo/addons/stock_restrict_lot/models/stock_move.py,sha256=lXtBPTeuVdu1WYWUuIONn7iCWk5FZtcuzYP9T594neo,5802
+odoo/addons/stock_restrict_lot/models/stock_picking.py,sha256=0l60llnOb3tnmk-3pNF21kP1aR8vO0qu5BbbeqD3T6A,417
+odoo/addons/stock_restrict_lot/models/stock_rule.py,sha256=1oOOLWBEC-inoLWHqhDxxH0Q9XXsInasxbIMLmYjQ5E,547
+odoo/addons/stock_restrict_lot/readme/CONTRIBUTORS.md,sha256=Yzf9SONTbOQxjFZA7UFGibcNzh3pOkl5ZkBgHifmQvc,146
+odoo/addons/stock_restrict_lot/readme/DESCRIPTION.md,sha256=jYTY5vmrg7LmJCnnJ2qwOpIVuvEaOepvYGAFy0vdlBA,289
+odoo/addons/stock_restrict_lot/static/description/icon.png,sha256=6xBPJauaFOF0KDHfHgQopSc28kKvxMaeoQFQWZtfZDo,9455
+odoo/addons/stock_restrict_lot/static/description/index.html,sha256=21V4T7wSjIshHFjANDH5MbOiozLaaoYXX329MjvfICo,13078
+odoo/addons/stock_restrict_lot/tests/__init__.py,sha256=5RimjaqBd8kD55SNoLqh-g9sCmmI28hPx7IQMUX6AOQ,32
+odoo/addons/stock_restrict_lot/tests/test_restrict_lot.py,sha256=k-tOml6ti2jDKBh3SLJ-Ow-fKoHTTPNplSjWqa-b4Ws,15501
+odoo/addons/stock_restrict_lot/views/stock_move_views.xml,sha256=w4zzez4f9fK-drmjnUh61ZZP4vyW_vM4ki4XcHPnFEA,1117
+odoo/addons/stock_restrict_lot/views/stock_picking.xml,sha256=s6ONtCnjx151Ob_BOyQ4BvkTs98xqcv7QoysZb8WQ3c,1225
+odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info/METADATA,sha256=BdWJg8pW9LmflQoEH28qw6L56yoiOEBMMtygdWXHi2k,4119
+odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info/WHEEL,sha256=9fEMia4zL7ZuZbnCOrcYogUhmn4XFIVaJ8G4YGI31xc,81
+odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info/top_level.txt,sha256=QE6RBQ0QX5f4eFuUcGgU5Kbq1A_qJcDs-e_vpr6pmfU,4
+odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info/RECORD,,

odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info/RECORD DELETED Viewed

@@ -1,24 +0,0 @@
-odoo/addons/stock_restrict_lot/README.rst,sha256=EFOZYRoPNMQz3M1wyDoD_ETAA3e3rLZ8MR623sh2mco,3494
-odoo/addons/stock_restrict_lot/__init__.py,sha256=X9EJGOE2GtZbS0G82PtSXmWSZ_R8jEM0rlJTDliQjp4,21
-odoo/addons/stock_restrict_lot/__manifest__.py,sha256=XJabscO10DZ25wSD43kOc_qJK3_7HfgE3ewB-kTBx4I,506
-odoo/addons/stock_restrict_lot/i18n/es.po,sha256=UKHL4vKheMhf0x2mZ_KwUmtjbwiCXnt-UJMeM82FwUs,2068
-odoo/addons/stock_restrict_lot/i18n/hr.po,sha256=YTZunKeyiqSlJhW2cMSi2dQ8jzrrFXk1gDCyPvl847Y,2111
-odoo/addons/stock_restrict_lot/i18n/it.po,sha256=p1GuTRBBNwL-2A5I00vhVf_9vekVVFpMBTOSishnong,2214
-odoo/addons/stock_restrict_lot/i18n/stock_restrict_lot.pot,sha256=AhJ4X9RXaEtapVvWPhQk3dHKAIADrVe5fT6Z8mBkpGY,1729
-odoo/addons/stock_restrict_lot/models/__init__.py,sha256=3nhRO6ipFBMbcrlWXMEMR62djB3_7q2TxcUM42xYMGg,108
-odoo/addons/stock_restrict_lot/models/product_product.py,sha256=sv-ZLxXqmf6UX4MII1lX7s1BsRgyiKOD_wDpo993KmU,888
-odoo/addons/stock_restrict_lot/models/stock_move.py,sha256=stXDy4srK6L2qTNQ6bHfoO24v3ZdmgGCJeq8RBvKxi8,4047
-odoo/addons/stock_restrict_lot/models/stock_picking.py,sha256=TrMC6cSDlvCNMyRE1fRzFVbA7wYxhdTUbUBpqqLF21E,417
-odoo/addons/stock_restrict_lot/models/stock_rule.py,sha256=NXXjB8rooKS0kpG3zvqpDDpLkmh7mvbnHLItDwdCec0,236
-odoo/addons/stock_restrict_lot/readme/CONTRIBUTORS.md,sha256=JBjjiC74D-iWs-cRLAY7BTnLuOA83Nh0fCtABQTadHs,98
-odoo/addons/stock_restrict_lot/readme/DESCRIPTION.md,sha256=5PwOcvRp7HF_bK6A2P20xcbXvvFZaCj6BYY7moybpO4,281
-odoo/addons/stock_restrict_lot/static/description/icon.png,sha256=6xBPJauaFOF0KDHfHgQopSc28kKvxMaeoQFQWZtfZDo,9455
-odoo/addons/stock_restrict_lot/static/description/index.html,sha256=YCf1HeHfrPlCPDW0kMqpLY-MTN7X9VJF__YLho_-Dbg,12933
-odoo/addons/stock_restrict_lot/tests/__init__.py,sha256=5RimjaqBd8kD55SNoLqh-g9sCmmI28hPx7IQMUX6AOQ,32
-odoo/addons/stock_restrict_lot/tests/test_restrict_lot.py,sha256=k2dsPmHa3IctRAr46SUpsBwhH9HS9lxZU-uD8hcLc98,11840
-odoo/addons/stock_restrict_lot/views/stock_move_views.xml,sha256=unNYn0AuYyBhxSnnPN6xf6X7J7S-G1KbXJVYF20scA4,1191
-odoo/addons/stock_restrict_lot/views/stock_picking.xml,sha256=X1N9S5g09NREC_DJAlM1CjZuV21nJgBxTANn_6kC64o,1299
-odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info/METADATA,sha256=yU1sCdCfXEnXKr4-291PcxfWHBBRtx6_vOSDSui1j2k,4073
-odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info/WHEEL,sha256=9fEMia4zL7ZuZbnCOrcYogUhmn4XFIVaJ8G4YGI31xc,81
-odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info/top_level.txt,sha256=QE6RBQ0QX5f4eFuUcGgU5Kbq1A_qJcDs-e_vpr6pmfU,4
-odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info/RECORD,,

{odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info → odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info}/WHEEL RENAMED Viewed

File without changes

{odoo_addon_stock_restrict_lot-17.0.1.2.0.dist-info → odoo_addon_stock_restrict_lot-18.0.1.0.0.2.dist-info}/top_level.txt RENAMED Viewed

File without changes

odoo-addon-stock-restrict-lot 17.0.1.2.0__py3-none-any.whl → 18.0.1.0.0.2__py3-none-any.whl

Potentially problematic release.

odoo-addon-stock-restrict-lot 17.0.1.2.0py3-none-any.whl → 18.0.1.0.0.2py3-none-any.whl