odoo-addon-shopfloor 18.0.0.1.2__py3-none-any.whl → 18.0.0.2.0__py3-none-any.whl

odoo/addons/shopfloor/README.rst

@@ -11,7 +11,7 @@ Shopfloor
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:01ad1a242b0c8a3b6ab8593cd6dedb31c5b6dd941b58e9af751833dad9cc69f3
+   !! source digest: sha256:534d443b8feb0fd2548ede9663e260ceb8dcfb20686a39d06022ee92bcc6d209
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png

odoo/addons/shopfloor/__manifest__.py

@@ -6,7 +6,7 @@
 {
     "name": "Shopfloor",
     "summary": "manage warehouse operations with barcode scanners",
-    "version": "18.0.0.1.2",
+    "version": "18.0.0.2.0",
     "development_status": "Beta",
     "category": "Inventory",
     "website": "https://github.com/OCA/stock-logistics-shopfloor",

odoo/addons/shopfloor/actions/message.py

@@ -742,7 +742,7 @@ class MessageAction(Component):
             "body": _("The picked quantity must be a value above zero."),
         }
 
-    def selected_lines_qty_done_higher_than_allowed(self, line):
+    def selected_lines_qty_picked_higher_than_allowed(self, line):
         """
         :param line: The selected line
         """
@@ -753,12 +753,12 @@ class MessageAction(Component):
                 "higher than the maximum allowed. "
                 "(%(product_name)s : %(quantity_done)s > %(quantity_reserved)s)",
                 product_name=line.product_id.name,
-                quantity_done=str(line.qty_done),
+                quantity_done=str(line.qty_picked),
                 quantity_reserved=str(line.quantity),
             ),
         }
 
