fix(deposit): page render and save a empty record

tu-graz-library · Oct 31, 2024 · b04e184 · b04e184
1 parent 4c69ee8
commit b04e184
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 11 deletions.
diff --git a/invenio_records_marc21/resources/deserializers/schema.py b/invenio_records_marc21/resources/deserializers/schema.py
@@ -31,6 +31,12 @@ class Meta:
     def load_metadata(self, value):
         """Load metadata."""
         fields = {}
+        metadata = {
+            "leader": value["leader"],
+        }
+
+        if "fields" not in value:
+            return metadata
 
         for field in value["fields"]:
             if int(field["id"]) < 10:
@@ -50,10 +56,8 @@ def load_metadata(self, value):
                         "subfields": subfields,
                     }
                 )
-        return {
-            "fields": fields,
-            "leader": value["leader"],
-        }
+        metadata["fields"] = fields
+        return metadata
 
     @pre_load
     def remove(self, data, **kwargs):

diff --git a/invenio_records_marc21/resources/serializers/deposit/schema.py b/invenio_records_marc21/resources/serializers/deposit/schema.py
@@ -20,8 +20,14 @@ class MetadataDepositField(Field):
 
     def _serialize(self, value, attr, obj, **kwargs):
         """Serialize metadata field."""
-        fields_ui = []
+        record = {
+            "fields": [],
+            "leader": value.get("leader"),
+        }
+        if "fields" not in value:
+            return record
 
+        fields_ui = []
         for category, fields in value["fields"].items():
             if isinstance(fields, str):
                 obj = {
@@ -45,11 +51,8 @@ def _serialize(self, value, attr, obj, **kwargs):
                     }
 
                     fields_ui.append(obj)
-
-        return {
-            "fields": fields_ui,
-            "leader": value.get("leader"),
-        }
+        record["fields"] = fields_ui
+        return record
 
 
 class Marc21DepositSchema(Marc21Schema):

diff --git a/invenio_records_marc21/ui/theme/deposit.py b/invenio_records_marc21/ui/theme/deposit.py
@@ -20,7 +20,7 @@
 def empty_record():
     """Create an empty record."""
     record = dump_empty(Marc21RecordSchema)
-    record["metadata"] = "<record><leader>00000nam a2200000zca4500</leader></record>"
+    record["metadata"] = {"leader": "00000nam a2200000zca4500", "fields": []}
     record["access"] = {"record": "public", "files": "public"}
     record["files"] = {"enabled": True}
     record["status"] = "draft"