feat: maintain placeholder order from template declaration (#83)

Author: metaforx

djangocms_rest/serializers/pages.py

@@ -131,17 +131,27 @@ def to_representation(self, page_content: PageContent) -> dict:
 
 
 class PageContentSerializer(BasePageSerializer, BasePageContentMixin):
+    """
+    Serialize the page content with the placeholders in the order they are declared in the template.
+    """
     placeholders = PlaceholderSerializer(many=True, required=False)
 
     def __init__(self, *args, **kwargs):
         super().__init__(*args, **kwargs)
         self.request = self.context.get("request")
 
     def to_representation(self, page_content: PageContent) -> dict:
-        declared_slots = [placeholder.slot for placeholder in get_declared_placeholders_for_obj(page_content)]
+        declared_placeholders = get_declared_placeholders_for_obj(page_content)
+        placeholder_map = {
+            placeholder.slot: placeholder
+            for placeholder in page_content.placeholders.all()
+        }
         placeholders = [
-            placeholder for placeholder in page_content.placeholders.all() if placeholder.slot in declared_slots
+            placeholder_map[declared.slot]
+            for declared in declared_placeholders
+            if declared.slot in placeholder_map
         ]
+
         data = self.get_base_representation(page_content)
         data["placeholders"] = PlaceholderSerializer(
             placeholders,