-    def line_scanned_qty_done_higher_than_allowed(self):
+    def line_scanned_qty_picked_higher_than_allowed(self):
         return {
             "message_type": "warning",
             "body": _(

odoo/addons/shopfloor/models/stock_move_line.py

@@ -117,13 +117,13 @@ class StockMoveLine(models.Model):
         elif qty_lesser:
             if not split_partial:
                 return (new_line, "lesser")
-            new_line = self._split_partial_quantity_to_be_done(
+            new_line = self._split_partial_quantity_to_be_picked(
                 qty_done, split_default_vals
             )
             return (new_line, "lesser")
         return (new_line, "full")
 
-    def _split_partial_quantity_to_be_done(self, quantity_done, split_default_vals):
+    def _split_partial_quantity_to_be_picked(self, quantity_done, split_default_vals):
         """Create a new move line with the remaining quantity to process."""
         # split the move line which will be processed later (maybe the user
         # has to pick some goods from another place because the location

odoo/addons/shopfloor/models/stock_picking.py

@@ -122,11 +122,8 @@ class StockPicking(models.Model):
         return new_picking.id
 
     def _put_in_pack(self, move_line_ids):
-        """
-        Marks the corresponding move lines as 'shopfloor_checkout_done'
-        when the package is created in the backend.
-
-        """
+        # Marks the corresponding move lines as 'shopfloor_checkout_done'
+        # when the package is created in the backend.
         new_package = super()._put_in_pack(move_line_ids)
         lines = move_line_ids.filtered(lambda p: p.result_package_id == new_package)
         lines.write({"shopfloor_checkout_done": True})

odoo/addons/shopfloor/services/checkout.py

@@ -426,9 +426,9 @@ class Checkout(Component):
                 continue
             if self.work.menu.no_prefill_qty and i == 0:
                 # For prefill quantity we only want to increment one line
-                line.qty_done += prefill_qty
+                line.qty_picked += prefill_qty
             elif not self.work.menu.no_prefill_qty:
-                line.qty_done = line.quantity
+                line.picked = True
             line.shopfloor_user_id = self.env.user
 
         picking = lines.mapped("picking_id")
@@ -440,7 +440,7 @@ class Checkout(Component):
 
     def _deselect_lines(self, lines):
         lines.filtered(lambda x: not x.shopfloor_checkout_done).write(
-            {"qty_done": 0, "shopfloor_user_id": False}
+            {"qty_picked": 0, "shopfloor_user_id": False}
         )
 
     def scan_line(self, picking_id, barcode, confirm_pack_all=False, confirm_lot=None):
@@ -454,7 +454,7 @@ class Checkout(Component):
         several packs contain it, the endpoint will ask to scan a pack; if the
         product is tracked by lot, to scan a lot.
 
-        Once move lines are found, their ``qty_done`` is set to their reserved
+        Once move lines are found, their ``qty_picked`` is set to their reserved
         quantity.
 
         Transitions:
@@ -828,28 +828,28 @@ class Checkout(Component):
         if not move_lines:
             message = self.msg_store.record_not_found()
         for move_line in move_lines:
-            qty_done = quantity_func(move_line)
-            if qty_done < 0:
+            qty_picked = quantity_func(move_line)
+            if qty_picked < 0:
                 message = {
                     "body": _("Negative quantity not allowed."),
                     "message_type": "error",
                 }
             else:
                 new_line = self.env["stock.move.line"]
-                if qty_done > 0:
+                if qty_picked > 0:
                     new_line, qty_check = move_line._split_qty_to_be_done(
-                        qty_done,
+                        qty_picked,
                         split_partial=False,
                         result_package_id=False,
                     )
-                move_line.qty_done = qty_done
+                move_line.qty_picked = qty_picked
                 if new_line:
                     selected_line_ids.append(new_line.id)
-                if qty_done > move_line.quantity:
+                if qty_picked > move_line.quantity:
                     return self._response_for_select_package(
                         picking,
                         self.env["stock.move.line"].browse(selected_line_ids).exists(),
-                        message=self.msg_store.line_scanned_qty_done_higher_than_allowed(),
+                        message=self.msg_store.line_scanned_qty_picked_higher_than_allowed(),
                     )
         return self._response_for_select_package(
             picking,
@@ -858,7 +858,7 @@ class Checkout(Component):
         )
 
     def reset_line_qty(self, picking_id, selected_line_ids, move_line_id):
-        """Reset qty_done of a move line to zero
+        """Reset picked quantity of a move line to zero
 
         Used to deselect a line in the "select_package" screen.
         The selected_line_ids parameter is used to keep the selection of lines
@@ -873,7 +873,7 @@ class Checkout(Component):
         )
 
     def set_line_qty(self, picking_id, selected_line_ids, move_line_id):
-        """Set qty_done of a move line to its reserved quantity
+        """Set picked quantity of a move line to its reserved quantity
 
         Used to select a line in the "select_package" screen.
         The selected_line_ids parameter is used to keep the selection of lines
@@ -888,7 +888,7 @@ class Checkout(Component):
         )
 
     def set_custom_qty(self, picking_id, selected_line_ids, move_line_id, qty_done):
-        """Change qty_done of a move line with a custom value
+        """Change picked quantity of a move line with a custom value
 
         The selected_line_ids parameter is used to keep the selection of lines
         stateless.
@@ -901,14 +901,14 @@ class Checkout(Component):
             picking_id, selected_line_ids, [move_line_id], lambda __: qty_done
         )
 
-    def _switch_line_qty_done(self, picking, selected_lines, switch_lines):
-        """Switch qty_done on lines and return to the 'select_package' state
+    def _switch_line_qty_picked(self, picking, selected_lines, switch_lines):
+        """Switch picked qty on lines and return to the 'select_package' state
 
