[MIG] base_location_nuts: Migration to 19.0

Borruso · Borruso · commit e882ace524c3 · 2026-02-09T11:28:06.000+01:00
diff --git a/base_location_nuts/README.rst b/base_location_nuts/README.rst
@@ -21,13 +21,13 @@ NUTS Regions
     :target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
     :alt: License: AGPL-3
 .. |badge3| image:: https://img.shields.io/badge/github-OCA%2Fpartner--contact-lightgray.png?logo=github
-    :target: https://github.com/OCA/partner-contact/tree/18.0/base_location_nuts
+    :target: https://github.com/OCA/partner-contact/tree/19.0/base_location_nuts
     :alt: OCA/partner-contact
 .. |badge4| image:: https://img.shields.io/badge/weblate-Translate%20me-F47D42.png
-    :target: https://translation.odoo-community.org/projects/partner-contact-18-0/partner-contact-18-0-base_location_nuts
+    :target: https://translation.odoo-community.org/projects/partner-contact-19-0/partner-contact-19-0-base_location_nuts
     :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/partner-contact&target_branch=18.0
+    :target: https://runboat.odoo-community.org/builds?repo=OCA/partner-contact&target_branch=19.0
     :alt: Try me on Runboat
 
 |badge1| |badge2| |badge3| |badge4| |badge5|
@@ -36,10 +36,10 @@ This module allows to import NUTS locations.
 
 Creates four new fields in Partner object, one per NUTS level
 
-- NUTS L1: Country level
-- NUTS L2: Normally state or big region level
-- NUTS L3: Normally substate or state level
-- NUTS L4: Normally small region or province level
+-  NUTS L1: Country level
+-  NUTS L2: Normally state or big region level
+-  NUTS L3: Normally substate or state level
+-  NUTS L4: Normally small region or province level
 
 This module allows to set the flag *Not updatable* in a NUTS region so
 that it gets no more updated nor deleted by the import wizard.
@@ -60,12 +60,12 @@ Installation
 We recommend to install another addon (one for each country) in order to
 relate NUTS with states defined by each localization addon, for example:
 
-- l10n_es_location_nuts : Spanish Provinces (NUTS level 4) related to
-  Partner State
-- l10n_de_location_nuts : German states (NUTS level 2) related to
-  Partner State
-- l10n_nl_location_nuts : Dutch provinces (NUTS level 3 and 4) related
-  to Partner State
+-  l10n_es_location_nuts : Spanish Provinces (NUTS level 4) related to
+   Partner State
+-  l10n_de_location_nuts : German states (NUTS level 2) related to
+   Partner State
+-  l10n_nl_location_nuts : Dutch provinces (NUTS level 3 and 4) related
+   to Partner State
 
 Configuration
 =============
@@ -96,7 +96,7 @@ Bug Tracker
 Bugs are tracked on `GitHub Issues <https://github.com/OCA/partner-contact/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/partner-contact/issues/new?body=module:%20base_location_nuts%0Aversion:%2018.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.
+`feedback <https://github.com/OCA/partner-contact/issues/new?body=module:%20base_location_nuts%0Aversion:%2019.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.
 
@@ -112,15 +112,15 @@ Authors
 Contributors
 ------------
 
-- Rafael Blasco <rafael.blasco@tecnativa.com>
-- Antonio Espinosa <antonio.espinosa@tecnativa.com>
-- Jairo Llopis <jairo.llopis@tecnativa.com>
-- David Vidal <david.vidal@tecnativa.com>
-- Simone Rubino <simone.rubino@agilebg.com>
-- Alexandre Díaz <alexandre.diaz@tecnativa.com>
-- Andrea Stirpe <a.stirpe@onestein.nl>
-- Eduardo López <elp@moduon.team>
-- Andrii Kompaniiets <andrii@moduon.team>
+-  Rafael Blasco <rafael.blasco@tecnativa.com>
+-  Antonio Espinosa <antonio.espinosa@tecnativa.com>
+-  Jairo Llopis <jairo.llopis@tecnativa.com>
+-  David Vidal <david.vidal@tecnativa.com>
+-  Simone Rubino <simone.rubino@agilebg.com>
+-  Alexandre Díaz <alexandre.diaz@tecnativa.com>
+-  Andrea Stirpe <a.stirpe@onestein.nl>
+-  Eduardo López <elp@moduon.team>
+-  Andrii Kompaniiets <andrii@moduon.team>
 
 Maintainers
 -----------
@@ -149,6 +149,6 @@ Current `maintainers <https://odoo-community.org/page/maintainer-role>`__:
 
 |maintainer-rafaelbn| |maintainer-edlopen| |maintainer-Andrii9090| 
 
