Changeset aaaa4b8 in flexograder for builder/exam_elements.py

Timestamp:

11/19/25 11:56:24 (5 months ago)

Author:

Enrico Schwass <ennoausberlin@…>

Branches:

fake-data, main, master

Children:

0792ddd

Parents:

7cc3672

Message:

flexograder entities updated to reflect changes on FlexoEntity

File:

• builder/exam_elements.py (modified) (9 diffs)

builder/exam_elements.py

@@ -5 +5 @@
 from dataclasses import dataclass, field
 from flexoentity import FlexoEntity, EntityType, EntityState, FlexOID, logger
-from .media_items import MediaItem, NullMediaItem, DownloadItem
+from builder.media_items import MediaItem, NullMediaItem, DownloadItem, media_factory
 
 
@@ -47 +47 @@
     text: str = ""
     topic: str = ""
-    media: List[MediaItem] = field(default_factory=lambda: [NullMediaItem()])
+    media: List[MediaItem] = field(default_factory=lambda: [])
 
     @classmethod
     def default(cls):
-        return cls()
+        return cls.with_domain_id(domain_id="DEF_DEFAULT")
 
     @property
@@ -98 +98 @@
     def media_to_html(self, prefix="media/") -> str:
         return "".join([m.to_html(prefix=prefix) for m in self.media if m])
+
+    @classmethod
+    def from_dict(cls, data):
+        if "flexo_id" not in data:
+            raise ValueError("Question is missing flexo_id")
+
+        flexo_id = FlexOID(data["flexo_id"])
+
+        # NOTE: cls(...) will call the correct subclass constructor
+        obj = cls(
+            text=data.get("text", ""),
+            topic=data.get("topic", ""),
+            flexo_id=flexo_id,
+        )
+
+        # restore provenance
+        obj.fingerprint = data.get("fingerprint", "")
+        obj.origin = data.get("origin")
+        if data.get("originator_id"):
+            obj.originator_id = UUID(data["originator_id"])
+        if data.get("owner_id"):
+            obj.owner_id = UUID(data["owner_id"])
+
+        return obj
 
     def to_dict(self):
@@ -105 +129 @@
             "qtype": self.qtype,  # avoid name clash with built-in type
             "text": self.text,
-            "media": [m.to_dict() for m in self.media if not isinstance(m, NullMediaItem)],
+            "media": [m.to_dict() for m in self.media],
             "state": self.state.name,  # explicit readable form
         })
@@ -111 +135 @@
 
     def has_media(self):
-        return any(not isinstance(item, NullMediaItem) for item in self.media)
+        return any(item for item in self.media)
 
     def get_media(self):
-        return [item for item in self.media if not isinstance(item, NullMediaItem)]
+        return [item for item in self.media]
 
 
@@ -154 +178 @@
         return d
 
+    @classmethod
+    def from_dict(cls, data):
+        obj = super().from_dict(data)
+        obj.media = [media_factory(m) for m in data.get("media", [])]
+        return obj
+
 
 @dataclass
@@ -187 +217 @@
         })
         return base
+
+    @classmethod
+    def from_dict(cls, data):
+        obj = super().from_dict(data)
+        obj.options = [
+            AnswerOption(o["id"], o["text"], o.get("points", 0.0))
+            for o in data.get("options", [])
+        ]
+
+        obj.media = [media_factory(m) for m in data.get("media", [])]
+        return obj
 
     def points_for(self, option_id):
@@ -280 +321 @@
 
     @classmethod
-    def from_dict(cls, data: dict) -> "TextQuestion":
-        validation = None
-        if "validation" in data and isinstance(data["validation"], dict):
-            validation = Validation(**data["validation"])
+    def from_dict(cls, data):
         obj = super().from_dict(data)
-        obj.validation = validation
+        v = data.get("validation")
+        obj.validation = Validation(**v) if v else None
+        obj.media = [media_factory(m) for m in data.get("media", [])]
         return obj
 
@@ -363 +403 @@
         return d
 
-    @staticmethod
-    def from_dict(data: dict) -> "CandidateIDQuestion":
-        q = CandidateIDQuestion(text=data["text"],
-                                topic=data.get("topic", ""),
-                                fields=data.get("fields", []),
-        )
-        flexo_id = data.get("id",
-                       FlexOID.safe_generate(domain="SYS",
-                                             entity_type=EntityType.ITEM,
-                                             entity_state=EntityState.DRAFT,
-                                             ))
-        q.flexo_id = flexo_id
-        return q
+    @classmethod
+    def from_dict(cls, data):
+        obj = super().from_dict(data)
+        obj.fields = data.get("fields", [])
+        obj.media = [media_factory(m) for m in data.get("media", [])]
+        return obj
 
 @dataclass