-        If at least one of the lines to switch has a qty_done, set them all
-        to zero. If all the lines to switch have a zero qty_done, switch them
+        If at least one of the lines to switch has a picked qty, set them all
+        to zero. If all the lines to switch have a zero picked qty, switch them
         to their quantity to deliver.
         """
-        if any(line.qty_done for line in switch_lines):
+        if any(line.qty_picked for line in switch_lines):
             return self._change_line_qty(
                 picking.id, selected_lines.ids, switch_lines.ids, lambda __: 0
             )
@@ -923,7 +923,7 @@ class Checkout(Component):
     def _increment_custom_qty(
         self, picking, selected_lines, increment_lines, qty_increment
     ):
-        """Increment the  qty_done of a move line with a custom value
+        """Increment the picked quantity of a move line with a custom value
 
         The selected_line parameter is used to keep the selection of lines
         stateless.
@@ -936,22 +936,31 @@ class Checkout(Component):
             picking.id,
             selected_lines.ids,
             increment_lines.ids,
-            lambda line: line.qty_done + qty_increment,
+            lambda line: line.qty_picked + qty_increment,
         )
 
     @staticmethod
     def _filter_lines_unpacked(move_line):
         return (
-            move_line.qty_done == 0 or move_line.shopfloor_user_id
+            not move_line.picked or move_line.shopfloor_user_id
         ) and not move_line.shopfloor_checkout_done
 
     @staticmethod
     def _filter_lines_to_pack(move_line):
-        return move_line.qty_done > 0 and not move_line.shopfloor_checkout_done
+        return move_line.picked and not move_line.shopfloor_checkout_done
 
     @staticmethod
     def _filter_lines_checkout_done(move_line):
-        return move_line.qty_done > 0 and move_line.shopfloor_checkout_done
+        # Starting from Odoo 18+, a move line part of a package level is not
+        # flagged as picked anymore (it'll be automatically at validation however).
+        # See '<stock.picking>._put_in_pack()' changes from
+        # https://github.com/odoo/odoo/commit/7dda6bb92715e
+        #
+        # Based on this new Odoo std behavior, we now consider a move line within
+        # a package level as picked (checkout is done).
+        return (
+            move_line.picked or move_line.package_level_id.is_done
+        ) and move_line.shopfloor_checkout_done
 
     def _is_package_allowed(self, picking, package):
         """Check if a package is allowed as a destination/delivery package.
@@ -966,7 +975,7 @@ class Checkout(Component):
         return package in existing_packages
 
     def _put_lines_in_package(self, picking, selected_lines, package):
-        """Put the current selected lines with a qty_done in a package
+        """Put the current selected lines with a picked quantity in a package
 
         Note: only packages which are already a delivery package for another
         line of the stock picking can be selected. Packages which are the
@@ -983,8 +992,8 @@ class Checkout(Component):
 
     def _put_lines_in_allowed_package(self, picking, lines_to_pack, package):
         for line in lines_to_pack:
-            if line.qty_done < line.quantity:
-                line._split_partial_quantity_to_be_done(line.qty_done, {})
+            if line.qty_picked < line.quantity:
+                line._split_partial_quantity_to_be_picked(line.qty_picked, {})
         lines_to_pack.write(
             {"result_package_id": package.id, "shopfloor_checkout_done": True}
         )
@@ -1032,8 +1041,8 @@ class Checkout(Component):
         of a selected line, the package is set to be the destination package of
         all the lines to pack.
 
-        When a product is scanned, it selects (set qty_done = reserved qty) or
-        deselects (set qty_done = 0) the move lines for this product. Only
+        When a product is scanned, it selects (set qty_picked = reserved qty) or
+        deselects (set qty_picked = 0) the move lines for this product. Only
         products not tracked by lot can use this.
 
         When a lot is scanned, it does the same as for the products but based
@@ -1043,7 +1052,7 @@ class Checkout(Component):
         package is created and set as destination of the lines to pack.
 
         Lines to pack are move lines in the list of ``selected_line_ids``
-        where ``qty_done`` > 0 and have not been packed yet
+        where ``picked`` is set and have not been packed yet
         (``shopfloor_checkout_done is False``).
 
         Transitions:
