odoo-addon-stock-move-location 16.0.1.4.1__py3-none-any.whl → 18.0.1.0.0.3__py3-none-any.whl

odoo/addons/stock_move_location/README.rst

@@ -7,7 +7,7 @@ Move Stock Location
    !! This file is generated by oca-gen-addon-readme !!
    !! changes will be overwritten.                   !!
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-   !! source digest: sha256:9f20ba56c774915b0e3598aabcaca4a43acd2ec1c36192be9fa99be476111ebd
+   !! source digest: sha256:d4e4ea6fb0a6f2f34b451c22d5ab139e4f867621b65786664f6b93d57558b66d
    !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 
 .. |badge1| image:: https://img.shields.io/badge/maturity-Beta-yellow.png
@@ -17,13 +17,13 @@ Move Stock Location
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fstock--logistics--warehouse-lightgray.png?logo=github
-    :target: https://github.com/OCA/stock-logistics-warehouse/tree/16.0/stock_move_location
+    :target: https://github.com/OCA/stock-logistics-warehouse/tree/18.0/stock_move_location
     :alt: OCA/stock-logistics-warehouse
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_move_location
+    :target: https://translation.odoo-community.org/projects/stock-logistics-warehouse-18-0/stock-logistics-warehouse-18-0-stock_move_location
     :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-warehouse&target_branch=16.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/stock-logistics-warehouse&target_branch=18.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -43,7 +43,6 @@ Usage
   where 2 locations can be specified.
 - Select origin and destination locations and press "IMMEDIATE TRANSFER"
   or "PLANNED TRANSFER"
-- Press ADD ALL button to add all products available
 - Those lines can be edited. Move quantity can't be more than a max
   available quantity
 - Move doesn't care about the reservations and will move stuff anyway
@@ -106,7 +105,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/stock-logistics-warehouse/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-warehouse/issues/new?body=module:%20stock_move_location%0Aversion:%2016.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/stock-logistics-warehouse/issues/new?body=module:%20stock_move_location%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.
 
@@ -133,6 +132,7 @@ Contributors
 
   - Sergio Teruel
   - João Marques
+  - Carlos Roca
 
 - Jacques-Etienne Baudoux <je@bcim.be>
 - Iryna Vyshnevska <i.vyshnevska@mobilunity.com>
@@ -143,6 +143,7 @@ Contributors
   - Aung Ko Ko Lin
 
 - Laurent Mignon <laurent.mignon@acsone.eu>
+- Maksym Yankin <maksym.yankin@camptocamp.com>
 
 Maintainers
 -----------
@@ -157,6 +158,6 @@ 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.
 
