From dca71363f4a3d4de9e3cb038fb9d76d6dc7034ea Mon Sep 17 00:00:00 2001
From: EdgarRetes <edgar.retes24@gmail.com>
Date: Wed, 20 Nov 2024 13:56:43 -0600
Subject: [PATCH] [MIG] kpi: Migration to 17.0

---
 kpi/__manifest__.py                     |  4 +--
 kpi/models/kpi.py                       | 14 ++++------
 kpi/models/kpi_category.py              |  4 +--
 kpi/models/kpi_history.py               |  6 ++---
 kpi/models/kpi_threshold.py             |  5 ++--
 kpi/models/kpi_threshold_range.py       | 18 +++++--------
 kpi/tests/test_kpi.py                   |  5 ++--
 kpi/views/kpi_category_views.xml        |  2 +-
 kpi/views/kpi_history_views.xml         | 14 +++++-----
 kpi/views/kpi_threshold_range_views.xml | 34 ++++++++++---------------
 kpi/views/kpi_threshold_views.xml       |  6 ++---
 kpi/views/kpi_views.xml                 | 21 +++++++++------
 kpi/views/menu.xml                      |  6 +++--
 13 files changed, 64 insertions(+), 75 deletions(-)

diff --git a/kpi/__manifest__.py b/kpi/__manifest__.py
index 3c629dd948..734689a727 100644
--- a/kpi/__manifest__.py
+++ b/kpi/__manifest__.py
@@ -3,12 +3,12 @@
 
 {
     "name": "Key Performance Indicator",
-    "version": "13.0.1.0.1",
+    "version": "17.0.1.0.0",
     "author": "Savoir-faire Linux,Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/reporting-engine",
     "license": "AGPL-3",
     "category": "Report",
-    "depends": ["base_external_dbsource"],
+    "depends": ["base_external_dbsource", "spreadsheet_dashboard"],
     "data": [
         "security/kpi_security.xml",
         "security/ir.model.access.csv",
diff --git a/kpi/models/kpi.py b/kpi/models/kpi.py
index 6b1f9312fc..b7ebb504aa 100644
--- a/kpi/models/kpi.py
+++ b/kpi/models/kpi.py
@@ -16,9 +16,9 @@
 
 def is_one_value(result):
     # check if sql query returns only one value
-    if type(result) is dict and "value" in result.dictfetchone():
+    if isinstance(result, dict) and "value" in result.dictfetchone():
         return True
-    elif type(result) is list and "value" in result[0]:
+    elif isinstance(result, list) and "value" in result[0]:
         return True
     else:
         return False
@@ -54,8 +54,8 @@ class KPI(models.Model):
     _name = "kpi"
     _description = "Key Performance Indicator"
 
-    name = fields.Char("Name", required=True)
-    description = fields.Text("Description")
+    name = fields.Char(required=True)
+    description = fields.Text()
     category_id = fields.Many2one(
         "kpi.category",
         "Category",
@@ -66,7 +66,7 @@ class KPI(models.Model):
         "Threshold",
         required=True,
     )
-    periodicity = fields.Integer("Periodicity", default=1)
+    periodicity = fields.Integer(default=1)
 
     periodicity_uom = fields.Selection(
         [
@@ -83,14 +83,11 @@ class KPI(models.Model):
 
     next_execution_date = fields.Datetime(
         "Next execution date",
-        readonly=True,
     )
     value = fields.Float(
-        string="Value",
         compute="_compute_display_last_kpi_value",
     )
     color = fields.Text(
-        "Color",
         compute="_compute_display_last_kpi_value",
     )
     last_execution = fields.Datetime(
@@ -122,7 +119,6 @@ class KPI(models.Model):
         "History",
     )
     active = fields.Boolean(
-        "Active",
         help=(
             "Only active KPIs will be updated by the scheduler based on"
             " the periodicity configuration."
diff --git a/kpi/models/kpi_category.py b/kpi/models/kpi_category.py
index 17c572792d..6491006c18 100644
--- a/kpi/models/kpi_category.py
+++ b/kpi/models/kpi_category.py
@@ -9,5 +9,5 @@ class KPICategory(models.Model):
 
     _name = "kpi.category"
     _description = "KPI Category"
-    name = fields.Char("Name", size=50, required=True)
-    description = fields.Text("Description")
+    name = fields.Char(required=True)
+    description = fields.Text()
diff --git a/kpi/models/kpi_history.py b/kpi/models/kpi_history.py
index 344d0c09cd..e222e02436 100644
--- a/kpi/models/kpi_history.py
+++ b/kpi/models/kpi_history.py
@@ -12,7 +12,6 @@ class KPIHistory(models.Model):
     _order = "date desc"
 
     name = fields.Char(
-        "Name",
         size=150,
         required=True,
         default=fields.Datetime.now(),
@@ -21,11 +20,10 @@ class KPIHistory(models.Model):
     date = fields.Datetime(
         "Execution Date",
         required=True,
-        readonly=True,
         default=lambda r: fields.Datetime.now(),
     )
-    value = fields.Float("Value", required=True, readonly=True)
-    color = fields.Text("Color", required=True, readonly=True, default="#FFFFFF")
+    value = fields.Float(required=True)
+    color = fields.Text(required=True, default="#FFFFFF")
     company_id = fields.Many2one(
         "res.company", "Company", default=lambda self: self.env.company
     )
diff --git a/kpi/models/kpi_threshold.py b/kpi/models/kpi_threshold.py
index 51fbdf9304..f4a492c0e4 100644
--- a/kpi/models/kpi_threshold.py
+++ b/kpi/models/kpi_threshold.py
@@ -34,7 +34,7 @@ def _compute_is_valid_threshold(self):
                     "Please make sure your ranges do not overlap."
                 )
 
-    name = fields.Char("Name", size=50, required=True)
+    name = fields.Char(required=True)
     range_ids = fields.Many2many(
         "kpi.threshold.range",
         "kpi_threshold_range_rel",
@@ -43,7 +43,6 @@ def _compute_is_valid_threshold(self):
         "Ranges",
     )
     valid = fields.Boolean(
-        string="Valid",
         required=True,
         compute="_compute_is_valid_threshold",
         default=True,
@@ -79,7 +78,7 @@ def create(self, data):
                             )
                     range_obj2 = self.env["kpi.threshold.range"]
                 range_obj1 = self.env["kpi.threshold.range"]
-        return super(KPIThreshold, self).create(data)
+        return super().create(data)
 
     def get_color(self, kpi_value):
         color = "#FFFFFF"
diff --git a/kpi/models/kpi_threshold_range.py b/kpi/models/kpi_threshold_range.py
index 434c4d4897..4c669170b8 100644
--- a/kpi/models/kpi_threshold_range.py
+++ b/kpi/models/kpi_threshold_range.py
@@ -9,9 +9,9 @@
 
 def is_one_value(result):
     # check if sql query returns only one value
-    if type(result) is dict and "value" in result.dictfetchone():
+    if isinstance(result, dict) and "value" in result.dictfetchone():
         return True
-    elif type(result) is list and "value" in result[0]:
+    elif isinstance(result, list) and "value" in result[0]:
         return True
     else:
         return False
@@ -58,16 +58,14 @@ def _selection_value_type(self):
             ("external", "SQL - External DB"),
         ]
 
-    name = fields.Char("Name", size=50, required=True)
+    name = fields.Char(required=True)
     valid = fields.Boolean(
-        string="Valid", required=True, compute="_compute_is_valid_range", default=True
+        required=True, compute="_compute_is_valid_range", default=True
     )
     invalid_message = fields.Char(
         string="Message", size=100, compute="_compute_is_valid_range"
     )
-    min_type = fields.Selection(
-        selection="_selection_value_type", string="Min Type", required=True
-    )
+    min_type = fields.Selection(selection="_selection_value_type", required=True)
     min_value = fields.Float(string="Minimum Value", compute="_compute_min_value")
     min_fixed_value = fields.Float("Minimum Fixed Value")
     min_code = fields.Text("Minimum Computation Code")
@@ -76,9 +74,7 @@ def _selection_value_type(self):
         "base.external.dbsource",
         "External DB Source Minimum",
     )
-    max_type = fields.Selection(
-        selection="_selection_value_type", string="Max Type", required=True
-    )
+    max_type = fields.Selection(selection="_selection_value_type", required=True)
     max_value = fields.Float(string="Maximum Value", compute="_compute_max_value")
     max_fixed_value = fields.Float("Maximum Fixed Value")
     max_code = fields.Text("Maximum Computation Code")
@@ -88,7 +84,7 @@ def _selection_value_type(self):
         "External DB Source Maximum",
     )
 
-    color = fields.Char(string="Color", help="Choose your color")
+    color = fields.Char(help="Choose your color")
 
     threshold_ids = fields.Many2many(
         "kpi.threshold",
diff --git a/kpi/tests/test_kpi.py b/kpi/tests/test_kpi.py
index 96c63eb421..4953efe3b2 100644
--- a/kpi/tests/test_kpi.py
+++ b/kpi/tests/test_kpi.py
@@ -4,8 +4,9 @@
 
 
 class TestKPI(TransactionCase):
-    def setUp(self):
-        super(TestKPI, self).setUp()
+    @classmethod
+    def setUpClass(cls):
+        super().setUpClass()
 
     def test_invalid_threshold_range(self):
         range1 = self.env["kpi.threshold.range"].create(
diff --git a/kpi/views/kpi_category_views.xml b/kpi/views/kpi_category_views.xml
index 466c5b80aa..c809068a03 100644
--- a/kpi/views/kpi_category_views.xml
+++ b/kpi/views/kpi_category_views.xml
@@ -7,7 +7,7 @@
         <field name="name">kpi.category.tree</field>
         <field name="model">kpi.category</field>
         <field name="arch" type="xml">
-            <tree string="Categories">
+            <tree>
                 <field name="name" />
             </tree>
         </field>
diff --git a/kpi/views/kpi_history_views.xml b/kpi/views/kpi_history_views.xml
index c2abf7ebd5..635368c4d0 100644
--- a/kpi/views/kpi_history_views.xml
+++ b/kpi/views/kpi_history_views.xml
@@ -7,11 +7,11 @@
         <field name="name">kpi.history.tree</field>
         <field name="model">kpi.history</field>
         <field name="arch" type="xml">
-            <tree string="KPI History">
+            <tree>
                 <field name="name" />
-                <field name="date" />
-                <field name="value" />
-                <field name="color" widget="color" />
+                <field name="date" readonly="1" />
+                <field name="value" readonly="1" />
+                <field name="color" widget="color" readonly="1" />
                 <field name="company_id" groups="base.group_multi_company" />
             </tree>
         </field>
@@ -25,9 +25,9 @@
                     <group col="4" colspan="4">
                         <field name="kpi_id" />
                         <field name="name" />
-                        <field name="date" />
-                        <field name="value" />
-                        <field name="color" widget="color" />
+                        <field name="date" readonly="1" />
+                        <field name="value" readonly="1" />
+                        <field name="color" widget="color" readonly="1" />
                         <field name="company_id" groups="base.group_multi_company" />
                     </group>
                 </sheet>
diff --git a/kpi/views/kpi_threshold_range_views.xml b/kpi/views/kpi_threshold_range_views.xml
index 1473a32dfb..5cf70aa0f4 100644
--- a/kpi/views/kpi_threshold_range_views.xml
+++ b/kpi/views/kpi_threshold_range_views.xml
@@ -7,11 +7,11 @@
         <field name="name">kpi.threshold.range.tree</field>
         <field name="model">kpi.threshold.range</field>
         <field name="arch" type="xml">
-            <tree string="Ranges" decoration-danger="invalid_message">
+            <tree decoration-danger="invalid_message">
                 <field name="name" />
                 <field name="min_value" />
                 <field name="max_value" />
-                <field name="color" widget="color" />
+                <field name="color" widget="color" readonly="1" />
                 <field name="invalid_message" />
                 <field name="company_id" groups="base.group_multi_company" />
             </tree>
@@ -25,7 +25,7 @@
                 <sheet>
                     <group col="6" colspan="6">
                         <field name="name" colspan="2" />
-                        <field name="color" colspan="2" widget="color" />
+                        <field name="color" colspan="2" widget="color" readonly="1" />
                         <field
                             name="company_id"
                             groups="base.group_multi_company"
@@ -39,23 +39,19 @@
                         <field
                             name="min_fixed_value"
                             colspan="3"
-                            attrs="{'invisible' : [('min_type', '!=', 'static')]}"
+                            invisible="min_type != 'static'"
                         />
                         <field
                             name="min_dbsource_id"
+                            invisible="min_type != 'external'"
                             colspan="3"
-                            attrs="{'invisible' : [('min_type', '!=', 'external')]}"
                         />
                         <field
                             name="min_code"
                             colspan="6"
-                            attrs="{'invisible' : [('min_type', 'not in', ('local','external','python'))]}"
-                        />
-                        <field
-                            name="min_error"
-                            colspan="6"
-                            attrs="{'invisible': [('min_error', '=', False)]}"
+                            invisible="min_type not in ('local', 'external', 'python')"
                         />
+                        <field name="min_error" colspan="6" invisible="not min_error" />
                         <newline />
                         <separator string="Maximum" />
                         <newline />
@@ -63,25 +59,21 @@
                         <field
                             name="max_fixed_value"
                             colspan="3"
-                            attrs="{'invisible' : [('max_type', '!=', 'static')]}"
+                            invisible="max_type != 'static'"
                         />
                         <field
                             name="max_dbsource_id"
                             colspan="3"
-                            attrs="{'invisible' : [('max_type', '!=', 'external')]}"
+                            invisible="max_type != 'external'"
                         />
                         <newline />
                         <field
                             name="max_code"
                             colspan="6"
-                            attrs="{'invisible' : [('max_type', 'not in', ('local','external','python'))]}"
+                            invisible="max_type not in ('local', 'external', 'python')"
                         />
                         <newline />
-                        <field
-                            name="max_error"
-                            colspan="6"
-                            attrs="{'invisible': [('max_error', '=', False)]}"
-                        />
+                        <field name="max_error" colspan="6" invisible="not max_error" />
                         <newline />
                     </group>
                     <group col="6" colspan="6">
@@ -89,13 +81,13 @@
                         <field name="threshold_ids" nolabel="1" colspan="4" />
                         <separator
                             string="Errors"
-                            attrs="{'invisible' : [('invalid_message', '=', False)]}"
+                            invisible="not invalid_message"
                             colspan="4"
                         />
                         <field
                             name="invalid_message"
                             nolabel="1"
-                            attrs="{'invisible' : [('invalid_message', '=', False)]}"
+                            invisible="not invalid_message"
                             colspan="4"
                         />
                     </group>
diff --git a/kpi/views/kpi_threshold_views.xml b/kpi/views/kpi_threshold_views.xml
index 280181ccfd..c69ac6e426 100644
--- a/kpi/views/kpi_threshold_views.xml
+++ b/kpi/views/kpi_threshold_views.xml
@@ -7,7 +7,7 @@
         <field name="name">kpi.threshold.tree</field>
         <field name="model">kpi.threshold</field>
         <field name="arch" type="xml">
-            <tree string="Thresholds" decoration-danger="invalid_message">
+            <tree decoration-danger="invalid_message">
                 <field name="name" />
                 <field name="invalid_message" />
                 <field name="company_id" groups="base.group_multi_company" />
@@ -38,13 +38,13 @@
                         <newline />
                         <separator
                             string="Errors"
-                            attrs="{'invisible' : [('invalid_message', '=', False)]}"
+                            invisible="not invalid_message"
                             colspan="4"
                         />
                         <field
                             name="invalid_message"
                             nolabel="1"
-                            attrs="{'invisible' : [('invalid_message', '=', False)]}"
+                            invisible="not invalid_message"
                             colspan="4"
                         />
                         <newline />
diff --git a/kpi/views/kpi_views.xml b/kpi/views/kpi_views.xml
index 34d68da09b..aff216e0d3 100644
--- a/kpi/views/kpi_views.xml
+++ b/kpi/views/kpi_views.xml
@@ -8,9 +8,9 @@
         <field name="model">kpi</field>
         <field eval="8" name="priority" />
         <field name="arch" type="xml">
-            <tree string="Key Performance Indicators">
+            <tree>
                 <field name="name" />
-                <field name="value" widget="progressbar" />
+                <field name="value" widget="progressbar" readonly="1" />
                 <field name="category_id" />
                 <field name="kpi_type" />
                 <field name="company_id" groups="base.group_multi_company" />
@@ -24,8 +24,8 @@
             <kanban class="o_kpi_kanban" create="false" edit="false" delete="false">
                 <field name="id" />
                 <field name="display_name" />
-                <field name="color" />
-                <field name="value" />
+                <field name="color" readonly="1" />
+                <field name="value" readonly="1" />
                 <field name="last_execution" />
                 <templates>
                     <t t-name="kanban-box">
@@ -41,7 +41,7 @@
                                     t-attf-style="color:#{record.color.raw_value}"
                                 >
                                     <strong>
-                                        <field name="value" />
+                                        <field name="value" readonly="1" />
                                     </strong>
                                 </div>
                                 <div
@@ -95,7 +95,7 @@
                             <field name="category_id" />
                         </group>
                         <group>
-                            <field name="value" colspan="2" />
+                            <field name="value" colspan="2" readonly="1" />
                             <field name="active" colspan="2" />
                             <field
                                 name="company_id"
@@ -115,17 +115,22 @@
                             <field name="history_ids" readonly="1" nolabel="1" />
                         </page>
                         <page string="Computation" groups="kpi.group_kpi_manager">
+                            <field name="kpi_type" invisible="1" />
                             <group col="6">
                                 <field name="periodicity" colspan="3" />
                                 <field name="periodicity_uom" colspan="3" />
-                                <field name="next_execution_date" colspan="3" />
+                                <field
+                                    name="next_execution_date"
+                                    colspan="3"
+                                    readonly="1"
+                                />
                                 <separator string="KPI Computation" colspan="6" />
                                 <newline />
                                 <field name="kpi_type" colspan="2" />
                                 <field
                                     name="dbsource_id"
                                     colspan="2"
-                                    attrs="{'invisible' : [('kpi_type', '!=', 'external')]}"
+                                    invisible="kpi_type != 'external'"
                                 />
                                 <newline />
                                 <field name="kpi_code" colspan="6" />
diff --git a/kpi/views/menu.xml b/kpi/views/menu.xml
index b951090e83..58e7b145df 100644
--- a/kpi/views/menu.xml
+++ b/kpi/views/menu.xml
@@ -2,19 +2,21 @@
 <!-- Copyright 2012 - Now  Savoir-faire Linux <https://www.savoirfairelinux.com/>
      License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl). -->
 <odoo>
+<!--        Change parent-->
     <menuitem
         id="menu_kpi_dasboard"
         name="Key Performance Indicators"
         action="open_kpi_dashboard"
         sequence="15"
-        parent="base.menu_reporting_dashboard"
+        parent="spreadsheet_dashboard.spreadsheet_dashboard_menu_dashboard"
         groups="base.group_user"
     />
     <!-- CONFIGURATION -->
+<!--        Change parent-->
     <menuitem
         id="menu_configuration_kpi"
+        parent="spreadsheet_dashboard.spreadsheet_dashboard_menu_configuration"
         name="Key Performance Indicators"
-        parent="base.menu_reporting_config"
         groups="kpi.group_kpi_manager"
         sequence="10"
     />