From 595aaba0ad3f7b25fd663f3deec63ef5b17e03ad Mon Sep 17 00:00:00 2001 From: Mathieu Benoit Date: Wed, 24 Sep 2025 03:15:39 -0400 Subject: [PATCH] [FIX] base_tier_validation: comment support approve_sequence_bypass - the comment will only be added to last review and remove repetition --- base_tier_validation/models/tier_validation.py | 6 +++++- base_tier_validation/wizard/comment_wizard.py | 5 ++++- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/base_tier_validation/models/tier_validation.py b/base_tier_validation/models/tier_validation.py index 782cca05d..1572a4417 100644 --- a/base_tier_validation/models/tier_validation.py +++ b/base_tier_validation/models/tier_validation.py @@ -650,7 +650,11 @@ def validate_tier(self): ) if self.has_comment: user_reviews = reviews.filtered( - lambda r: r.status == "pending" and (self.env.user in r.reviewer_ids) + lambda r: ( + r.status == "pending" + or (r.approve_sequence_bypass and r.status != "approved") + ) + and (self.env.user in r.reviewer_ids) ) return self._add_comment("validate", user_reviews) self._validate_tier(reviews) diff --git a/base_tier_validation/wizard/comment_wizard.py b/base_tier_validation/wizard/comment_wizard.py index 5c5a6c933..d87a126c3 100644 --- a/base_tier_validation/wizard/comment_wizard.py +++ b/base_tier_validation/wizard/comment_wizard.py @@ -17,7 +17,10 @@ class CommentWizard(models.TransientModel): def add_comment(self): self.ensure_one() rec = self.env[self.res_model].browse(self.res_id) - self.review_ids.write({"comment": self.comment}) + # Write only comment on last review and not on all + self.review_ids.sorted(lambda x: x.sequence)[-1].write( + {"comment": self.comment} + ) if self.validate_reject == "validate": rec._validate_tier(self.review_ids) if self.validate_reject == "reject":