[MIG] base_ir_actions_sequence: Migration to 18.0

Author: LauraCForgeFlow

base_ir_actions_sequence/__manifest__.py

@@ -2,7 +2,7 @@
 # License LGPL-3.0 or later (http://www.gnu.org/licenses/lgpl).
 {
     "name": "Base Actions Sequence",
-    "version": "13.0.1.0.1",
+    "version": "18.0.1.0.0",
     "category": "Extra Tools",
     "author": "ForgeFlow, Odoo Community Association (OCA)",
     "website": "https://github.com/OCA/server-ux",

base_ir_actions_sequence/models/ir_actions.py

@@ -1,21 +1,19 @@
-from collections import defaultdict
-
-from odoo import api, fields, models, tools
+from odoo import fields, models, tools
+from odoo.tools import frozendict
 
 
 class IrActions(models.Model):
     _inherit = "ir.actions.actions"
 
-    priority = fields.Integer(string="Sequence", default=16, required=True)
+    sequence = fields.Integer(default=16, required=True)
 
-    @api.model
-    @tools.ormcache("frozenset(self.env.user.groups_id.ids)", "model_name")
-    def get_bindings(self, model_name):
-        """
-        Order Report Actions by priority
-        """
-        res = super().get_bindings(model_name)
-        ordered_res = defaultdict(list)
-        for key, values in res.items():
-            ordered_res[key] = sorted(values, key=lambda x: (x["priority"], x["id"]))
-        return ordered_res
+    @tools.ormcache("model_name", "self.env.lang")
+    def _get_bindings(self, model_name):
+        result = dict(super()._get_bindings(model_name))
+        # Only for 'report' type, since for 'action' type the bindings are
+        # managed in the original _get_bindings method
+        if result.get("report"):
+            result["report"] = tuple(
+                sorted(result["report"], key=lambda vals: vals.get("sequence", 0))
+            )
+        return frozendict(result)

base_ir_actions_sequence/views/actions_views.xml

@@ -9,7 +9,7 @@
         <field name="inherit_id" ref="base.view_server_action_form" />
         <field name="arch" type="xml">
             <field name="model_id" position="after">
-                <field name="priority" />
+                <field name="sequence" />
             </field>
         </field>
     </record>
@@ -19,7 +19,7 @@
         <field name="inherit_id" ref="base.view_server_action_tree" />
         <field name="arch" type="xml">
             <field name="usage" position="after">
-                <field name="priority" optional="show" />
+                <field name="sequence" optional="show" />
             </field>
         </field>
     </record>
@@ -32,7 +32,7 @@
         <field name="inherit_id" ref="base.view_window_action_form" />
         <field name="arch" type="xml">
             <field name="res_model" position="after">
-                <field name="priority" />
+                <field name="sequence" />
             </field>
         </field>
     </record>
@@ -44,7 +44,7 @@
         <field name="inherit_id" ref="base.view_window_action_tree" />
         <field name="arch" type="xml">
             <field name="context" position="after">
-                <field name="priority" optional="show" />
+                <field name="sequence" optional="show" />
             </field>
         </field>
     </record>
@@ -55,7 +55,7 @@
         <field name="inherit_id" ref="base.act_report_xml_view" />
         <field name="arch" type="xml">
             <field name="report_type" position="after">
-                <field name="priority" />
+                <field name="sequence" />
             </field>
         </field>
     </record>
@@ -67,7 +67,7 @@
         <field name="inherit_id" ref="base.act_report_xml_view_tree" />
         <field name="arch" type="xml">
             <field name="report_type" position="after">
-                <field name="priority" optional="show" />
+                <field name="sequence" optional="show" />
             </field>
         </field>
     </record>