Changeset 3cf10cb in flexoentity

Timestamp:

11/19/25 12:55:23 (2 months ago)

Author:

Enrico Schwass <ennoausberlin@…>

Branches:

master

Children:

bd1554a

Parents:

8840db7

Message:

simplify allowed_transitions and use enums instead of strings

File:

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

flexoentity/flexo_entity.py

@@ -62 +62 @@
 from flexoentity.id_factory import FlexOID
 from flexoentity import canonical_seed
+
 
 # ──────────────────────────────────────────────────────────────────────────────
@@ -105 +106 @@
     OBSOLETE = "O"
 
+    def allowed_transitions(self) -> list[str]:
+        """Return allowed state names for this state."""
+        return TRANSITIONS[self]
+
     def __str__(self):
         return self.name
+
+TRANSITIONS = {
+    EntityState.DRAFT: [EntityState.APPROVED],
+    EntityState.APPROVED: [EntityState.APPROVED_AND_SIGNED, EntityState.PUBLISHED,
+                           EntityState.OBSOLETE],
+    EntityState.APPROVED_AND_SIGNED: [EntityState.PUBLISHED, EntityState.OBSOLETE],
+    EntityState.PUBLISHED: [EntityState.OBSOLETE],
+    EntityState.OBSOLETE: [],
+}
+
 
 
@@ -507 +522 @@
     def allowed_transitions(self) -> list[str]:
         """Return a list of possible next state names (for GUI use)."""
-        match self.state:
-            case EntityState.DRAFT:
-                return ["APPROVED"]
-            case EntityState.APPROVED:
-                return ["APPROVED_AND_SIGNED", "PUBLISHED", "OBSOLETE"]
-            case EntityState.APPROVED_AND_SIGNED:
-                return ["PUBLISHED", "OBSOLETE"]
-            case EntityState.PUBLISHED:
-                return ["OBSOLETE"]
-            case EntityState.OBSOLETE:
-                return []
-            case _:
-                return []
+        return self.state.allowed_transitions()
 
     def apply_state_change(self, new_state):