Handle missing infos in prenotazioniFolder serializer

RedTurtle · Nov 9, 2023 · 31f0592 · 31f0592
1 parent 932795d
commit 31f0592
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 9 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -4,7 +4,8 @@ Changelog
 1.1.11 (unreleased)
 -------------------
 
-- Nothing changed yet.
+- Handle missing infos in prenotazioniFolder serializer.
+  [cekk]
 
 
 1.1.10 (2023-10-16)

diff --git a/src/design/plone/ioprenoto/restapi/serializers/ovverrides/prenotazioniFolder.py b/src/design/plone/ioprenoto/restapi/serializers/ovverrides/prenotazioniFolder.py
@@ -26,11 +26,11 @@ def __call__(self, *args, **kwargs):
         ):
             # XXX: this is a workaround to avoid the Unauthorized exception
             return {
-                "@components": resp["@components"],
-                "@id": resp["@id"],
-                "@type": resp["@type"],
-                "layout": resp["layout"],
-                "title": resp["title"],
+                "@components": resp.get("@components", {}),
+                "@id": resp.get("@id", ""),
+                "@type": resp.get("@type", ""),
+                "layout": resp.get("@layout", ""),
+                "title": resp.get("@title", ""),
                 "error": "Unauthorized",
                 "anonymous": api.user.is_anonymous(),
             }

diff --git a/src/design/plone/ioprenoto/tests/test_prenotazione_add.py b/src/design/plone/ioprenoto/tests/test_prenotazione_add.py
@@ -23,17 +23,24 @@ def setUp(self):
             title="Prenota foo",
             description="",
             daData=date.today(),
-            booking_types=[
-                {"name": "Type A", "duration": "30"},
-            ],
             gates=["Gate A"],
         )
         week_table = self.folder_prenotazioni.week_table
         for row in week_table:
             row["morning_start"] = "0700"
             row["morning_end"] = "1000"
         self.folder_prenotazioni.week_table = week_table
+
+        booking_type_a = api.content.create(
+            type="PrenotazioneType",
+            title="Type A",
+            duration=30,
+            container=self.folder_prenotazioni,
+            gates=["all"],
+        )
+
         api.content.transition(obj=self.folder_prenotazioni, transition="publish")
+        api.content.transition(obj=booking_type_a, transition="publish")
         transaction.commit()
 
         self.api_session = RelativeSession(self.portal_url)