Changeset 52ccac6 in flexoentity

Timestamp:

10/27/25 11:34:39 (3 months ago)

Author:

Enrico Schwass <ennoausberlin@…>

Branches:

master

Children:

ca39274

Parents:

a3227e7

Message:

fix obsolete state handling

File:

• flexoentity/flexo_entity.py (modified) (4 diffs)

flexoentity/flexo_entity.py

@@ -247 +247 @@
         # special case: marking obsolete
         if target_state == EntityState.OBSOLETE:
+            self.flexo_id = FlexOID(self.flexo_id.flexo_id[:-1] + "O") 
             self.state = target_state
             return
@@ -334 +335 @@
 
     def obsolete(self):
+        allowed = self.allowed_transitions()
+        if "OBSOLETE" not in allowed:
+            raise ValueError(
+                f"Illegal state transition: {self.state.name} -> OBSOLETE. "
+                f"Allowed: {', '.join(allowed) or 'none'}"
+            )
         if self.state != EntityState.OBSOLETE:
             self._transition(EntityState.OBSOLETE)
@@ -387 +394 @@
         if new_state == EntityState.APPROVED:
             self.approve()
-        elif new_state == EntityState.SIGNED:
+        elif new_state == EntityState.APPROVED_AND_SIGNED:
             self.sign()
         elif new_state == EntityState.PUBLISHED:
@@ -394 +401 @@
             self.obsolete()
         else:
-            raise RuntimeError(f"No handler for state transition to {new_state.name}")
+            raise RuntimeError(f"No handler for state transition to {new_state}")