@@ -1104,7 +1113,7 @@ class Checkout(Component):
         If none are found, return the first line for that product.
         """
         return next(
-            (line for line in product_lines if line.qty_done < line.quantity),
+            (line for line in product_lines if line.qty_picked < line.quantity),
             fields.first(product_lines),
         )
 
@@ -1126,7 +1135,7 @@ class Checkout(Component):
                 self._find_line_to_increment(product_lines),
                 quantity_increment,
             )
-        return self._switch_line_qty_done(picking, selected_lines, product_lines)
+        return self._switch_line_qty_picked(picking, selected_lines, product_lines)
 
     def _scan_package_action_from_packaging(
         self, picking, selected_lines, packaging, **kw
@@ -1141,7 +1150,7 @@ class Checkout(Component):
             return self._increment_custom_qty(
                 picking, selected_lines, self._find_line_to_increment(lot_lines), 1
             )
-        return self._switch_line_qty_done(picking, selected_lines, lot_lines)
+        return self._switch_line_qty_picked(picking, selected_lines, lot_lines)
 
     def _scan_package_action_from_serial(self, picking, selection_lines, lot, **kw):
         # Search serial number is actually the same as searching for lot (as of v14...)
@@ -1221,7 +1230,7 @@ class Checkout(Component):
                 selected_lines,
                 message=self.msg_store.no_delivery_packaging_available(),
             )
-        response = self._check_allowed_qty_done(picking, selected_lines)
+        response = self._check_allowed_qty_picked(picking, selected_lines)
         if response:
             return response
         return self._response_for_select_delivery_packaging(picking, delivery_packaging)
@@ -1233,8 +1242,8 @@ class Checkout(Component):
         the selected lines.
 
         Selected lines are move lines in the list of ``move_line_ids`` where
-        ``qty_done`` > 0 and have no destination package
-        (shopfloor_checkout_done is False).
+        ``picked`` is set and have no destination package
+        (``shopfloor_checkout_done`` is not set).
 
         Transitions:
         * select_line: goes back to selection of lines to work on next lines
@@ -1253,8 +1262,8 @@ class Checkout(Component):
         """Process all selected lines without any package.
 
         Selected lines are move lines in the list of ``move_line_ids`` where
-        ``qty_done`` > 0 and have no destination package
-        (shopfloor_checkout_done is False).
+        ``picked`` is set and have no destination package
+        (``shopfloor_checkout_done`` is False).
 
         Transitions:
         * select_line: goes back to selection of lines to work on next lines
@@ -1266,13 +1275,11 @@ class Checkout(Component):
         if message:
             return self._response_for_select_document(message=message)
         selected_lines = self.env["stock.move.line"].browse(selected_line_ids).exists()
