Fix sorting bugs in @scadenziario-day (#291)

Co-authored-by: Luca <[email protected]>
RedTurtle · Jan 13, 2025 · 436094f · 436094f
1 parent c46ae65
commit 436094f
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/CHANGES.rst b/CHANGES.rst
@@ -8,6 +8,8 @@ Changelog
   [mamico]
 - add SEO behavior to File
   [mamico]
+- Fix sorting in @scadenziario-day endpoint.
+  [folix-01]
 
 6.3.1 (2024-12-12)
 ------------------

diff --git a/src/design/plone/contenttypes/restapi/services/scadenziario/post.py b/src/design/plone/contenttypes/restapi/services/scadenziario/post.py
@@ -194,8 +194,8 @@ def reply(self):
         if query is None:
             raise Exception("No query supplied")
 
-        if sort_order:
-            sort_order = "descending" if sort_order else "ascending"
+        if sort_order not in {"descending", "ascending"}:
+            sort_order = "ascending"
 
         # results = querybuilder(**querybuilder_parameters)
         # Seems that origina querybuilder is not able to handle event search on
@@ -286,7 +286,6 @@ def reply(self):
                         }
                     )
 
-                results_to_be_returned[key].sort(key=lambda x: x["title"])
         return {
             "@id": self.request.get("URL"),
             "items": results_to_be_returned,