-This module is part of the `OCA/partner-contact <https://github.com/OCA/partner-contact/tree/18.0/base_location_nuts>`_ project on GitHub.
+This module is part of the `OCA/partner-contact <https://github.com/OCA/partner-contact/tree/19.0/base_location_nuts>`_ project on GitHub.
 
 You are welcome to contribute. To learn how please visit https://odoo-community.org/page/Contribute.
diff --git a/base_location_nuts/__manifest__.py b/base_location_nuts/__manifest__.py
@@ -7,7 +7,7 @@
 {
     "name": "NUTS Regions",
     "category": "Localization/Europe",
-    "version": "18.0.1.2.0",
+    "version": "19.0.1.0.0",
     "depends": ["contacts"],
     "data": [
         "views/res_country_view.xml",
diff --git a/base_location_nuts/models/res_partner.py b/base_location_nuts/models/res_partner.py
@@ -4,7 +4,7 @@
 # License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl.html).
 
 from odoo import api, fields, models
-from odoo.osv.expression import AND, OR
+from odoo.fields import Domain
 
 
 class ResPartner(models.Model):
@@ -40,35 +40,34 @@ class ResPartner(models.Model):
     def _compute_allowed_nuts(self):
         Nuts = self.env["res.partner.nuts"]
         for partner in self:
-            domain = []
+            domain = Domain(Domain.TRUE)
             for level in range(1, 4):
-                nuts = partner["nuts%d_id" % level]
+                nuts = partner[f"nuts{level}_id"]
                 if nuts:
                     if not domain:
-                        domain = OR(
-                            [
-                                [
-                                    ("parent_id", "=", nuts.id),
-                                    ("level", "=", level + 1),
-                                ],
-                            ]
+                        domain = Domain(
+                            [("parent_id", "=", nuts.id), ("level", "=", level + 1)]
                         )
                     else:
-                        domain = OR(
+                        domain = Domain.OR(
                             [
-                                [
-                                    ("parent_id", "=", nuts.id),
-                                    ("level", "=", level + 1),
-                                ],
+                                Domain(
+                                    [
+                                        ("parent_id", "=", nuts.id),
+                                        ("level", "=", level + 1),
+                                    ]
+                                ),
                                 domain,
                             ]
                         )
             if partner.country_id:
-                domain = AND([[("country_id", "=", partner.country_id.id)], domain])
+                domain = Domain.AND(
+                    [Domain("country_id", "=", partner.country_id.id), domain]
+                )
             partner.allowed_nut_ids = Nuts.search(domain)
 
     def _onchange_nuts(self, level):
-        field = self["nuts%d_id" % level]
+        field = self[f"nuts{level}_id"]
         country_id = field.country_id
         state_id = field.state_id
         if country_id and self.country_id != country_id:
@@ -78,7 +77,7 @@ def _onchange_nuts(self, level):
         if level > 1:
             parent_id = field.parent_id
             if parent_id:
-                nuts_parent_level = "nuts%d_id" % (level - 1)
+                nuts_parent_level = f"nuts{level - 1}_id"
                 parent_field = self[nuts_parent_level]
                 if parent_field != parent_id:
                     self[nuts_parent_level] = parent_id
@@ -127,7 +126,7 @@ def onchange_state_id_base_location_nuts(self):
                     limit=1,
                 )
                 if nuts_state:
-                    field = "nuts%d_id" % self.country_id.state_level
+                    field = f"nuts{self.country_id.state_level}_id"
                     self[field] = nuts_state
 
     @api.model
diff --git a/base_location_nuts/static/description/index.html b/base_location_nuts/static/description/index.html
@@ -374,7 +374,7 @@ <h1>NUTS Regions</h1>
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
 !! source digest: sha256:c6e27ffb13e1a827941bf8a8fb29f8b71115cd12f2f6fb02593d91af19803ee8
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/partner-contact/tree/18.0/base_location_nuts"><img alt="OCA/partner-contact" src="https://img.shields.io/badge/github-OCA%2Fpartner--contact-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/partner-contact-18-0/partner-contact-18-0-base_location_nuts"><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/partner-contact&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><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Beta" src="https://img.shields.io/badge/maturity-Beta-yellow.png" /></a> <a class="reference external image-reference" href="http://www.gnu.org/licenses/agpl-3.0-standalone.html"><img alt="License: AGPL-3" src="https://img.shields.io/badge/license-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/partner-contact/tree/19.0/base_location_nuts"><img alt="OCA/partner-contact" src="https://img.shields.io/badge/github-OCA%2Fpartner--contact-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/partner-contact-19-0/partner-contact-19-0-base_location_nuts"><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/partner-contact&amp;target_branch=19.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 import NUTS locations.</p>
 <p>Creates four new fields in Partner object, one per NUTS level</p>
 <ul class="simple">
@@ -441,7 +441,7 @@ <h2><a class="toc-backref" href="#toc-entry-4">Bug Tracker</a></h2>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/partner-contact/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/partner-contact/issues/new?body=module:%20base_location_nuts%0Aversion:%2018.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/partner-contact/issues/new?body=module:%20base_location_nuts%0Aversion:%2019.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">
@@ -478,7 +478,7 @@ <h3><a class="toc-backref" href="#toc-entry-8">Maintainers</a></h3>
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainers</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/rafaelbn"><img alt="rafaelbn" src="https://github.com/rafaelbn.png?size=40px" /></a> <a class="reference external image-reference" href="https://github.com/edlopen"><img alt="edlopen" src="https://github.com/edlopen.png?size=40px" /></a> <a class="reference external image-reference" href="https://github.com/Andrii9090"><img alt="Andrii9090" src="https://github.com/Andrii9090.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/partner-contact/tree/18.0/base_location_nuts">OCA/partner-contact</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/partner-contact/tree/19.0/base_location_nuts">OCA/partner-contact</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>
diff --git a/base_location_nuts/views/res_country_view.xml b/base_location_nuts/views/res_country_view.xml
@@ -5,7 +5,7 @@
         <field name="model">res.country</field>
         <field name="inherit_id" ref="base.view_country_form" />
         <field name="arch" type="xml">
-            <xpath expr="/form">
+            <xpath expr="//form/sheet">
                 <group name="nuts" string="NUTS">
                     <group>
                         <field name="state_level" />
diff --git a/base_location_nuts/views/res_partner_nuts_view.xml b/base_location_nuts/views/res_partner_nuts_view.xml
@@ -34,6 +34,7 @@
     </record>
     <record id="res_partner_nuts_action" model="ir.actions.act_window">
         <field name="name">NUTS Items</field>
+        <field name="path">nuts-items</field>
         <field name="res_model">res.partner.nuts</field>
         <field name="view_mode">list,form</field>
         <field name="help">You must click at import wizard to populate NUTS items
@@ -48,7 +49,7 @@
                 <field name="name" />
                 <field name="country_id" />
                 <field name="state_id" />
-                <group expand="0" string="Group By">
+                <group>
                     <filter
                         name="country"
                         string="Country"
diff --git a/base_location_nuts/wizard/nuts_import.py b/base_location_nuts/wizard/nuts_import.py
@@ -8,7 +8,8 @@
 
 import requests
 
-from odoo import _, api, models
+from odoo import api, models
+from odoo.fields import Domain
 
 logger = logging.getLogger(__name__)
 
@@ -159,13 +160,14 @@ def _delete_obsolete_nuts(self, new_nuts_ids):
             ntd2 = nuts_to_delete.filtered(lambda nut: nut.level == level)  # noqa: B023
             ntd2.unlink()
             nuts_to_delete -= ntd2
-        logger.info("%d NUTS entries deleted" % len(nuts_to_delete))
+        logger.info(f"{len(nuts_to_delete)} NUTS entries deleted")
 
     def _create_partner_nuts(self, nuts_data):
         """Creates or updates NUTS regions from the given GET request."""
         nuts_ids = self.env["res.partner.nuts"]
         nuts_json = nuts_data.json()
         bindings = nuts_json.get("results", {}).get("bindings", [])
+        # pylint: disable=no-search-all
         all_nuts = nuts_ids.search_read([], ["code"])
         nuts_map = {nut.get("code"): nut.get("id") for nut in all_nuts}
         for binding in bindings:
@@ -225,10 +227,10 @@ def action_partner_nuts(self):
         nuts_ids = self.import_update_partner_nuts()
         tree_view_id = self.env.ref("base_location_nuts.res_partner_nuts_tree").id
         return {
-            "name": _("Partner NUTS by EU"),
+            "name": self.env._("Partner NUTS by EU"),
             "view_mode": "list",
             "res_model": "res.partner.nuts",
             "view_id": tree_view_id,
             "type": "ir.actions.act_window",
-            "domain": [("id", "=", nuts_ids.ids)],
+            "domain": Domain("id", "=", nuts_ids.ids),
         }
diff --git a/base_location_nuts/wizard/nuts_import_view.xml b/base_location_nuts/wizard/nuts_import_view.xml
@@ -28,7 +28,7 @@
     </record>
     <record id="nuts_import_action" model="ir.actions.act_window">
         <field name="name">Import NUTS 2024 from SHOWVOC</field>
-        <field name="type">ir.actions.act_window</field>
+        <field name="path">import-nuts</field>
         <field name="res_model">nuts.import</field>
         <field name="view_id" ref="nuts_import_form" />
         <field name="view_mode">form</field>

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@`
`7`	`7`	`{`
`8`	`8`	`"name": "NUTS Regions",`
`9`	`9`	`"category": "Localization/Europe",`
`10`		`- "version": "18.0.1.2.0",`
	`10`	`+ "version": "19.0.1.0.0",`
`11`	`11`	`"depends": ["contacts"],`
`12`	`12`	`"data": [`
`13`	`13`	`"views/res_country_view.xml",`