odoo-addon-shipment-advice 16.0.1.7.2.3__py3-none-any.whl → 18.0.1.0.0.4__py3-none-any.whl

odoo/addons/shipment_advice/models/shipment_advice.py

@@ -3,7 +3,7 @@
 # Copyright 2025 Jacques-Etienne Baudoux (BCIM) <je@bcim.be>
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
 
-from odoo import _, api, fields, models
+from odoo import api, fields, models
 from odoo.exceptions import UserError
 
 from odoo.addons.queue_job.delay import chain, group
@@ -17,9 +17,8 @@ class ShipmentAdvice(models.Model):
     _order = "arrival_date DESC, id DESC"
 
     def _default_warehouse_id(self):
-        company = self.env.context.get("default_company_id") or self.env.company.id
-        wh = self.env["stock.warehouse"].search([("company_id", "=", company)], limit=1)
-        return wh.id
+        wh = self.env.ref("stock.warehouse0", raise_if_not_found=False)
+        return wh.id or False
 
     name = fields.Char(
         default="/", copy=False, index=True, required=True, readonly=True
@@ -43,9 +42,7 @@ class ShipmentAdvice(models.Model):
         ondelete="cascade",
         string="Warehouse",
         required=True,
-        states={"draft": [("readonly", False)]},
-        readonly=True,
-        check_company=False,
+        check_company=True,
         default=_default_warehouse_id,
     )
     company_id = fields.Many2one(
@@ -68,26 +65,16 @@ class ShipmentAdvice(models.Model):
         comodel_name="stock.dock",
         ondelete="restrict",
         string="Loading dock",
-        states={"draft": [("readonly", False)], "confirmed": [("readonly", False)]},
-        readonly=True,
         index=True,
         domain="[('warehouse_id', '=', warehouse_id)]",
     )
     arrival_date = fields.Datetime(
-        states={"draft": [("readonly", False)], "confirmed": [("readonly", False)]},
-        readonly=True,
         help=(
             "When will the shipment arrive at the (un)loading dock. It is a "
             "planned date until in progress, then it represents the real one."
         ),
     )
     departure_date = fields.Datetime(
-        states={
-            "draft": [("readonly", False)],
-            "confirmed": [("readonly", False)],
-            "in_progress": [("readonly", False)],
-        },
-        readonly=True,
         help=(
             "When will the shipment leave the (un)loading dock. It is a "
             "planned date until in progress, then it represents the real one."
@@ -95,12 +82,6 @@ class ShipmentAdvice(models.Model):
     )
     ref = fields.Char(
         string="Consignment/Truck Ref.",
-        states={
-            "draft": [("readonly", False)],
-            "confirmed": [("readonly", False)],
-            "in_progress": [("readonly", False)],
-        },
-        readonly=True,
     )
     total_load = fields.Float(
         string="Total load (kg)",
@@ -111,12 +92,6 @@ class ShipmentAdvice(models.Model):
         comodel_name="stock.move",
         inverse_name="shipment_advice_id",
         string="Planned content list",
-        states={
-            "draft": [("readonly", False)],
-            "confirmed": [("readonly", False)],
-            "in_progress": [("readonly", False)],
-        },
-        readonly=True,
         check_company=True,
     )
     planned_moves_count = fields.Integer(compute="_compute_count")
@@ -130,25 +105,13 @@ class ShipmentAdvice(models.Model):
         comodel_name="stock.move.line",
         inverse_name="shipment_advice_id",
         string="Loaded content list",
-        states={
-            "draft": [("readonly", False)],
-            "confirmed": [("readonly", False)],
-            "in_progress": [("readonly", False)],
-        },
-        readonly=True,
         check_company=True,
     )
     loaded_move_line_without_package_ids = fields.One2many(
         comodel_name="stock.move.line",
         inverse_name="shipment_advice_id",
         string="Loaded content list (w/o packages)",
-        states={
-            "draft": [("readonly", False)],
-            "confirmed": [("readonly", False)],
-            "in_progress": [("readonly", False)],
-        },
         domain=[("package_level_id", "=", False)],
-        readonly=True,
         check_company=True,
     )
     loaded_move_lines_without_package_count = fields.Integer(compute="_compute_count")