-This module is part of the `OCA/stock-logistics-warehouse <https://github.com/OCA/stock-logistics-warehouse/tree/16.0/stock_move_location>`_ project on GitHub.
+This module is part of the `OCA/stock-logistics-warehouse <https://github.com/OCA/stock-logistics-warehouse/tree/18.0/stock_move_location>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

odoo/addons/stock_move_location/__manifest__.py

@@ -5,7 +5,7 @@
 
 {
     "name": "Move Stock Location",
-    "version": "16.0.1.4.1",
+    "version": "18.0.1.0.0",
     "author": "Julius Network Solutions, "
     "BCIM,"
     "Camptocamp,"

odoo/addons/stock_move_location/i18n/stock_move_location.pot

@@ -4,7 +4,7 @@
 #
 msgid ""
 msgstr ""
-"Project-Id-Version: Odoo Server 16.0\n"
+"Project-Id-Version: Odoo Server 18.0\n"
 "Report-Msgid-Bugs-To: \n"
 "Last-Translator: \n"
 "Language-Team: \n"
@@ -28,6 +28,11 @@ msgstr ""
 msgid "Cancel"
 msgstr ""
 
+#. module: stock_move_location
+#: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location__company_id
+msgid "Company"
+msgstr ""
+
 #. module: stock_move_location
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location__picking_id
 msgid "Connected Picking"
@@ -69,9 +74,20 @@ msgstr ""
 
 #. module: stock_move_location
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location__edit_locations
+#: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
 msgid "Edit Locations"
 msgstr ""
 
+#. module: stock_move_location
+#: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location__exclude_reserved_qty
+msgid "Exclude Reserved Qty"
+msgstr ""
+
+#. module: stock_move_location
+#: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
+msgid "Exclude Reserved Quantities"
+msgstr ""
+
 #. module: stock_move_location
 #: model_terms:ir.ui.view,arch_db:stock_move_location.view_picking_form
 msgid "Fill with stock"
@@ -93,12 +109,6 @@ msgstr ""
 msgid "Immediate Transfer"
 msgstr ""
 
-#. module: stock_move_location
-#: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location____last_update
-#: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location_line____last_update
-msgid "Last Modified on"
-msgstr ""
-
 #. module: stock_move_location
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location__write_uid
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location_line__write_uid
@@ -111,6 +121,11 @@ msgstr ""
 msgid "Last Updated on"
 msgstr ""
 
+#. module: stock_move_location
+#: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
+msgid "Lot/SN:"
+msgstr ""
+
 #. module: stock_move_location
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location_line__lot_id
 msgid "Lot/Serial Number"
@@ -145,7 +160,6 @@ msgstr ""
 #. module: stock_move_location
 #. odoo-python
 #: code:addons/stock_move_location/wizard/stock_move_location_line.py:0
-#, python-format
 msgid "Move quantity can not exceed max quantity or be negative"
 msgstr ""
 
@@ -157,7 +171,6 @@ msgstr ""
 #. module: stock_move_location
 #. odoo-python
 #: code:addons/stock_move_location/models/stock_picking.py:0
-#, python-format
 msgid "Moves lines already exists"
 msgstr ""
 
@@ -196,7 +209,6 @@ msgstr ""
 #. module: stock_move_location
 #. odoo-python
 #: code:addons/stock_move_location/models/stock_picking.py:0
-#, python-format
 msgid "Please choose a source end location"
 msgstr ""
 
@@ -210,6 +222,11 @@ msgstr ""
 msgid "Product Unit of Measure"
 msgstr ""
 
+#. module: stock_move_location
+#: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
+msgid "Quantity"
+msgstr ""
+
 #. module: stock_move_location
 #: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location_line__move_quantity
 msgid "Quantity to move"
@@ -237,11 +254,21 @@ msgstr ""
 msgid "Stock Move"
 msgstr ""
 
+#. module: stock_move_location
+#: model:ir.model.fields,field_description:stock_move_location.field_wiz_stock_move_location_line__total_quantity
+msgid "Total existence quantity"
+msgstr ""
+
 #. module: stock_move_location
 #: model:ir.model,name:stock_move_location.model_stock_picking
 msgid "Transfer"
 msgstr ""
 
+#. module: stock_move_location
+#: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
+msgid "Unit of Measure"
+msgstr ""
+
 #. module: stock_move_location
 #: model_terms:ir.ui.view,arch_db:stock_move_location.view_wiz_stock_move_location_form_stock_move_location
 msgid "UoM"

odoo/addons/stock_move_location/init_hook.py

@@ -1,9 +1,5 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
-from odoo import SUPERUSER_ID, api
-
-
-def enable_multi_locations(cr, registry):
-    env = api.Environment(cr, SUPERUSER_ID, {})
+def enable_multi_locations(env):
     ResConfig = env["res.config.settings"]
     default_values = ResConfig.default_get(list(ResConfig.fields_get()))
     default_values.update({"group_stock_multi_locations": True})

odoo/addons/stock_move_location/models/stock_picking.py

@@ -17,7 +17,6 @@ class StockPicking(models.Model):
         context = {
             "active_ids": self._get_movable_quants().ids,
             "active_model": "stock.quant",
-            "only_reserved_qty": True,
             "planned": True,
         }
         move_wizard = (
@@ -32,7 +31,6 @@ class StockPicking(models.Model):
                 }
             )
         )
-        move_wizard._onchange_destination_location_id()
         move_wizard.action_move_location()
         return True
 
@@ -43,9 +41,13 @@ class StockPicking(models.Model):
             raise UserError(_("Moves lines already exists"))
 
     def _get_movable_quants(self):
-        return self.env["stock.quant"].search(
-            [
-                ("location_id", "=", self.location_id.id),
-                ("quantity", ">", 0.0),
-            ]
+        return (
+            self.env["stock.quant"]
+            .search(
+                [
+                    ("location_id", "=", self.location_id.id),
+                    ("quantity", ">", 0.0),
+                ]
+            )
+            .filtered(lambda quant: quant.quantity - quant.reserved_quantity > 0.0)
         )

odoo/addons/stock_move_location/readme/CONTRIBUTORS.md

@@ -7,6 +7,7 @@
 - Tecnativa \<tecnativa.com\>
   - Sergio Teruel
   - João Marques
+  - Carlos Roca
 - Jacques-Etienne Baudoux \<<je@bcim.be>\>
 - Iryna Vyshnevska \<<i.vyshnevska@mobilunity.com>\>
 - Alexei Rivera \<<arivera@archeti.com>\>
@@ -14,3 +15,4 @@
 - Quartile \<<https://www.quartile.co>\>
   - Aung Ko Ko Lin
 - Laurent Mignon \<<laurent.mignon@acsone.eu>\>
+- Maksym Yankin \<<maksym.yankin@camptocamp.com>\>

odoo/addons/stock_move_location/readme/USAGE.md

@@ -2,7 +2,6 @@
   can be specified.
 - Select origin and destination locations and press "IMMEDIATE TRANSFER" or "PLANNED
   TRANSFER"
-- Press ADD ALL button to add all products available
 - Those lines can be edited. Move quantity can't be more than a max available quantity
 - Move doesn't care about the reservations and will move stuff anyway
 - If during your operation with the wizard the real quantity will change it will move

odoo/addons/stock_move_location/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:9f20ba56c774915b0e3598aabcaca4a43acd2ec1c36192be9fa99be476111ebd
+!! source digest: sha256:d4e4ea6fb0a6f2f34b451c22d5ab139e4f867621b65786664f6b93d57558b66d
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<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-warehouse/tree/16.0/stock_move_location"><img alt="OCA/stock-logistics-warehouse" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--warehouse-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-warehouse-16-0/stock-logistics-warehouse-16-0-stock_move_location"><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-warehouse&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-warehouse/tree/18.0/stock_move_location"><img alt="OCA/stock-logistics-warehouse" src="https://img.shields.io/badge/github-OCA%2Fstock--logistics--warehouse-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/stock-logistics-warehouse-18-0/stock-logistics-warehouse-18-0-stock_move_location"><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-warehouse&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 allows to move entire location of products from one place to
 another and move only selected quantities.</p>
 <p><strong>Table of contents</strong></p>
@@ -393,7 +393,6 @@ another and move only selected quantities.</p>
 where 2 locations can be specified.</li>
 <li>Select origin and destination locations and press “IMMEDIATE TRANSFER”
 or “PLANNED TRANSFER”</li>
-<li>Press ADD ALL button to add all products available</li>
 <li>Those lines can be edited. Move quantity can’t be more than a max
 available quantity</li>
 <li>Move doesn’t care about the reservations and will move stuff anyway</li>
@@ -447,7 +446,7 @@ filled in.</li>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/stock-logistics-warehouse/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-warehouse/issues/new?body=module:%20stock_move_location%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-warehouse/issues/new?body=module:%20stock_move_location%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">
@@ -472,6 +471,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 <li>Tecnativa &lt;tecnativa.com&gt;<ul>
 <li>Sergio Teruel</li>
 <li>João Marques</li>
+<li>Carlos Roca</li>
 </ul>
 </li>
 <li>Jacques-Etienne Baudoux &lt;<a class="reference external" href="mailto:je&#64;bcim.be">je&#64;bcim.be</a>&gt;</li>
@@ -483,6 +483,7 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 </ul>
 </li>
 <li>Laurent Mignon &lt;<a class="reference external" href="mailto:laurent.mignon&#64;acsone.eu">laurent.mignon&#64;acsone.eu</a>&gt;</li>
+<li>Maksym Yankin &lt;<a class="reference external" href="mailto:maksym.yankin&#64;camptocamp.com">maksym.yankin&#64;camptocamp.com</a>&gt;</li>
 </ul>
 </div>
 <div class="section" id="maintainers">
@@ -494,7 +495,7 @@ 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-warehouse/tree/16.0/stock_move_location">OCA/stock-logistics-warehouse</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/stock-logistics-warehouse/tree/18.0/stock_move_location">OCA/stock-logistics-warehouse</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_move_location/tests/test_common.py

@@ -1,17 +1,17 @@
 # Copyright (C) 2011 Julius Network Solutions SARL <contact@julius.fr>
 # Copyright 2018 Camptocamp SA
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl)
+from odoo.tests import Form
 
-from odoo.tests import Form, common
+from odoo.addons.base.tests.common import BaseCommon
 
 
-class TestsCommon(common.TransactionCase):
+class TestsCommon(BaseCommon):
     @classmethod
     def setUpClass(cls):
         super().setUpClass()
-        cls.env = cls.env(context=dict(cls.env.context, tracking_disable=True))
         cls.location_obj = cls.env["stock.location"]
-        product_obj = cls.env["product.product"]
+        cls.product_obj = cls.env["product.product"]
         cls.wizard_obj = cls.env["wiz.stock.move.location"]
         cls.quant_obj = cls.env["stock.quant"]
         cls.company = cls.env.ref("base.main_company")
@@ -43,14 +43,14 @@ class TestsCommon(common.TransactionCase):
             }
         )
         cls.uom_unit = cls.env.ref("uom.product_uom_unit")
-        cls.product_no_lots = product_obj.create(
-            {"name": "Pineapple", "type": "product", "tracking": "none"}
+        cls.product_no_lots = cls.product_obj.create(
+            {"name": "Pineapple", "is_storable": True, "tracking": "none"}
         )
-        cls.product_lots = product_obj.create(
-            {"name": "Apple", "type": "product", "tracking": "lot"}
+        cls.product_lots = cls.product_obj.create(
+            {"name": "Apple", "is_storable": True, "tracking": "lot"}
         )
-        cls.product_package = product_obj.create(
-            {"name": "Orange", "type": "product", "tracking": "lot"}
+        cls.product_package = cls.product_obj.create(
+            {"name": "Orange", "is_storable": True, "tracking": "lot"}
         )
         cls.lot1 = cls.env["stock.lot"].create(
             {
@@ -73,8 +73,8 @@ class TestsCommon(common.TransactionCase):
                 "company_id": cls.company.id,
             }
         )
-        cls.product_package = product_obj.create(
-            {"name": "Orange", "type": "product", "tracking": "lot"}
+        cls.product_package = cls.product_obj.create(
+            {"name": "Orange", "is_storable": True, "tracking": "lot"}
         )
         cls.lot4 = cls.env["stock.lot"].create(
             {
@@ -157,12 +157,15 @@ class TestsCommon(common.TransactionCase):
             amount,
         )
 
-    def _create_wizard(self, origin_location, destination_location):
+    def _create_wizard(
+        self, origin_location, destination_location, exclude_reserved_qty=False
+    ):
         move_location_wizard = self.env["wiz.stock.move.location"]
         return move_location_wizard.create(
             {
                 "origin_location_id": origin_location.id,
                 "destination_location_id": destination_location.id,
+                "exclude_reserved_qty": exclude_reserved_qty,
             }
         )
 

odoo/addons/stock_move_location/tests/test_move_location.py

@@ -84,20 +84,18 @@ class TestMoveLocation(TestsCommon):
         wizard = self._create_wizard(self.internal_loc_1, self.internal_loc_2)
         wizard.onchange_origin_location()
         self.assertEqual(len(wizard.stock_move_location_line_ids), 7)
-        wizard._onchange_destination_location_id()
-        self.assertEqual(len(wizard.stock_move_location_line_ids), 7)
         dest_location_line = wizard.stock_move_location_line_ids.mapped(
             "destination_location_id"
         )
         self.assertEqual(dest_location_line, wizard.destination_location_id)
-        wizard._onchange_origin_location_id()
+        wizard.clear_lines()
         self.assertEqual(len(wizard.stock_move_location_line_ids), 0)
 
     def test_wizard_onchange_origin_location(self):
         """Test a product that have existing quants with undefined quantity."""
 
         product_not_available = self.env["product.product"].create(
-            {"name": "Mango", "type": "product", "tracking": "none"}
+            {"name": "Mango", "is_storable": True, "tracking": "none"}
         )
         self.quant_obj.create(
             {
@@ -137,7 +135,7 @@ class TestMoveLocation(TestsCommon):
         )
         picking_lines = sorted(
             [
-                (line.product_id.id, line.lot_id.id, line.reserved_uom_qty)
+                (line.product_id.id, line.lot_id.id, line.quantity)
                 for line in picking.move_line_ids
             ],
             key=lambda x: (x[0], x[1]),
@@ -148,13 +146,13 @@ class TestMoveLocation(TestsCommon):
             "Mismatch between move location lines and move lines",
         )
         self.assertEqual(
-            sorted(picking.move_line_ids.mapped("reserved_uom_qty")),
+            sorted(picking.move_line_ids.mapped("quantity")),
             [1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 123.0],
         )
 
     def test_planned_transfer_strict(self):
         product = self.env["product.product"].create(
-            {"name": "Test", "type": "product", "tracking": "lot"}
+            {"name": "Test", "is_storable": True, "tracking": "lot"}
         )
         lot = self.env["stock.lot"].create(
             {
@@ -193,14 +191,14 @@ class TestMoveLocation(TestsCommon):
             location_line.move_quantity,
         ]
         line = picking.move_line_ids
-        picking_lines = [line.product_id.id, line.lot_id.id, line.reserved_uom_qty]
+        picking_lines = [line.product_id.id, line.lot_id.id, line.quantity]
         self.assertEqual(
             wizard_lines,
             picking_lines,
             "Mismatch between move location lines and move lines",
         )
         self.assertEqual(
-            picking.move_line_ids.reserved_uom_qty,
+            picking.move_line_ids.quantity,
             10.0,
         )
 
@@ -214,14 +212,10 @@ class TestMoveLocation(TestsCommon):
         location_lines.unlink()
         wizard.action_move_location()
         picking = wizard.picking_id
-        self.assertEqual(picking.state, "assigned")
-        self.assertEqual(
-            len(wizard.stock_move_location_line_ids), len(picking.move_line_ids)
-        )
-        self.assertEqual(
-            picking.move_line_ids.mapped("reserved_uom_qty"),
-            [0.0],
-        )
+        # Planned transfer state is "confirmed"
+        # move lines (quantity is zero) are removed
+        self.assertEqual(picking.state, "confirmed")
+        self.assertFalse(picking.move_line_ids)
 
     def test_quant_transfer(self):
         """Test quants transfer."""
@@ -241,10 +235,8 @@ class TestMoveLocation(TestsCommon):
         wizard.onchange_origin_location()
         self.assertEqual(len(lines), 3)
         wizard.destination_location_id = self.internal_loc_1
-        wizard._onchange_destination_location_id()
         self.assertEqual(lines.mapped("destination_location_id"), self.internal_loc_1)
         wizard.origin_location_id = self.internal_loc_2
-        wizard._onchange_destination_location_id()
         self.assertEqual(len(lines), 3)
 
     def test_readonly_location_computation(self):
@@ -276,7 +268,7 @@ class TestMoveLocation(TestsCommon):
             lambda p: p.product_id == self.product_no_lots
         )[0]
         self.assertEqual(
-            putaway_line.destination_location_id, self.internal_loc_2_shelf
+            putaway_line.destination_location_id, wizard.destination_location_id
         )
         picking_action = wizard.action_move_location()
         picking = self.env["stock.picking"].browse(picking_action["res_id"])
@@ -302,6 +294,7 @@ class TestMoveLocation(TestsCommon):
 
         # Create and assign a delivery picking to reserve some quantities
         delivery_picking = self._create_picking(delivery_order_type)
+        # delivery_picking.location_id = wh_stock_shelf_1
         delivery_move = self.env["stock.move"].create(
             {
                 "name": "Delivery move",
@@ -315,12 +308,14 @@ class TestMoveLocation(TestsCommon):
         )
         delivery_picking.action_confirm()
         self.assertEqual(delivery_picking.state, "assigned")
+        self.assertEqual(delivery_move.move_line_ids.location_id, wh_stock_shelf_1)
 
         # Move all quantities to other location using module's wizard
         wizard = self._create_wizard(wh_stock_shelf_1, wh_stock_shelf_2)
         wizard.onchange_origin_location()
         wizard.action_move_location()
-        self.assertEqual(delivery_picking.state, "confirmed")
+        self.assertEqual(delivery_picking.state, "assigned")
+        self.assertEqual(delivery_move.move_line_ids.location_id, wh_stock_shelf_2)
 
         # Do a planned transfer to move quantities to other location
         #  without using module's wizard
@@ -344,8 +339,8 @@ class TestMoveLocation(TestsCommon):
         self.assertEqual(delivery_picking.state, "confirmed")
         internal_picking.action_confirm()
         internal_picking.action_assign()
-        internal_picking.move_line_ids.qty_done = (
-            internal_picking.move_line_ids.reserved_uom_qty
+        internal_picking.move_line_ids.quantity = (
+            internal_picking.move_line_ids.quantity
         )
         internal_picking.button_validate()
         self.assertEqual(internal_picking.state, "done")
@@ -354,5 +349,5 @@ class TestMoveLocation(TestsCommon):
         self.assertEqual(delivery_picking.state, "assigned")
         # The old reserved quantities must be in new location after confirm wizard
         self.assertEqual(len(delivery_move.move_line_ids), 1)
-        self.assertEqual(delivery_move.move_line_ids.reserved_uom_qty, 20.0)
+        self.assertEqual(delivery_move.move_line_ids.quantity, 20.0)
         self.assertEqual(delivery_move.move_line_ids.location_id, wh_stock_shelf_3)

odoo/addons/stock_move_location/tests/test_stock_fillwithstock.py

@@ -2,63 +2,58 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).
 
 
-import odoo.tests.common as common
+from odoo.addons.base.tests.common import BaseCommon
 
 
-class TestFillwithStock(common.TransactionCase):
-    def setUp(self):
-        super(TestFillwithStock, self).setUp()
-        self.env = self.env(
-            context=dict(
-                self.env.context,
-                tracking_disable=True,
-            )
-        )
+class TestFillwithStock(BaseCommon):
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
 
-        self.stock_location = self.env.ref("stock.stock_location_stock")
-        self.pack_location = self.env.ref("stock.location_pack_zone")
+        cls.stock_location = cls.env.ref("stock.stock_location_stock")
+        cls.pack_location = cls.env.ref("stock.location_pack_zone")
 
-        self.shelf1_location = self.env["stock.location"].create(
+        cls.shelf1_location = cls.env["stock.location"].create(
             {
                 "name": "Test location",
                 "usage": "internal",
-                "location_id": self.stock_location.id,
+                "location_id": cls.stock_location.id,
             }
         )
 
-        self.product1 = self.env["product.product"].create(
+        cls.product1 = cls.env["product.product"].create(
             {
                 "name": "Product A",
-                "type": "product",
+                "is_storable": True,
             }
         )
-        self.product2 = self.env["product.product"].create(
+        cls.product2 = cls.env["product.product"].create(
             {
                 "name": "Product B",
-                "type": "product",
+                "is_storable": True,
             }
         )
 
-        self.env["stock.quant"].create(
+        cls.env["stock.quant"].create(
             {
-                "product_id": self.product1.id,
-                "location_id": self.shelf1_location.id,
+                "product_id": cls.product1.id,
+                "location_id": cls.shelf1_location.id,
                 "quantity": 5.0,
                 "reserved_quantity": 0.0,
             }
         )
-        self.env["stock.quant"].create(
+        cls.env["stock.quant"].create(
             {
-                "product_id": self.product1.id,
-                "location_id": self.shelf1_location.id,
+                "product_id": cls.product1.id,
+                "location_id": cls.shelf1_location.id,
                 "quantity": 10.0,
                 "reserved_quantity": 5.0,
             }
         )
-        self.env["stock.quant"].create(
+        cls.env["stock.quant"].create(
             {
-                "product_id": self.product2.id,
-                "location_id": self.shelf1_location.id,
+                "product_id": cls.product2.id,
+                "location_id": cls.shelf1_location.id,
                 "quantity": 5.0,
                 "reserved_quantity": 0.0,
             }
@@ -80,7 +75,7 @@ class TestFillwithStock(common.TransactionCase):
             picking_stock_pack.move_ids.filtered(
                 lambda m: m.product_id == self.product1
             ).product_uom_qty,
-            10.0,
+            15.0,
         )
         self.assertEqual(
             picking_stock_pack.move_ids.filtered(

odoo/addons/stock_move_location/views/stock_picking.xml

@@ -1,6 +1,5 @@
 <?xml version="1.0" ?>
 <odoo>
-
     <record id="view_picking_form" model="ir.ui.view">
         <field name="name">stock.picking.form.fillwithstock</field>
         <field name="model">stock.picking</field>
@@ -9,7 +8,7 @@
             <button name="action_confirm" position="before">
                 <button
                     name="button_fillwithstock"
-                    states="draft"
+                    invisible="state != 'draft'"
                     string="Fill with stock"
                     type="object"
                     class="oe_highlight"
@@ -18,5 +17,4 @@
             </button>
         </field>
     </record>
-
 </odoo>