-        selected_lines_with_qty_done = selected_lines.filtered(
-            lambda line: line.qty_done > 0
-        )
-        selected_lines_with_qty_done.write(
+        picked_selected_lines = selected_lines.filtered(lambda line: line.picked)
+        picked_selected_lines.write(
             {"shopfloor_checkout_done": True, "result_package_id": False}
         )
-        response = self._check_allowed_qty_done(picking, selected_lines)
+        response = self._check_allowed_qty_picked(picking, selected_lines)
         if response:
             return response
         return self._response_for_select_line(
@@ -1298,21 +1305,21 @@ class Checkout(Component):
         if message:
             return self._response_for_select_document(message=message)
         lines = self.env["stock.move.line"].browse(selected_line_ids).exists()
-        response = self._check_allowed_qty_done(picking, lines)
+        response = self._check_allowed_qty_picked(picking, lines)
         if response:
             return response
         return self._response_for_select_dest_package(picking, lines)
 
-    def _check_allowed_qty_done(self, picking, lines):
+    def _check_allowed_qty_picked(self, picking, lines):
         for line in lines:
-            # Do not allow to proceed if the qty_done of
+            # Do not allow to proceed if the qty_picked of
             # any of the selected lines
             # is higher than the quantity to do.
-            if line.qty_done > line.quantity:
+            if line.qty_picked > line.quantity:
                 return self._response_for_select_package(
                     picking,
                     lines,
-                    message=self.msg_store.selected_lines_qty_done_higher_than_allowed(
+                    message=self.msg_store.selected_lines_qty_picked_higher_than_allowed(
                         line
                     ),
                 )
@@ -1329,7 +1336,7 @@ class Checkout(Component):
     def scan_dest_package(self, picking_id, selected_line_ids, barcode):
         """Scan destination package for lines
 
-        Set the destination package on the selected lines with a `qty_done` if
+        Set the destination package on the `picked` selected lines if
         the package is valid. It is valid when one of:
 
         * it is already the destination package of another line of the stock.picking
@@ -1459,7 +1466,7 @@ class Checkout(Component):
 
         All the move lines with the package as ``result_package_id`` have their
         ``result_package_id`` reset to the source package (default odoo behavior)
-        and their ``qty_done`` set to 0.
+        and their ``qty_picked`` set to 0.
 
         It flags ``shopfloor_checkout_done`` to False
         so they have to be processed again.
@@ -1488,14 +1495,17 @@ class Checkout(Component):
             for move_line in move_lines:
                 move_line.write(
                     {
-                        "qty_done": 0,
+                        "qty_picked": 0,
+                        "picked": False,
                         "result_package_id": move_line.package_id,
                         "shopfloor_checkout_done": False,
                     }
                 )
             msg = _("Package cancelled")
         if line:
-            line.write({"qty_done": 0, "shopfloor_checkout_done": False})
+            line.write(
+                {"qty_picked": 0, "picked": False, "shopfloor_checkout_done": False}
+            )
             msg = _("Line cancelled")
         return self._response_for_select_line(
             picking, message={"message_type": "success", "body": msg}
@@ -1504,7 +1514,7 @@ class Checkout(Component):
     def done(self, picking_id, confirmation=False):
         """Set the moves as done
 
-        If some lines have not the full ``qty_done`` or no destination package set,
+        If some lines are not fully ``picked`` or no destination package set,
         a confirmation is asked to the user.
 
         Transitions:
@@ -1519,7 +1529,7 @@ class Checkout(Component):
             return self._response_for_select_document(message=message)
         lines = picking.move_line_ids
         if not confirmation:
-            if not all(line.qty_done == line.quantity for line in lines):
+            if not all(line.qty_picked == line.quantity for line in lines):
                 return self._response_for_summary(
                     picking,
                     need_confirm=True,

odoo/addons/shopfloor/services/cluster_picking.py

@@ -4,8 +4,9 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 from odoo import _, fields
 from odoo.osv import expression
+from odoo.tools import str2bool
 
-from odoo.addons.base_rest.components.service import to_bool, to_int
+from odoo.addons.base_rest.components.service import to_int
 from odoo.addons.component.core import Component
 
 from ..utils import to_float
@@ -1385,7 +1386,7 @@ class ShopfloorClusterPickingValidator(Component):
         return {
             "picking_batch_id": {"coerce": to_int, "required": True, "type": "integer"},
             "move_line_id": {"coerce": to_int, "required": True, "type": "integer"},
-            "zero": {"coerce": to_bool, "required": True, "type": "boolean"},
+            "zero": {"coerce": str2bool, "required": True, "type": "boolean"},
         }
 
     def skip_line(self):

odoo/addons/shopfloor/services/delivery.py

@@ -2,9 +2,10 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 from odoo import _, fields
 from odoo.osv import expression
+from odoo.tools import str2bool
 from odoo.tools.float_utils import float_is_zero
 
-from odoo.addons.base_rest.components.service import to_bool, to_int
+from odoo.addons.base_rest.components.service import to_int
 from odoo.addons.component.core import Component
 
 
@@ -211,25 +212,23 @@ class Delivery(Component):
         if product_qty:  # defined with lot/product/packaging scan
             # With a product_qty we process only one move line,
             # so one move to deal with regarding the qty
-            qty_done = lines.move_id.product_id.uom_id._compute_quantity(
+            qty_picked = lines.move_id.product_id.uom_id._compute_quantity(
                 product_qty, lines.move_id.product_uom
             )
-            lines.qty_done += qty_done
+            for line in lines:
+                line.qty_picked += qty_picked
             return self._action_picking_done(
                 lines.picking_id, force=allow_prepackaged_product
             )
         for line in lines:
             # note: the package level is automatically set to "is_done" when
-            # the qty_done is full
-            line.qty_done = line.quantity
+            # the line is fully picked
+            line.picked = True
         picking = fields.first(lines.mapped("picking_id"))
         return self._action_picking_done(picking, force=allow_prepackaged_product)
 
     def _reset_lines(self, lines):
-        for line in lines:
-            # note: the package level "is_done" field is automatically unset
-            # when the qty_done is not full
-            line.qty_done = 0
+        lines.picked = False
 
     def _deliver_package(self, picking, package, location):
         lines = package.move_line_ids.filtered(
@@ -256,7 +255,7 @@ class Delivery(Component):
         message = self._check_picking_type(lines.mapped("picking_id"))
         if message:
             return self._response_for_deliver(location=location, message=message)
-        # TODO add a message if any of the lines already had a qty_done > 0
+        # TODO add a message if any of the lines are already picked
         new_picking = fields.first(lines.mapped("picking_id"))
         if self._set_lines_done(lines):
             return self._response_for_deliver(
@@ -269,7 +268,7 @@ class Delivery(Component):
         # in the picking type, and then use IN (ids)
         domain = []
         if no_qty_done:
-            domain.append(("qty_done", "=", 0))
+            domain.append(("picked", "=", False))
         return domain
 
     def _lines_from_lot_domain(
@@ -419,8 +418,8 @@ class Delivery(Component):
                     location=location,
                     message=self.msg_store.product_not_unitary_in_package_scan_package(),
                 )
-        # We focus only on lines on which we can increase the 'qty_done'
-        lines = lines.filtered(lambda x: (x.qty_done + product_qty) <= x.quantity)
+        # We focus only on lines on which we can increase the 'qty_picked'
+        lines = lines.filtered(lambda x: (x.qty_picked + product_qty) <= x.quantity)
         # Filter lines to keep only ones from one delivery operation
         # (we do not want to process lines of another delivery operation)
         lines = lines._filter_on_picking(picking)
@@ -716,7 +715,7 @@ class Delivery(Component):
                 "Product Unit of Measure"
             )
             no_quantities_done = all(
-                float_is_zero(move_line.qty_done, precision_digits=precision_digits)
+                float_is_zero(move_line.qty_picked, precision_digits=precision_digits)
                 for move_line in picking.move_line_ids.filtered(
                     lambda m: m.state not in ("done", "cancel")
                 )
@@ -799,7 +798,7 @@ class ShopfloorDeliveryValidator(Component):
     def done(self):
         return {
             "picking_id": {"coerce": to_int, "required": True, "type": "integer"},
-            "confirm": {"coerce": to_bool, "required": False, "type": "boolean"},
+            "confirm": {"coerce": str2bool, "required": False, "type": "boolean"},
         }
 
 

odoo/addons/shopfloor/services/zone_picking.py

@@ -6,9 +6,10 @@ import functools
 from collections import defaultdict
 
 from odoo.fields import first
+from odoo.tools import str2bool
 from odoo.tools.float_utils import float_compare, float_is_zero
 
-from odoo.addons.base_rest.components.service import to_bool, to_int
+from odoo.addons.base_rest.components.service import to_int
 from odoo.addons.component.core import Component
 
 from ..exceptions import ConcurentWorkOnTransfer
@@ -1802,7 +1803,7 @@ class ShopfloorZonePickingValidator(Component):
     def is_zero(self):
         return {
             "move_line_id": {"coerce": to_int, "required": True, "type": "integer"},
-            "zero": {"coerce": to_bool, "required": True, "type": "boolean"},
+            "zero": {"coerce": str2bool, "required": True, "type": "boolean"},
         }
 
     def stock_issue(self):

odoo/addons/shopfloor/static/description/index.html

@@ -372,7 +372,7 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:01ad1a242b0c8a3b6ab8593cd6dedb31c5b6dd941b58e9af751833dad9cc69f3
+!! source digest: sha256:534d443b8feb0fd2548ede9663e260ceb8dcfb20686a39d06022ee92bcc6d209
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
 <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/stock-logistics-shopfloor/tree/18.0/shopfloor"><img alt="OCA/stock-logistics-shopfloor" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--shopfloor-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-shopfloor-18-0/stock-logistics-shopfloor-18-0-shopfloor"><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-shopfloor&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>Shopfloor is a barcode scanner application for internal warehouse

odoo/addons/shopfloor/tests/__init__.py

@@ -23,37 +23,38 @@ from . import test_actions_stock
 # from . import test_cluster_picking_stock_issue
 # from . import test_cluster_picking_change_pack_lot
 # from . import test_cluster_picking_unload
-# from . import test_checkout_base
-# from . import test_checkout_scan
-# from . import test_checkout_select
-# from . import test_checkout_scan_line
-# from . import test_checkout_scan_line_no_prefill_qty
-# from . import test_checkout_scan_line_base
-# from . import test_checkout_scan_dest_location
-# from . import test_checkout_select_line
-# from . import test_checkout_select_package_base
-# from . import test_checkout_set_qty
-# from . import test_checkout_scan_package_action
-# from . import test_checkout_scan_package_action_no_prefill_qty
-# from . import test_checkout_new_package
-# from . import test_checkout_no_package
-# from . import test_checkout_auto_post
-# from . import test_checkout_list_delivery_packaging
-# from . import test_checkout_list_package
-# from . import test_checkout_summary
-# from . import test_checkout_change_packaging
-# from . import test_checkout_cancel_line
-# from . import test_checkout_done
-# from . import test_delivery_base
-# from . import test_delivery_done
-# from . import test_delivery_scan_deliver
-# from . import test_delivery_reset_qty_done_line
-# from . import test_delivery_reset_qty_done_pack
-# from . import test_delivery_set_qty_done_pack
-# from . import test_delivery_set_qty_done_line
-# from . import test_delivery_sublocation
-# from . import test_delivery_list_stock_picking
-# from . import test_delivery_select
+from . import test_checkout_base
+from . import test_checkout_scan
+from . import test_checkout_select
+from . import test_checkout_scan_line
+from . import test_checkout_scan_line_no_prefill_qty
+from . import test_checkout_scan_line_base
+from . import test_checkout_scan_dest_location
+from . import test_checkout_select_line
+from . import test_checkout_select_package_base
+from . import test_checkout_set_qty
+from . import test_checkout_scan_package_action
+from . import test_checkout_scan_package_action_no_prefill_qty
+from . import test_checkout_new_package
+from . import test_checkout_no_package
+from . import test_checkout_auto_post
+from . import test_checkout_list_delivery_packaging
+from . import test_checkout_list_package
+from . import test_checkout_summary
+from . import test_checkout_change_packaging
+from . import test_checkout_cancel_line
+from . import test_checkout_done
+from . import test_delivery_base
+from . import test_delivery_done
+from . import test_delivery_scan_deliver
+from . import test_delivery_reset_qty_done_line
+from . import test_delivery_reset_qty_done_pack
+from . import test_delivery_set_qty_done_pack
+from . import test_delivery_set_qty_done_line
+from . import test_delivery_sublocation
+from . import test_delivery_list_stock_picking
+from . import test_delivery_select
+
 # from . import test_location_content_transfer_base
 # from . import test_location_content_transfer_start
 # from . import test_location_content_transfer_get_work

odoo/addons/shopfloor/tests/common.py

@@ -182,8 +182,10 @@ class CommonCase(BaseCommonCase):
         )
         # this method adds the quantity to the current quantity, so remove it
         quantity -= sum(quants.mapped("quantity"))
+        if not quantity:
+            return
         cls.env["stock.quant"]._update_available_quantity(
-            product, location, quantity, package_id=package, lot_id=lot
+            product, location, quantity=quantity, package_id=package, lot_id=lot
         )
 
     @classmethod