odoo-addon-stock-move-location 16.0.1.4.0.2__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:b188235583d8ffee97982515cd164a26620734c9758c886ffae1b371d3843acc
+   !! 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|
@@ -39,27 +39,25 @@ another and move only selected quantities.
 Usage
 =====
 
--  A new menu item Operations > Move from location... opens a wizard
-   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
--  If during your operation with the wizard the real quantity will
-   change it will move only the available quantity at the button press
--  Products will be moved and a form view of picking that did that will
-   show up
--  If "PLANNED TRANSFER" is used - the picking won't be validated
-   automatically
+- A new menu item Operations > Move from location... opens a wizard
+  where 2 locations can be specified.
+- Select origin and destination locations and press "IMMEDIATE TRANSFER"
+  or "PLANNED TRANSFER"
+- 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 only the available quantity at the button press
+- Products will be moved and a form view of picking that did that will
+  show up
+- If "PLANNED TRANSFER" is used - the picking won't be validated
+  automatically
 
 If you want to transfer a full quant:
 
--  Go to Inventory > Products > Products and click "On hand" smart
-   button or Inventory > Reporting > Inventory, the quants view will be
-   opened.
--  Select the quantities which you want move to another location
+- Go to Inventory > Products > Products and click "On hand" smart button
+  or Inventory > Reporting > Inventory, the quants view will be opened.
+- Select the quantities which you want move to another location
 
 If you go to the Inventory Dashboard you can see the button "Move from
 location" in each of the picking types (only applicable to internal and
@@ -92,14 +90,14 @@ Known issues / Roadmap
 Change the current implementation (suggested by Denis Roussel from
 ACSONE):
 
--  A new parameter on stock picking types : 'Product Change Location'
-   (with a little help).
--  With this, go to the dashboard, create a picking with that type.
--  Add a button on the picking form which is visible with that type that
-   fill in the picking as now
--  Nice to have: add a magic button on locations that with context
-   creates a new picking of that type with the origin location already
-   filled in.
+- A new parameter on stock picking types : 'Product Change Location'
+  (with a little help).
+- With this, go to the dashboard, create a picking with that type.
+- Add a button on the picking form which is visible with that type that
+  fill in the picking as now
+- Nice to have: add a magic button on locations that with context
+  creates a new picking of that type with the origin location already
+  filled in.
 
 Bug Tracker
 ===========
@@ -107,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.
 
@@ -124,26 +122,28 @@ Authors
 Contributors
 ------------
 
--  Mathieu Vatel <mathieu@julius.fr>
--  Mykhailo Panarin <m.panarin@mobilunity.com>
--  Joan Sisquella <joan.sisquella@forgeflow.com>
--  Jordi Ballester Alomar <jordi.ballester@forgeflow.com>
--  Lois Rilo <lois.rilo@forgeflow.com>
--  Héctor Villarreal <hector.villarreal@forgeflow.com>
--  Tecnativa <tecnativa.com>
+- Mathieu Vatel <mathieu@julius.fr>
+- Mykhailo Panarin <m.panarin@mobilunity.com>
+- Joan Sisquella <joan.sisquella@forgeflow.com>
+- Jordi Ballester Alomar <jordi.ballester@forgeflow.com>
+- Lois Rilo <lois.rilo@forgeflow.com>
+- Héctor Villarreal <hector.villarreal@forgeflow.com>
+- Tecnativa <tecnativa.com>
 
-   -  Sergio Teruel
-   -  João Marques
+  - Sergio Teruel
+  - João Marques
+  - Carlos Roca
 
--  Jacques-Etienne Baudoux <je@bcim.be>
--  Iryna Vyshnevska <i.vyshnevska@mobilunity.com>
--  Alexei Rivera <arivera@archeti.com>
--  Abraham Anes <abraham@studio73.es>
--  Quartile <https://www.quartile.co>
+- Jacques-Etienne Baudoux <je@bcim.be>
+- Iryna Vyshnevska <i.vyshnevska@mobilunity.com>
+- Alexei Rivera <arivera@archeti.com>
+- Abraham Anes <abraham@studio73.es>
+- Quartile <https://www.quartile.co>
 
-   -  Aung Ko Ko Lin
+  - Aung Ko Ko Lin
 
--  Laurent Mignon <laurent.mignon@acsone.eu>
+- Laurent Mignon <laurent.mignon@acsone.eu>
+- Maksym Yankin <maksym.yankin@camptocamp.com>
 
 Maintainers
 -----------
@@ -158,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.0",
+    "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:b188235583d8ffee97982515cd164a26620734c9758c886ffae1b371d3843acc
+!! 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>
@@ -391,14 +391,13 @@ another and move only selected quantities.</p>
 <ul class="simple">
 <li>A new menu item Operations &gt; Move from location… opens a wizard
 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>Select origin and destination locations and press “IMMEDIATE TRANSFER”
+or “PLANNED TRANSFER”</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>
-<li>If during your operation with the wizard the real quantity will
-change it will move only the available quantity at the button press</li>
+<li>If during your operation with the wizard the real quantity will change
+it will move only the available quantity at the button press</li>
 <li>Products will be moved and a form view of picking that did that will
 show up</li>
 <li>If “PLANNED TRANSFER” is used - the picking won’t be validated
@@ -406,9 +405,8 @@ automatically</li>
 </ul>
 <p>If you want to transfer a full quant:</p>
 <ul class="simple">
-<li>Go to Inventory &gt; Products &gt; Products and click “On hand” smart
-button or Inventory &gt; Reporting &gt; Inventory, the quants view will be
-opened.</li>
+<li>Go to Inventory &gt; Products &gt; Products and click “On hand” smart button
+or Inventory &gt; Reporting &gt; Inventory, the quants view will be opened.</li>
 <li>Select the quantities which you want move to another location</li>
 </ul>
 <p>If you go to the Inventory Dashboard you can see the button “Move from
@@ -448,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">
@@ -473,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>
@@ -484,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">
@@ -495,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)