[MIG] purchase_location_by_line: Migration to 12.0

Author: MiquelRForgeFlow

purchase_location_by_line/README.rst

@@ -14,13 +14,13 @@ Purchase Location by Line
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fpurchase--workflow-lightgray.png?logo=github
-    :target: https://github.com/OCA/purchase-workflow/tree/11.0/purchase_location_by_line
+    :target: https://github.com/OCA/purchase-workflow/tree/12.0/purchase_location_by_line
     :alt: OCA/purchase-workflow
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/purchase-workflow-11-0/purchase-workflow-11-0-purchase_location_by_line
+    :target: https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_location_by_line
     :alt: Translate me on Weblate
 .. |badge5| image:: https://img.shields.io/badge/runbot-Try%20me-875A7B.png
-    :target: https://runbot.odoo-community.org/runbot/142/11.0
+    :target: https://runbot.odoo-community.org/runbot/142/12.0
     :alt: Try me on Runbot
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -44,7 +44,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/purchase-workflow/issues>`_.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us smashing it by providing a detailed and welcomed
-`feedback <https://github.com/OCA/purchase-workflow/issues/new?body=module:%20purchase_location_by_line%0Aversion:%2011.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/purchase-workflow/issues/new?body=module:%20purchase_location_by_line%0Aversion:%2012.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.
 