@@ -275,14 +238,15 @@ class ShipmentAdvice(models.Model):
         for shipment in self:
             if shipment.state != "draft":
                 raise UserError(
-                    _("Shipment {} is not draft, operation aborted.").format(
-                        shipment.name
+                    self.env._(
+                        "Shipment %s is not draft, operation aborted.", shipment.name
                     )
                 )
             if not shipment.arrival_date:
                 raise UserError(
-                    _("Arrival date should be set on the shipment advice {}.").format(
-                        shipment.name
+                    self.env._(
+                        "Arrival date should be set on the shipment advice %s.",
+                        shipment.name,
                     )
                 )
             shipment.state = "confirmed"
@@ -292,14 +256,15 @@ class ShipmentAdvice(models.Model):
         for shipment in self:
             if shipment.state != "confirmed":
                 raise UserError(
-                    _("Shipment {} is not confirmed, operation aborted.").format(
-                        shipment.name
+                    self.env._(
+                        "Shipment %s is not confirmed, operation aborted.",
+                        shipment.name,
                     )
                 )
             if not shipment.dock_id:
                 raise UserError(
-                    _("Dock should be set on the shipment advice {}.").format(
-                        shipment.name
+                    self.env._(
+                        "Dock should be set on the shipment advice %s.", shipment.name
                     )
                 )
             if not shipment.arrival_date:
@@ -311,7 +276,7 @@ class ShipmentAdvice(models.Model):
         """Lock records for the current SQL transaction."""
         if not records:
             return
-        sql = "SELECT id FROM %s WHERE ID IN %%s FOR UPDATE" % records._table
+        sql = f"SELECT id FROM {records._table} WHERE ID IN %s FOR UPDATE"
         self.env.cr.execute(sql, (tuple(records.ids),), log_exceptions=False)
 
     def action_done(self):
@@ -345,7 +310,7 @@ class ShipmentAdvice(models.Model):
                     *[
                         self.delayable(
                             identity_key=identity_exact,
-                            description=_(
+                            description=self.env._(
                                 "%(sa)s: %(pick)s background validation",
                                 sa=self.name,
                                 pick=picking.name,
@@ -367,8 +332,8 @@ class ShipmentAdvice(models.Model):
         for shipment in self:
             if shipment.state not in ("in_progress", "error"):
                 raise UserError(
-                    _("Shipment {} is not started, operation aborted.").format(
-                        shipment.name
+                    self.env._(
+                        "Shipment %s is not started, operation aborted.", shipment.name
                     )
                 )
 
@@ -401,7 +366,7 @@ class ShipmentAdvice(models.Model):
             return
         moves_to_unplan = (
             self.loaded_move_line_ids.move_id | self.planned_move_ids
-        ).filtered(lambda m: m.state not in ("cancel", "done") and not m.quantity_done)
+        ).filtered(lambda m: m.state not in ("cancel", "done") and not m.picked)
         moves_to_unplan.shipment_advice_id = False
 
     def _postprocess_action_done(self):
@@ -414,7 +379,7 @@ class ShipmentAdvice(models.Model):
             self.write(
                 {
                     "state": "error",
-                    "error_message": _(
+                    "error_message": self.env._(
                         "One of the pickings to process failed to validate"
                     ),
                 }
@@ -430,7 +395,7 @@ class ShipmentAdvice(models.Model):
 
     @api.model
     def _get_error_message(self, error, related_object):
-        return _(
+        return self.env._(
             "An error occurred while processing:\n- %(related_object_name)s: %(error)s",
             related_object_name=related_object.display_name,
             error=str(error),
@@ -459,8 +424,8 @@ class ShipmentAdvice(models.Model):
         for shipment in self:
             if shipment.state not in ("confirmed", "in_progress"):
                 raise UserError(
-                    _("Shipment {} is not started, operation aborted.").format(
-                        shipment.name
+                    self.env._(
+                        "Shipment %s is not started, operation aborted.", shipment.name
                     )
                 )
             shipment.state = "cancel"
@@ -469,8 +434,8 @@ class ShipmentAdvice(models.Model):
         for shipment in self:
             if shipment.state != "cancel":
                 raise UserError(
-                    _("Shipment {} is not canceled, operation aborted.").format(
-                        shipment.name
+                    self.env._(
+                        "Shipment %s is not canceled, operation aborted.", shipment.name
                     )
                 )
             shipment.state = "draft"
@@ -485,7 +450,7 @@ class ShipmentAdvice(models.Model):
         action_xmlid = "stock.stock_move_action"
         action = self.env["ir.actions.act_window"]._for_xml_id(action_xmlid)
         action["views"] = [
-            (self.env.ref("stock.view_picking_move_tree").id, "tree"),
+            (self.env.ref("stock.view_picking_move_tree").id, "list"),
         ]
         action["domain"] = [("id", "in", self.planned_move_ids.ids)]
         action["context"] = {}  # Disable filters
@@ -540,21 +505,21 @@ class ShipmentAdvice(models.Model):
     def button_open_deliveries_in_progress(self):
         action_xmlid = "stock.action_picking_tree_all"
         action = self.env["ir.actions.act_window"]._for_xml_id(action_xmlid)
-        view_tree = self.env.ref(
+        view_list = self.env.ref(
             "shipment_advice.stock_picking_loading_progress_view_tree"
         )
-        tree_view_index = action["views"].index((False, "tree"))
-        action["views"][tree_view_index] = (view_tree.id, "tree")
+        list_view_index = action["views"].index((False, "list"))
+        action["views"][list_view_index] = (view_list.id, "list")
         action["domain"] = self._domain_open_deliveries_in_progress()
         return action
 
     def button_open_receptions_in_progress(self):
         action_xmlid = "stock.action_picking_tree_all"
         action = self.env["ir.actions.act_window"]._for_xml_id(action_xmlid)
-        view_tree = self.env.ref(
+        view_list = self.env.ref(
             "shipment_advice.stock_picking_loading_progress_view_tree"
         )
-        tree_view_index = action["views"].index((False, "tree"))
-        action["views"][tree_view_index] = (view_tree.id, "tree")
+        list_view_index = action["views"].index((False, "list"))
+        action["views"][list_view_index] = (view_list.id, "list")
         action["domain"] = [("id", "in", self.planned_picking_ids.ids)]
         return action

odoo/addons/shipment_advice/models/stock_move_line.py

@@ -1,7 +1,7 @@
 # Copyright 2021 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
 
-from odoo import _, fields, models
+from odoo import fields, models
 from odoo.exceptions import UserError
 from odoo.tools import float_is_zero
 
@@ -31,8 +31,9 @@ class StockMoveLine(models.Model):
         for move_line in self:
             if move_line.package_level_id:
                 package_lines = move_line.package_level_id.move_line_ids.filtered(
-                    lambda l: l.state in ("partially_available", "assigned")
+                    lambda ml: ml.state in ("partially_available", "assigned")
                 )
+
                 if not set(package_lines.ids).issubset(set(self.ids)):
                     return False
         return True
@@ -46,14 +47,14 @@ class StockMoveLine(models.Model):
             products = move_lines.product_id.mapped("display_name")
             packages = move_lines.package_id.mapped("display_name")
             raise UserError(
-                _(
+                self.env._(
                     "You cannot load this move line alone, you have to "
                     "move the whole package content.\n%(info)s",
                     info="\n".join(
                         [
-                            _("Transfers: %s", ", ".join(pickings)),
-                            _("Products: %s", ", ".join(products)),
-                            _("Packages: %s", ", ".join(packages)),
+                            self.env._("Transfers: %s", ", ".join(pickings)),
+                            self.env._("Products: %s", ", ".join(products)),
+                            self.env._("Packages: %s", ", ".join(packages)),
                         ]
                     ),
                 )
@@ -63,22 +64,25 @@ class StockMoveLine(models.Model):
             planned_shipment = move_line.move_id.shipment_advice_id
             if planned_shipment and planned_shipment != shipment_advice:
                 raise UserError(
-                    _(
+                    self.env._(
                         "You cannot load this into this shipment as it has been "
-                        "planned to be loaded in {}"
-                    ).format(planned_shipment.name)
+                        "planned to be loaded in %s",
+                        planned_shipment.name,
+                    )
                 )
             # If no planned shipment, allow the loading only if the shipment
             # is not a planned one
             elif not planned_shipment and shipment_advice.planned_move_ids:
                 raise UserError(
-                    _(
+                    self.env._(
                         "You cannot load this into this shipment because its "
                         "content is planned already.\n%(info)s",
                         info="\n".join(
                             [
-                                _("Transfer: %s", move_line.picking_id.name),
-                                _("Product: %s", move_line.product_id.display_name),
+                                self.env._("Transfer: %s", move_line.picking_id.name),
+                                self.env._(
+                                    "Product: %s", move_line.product_id.display_name
+                                ),
                             ]
                         ),
                     )
@@ -88,7 +92,7 @@ class StockMoveLine(models.Model):
                 and move_line.shipment_advice_id != shipment_advice
             ):
                 raise UserError(
-                    _(
+                    self.env._(
                         "This move is already loaded  in another shipment."
                         "\nProduct: %(product)s.\nPicking: %(picking)s",
                         product=move_line.product_id.display_name,
@@ -97,31 +101,29 @@ class StockMoveLine(models.Model):
                 )
             move_line.shipment_advice_id = shipment_advice
             uom = move_line.product_uom_id or move_line.product_id.uom_id
-            if float_is_zero(
-                move_line.reserved_uom_qty, precision_rounding=uom.rounding
-            ):
+            if float_is_zero(move_line.quantity, precision_rounding=uom.rounding):
                 raise UserError(
-                    _(
+                    self.env._(
                         "Nothing to load for %(product)s.\nPicking: %(picking)s",
                         product=move_line.product_id.display_name,
                         picking=move_line.picking_id.name,
                     )
                 )
             if move_line.state in ("partially_available", "assigned"):
-                move_line.qty_done = move_line.reserved_uom_qty
+                move_line.picked = True
 
     def _unload_from_shipment(self):
         """Unload the move lines from their related shipment advice."""
         if not self._check_entire_package():
             raise UserError(
-                _(
+                self.env._(
                     "You cannot unload this move line alone, you have to "
                     "unload the whole package content."
                 )
             )
         self.shipment_advice_id = False
-        self.qty_done = 0
+        self.picked = False
 
     def _is_loaded_in_shipment(self):
         """Return `True` if the move lines are loaded in a shipment."""
-        return all([line.qty_done and line.shipment_advice_id for line in self])
+        return all([line.picked and line.shipment_advice_id for line in self])

odoo/addons/shipment_advice/models/stock_package_level.py

@@ -9,7 +9,15 @@ class StockPackageLevel(models.Model):
 
     shipment_advice_id = fields.Many2one(related="move_line_ids.shipment_advice_id")
     package_shipping_weight = fields.Float(related="package_id.shipping_weight")
-    package_weight_uom_name = fields.Char(related="package_id.weight_uom_name")
+    package_shipping_weight_uom_name = fields.Char(
+        compute="_compute_package_shipping_weight_uom_name", store=False
+    )
+
+    def _compute_package_shipping_weight_uom_name(self):
+        for package_level in self:
+            package_level.package_shipping_weight_uom_name = self.env[
+                "product.template"
+            ]._get_weight_uom_name_from_ir_config_parameter()
 
     def button_load_in_shipment(self):
         action_xmlid = "shipment_advice.wizard_load_shipment_picking_action"
@@ -28,7 +36,10 @@ class StockPackageLevel(models.Model):
 
     def _unload_from_shipment(self):
         """Unload the package levels from their related shipment advice."""
+        # Workaround: Odoo's stock.package_level.is_done computed field doesn't include
+        # move_line_ids.picked in its @api.depends, so we manually invalidate the cache
         self.move_line_ids._unload_from_shipment()
+        self.invalidate_recordset(fnames=["is_done"])
 
     def _is_loaded_in_shipment(self):
         """Return `True` if the package levels are loaded in a shipment."""

odoo/addons/shipment_advice/models/stock_picking.py

@@ -79,7 +79,7 @@ class StockPicking(models.Model):
             # otherwise overloaded container would be marked as partially loaded
             picking.is_fully_loaded_in_shipment = (
                 all(
-                    line.shipment_advice_id and line.qty_done >= line.reserved_uom_qty
+                    line.shipment_advice_id and line.picked
                     for line in picking.move_line_ids
                 )
                 if picking.move_line_ids
@@ -88,7 +88,7 @@ class StockPicking(models.Model):
             picking.is_partially_loaded_in_shipment = (
                 not picking.is_fully_loaded_in_shipment
                 and any(
-                    line.shipment_advice_id and line.qty_done > 0
+                    line.shipment_advice_id and line.picked
                     for line in picking.move_line_ids
                 )
             )
@@ -131,7 +131,7 @@ class StockPicking(models.Model):
                     [
                         ml
                         for ml in picking.move_line_ids_without_package
-                        if ml.shipment_advice_id and ml.qty_done > 0
+                        if ml.shipment_advice_id and ml.picked
                     ]
                 )
                 picking.loaded_move_lines_progress_f = (
@@ -147,7 +147,7 @@ class StockPicking(models.Model):
                 picking.loaded_weight = sum(
                     ml.result_package_id.shipping_weight or ml.move_id.weight
                     for ml in picking.move_line_ids_without_package
-                    if ml.shipment_advice_id and ml.qty_done > 0
+                    if ml.shipment_advice_id and ml.picked
                 ) + sum(
                     pl.package_id.shipping_weight
                     for pl in picking.package_level_ids
@@ -163,9 +163,11 @@ class StockPicking(models.Model):
             waiting_moves = picking.move_ids.filtered(
                 lambda ml: ml.state not in ["done", "cancel"]
             )
+
             picking.loaded_waiting_quantity = sum(
                 waiting_moves.mapped("product_qty")
-            ) - sum(waiting_moves.mapped("reserved_availability"))
+            ) - sum(waiting_moves.mapped("quantity"))
+
             # Overall progress based on the operation type
             if picking.picking_type_id.show_entire_packs:
                 picking.loaded_progress_f = picking.loaded_packages_progress_f

odoo/addons/shipment_advice/readme/CONTRIBUTORS.md

@@ -0,0 +1,12 @@
+- Sébastien Alix \<sebastien.alix@camptocamp.com\>
+- Guewen Baconnier \<guewen.baconnier@camptocamp.com\>
+- Simone Orsi \<simahawk@gmail.com\>
+- [Trobz](https://trobz.com):
+  - Dung Tran \<dungtd@trobz.com\>
+- Michael Tietz (MT Software) \<mtietz@mt-software.de\>
+- Jacques-Etienne Baudoux \<je@bcim.be\>
+
+## Design
+
+- Joël Grand-Guillaume \<joel.grandguillaume@camptocamp.com\>
+- Jacques-Etienne Baudoux \<je@bcim.be\>

odoo/addons/shipment_advice/report/report_shipment_advice.xml

@@ -138,7 +138,7 @@
                                     <span t-field="line.product_id" />
                                 </td>
                                 <td>
-                                    <span t-field="line.qty_done" />
+                                    <span t-field="line.quantity" />
                                 </td>
                                 <td>
                                     <span t-field="line.picking_id" />

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

@@ -367,9 +367,9 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:42b81c5fcdcb772ae5220e2758962658d87c1c83d548a3fb56b665d67cb819a6
+!! source digest: sha256:95c96f19154e5730489d76cddc8ddbd8a5a8e273db29d5aa9c871fc51fa22d33
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<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/stock-logistics-transport/tree/16.0/shipment_advice"><img alt="OCA/stock-logistics-transport" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--transport-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-transport-16-0/stock-logistics-transport-16-0-shipment_advice"><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-transport&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/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/stock-logistics-transport/tree/18.0/shipment_advice"><img alt="OCA/stock-logistics-transport" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--transport-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-transport-18-0/stock-logistics-transport-18-0-shipment_advice"><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-transport&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>Manage your (un)loading process through shipment advices.</p>
 <p><strong>Table of contents</strong></p>
 <div class="contents local topic" id="contents">
@@ -377,10 +377,11 @@ ul.auto-toc {
 <li><a class="reference internal" href="#bug-tracker" id="toc-entry-1">Bug Tracker</a></li>
 <li><a class="reference internal" href="#credits" id="toc-entry-2">Credits</a><ul>
 <li><a class="reference internal" href="#authors" id="toc-entry-3">Authors</a></li>
-<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a></li>
+<li><a class="reference internal" href="#contributors" id="toc-entry-4">Contributors</a><ul>
 <li><a class="reference internal" href="#design" id="toc-entry-5">Design</a></li>
-<li><a class="reference internal" href="#other-credits" id="toc-entry-6">Other credits</a></li>
-<li><a class="reference internal" href="#maintainers" id="toc-entry-7">Maintainers</a></li>
+</ul>
+</li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-6">Maintainers</a></li>
 </ul>
 </li>
 </ul>
@@ -390,7 +391,7 @@ ul.auto-toc {
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/stock-logistics-transport/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-transport/issues/new?body=module:%20shipment_advice%0Aversion:%2016.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-transport/issues/new?body=module:%20shipment_advice%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">
@@ -399,6 +400,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <h2><a class="toc-backref" href="#toc-entry-3">Authors</a></h2>
 <ul class="simple">
 <li>Camptocamp</li>
+<li>BCIM</li>
 </ul>
 </div>
 <div class="section" id="contributors">
@@ -407,30 +409,23 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <li>Sébastien Alix &lt;<a class="reference external" href="mailto:sebastien.alix&#64;camptocamp.com">sebastien.alix&#64;camptocamp.com</a>&gt;</li>
 <li>Guewen Baconnier &lt;<a class="reference external" href="mailto:guewen.baconnier&#64;camptocamp.com">guewen.baconnier&#64;camptocamp.com</a>&gt;</li>
 <li>Simone Orsi &lt;<a class="reference external" href="mailto:simahawk&#64;gmail.com">simahawk&#64;gmail.com</a>&gt;</li>
-<li><a class="reference external" href="https://trobz.com">Trobz</a>:
-* Dung Tran &lt;<a class="reference external" href="mailto:dungtd&#64;trobz.com">dungtd&#64;trobz.com</a>&gt;</li>
+<li><a class="reference external" href="https://trobz.com">Trobz</a>:<ul>
+<li>Dung Tran &lt;<a class="reference external" href="mailto:dungtd&#64;trobz.com">dungtd&#64;trobz.com</a>&gt;</li>
+</ul>
+</li>
 <li>Michael Tietz (MT Software) &lt;<a class="reference external" href="mailto:mtietz&#64;mt-software.de">mtietz&#64;mt-software.de</a>&gt;</li>
 <li>Jacques-Etienne Baudoux &lt;<a class="reference external" href="mailto:je&#64;bcim.be">je&#64;bcim.be</a>&gt;</li>
 </ul>
-</div>
 <div class="section" id="design">
-<h2><a class="toc-backref" href="#toc-entry-5">Design</a></h2>
+<h3><a class="toc-backref" href="#toc-entry-5">Design</a></h3>
 <ul class="simple">
 <li>Joël Grand-Guillaume &lt;<a class="reference external" href="mailto:joel.grandguillaume&#64;camptocamp.com">joel.grandguillaume&#64;camptocamp.com</a>&gt;</li>
 <li>Jacques-Etienne Baudoux &lt;<a class="reference external" href="mailto:je&#64;bcim.be">je&#64;bcim.be</a>&gt;</li>
 </ul>
 </div>
-<div class="section" id="other-credits">
-<h2><a class="toc-backref" href="#toc-entry-6">Other credits</a></h2>
-<p><strong>Financial support</strong></p>
-<ul class="simple">
-<li>Cosanum</li>
-<li>Camptocamp R&amp;D</li>
-</ul>
-<p>The migration of this module from 13.0 to 14.0 was financially supported by Camptocamp</p>
 </div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#toc-entry-7">Maintainers</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-6">Maintainers</a></h2>
 <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" />
@@ -438,7 +433,9 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <p>OCA, or the Odoo Community Association, is a nonprofit organization whose
 mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/stock-logistics-transport/tree/16.0/shipment_advice">OCA/stock-logistics-transport</a> project on GitHub.</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/jbaudoux"><img alt="jbaudoux" src="https://github.com/jbaudoux.png?size=40px" /></a></p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/stock-logistics-transport/tree/18.0/shipment_advice">OCA/stock-logistics-transport</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/shipment_advice/tests/common.py

@@ -3,14 +3,15 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
 
 from odoo import fields
-from odoo.tests.common import Form, TransactionCase, new_test_user
+from odoo.tests import Form, new_test_user
 
+from odoo.addons.base.tests.common import BaseCommon
 
-class Common(TransactionCase):
+
+class Common(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
         # Configuration
         cls.dock = cls.env.ref("shipment_advice.stock_dock_demo")
         cls.picking_type_out = cls.env.ref("stock.picking_type_out")
@@ -110,8 +111,6 @@ class Common(TransactionCase):
                 "warehouse_id": picking_type.warehouse_id.id,
                 "picking_type_id": picking_type.id,
                 "group_id": group and group.id or False,
-                # "procure_method": "make_to_order",
-                # "state": "draft",
             }
         )
         move._assign_picking()
@@ -198,10 +197,10 @@ class Common(TransactionCase):
         return wiz
 
     @classmethod
-    def validate_picking(cls, picking, qty_done=None):
+    def validate_picking(cls, picking):
         picking.ensure_one()
         for ml in picking.move_line_ids:
-            ml.qty_done = qty_done or ml.reserved_uom_qty
+            ml.picked = True
         action_data = picking.button_validate()
         if action_data is True:
             return cls.env["stock.picking"]

odoo/addons/shipment_advice/tests/test_shipment_advice.py

@@ -44,7 +44,7 @@ class TestShipmentAdvice(Common):
         self.plan_records_in_shipment(self.shipment_advice_in, picking)
         self.progress_shipment_advice(self.shipment_advice_in)
         for ml in picking.move_line_ids:
-            ml.qty_done = ml.reserved_uom_qty
+            ml.picked = True
         picking._action_done()
         self.shipment_advice_in.action_done()
         self.assertEqual(self.shipment_advice_in.state, "done")
@@ -66,7 +66,7 @@ class TestShipmentAdvice(Common):
         self.progress_shipment_advice(self.shipment_advice_in)
         # Receive it (making its related transfer partially received)
         for ml in self.move_product_in1.move_line_ids:
-            ml.qty_done = ml.reserved_uom_qty
+            ml.picked = True
         self.assertEqual(picking, self.move_product_in2.picking_id)
         # When validating the shipment, a backorder is created for unprocessed moves
         self.shipment_advice_in.action_done()