odoo-addon-base-tier-validation 17.0.2.3.2.1__py3-none-any.whl → 18.0.1.0.0.3__py3-none-any.whl

odoo/addons/base_tier_validation/models/tier_review.py

@@ -5,7 +5,7 @@ import logging
 
 import pytz
 
-from odoo import _, api, fields, models
+from odoo import api, fields, models
 from odoo.exceptions import ValidationError
 
 _logger = logging.getLogger(__name__)
@@ -15,7 +15,7 @@ class TierReview(models.Model):
     _name = "tier.review"
     _description = "Tier Review"
 
-    name = fields.Char(related="definition_id.name", readonly=True)
+    name = fields.Char(related="definition_id.name")
     status = fields.Selection(
         [
             ("waiting", "Waiting"),
@@ -32,14 +32,10 @@ class TierReview(models.Model):
         related="definition_id.company_id",
         store=True,
     )
-    review_type = fields.Selection(related="definition_id.review_type", readonly=True)
-    reviewer_id = fields.Many2one(related="definition_id.reviewer_id", readonly=True)
-    reviewer_group_id = fields.Many2one(
-        related="definition_id.reviewer_group_id", readonly=True
-    )
-    reviewer_field_id = fields.Many2one(
-        related="definition_id.reviewer_field_id", readonly=True
-    )
+    review_type = fields.Selection(related="definition_id.review_type")
+    reviewer_id = fields.Many2one(related="definition_id.reviewer_id")
+    reviewer_group_id = fields.Many2one(related="definition_id.reviewer_group_id")
+    reviewer_field_id = fields.Many2one(related="definition_id.reviewer_field_id")
     reviewer_ids = fields.Many2many(
         string="Reviewers",
         comodel_name="res.users",
@@ -55,7 +51,7 @@ class TierReview(models.Model):
     reviewed_formated_date = fields.Char(
         string="Validation Formated Date", compute="_compute_reviewed_formated_date"
     )
-    has_comment = fields.Boolean(related="definition_id.has_comment", readonly=True)
+    has_comment = fields.Boolean(related="definition_id.has_comment")
     comment = fields.Char(string="Comments")
     can_review = fields.Boolean(
         compute="_compute_can_review",
@@ -63,11 +59,9 @@ class TierReview(models.Model):
         help="""Can review will be marked if the review is pending and the
         approve sequence has been achieved""",
     )
-    approve_sequence = fields.Boolean(
-        related="definition_id.approve_sequence", readonly=True
-    )
+    approve_sequence = fields.Boolean(related="definition_id.approve_sequence")
     approve_sequence_bypass = fields.Boolean(
-        related="definition_id.approve_sequence_bypass", readonly=True
+        related="definition_id.approve_sequence_bypass"
     )
     last_reminder_date = fields.Datetime(readonly=True)
 
@@ -141,7 +135,7 @@ class TierReview(models.Model):
         for rec in self:
             todo_by = False
             if rec.reviewer_group_id:
-                todo_by = _("Group %s") % rec.reviewer_group_id.name
+                todo_by = self.env._("Group %s", rec.reviewer_group_id.name)
             else:
                 todo_by = ", ".join(rec.reviewer_ids[:num_show].mapped("display_name"))
                 num_users = len(rec.reviewer_ids)
@@ -157,7 +151,9 @@ class TierReview(models.Model):
             resource = self.env[self.model].browse(self.res_id)
             reviewer_field = getattr(resource, self.reviewer_field_id.name, False)
             if not reviewer_field or not reviewer_field._name == "res.users":
-                raise ValidationError(_("There are no res.users in the selected field"))
+                raise ValidationError(
+                    self.env._("There are no res.users in the selected field")
+                )
         return reviewer_field
 
     def _notify_pending_status(self, review_ids):
@@ -173,7 +169,7 @@ class TierReview(models.Model):
 
     def _notify_review_reminder_body(self):
         delay = (fields.Datetime.now() - self.create_date).days
-        return _("A review has been requested %s days ago.") % (delay)
+        return self.env._("A review has been requested %s days ago.", delay)
 
     def _send_review_reminder(self):
         record = self.env[self.model].browse(self.res_id)
@@ -183,7 +179,7 @@ class TierReview(models.Model):
         elif hasattr(record, "message_post"):
             self._notify_review_reminder(record)
         else:
-            msg = "Could not send reminder for record %s" % record
+            msg = f"Could not send reminder for record {record}"
             _logger.exception(msg)
         self.last_reminder_date = fields.Datetime.now()
 

odoo/addons/base_tier_validation/models/tier_validation.py

@@ -7,8 +7,9 @@ from ast import literal_eval
 from lxml import etree
 from psycopg2.extensions import AsIs
 
-from odoo import _, api, fields, models
+from odoo import api, fields, models
 from odoo.exceptions import ValidationError
+from odoo.tools import SQL
 from odoo.tools.misc import frozendict
 
 BASE_EXCEPTION_FIELDS = ["message_follower_ids", "access_token"]
@@ -169,22 +170,21 @@ class TierValidation(models.AbstractModel):
         return self._description
 
     def _get_to_validate_message(self):
-        return (
-            """<i class="fa fa-info-circle" /> %s"""
-            % _("This %s needs to be validated")
-            % self._get_to_validate_message_name()
-        )
+        return f"""<i class="fa fa-info-circle"></i> {self.env._(
+            "This %s needs to be validated",
+            self._get_to_validate_message_name()
+        )}"""
 
     def _get_validated_message(self):
-        msg = """<i class="fa fa-thumbs-up" /> %s""" % _(
-            """Operation has been <b>validated</b>!"""
-        )
+        msg = f"""<i class="fa fa-thumbs-up"></i> {self.env._(
+            "Operation has been <b>validated</b>!"
+        )}"""
         return self.validated and msg or ""
 
     def _get_rejected_message(self):
-        msg = """<i class="fa fa-thumbs-down" /> %s""" % _(
-            """Operation has been <b>rejected</b>."""
-        )
+        msg = f"""<i class="fa fa-thumbs-down"></i> {self.env._(
+            "Operation has been <b>rejected</b>."
+        )}"""
         return self.rejected and msg or ""
 
     def _compute_validated_rejected(self):
@@ -221,7 +221,7 @@ class TierValidation(models.AbstractModel):
             review = rec.review_ids.sorted("sequence").filtered(
                 lambda x: x.status == "pending"
             )[:1]
-            rec.next_review = review and _("Next: %s") % review.name or ""
+            rec.next_review = review and self.env._("Next: %s", review.name or "")
 
     def _compute_hide_reviews(self):
         for rec in self:
@@ -368,11 +368,12 @@ class TierValidation(models.AbstractModel):
         self._tier_validation_check_write_remove_reviews(vals)
         return super().write(vals)
 
-    def _write(self, vals):
-        if self._tier_validation_state_field_is_computed:
-            self._tier_validation_check_state_on_write(vals)
-            self._tier_validation_check_write_remove_reviews(vals)
-        return super()._write(vals)
+    def _write_multi(self, vals_list):
+        for rec, vals in zip(self, vals_list, strict=False):
+            if rec._tier_validation_state_field_is_computed:
+                rec._tier_validation_check_state_on_write(vals)
+                rec._tier_validation_check_write_remove_reviews(vals)
+        return super()._write_multi(vals_list)
 
     def _tier_validation_get_current_state_value(self):
         """Get the current value from the cache or the database.
@@ -383,12 +384,12 @@ class TierValidation(models.AbstractModel):
         self.ensure_one()
         if self._tier_validation_state_field_is_computed and isinstance(self.id, int):
             self.env.cr.execute(
-                "select %(field)s from %(table)s where id = %(res_id)s",
-                {
-                    "field": AsIs(self._state_field),
-                    "table": AsIs(self._table),
-                    "res_id": self.id,
-                },
+                SQL(
+                    "select %(field)s from %(table)s where id = %(res_id)s",
+                    field=AsIs(self._state_field),
+                    table=AsIs(self._table),
+                    res_id=self.id,
+                )
             )
             rows = self.env.cr.fetchall()
             if rows:
@@ -407,16 +408,16 @@ class TierValidation(models.AbstractModel):
                             lambda r: r.status == "pending"
                         ).mapped("name")
                         raise ValidationError(
-                            _(
+                            self.env._(
                                 "This action needs to be validated for at least "
                                 "one record. Reviews pending:\n - %s "
-                                "\nPlease request a validation."
+                                "\nPlease request a validation.",
+                                "\n - ".join(pending_reviews),
                             )
-                            % "\n - ".join(pending_reviews)
                         )
                 if rec.review_ids and not rec.validated:
                     raise ValidationError(
-                        _(
+                        self.env._(
                             "A validation process is still open for at least "
                             "one record."
                         )
@@ -437,16 +438,16 @@ class TierValidation(models.AbstractModel):
                 ) = rec._get_fields_to_write_validation(
                     vals, rec._get_under_validation_exceptions
                 )
+                not_allowed_fields_str = "\n- ".join(not_allowed_fields)
+                allowed_fields_str = "\n- ".join(allowed_fields)
                 raise ValidationError(
-                    _(
+                    self.env._(
                         "You are not allowed to write those fields under validation.\n"
-                        "- %(not_allowed_fields)s\n\n"
-                        "Only those fields can be modified:\n- %(allowed_fields)s"
+                        "- %(not_allowed_fields_str)s\n\n"
+                        "Only those fields can be modified:\n- %(allowed_fields_str)s",
+                        not_allowed_fields_str=not_allowed_fields_str,
+                        allowed_fields_str=allowed_fields_str,
                     )
-                    % {
-                        "not_allowed_fields": "\n- ".join(not_allowed_fields),
-                        "allowed_fields": "\n- ".join(allowed_fields),
-                    }
                 )
 
             # Write after validation. Check only if Tier Validation Exception is created
@@ -464,16 +465,16 @@ class TierValidation(models.AbstractModel):
                 ) = rec._get_fields_to_write_validation(
                     vals, rec._get_after_validation_exceptions
                 )
+                not_allowed_fields_str = "\n- ".join(not_allowed_fields)
+                allowed_fields_str = "\n- ".join(allowed_fields)
                 raise ValidationError(
-                    _(
+                    self.env._(
                         "You are not allowed to write those fields after validation.\n"
-                        "- %(not_allowed_fields)s\n\n"
-                        "Only those fields can be modified:\n- %(allowed_fields)s"
+                        "- %(not_allowed_fields_str)s\n\n"
+                        "Only those fields can be modified:\n- %(allowed_fields_str)s",
+                        not_allowed_fields_str=not_allowed_fields_str,
+                        allowed_fields_str=allowed_fields_str,
                     )
-                    % {
-                        "not_allowed_fields": "\n- ".join(not_allowed_fields),
-                        "allowed_fields": "\n- ".join(allowed_fields),
-                    }
                 )
 
     def _tier_validation_check_write_remove_reviews(self, vals):
@@ -576,13 +577,13 @@ class TierValidation(models.AbstractModel):
         )
         if has_comment:
             comment = has_comment.mapped("comment")[0]
-            return _("A review was accepted. (%s)") % comment
-        return _("A review was accepted")
+            return self.env._("A review was accepted. (%s)", comment)
+        return self.env._("A review was accepted")
 
     def _add_comment(self, validate_reject, reviews):
         wizard = self.env.ref("base_tier_validation.view_comment_wizard")
         return {
-            "name": _("Comment"),
+            "name": self.env._("Comment"),
             "type": "ir.actions.act_window",
             "view_mode": "form",
             "res_model": "comment.wizard",
@@ -626,11 +627,12 @@ class TierValidation(models.AbstractModel):
         )
         if has_comment:
             comment = has_comment.mapped("comment")[0]
-            return _("A review was rejected by %(user)s. (%(comment)s)") % {
-                "user": self.env.user.name,
-                "comment": comment,
-            }
-        return _("A review was rejected by %s.") % (self.env.user.name)
+            return self.env._(
+                "A review was rejected by %(user)s. (%(comment)s)",
+                user=self.env.user.name,
+                comment=comment,
+            )
+        return self.env._("A review was rejected by %s.", self.env.user.name)
 
     def _notify_rejected_review(self):
         post = "message_post"
@@ -672,12 +674,13 @@ class TierValidation(models.AbstractModel):
                 rec._notify_rejected_review()
 
     def _notify_created_review_body(self):
-        return _("A record to be reviewed has been created by %s.") % (
-            self.env.user.name
+        return self.env._(
+            "A record to be reviewed has been created by %s.",
+            self.env.user.name,
         )
 
     def _notify_requested_review_body(self):
-        return _("A review has been requested by %s.") % (self.env.user.name)
+        return self.env._("A review has been requested by %s.", self.env.user.name)
 
     def _notify_review_requested(self, tier_reviews):
         """method to notify when tier validation is created"""
@@ -732,7 +735,7 @@ class TierValidation(models.AbstractModel):
         return created_trs
 
     def _notify_restarted_review_body(self):
-        return _("The review has been reset by %s.") % (self.env.user.name)
+        return self.env._("The review has been reset by %s.", self.env.user.name)
 
     def _notify_restarted_review(self):
         post = "message_post"
@@ -777,10 +780,8 @@ class TierValidation(models.AbstractModel):
     @api.model
     def _update_counter(self, review_counter):
         self.review_ids._compute_can_review()
-        notifications = []
         channel = "base.tier.validation/updated"
-        notifications.append([self.env.user.partner_id, channel, review_counter])
-        self.env["bus.bus"]._sendmany(notifications)
+        self.env.user.partner_id._bus_send(channel, review_counter)
 
     def unlink(self):
         self.mapped("review_ids").unlink()

odoo/addons/base_tier_validation/models/tier_validation_exception.py

@@ -1,7 +1,7 @@
 # Copyright 2024 Moduon Team (https://www.moduon.team)
 # License AGPL-3.0 or later (https://www.gnu.org/licenses/agpl).
 
-from odoo import _, api, exceptions, fields, models
+from odoo import api, exceptions, fields, models
 
 from .tier_validation import BASE_EXCEPTION_FIELDS
 
@@ -9,7 +9,6 @@ from .tier_validation import BASE_EXCEPTION_FIELDS
 class TierValidationException(models.Model):
     _name = "tier.validation.exception"
     _description = "Tier Validation Exceptions"
-    _rec_name = "name"
 
     @api.model
     def _get_tier_validation_model_names(self):
@@ -30,7 +29,6 @@ class TierValidationException(models.Model):
         related="model_id.model",
         string="Model Name",
         store=True,
-        readonly=True,
         index=True,
     )
     field_ids = fields.Many2many(
@@ -64,17 +62,21 @@ class TierValidationException(models.Model):
 
     @api.depends("model_id")
     def _compute_valid_model_field_ids(self):
-        for record in self:
-            record.valid_model_field_ids = (
-                self.env["ir.model.fields"]
-                .sudo()
-                .search(
-                    [
-                        ("model", "=", record.model_name),
-                        ("name", "not in", BASE_EXCEPTION_FIELDS),
-                    ]
-                )
+        model_names = self.mapped("model_name")
+        valid_model_fields = dict(
+            self.env["ir.model.fields"]
+            .sudo()
+            ._read_group(
+                domain=[
+                    ("model", "in", model_names),
+                    ("name", "not in", BASE_EXCEPTION_FIELDS),
+                ],
+                groupby=["model"],
+                aggregates=["id:array_agg"],
             )
+        )
+        for record in self:
+            record.valid_model_field_ids = valid_model_fields.get(record.model_name, [])
 
     @api.constrains(
         "allowed_to_write_under_validation", "allowed_to_write_after_validation"
@@ -85,7 +87,7 @@ class TierValidationException(models.Model):
             and not self.allowed_to_write_after_validation
         ):
             raise exceptions.ValidationError(
-                _(
+                self.env._(
                     "At least one of these fields must be checked! "
                     "Write under Validation, Write after Validation"
                 )

odoo/addons/base_tier_validation/readme/CONTRIBUTORS.md

@@ -14,3 +14,5 @@
 - [XCG Consulting](https://xcg-consulting.fr):
   - Houzéfa Abbasbhay
 - Stefan Rijnhart \<<stefan@opener.amsterdam>\>
+- Kevin Khao \<<kevinkhao@gmail.com>\>
+- Do Anh Duy \<<duyda@trobz.com>\>

odoo/addons/base_tier_validation/readme/CREDITS.md

@@ -0,0 +1 @@
+The migration of this module from 17.0 to 18.0 was financially supported by Camptocamp.

odoo/addons/base_tier_validation/security/ir.model.access.csv

@@ -7,5 +7,5 @@ access_tier_review,access.tier.review,model_tier_review,base.group_user,1,1,1,1
 access_tier_definition_all,tier.definition.all,model_tier_definition,base.group_user,1,0,0,0
 access_tier_definition_settings,tier.definition.settings,model_tier_definition,base.group_erp_manager,1,1,1,1
 access_comment_wizard,access.comment.wizard,model_comment_wizard,base.group_user,1,1,1,1
-access_tier_validation_exceptions_all,tier.validation.exceptions,model_tier_validation_exception,,1,0,0,0
+access_tier_validation_exceptions_all,tier.validation.exceptions,model_tier_validation_exception,base.group_user,1,0,0,0
 access_tier_validation_exceptions_settings,tier.validation.exceptions,model_tier_validation_exception,base.group_system,1,1,1,1

odoo/addons/base_tier_validation/security/tier_validation_security.xml

@@ -4,16 +4,12 @@
         <field name="name">Tier Definition multi-company</field>
         <field name="model_id" ref="model_tier_definition" />
         <field name="global" eval="True" />
-        <field name="domain_force">
-            ['|',('company_id','=',False),('company_id', 'in', company_ids)]
-        </field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
     <record id="tier_review_comp_rule" model="ir.rule">
         <field name="name">Tier Review multi-company</field>
         <field name="model_id" ref="model_tier_review" />
         <field name="global" eval="True" />
-        <field name="domain_force">
-            ['|',('company_id','=',False),('company_id', 'in', company_ids)]
-        </field>
+        <field name="domain_force">[('company_id', 'in', company_ids + [False])]</field>
     </record>
 </odoo>

odoo/addons/base_tier_validation/static/description/index.html

@@ -367,9 +367,9 @@ ul.auto-toc {
 !! This file is generated by oca-gen-addon-readme !!
 !! changes will be overwritten.                   !!
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
-!! source digest: sha256:3687572682629edee22a15d36a33201f134bc3f2147eaaaee4bd3bb567a0c8ab
+!! source digest: sha256:7b3ca7f03e3d92d3cdaeb8de45ef9b88d126b246ba08f2f16d0b17f0121d1c11
 !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! -->
-<p><a class="reference external image-reference" href="https://odoo-community.org/page/development-status"><img alt="Mature" src="https://img.shields.io/badge/maturity-Mature-brightgreen.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/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/server-ux/tree/17.0/base_tier_validation"><img alt="OCA/server-ux" src="https://img.shields.io/badge/github-OCA%2Fserver--ux-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/server-ux-17-0/server-ux-17-0-base_tier_validation"><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/server-ux&amp;target_branch=17.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="Mature" src="https://img.shields.io/badge/maturity-Mature-brightgreen.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/licence-AGPL--3-blue.png" /></a> <a class="reference external image-reference" href="https://github.com/OCA/server-ux/tree/18.0/base_tier_validation"><img alt="OCA/server-ux" src="https://img.shields.io/badge/github-OCA%2Fserver--ux-lightgray.png?logo=github" /></a> <a class="reference external image-reference" href="https://translation.odoo-community.org/projects/server-ux-18-0/server-ux-18-0-base_tier_validation"><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/server-ux&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>Validating some operations is a common need across different areas in a
 company and sometimes it also involves several people and stages in the
 process. With this module you will be able to define your custom
@@ -414,7 +414,8 @@ compute method in <tt class="docutils literal">_get_after_validation_exceptions<
 <li><a class="reference internal" href="#credits" id="toc-entry-20">Credits</a><ul>
 <li><a class="reference internal" href="#authors" id="toc-entry-21">Authors</a></li>
 <li><a class="reference internal" href="#contributors" id="toc-entry-22">Contributors</a></li>
-<li><a class="reference internal" href="#maintainers" id="toc-entry-23">Maintainers</a></li>
+<li><a class="reference internal" href="#other-credits" id="toc-entry-23">Other credits</a></li>
+<li><a class="reference internal" href="#maintainers" id="toc-entry-24">Maintainers</a></li>
 </ul>
 </li>
 </ul>
@@ -589,7 +590,7 @@ sequence, but also other sequence for the same approver</li>
 <p>Bugs are tracked on <a class="reference external" href="https://github.com/OCA/server-ux/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/server-ux/issues/new?body=module:%20base_tier_validation%0Aversion:%2017.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/server-ux/issues/new?body=module:%20base_tier_validation%0Aversion:%2018.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">
@@ -621,10 +622,17 @@ If you spotted it first, help us to smash it by providing a detailed and welcome
 </ul>
 </li>
 <li>Stefan Rijnhart &lt;<a class="reference external" href="mailto:stefan&#64;opener.amsterdam">stefan&#64;opener.amsterdam</a>&gt;</li>
+<li>Kevin Khao &lt;<a class="reference external" href="mailto:kevinkhao&#64;gmail.com">kevinkhao&#64;gmail.com</a>&gt;</li>
+<li>Do Anh Duy &lt;<a class="reference external" href="mailto:duyda&#64;trobz.com">duyda&#64;trobz.com</a>&gt;</li>
 </ul>
 </div>
+<div class="section" id="other-credits">
+<h2><a class="toc-backref" href="#toc-entry-23">Other credits</a></h2>
+<p>The migration of this module from 17.0 to 18.0 was financially supported
+by Camptocamp.</p>
+</div>
 <div class="section" id="maintainers">
-<h2><a class="toc-backref" href="#toc-entry-23">Maintainers</a></h2>
+<h2><a class="toc-backref" href="#toc-entry-24">Maintainers</a></h2>
 <p>This module is maintained by the OCA.</p>
 <a class="reference external image-reference" href="https://odoo-community.org">
 <img alt="Odoo Community Association" src="https://odoo-community.org/logo.png" />
@@ -634,7 +642,7 @@ mission is to support the collaborative development of Odoo features and
 promote its widespread use.</p>
 <p>Current <a class="reference external" href="https://odoo-community.org/page/maintainer-role">maintainer</a>:</p>
 <p><a class="reference external image-reference" href="https://github.com/LoisRForgeFlow"><img alt="LoisRForgeFlow" src="https://github.com/LoisRForgeFlow.png?size=40px" /></a></p>
-<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-ux/tree/17.0/base_tier_validation">OCA/server-ux</a> project on GitHub.</p>
+<p>This module is part of the <a class="reference external" href="https://github.com/OCA/server-ux/tree/18.0/base_tier_validation">OCA/server-ux</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>

odoo/addons/base_tier_validation/static/src/components/tier_review_menu/tier_review_menu.esm.js

@@ -1,22 +1,25 @@
-/* @odoo-module */
-
 import {Component, useState} from "@odoo/owl";
 import {Dropdown} from "@web/core/dropdown/dropdown";
-import {DropdownItem} from "@web/core/dropdown/dropdown_item";
 import {registry} from "@web/core/registry";
 import {useDiscussSystray} from "@mail/utils/common/hooks";
+import {useDropdownState} from "@web/core/dropdown/dropdown_hooks";
 import {useService} from "@web/core/utils/hooks";
 
-const systrayRegistry = registry.category("systray");
-
 export class TierReviewMenu extends Component {
+    static components = {Dropdown};
+    static props = [];
+    static template = "base_tier_validation.TierReviewMenu";
+
     setup() {
+        super.setup();
         this.discussSystray = useDiscussSystray();
         this.orm = useService("orm");
         this.store = useState(useService("mail.store"));
         this.action = useService("action");
+        this.dropdown = useDropdownState();
         this.fetchSystrayReviewer();
     }
+
     async fetchSystrayReviewer() {
         const groups = await this.orm.call("res.users", "review_user_count");
         let total = 0;
@@ -26,9 +29,7 @@ export class TierReviewMenu extends Component {
         this.store.tierReviewCounter = total;
         this.store.tierReviewGroups = groups;
     }
-    onBeforeOpen() {
-        this.fetchSystrayReviewer();
-    }
+
     availableViews() {
         return [
             [false, "kanban"],
@@ -37,11 +38,11 @@ export class TierReviewMenu extends Component {
             [false, "activity"],
         ];
     }
+
     openReviewGroup(group) {
-        document.body.click();
-        // Hack to close dropdown
+        this.dropdown.close();
         const context = {};
-        var domain = [["can_review", "=", true]];
+        const domain = [["can_review", "=", true]];
         if (group.active_field) {
             domain.push(["active", "in", [true, false]]);
         }
@@ -64,10 +65,10 @@ export class TierReviewMenu extends Component {
     }
 }
 
-TierReviewMenu.template = "base_tier_validation.TierReviewMenu";
-TierReviewMenu.components = {Dropdown, DropdownItem};
-TierReviewMenu.props = [];
-
-export const systrayItem = {Component: TierReviewMenu};
+export const systrayItem = {
+    Component: TierReviewMenu,
+};
 
-systrayRegistry.add("base_tier_validation.ReviewerMenu", systrayItem, {sequence: 99});
+registry
+    .category("systray")
+    .add("base_tier_validation.ReviewerMenu", systrayItem, {sequence: 99});