@@ -75,6 +75,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/purchase-workflow <https://github.com/OCA/purchase-workflow/tree/11.0/purchase_location_by_line>`_ project on GitHub.
+This module is part of the `OCA/purchase-workflow <https://github.com/OCA/purchase-workflow/tree/12.0/purchase_location_by_line>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.

purchase_location_by_line/__init__.py

@@ -1 +1,3 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
 from . import models

purchase_location_by_line/__manifest__.py

@@ -7,13 +7,13 @@
     "name": "Purchase Location by Line",
     "summary": "Allows to define a specific destination location on each PO "
                "line",
-    "version": "11.0.1.0.0",
+    "version": "12.0.1.0.0",
     "author": "Eficent, "
               "Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/purchase-workflow",
     "category": "Purchase Management",
     "depends": [
-        "purchase",
+        "purchase_stock",
         "purchase_delivery_split_date"
     ],
     "license": "AGPL-3",

purchase_location_by_line/models/__init__.py

@@ -1,2 +1,4 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
 from . import purchase
 from . import stock_picking

purchase_location_by_line/models/purchase.py

@@ -10,16 +10,16 @@ class PurchaseOrderLine(models.Model):
     _inherit = 'purchase.order.line'
 
     location_dest_id = fields.Many2one(
-        'stock.location', 'Destination', domain=[('usage', 'in',
-                                                  ['internal', 'transit'])])
+        comodel_name='stock.location', string='Destination',
+        domain=[('usage', 'in', ['internal', 'transit'])])
 
     @api.model
     def _first_picking_copy_vals(self, key, lines):
         """The data to be copied to new pickings is updated with data from the
         grouping key.  This method is designed for extensibility, so that
         other modules can store more data based on new keys."""
-        vals = super(PurchaseOrderLine, self)._first_picking_copy_vals(key,
-                                                                       lines)
+        vals = super(PurchaseOrderLine, self)._first_picking_copy_vals(
+            key, lines)
         for key_element in key:
             if 'location_dest_id' in key_element.keys():
                 vals['location_dest_id'] = key_element['location_dest_id'].id
@@ -32,8 +32,8 @@ def _get_group_keys(self, order, line, picking=False):
         dictionary element with the field that you want to group by. This
         method is designed for extensibility, so that other modules can add
         additional keys or replace them by others."""
-        key = super(PurchaseOrderLine, self)._get_group_keys(order, line,
-                                                             picking=picking)
+        key = super(PurchaseOrderLine, self)._get_group_keys(
+            order, line, picking=picking)
         default_picking_location_id = line.order_id._get_destination_location()
         default_picking_location = self.env['stock.location'].browse(
             default_picking_location_id)

purchase_location_by_line/static/description/index.html

@@ -367,7 +367,7 @@ <h1 class="title">Purchase Location by Line</h1>
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external" 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" 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" href="https://github.com/OCA/purchase-workflow/tree/11.0/purchase_location_by_line"><img alt="OCA/purchase-workflow" src="https://img.shields.io/badge/github-OCA%2Fpurchase--workflow-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/purchase-workflow-11-0/purchase-workflow-11-0-purchase_location_by_line"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runbot.odoo-community.org/runbot/142/11.0"><img alt="Try me on Runbot" src="https://img.shields.io/badge/runbot-Try%20me-875A7B.png" /></a></p>
+<p><a class="reference external" 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" 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" href="https://github.com/OCA/purchase-workflow/tree/12.0/purchase_location_by_line"><img alt="OCA/purchase-workflow" src="https://img.shields.io/badge/github-OCA%2Fpurchase--workflow-lightgray.png?logo=github" /></a> <a class="reference external" href="https://translation.odoo-community.org/projects/purchase-workflow-12-0/purchase-workflow-12-0-purchase_location_by_line"><img alt="Translate me on Weblate" src="https://img.shields.io/badge/weblate-Translate%20me-F47D42.png" /></a> <a class="reference external" href="https://runbot.odoo-community.org/runbot/142/12.0"><img alt="Try me on Runbot" src="https://img.shields.io/badge/runbot-Try%20me-875A7B.png" /></a></p>
 <p>Allows to define a specific destination location on each Purchase Order line.
 When the PO is confirmed, it will generate one Incoming Shipment per
 combination of destination location and expected date indicated in the
@@ -391,7 +391,7 @@ <h1><a class="toc-backref" href="#id1">Bug Tracker</a></h1>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/purchase-workflow/issues">GitHub Issues</a>.
 In case of trouble, please check there if your issue has already been reported.
 If you spotted it first, help us smashing it by providing a detailed and welcomed
-<a class="reference external" href="https://github.com/OCA/purchase-workflow/issues/new?body=module:%20purchase_location_by_line%0Aversion:%2011.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/purchase-workflow/issues/new?body=module:%20purchase_location_by_line%0Aversion:%2012.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">
@@ -416,7 +416,7 @@ <h2><a class="toc-backref" href="#id5">Maintainers</a></h2>
 <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/purchase-workflow/tree/11.0/purchase_location_by_line">OCA/purchase-workflow</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/purchase-workflow/tree/12.0/purchase_location_by_line">OCA/purchase-workflow</a> project on GitHub.</p>
 <p>You are welcome to contribute. To learn how please visit <a class="reference external" href="https://odoo-community.org/page/Contribute">https://odoo-community.org/page/Contribute</a>.</p>
 </div>
 </div>

purchase_location_by_line/tests/__init__.py

@@ -1 +1,3 @@
+# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
+
 from . import test_purchase_delivery

purchase_location_by_line/tests/test_purchase_delivery.py

@@ -63,7 +63,7 @@ def test_check_single_date(self):
             "There must be 1 picking for the PO when confirmed")
 
         self.assertEquals(
-            self.po.picking_ids[0].scheduled_date[:10], self.date_sooner,
+            str(self.po.picking_ids[0].scheduled_date)[:10], self.date_sooner,
             "The picking must be planned at the expected date")
 
     def test_check_multiple_dates(self):
@@ -85,10 +85,10 @@ def test_check_multiple_dates(self):
         sorted_pickings = sorted(
             self.po.picking_ids, key=lambda x: x.scheduled_date)
         self.assertEquals(
-            sorted_pickings[0].scheduled_date[:10], self.date_sooner,
+            str(sorted_pickings[0].scheduled_date)[:10], self.date_sooner,
             "The first picking must be planned at the soonest date")
         self.assertEquals(
-            sorted_pickings[1].scheduled_date[:10], self.date_later,
+            str(sorted_pickings[1].scheduled_date)[:10], self.date_later,
             "The second picking must be planned at the latest date")
 
         l2_picking = self.po.picking_ids.filtered(
@@ -145,10 +145,10 @@ def test_check_multiple_locations_multiple_dates(self):
         sorted_pickings = sorted(
             self.po.picking_ids, key=lambda x: x.scheduled_date)
         self.assertEquals(
-            sorted_pickings[0].scheduled_date[:10], self.date_sooner,
+            str(sorted_pickings[0].scheduled_date)[:10], self.date_sooner,
             "The first picking must be planned at the soonest date")
         self.assertEquals(
-            sorted_pickings[2].scheduled_date[:10], self.date_later,
+            str(sorted_pickings[2].scheduled_date)[:10], self.date_later,
             "The second picking must be planned at the latest date")
 
     def test_check_multiple_locations_multiple_dates_02(self):
@@ -178,8 +178,8 @@ def test_check_multiple_locations_multiple_dates_02(self):
         sorted_pickings = sorted(
             self.po.picking_ids, key=lambda x: x.scheduled_date)
         self.assertEquals(
-            sorted_pickings[0].scheduled_date[:10], self.date_sooner,
+            str(sorted_pickings[0].scheduled_date)[:10], self.date_sooner,
             "The first picking must be planned at the soonest date")
         self.assertEquals(
-            sorted_pickings[2].scheduled_date[:10], self.date_later,
+            str(sorted_pickings[2].scheduled_date)[:10], self.date_later,
             "The second picking must be planned at the latest date")