fixup! edi_oca: consumer mixin trigger state event

thienvh332 · Nov 29, 2024 · f1da5a5 · f1da5a5
1 parent 948d868
commit f1da5a5
Showing 1 changed file with 9 additions and 2 deletions.
diff --git a/edi_oca/models/edi_exchange_consumer_mixin.py b/edi_oca/models/edi_exchange_consumer_mixin.py
@@ -408,9 +408,16 @@ def _edi_send_via_email(
     def write(self, vals):
         # Generic event to match a state change
         # TODO: this can be added to component_event for models having the state field
-        if "state" in vals and "state" in self._fields:
+        state_change = "state" in vals and "state" in self._fields
+        if state_change:
+            for rec in self:
+                rec._event(f"on_edi_{self._table}_before_state_change").notify(
+                    rec, state=vals["state"]
+                )
+        res = super().write(vals)
+        if state_change:
             for rec in self:
                 rec._event(f"on_edi_{self._table}_state_change").notify(
                     rec, state=vals["state"]
                 )
-        return super().write(vals)
+        return res