[MIG] document_page: Migration to 19.0

Author: gregory-rutecki

document_page/__manifest__.py

@@ -4,7 +4,7 @@
 
 {
     "name": "Document Page",
-    "version": "18.0.2.1.0",
+    "version": "19.0.2.1.0",
     "category": "Knowledge Management",
     "author": "OpenERP SA, Odoo Community Association (OCA)",
     "images": [
@@ -17,7 +17,7 @@
     ],
     "website": "https://github.com/OCA/knowledge",
     "license": "AGPL-3",
-    "depends": ["mail", "document_knowledge", "web_editor"],
+    "depends": ["mail", "document_knowledge", "html_editor"],
     "data": [
         "security/document_page_security.xml",
         "security/ir.model.access.csv",

document_page/demo/document_page.xml

@@ -3,7 +3,7 @@
     <record id="base.user_demo" model="res.users">
         <field
             eval="[(4, ref('document_knowledge.group_document_user'))]"
-            name="groups_id"
+            name="group_ids"
         />
     </record>
 

document_page/models/document_page_history.py

@@ -4,7 +4,7 @@
 
 from odoo import api, fields, models
 
-from odoo.addons.web_editor.models.diff_utils import (
+from odoo.addons.html_editor.models.diff_utils import (
     generate_comparison,
 )
 
@@ -54,4 +54,4 @@ def _get_diff(self, v1, v2):
     @api.depends("page_id")
     def _compute_display_name(self):
         for rec in self:
-            rec.display_name = rec.id, "%s #%i" % (rec.page_id.name, rec.id)
+            rec.display_name = f"{rec.page_id.name} #{rec.id}"

document_page/models/ir_ui_menu.py

@@ -15,12 +15,12 @@ def _visible_menu_ids(self, debug=False):
                 self.with_context(**{"ir.ui.menu.full_list": True}).search([]).sudo()
             )
             groups = (
-                self.env.user.groups_id
+                self.env.user.group_ids
                 if not debug
-                else self.env.user.groups_id - self.env.ref("base.group_no_one")
+                else self.env.user.group_ids - self.env.ref("base.group_no_one")
             )
             authorized_menus = menus.filtered(
-                lambda m: not m.groups_id or m.groups_id and groups
+                lambda m: not m.group_ids or m.group_ids and groups
             )
             authorized_folder_menus = authorized_menus.filtered(lambda m: not m.action)
             visible_ids = visible_ids.union(authorized_folder_menus.ids)

document_page/security/document_page_security.xml

@@ -2,17 +2,17 @@
 <odoo>
     <record id="group_document_editor" model="res.groups">
         <field name="name">Editor</field>
-        <field name="category_id" ref="document_knowledge.module_category_knowledge" />
+        <field name="privilege_id" ref="document_knowledge.res_groups_privilege_knowledge" />
         <field
             name="implied_ids"
             eval="[(4, ref('document_knowledge.group_document_user'))]"
         />
     </record>
     <record id="group_document_manager" model="res.groups">
         <field name="name">Manager</field>
-        <field name="category_id" ref="document_knowledge.module_category_knowledge" />
+        <field name="privilege_id" ref="document_knowledge.res_groups_privilege_knowledge" />
         <field name="implied_ids" eval="[(4, ref('group_document_editor'))]" />
-        <field name="users" eval="[(4, ref('base.user_admin'))]" />
+        <field name="user_ids" eval="[Command.link(ref('base.user_admin'))]" />
     </record>
     <record model="ir.rule" id="document_page_rule">
         <field name="name">document_page multi-company</field>

document_page/static/src/js/document_page_kanban_controller.esm.js

@@ -5,7 +5,6 @@ export class DocumentPageKanbanController extends KanbanController {
      * @param {Object} record
      */
     async openRecord(record) {
-        // eslint-disable-next-line no-undef
         const element = document.querySelector(
             `.o_kanban_record[data-id="${record.id}"] .o_document_page_kanban_boxes a`
         );

document_page/views/document_page.xml

@@ -156,7 +156,7 @@
                 <field name="create_uid" />
                 <field name="content_uid" />
                 <field name="name" string="Title" />
-                <group expand="0" string="Group By...">
+                <group>
                     <filter
                         name="group_by_category"
                         string="Category"

document_page/views/document_page_category.xml

@@ -208,7 +208,7 @@
                 <field name="create_uid" />
                 <field name="content_uid" />
                 <field name="company_id" groups="base.group_multi_company" />
-                <group expand="0" string="Group By...">
+                <group>
                     <filter
                         name="group_by_category"
                         string="Category"

document_page/views/document_page_history.xml

@@ -25,7 +25,7 @@
                 <field name="page_id" required="True" />
                 <field name="content" required="True" />
                 <field name="create_uid" />
-                <group expand="0" string="Group By...">
+                <group>
                     <filter
                         name="group_by_author"
                         string="Author"

document_page/wizard/document_page_create_menu.py

@@ -42,7 +42,7 @@ def document_page_menu_create(self):
             "type": "ir.actions.act_window",
             "target": "current",
         }
-        value["domain"] = "[('parent_id','=',%d)]" % page.id
+        value["domain"] = f"[('parent_id','=',{page.id})]"
         value["res_id"] = page.id
 
         # only the super user is allowed to create menu due to security rules