[FIX] base_tier_validation: ensure skip_validation_check bypasses validations

Author: Jord38

base_tier_validation/__manifest__.py

@@ -3,7 +3,7 @@
 {
     "name": "Base Tier Validation",
     "summary": "Implement a validation process based on tiers.",
-    "version": "17.0.4.0.0",
+    "version": "17.0.4.0.1",
     "development_status": "Mature",
     "maintainers": ["LoisRForgeFlow"],
     "category": "Tools",

base_tier_validation/models/tier_validation.py

@@ -371,13 +371,16 @@ def _check_tier_state_transition(self, vals):
         ) not in (self._state_to + [self._cancel_state])
 
     def write(self, vals):
-        self._tier_validation_check_state_on_write(vals)
-        self._tier_validation_check_write_allowed(vals)
-        self._tier_validation_check_write_remove_reviews(vals)
+        if not self._context.get("skip_validation_check"):
+            self._tier_validation_check_state_on_write(vals)
+            self._tier_validation_check_write_allowed(vals)
+            self._tier_validation_check_write_remove_reviews(vals)
         return super().write(vals)
 
     def _write(self, vals):
-        if self._tier_validation_state_field_is_computed:
+        if self._tier_validation_state_field_is_computed and not self._context.get(
+            "skip_validation_check"
+        ):
             self._tier_validation_check_state_on_write(vals)
             self._tier_validation_check_write_remove_reviews(vals)
         return super()._write(vals)
@@ -437,7 +440,6 @@ def _tier_validation_check_write_allowed(self, vals):
                 rec.review_ids
                 and rec._check_tier_state_transition(vals)
                 and not rec._check_allow_write_under_validation(vals)
-                and not rec._context.get("skip_validation_check")
             ):
                 (
                     allowed_fields,
@@ -464,7 +466,6 @@ def _tier_validation_check_write_allowed(self, vals):
                 and rec._tier_validation_get_current_state_value()
                 in (self._state_to + [self._cancel_state])
                 and not rec._check_allow_write_after_validation(vals)
-                and not rec._context.get("skip_validation_check")
             ):
                 (
                     allowed_fields,