[MIG] base_kanban_stage: Migration to 19.0

Author: JasminSForgeFlow

base_kanban_stage/__manifest__.py

@@ -4,7 +4,7 @@
 {
     "name": "Kanban - Stage Support",
     "summary": "Provides stage model and abstract logic for inheritance",
-    "version": "16.0.1.0.0",
+    "version": "19.0.1.0.0",
     "author": "LasLabs, Odoo Community Association (OCA)",
     "category": "base",
     "depends": ["base"],

base_kanban_stage/models/base_kanban_abstract.py

@@ -111,6 +111,6 @@ def _valid_field_parameter(self, field, name):
         # allow tracking on models inheriting from 'base.kanban.stage'
         return name == "tracking" or super()._valid_field_parameter(field, name)
 
-    def _read_group_stage_ids(self, stages, domain, order):
+    def _read_group_stage_ids(self, stages, domain):
         search_domain = [("res_model_id.model", "=", self._name)]
-        return stages.search(search_domain, order=order)
+        return stages.search(search_domain)

base_kanban_stage/models/ir_model.py

@@ -1,7 +1,7 @@
-# Copyright 2019 Eficent Business and IT Consulting Services S.L.
+# Copyright 2025 ForgeFlow S.L.
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
 
-from odoo import _, api, fields, models
+from odoo import api, fields, models
 from odoo.exceptions import UserError
 
 
@@ -17,9 +17,11 @@ class IrModel(models.Model):
     def write(self, vals):
         if self and "is_kanban" in vals:
             if not all(rec.state == "manual" for rec in self):
-                raise UserError(_("Only custom models can be modified."))
+                raise UserError(self.env._("Only custom models can be modified."))
             if not all(rec.is_kanban <= vals["is_kanban"] for rec in self):
-                raise UserError(_('Field "Kanban" cannot be changed to "False".'))
+                raise UserError(
+                    self.env._('Field "Kanban" cannot be changed to "False".')
+                )
             res = super().write(vals)
             # setup models; this reloads custom models in registry
             self.pool.setup_models(self._cr)
@@ -38,8 +40,8 @@ def _reflect_model_params(self, model):
         return vals
 
     @api.model
-    def _instanciate(self, model_data):
-        model_class = super()._instanciate(model_data)
+    def _instanciate_attrs(self, model_data):
+        model_class = super()._instanciate_attrs(model_data)
         if model_data.get("is_kanban") and model_class._name != "base.kanban.abstract":
             parents = model_class._inherit or []
             parents = [parents] if isinstance(parents, (str,)) else parents

base_kanban_stage/tests/test_base_kanban_abstract.py

@@ -1,53 +1,47 @@
 # Copyright 2016-2017 LasLabs Inc.
-# Copyright 2019 ForgeFlow.
+# Copyright 2025 ForgeFlow.
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
 
-from collections import deque
-
-from odoo import api, models
+from odoo import models
 from odoo.tests.common import TransactionCase
 
 
 class BaseKanbanAbstractTester(models.Model):
     _name = "base.kanban.abstract.tester"
     _description = "base kanban abstract tester"
     _inherit = "base.kanban.abstract"
+    _order = "id asc"
 
 
 class TestBaseKanbanAbstract(TransactionCase):
-    def _init_test_model(self, model_cls):
-        model_cls._build_model(self.registry, self.cr)
-        model = self.env[model_cls._name]
-        # setup_models():
-        model._prepare_setup()
-        model._setup_base()
-        model._setup_fields()
-        model._setup_complete()
-        # init_models():
-        self.registry._post_init_queue = deque()
-        self.registry._foreign_keys = {}
-        model._auto_init()
-        model.init()
-        self.env["ir.model"]._reflect_models([model._name])
-        self.env["ir.model.fields"]._reflect_fields([model._name])
-        self.env["ir.model.fields.selection"]._reflect_selections([model._name])
-        self.env["ir.model.constraint"]._reflect_constraints([model._name])
-        while self.registry._post_init_queue:
-            func = self.registry._post_init_queue.popleft()
-            func()
-        del self.registry._post_init_queue
-        del self.registry._foreign_keys
-        return model
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
 
-    def setUp(self):
-        super().setUp()
+        # Manually register and build the test model
+        from odoo.orm import model_classes
+
+        registry = cls.env.registry
+        cr = cls.env.cr
+
+        # Build the model if not already in registry
+        if BaseKanbanAbstractTester._name not in registry.models:
+            # Add the model definition to registry
+            model_classes.add_to_registry(registry, BaseKanbanAbstractTester)
 
-        self.registry.enter_test_mode(self.cr)
-        self.old_cursor = self.cr
-        self.cr = self.registry.cursor()
-        self.env = api.Environment(self.cr, self.uid, {})
-        self.test_model = self._init_test_model(BaseKanbanAbstractTester)
+            # Setup models
+            registry._setup_models__(cr, [BaseKanbanAbstractTester._name])
 
+            # Initialize the model
+            with cls.muted_registry_logger:
+                registry.init_models(
+                    cr, [BaseKanbanAbstractTester._name], {}, install=False
+                )
+
+        cls.test_model = cls.env[BaseKanbanAbstractTester._name]
+
+    def setUp(self):
+        super().setUp()
         test_model_record = self.env["ir.model"].search(
             [
                 ("model", "=", self.test_model._name),
@@ -71,12 +65,8 @@ def setUp(self):
         )
 
     def tearDown(self):
-        self.registry.leave_test_mode()
         self.registry[self.test_model._name]._abstract = True
         self.registry[self.test_model._name]._auto = False
-        self.cr = self.old_cursor
-        self.env = api.Environment(self.cr, self.uid, {})
-
         super().tearDown()
 
     def test_default_stage_id_no_stages(self):
@@ -99,12 +89,11 @@ def test_default_stage_id_available_stages(self):
     def test_read_group_stage_ids(self):
         """It should return all corresponding stages in requested sort order"""
         result = self.test_model._read_group_stage_ids(
-            self.env["base.kanban.stage"], None, "id"
+            self.env["base.kanban.stage"], None
         )
 
         expected = self.env["base.kanban.stage"].search(
             [("res_model_id.model", "=", self.test_model._name)],
-            order="id",
         )
         self.assertEqual(result[0], expected[0])
         self.assertEqual(result[1], expected[1])

base_kanban_stage/views/base_kanban_stage.xml

@@ -32,11 +32,11 @@
         <field name="name">Kanban Stages - Tree View</field>
         <field name="model">base.kanban.stage</field>
         <field name="arch" type="xml">
-            <tree name="Kanban Stages">
+            <list name="Kanban Stages">
                 <field name="name" />
                 <field name="res_model_id" />
                 <field name="sequence" />
-            </tree>
+            </list>
         </field>
     </record>
     <record id="base_kanban_stage_view_search" model="ir.ui.view">
@@ -54,7 +54,7 @@
         <field name="name">Kanban Stages</field>
         <field name="res_model">base.kanban.stage</field>
         <field name="type">ir.actions.act_window</field>
-        <field name="view_mode">tree,form</field>
+        <field name="view_mode">list,form</field>
     </record>
     <menuitem
         id="base_kanban_stage_root_menu"

base_kanban_stage/views/ir_model_views.xml

@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8" ?>
 <!--
-    Copyright 2019 Eficent Business and IT Consulting Services S.L.
+    Copyright 2025 ForgeFlow S.L.
     License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl.html).
 -->
 <odoo>
@@ -11,7 +11,7 @@
             <field name="transient" position="after">
                 <field
                     name="is_kanban"
-                    attrs="{'readonly': [('state','!=', 'manual')]}"
+                    readonly="state != 'manual'"
                     groups="base.group_no_one"
                 />
             </field>