34091 task update pageapi to expose style properties object support

dotCMS/src/main/java/com/dotcms/rest/api/v1/page/PageContainerForm.java

@@ -84,35 +84,70 @@ public PageContainerForm deserialize(final JsonParser jsonParser,
 
                 // Parse styleProperties for each contentlet (optional field)
                 final JsonNode stylePropertiesNode = jsonElement.get(STYLE_PROPERTIES_ATTRIBUTE_NAME);
-                if (stylePropertiesNode != null && stylePropertiesNode.isObject()) {
-                    stylePropertiesNode.fields().forEachRemaining(entry -> {
-                        final String contentletId = entry.getKey();
-                        final JsonNode styleProps = entry.getValue();
-                        if (styleProps != null && styleProps.isObject()) {
-                            final Map<String, Object> propsMap = new HashMap<>();
-                            styleProps.fields().forEachRemaining(prop -> {
-                                final JsonNode propValue = prop.getValue();
-                                // Handle different JSON value types
-                                if (propValue.isTextual()) {
-                                    propsMap.put(prop.getKey(), propValue.asText());
-                                } else if (propValue.isNumber()) {
-                                    propsMap.put(prop.getKey(), propValue.numberValue());
-                                } else if (propValue.isBoolean()) {
-                                    propsMap.put(prop.getKey(), propValue.asBoolean());
-                                } else {
-                                    propsMap.put(prop.getKey(), propValue.toString());
-                                }
-                            });
-                            containerEntry.setStyleProperties(contentletId, propsMap);
-                        }
-                    });
+                if (UtilMethods.isSet(stylePropertiesNode) && stylePropertiesNode.isObject()) {
+                    processStyleProperties(stylePropertiesNode, containerEntry);
                 }
 
                 entries.add(containerEntry);
             }
 
             return new PageContainerForm(entries, jsonNode.toString());
         }
+
+        /**
+         * Processes the style properties for the container entry.
+         * It converts the JSON node to a Map<String, Object> and sets it to the container entry.
+         * @param stylePropertiesNode The JSON node containing the style properties.
+         * @param containerEntry The container entry to set the style properties.
+         */
+        private void processStyleProperties(final JsonNode stylePropertiesNode, final ContainerEntry containerEntry) {
+            stylePropertiesNode.fields().forEachRemaining(entry -> {
+                final String contentletId = entry.getKey();
+                final JsonNode styleProps = entry.getValue();
+                if (styleProps != null && styleProps.isObject()) {
+                    final Map<String, Object> propsMap = new HashMap<>();
+                    styleProps.fields().forEachRemaining(prop -> {
+                        final JsonNode propValue = prop.getValue();
+                        propsMap.put(prop.getKey(), convertJsonNodeToObject(propValue));
+                    });
+                    containerEntry.setStyleProperties(contentletId, propsMap);
+                }
+            });
+        }
+
+        /**
+         * Recursively converts a JsonNode to its corresponding Java object type.
+         * Handles all JSON types: primitives, objects, arrays, and null.
+         *
+         * @param node The JsonNode to convert
+         * @return The converted Java object (String, Number, Boolean, Map, List, or null)
+         */
+        private Object convertJsonNodeToObject(final JsonNode node) {
+            if (node == null || node.isNull()) {
+                return null;
+            } else if (node.isBoolean()) {
+                return node.asBoolean();
+            } else if (node.isInt()) {
+                return node.asInt();
+            } else if (node.isLong()) {
+                return node.asLong();
+            } else if (node.isDouble() || node.isFloat()) {
+                return node.asDouble();
+            } else if (node.isArray()) {
+                final List<Object> list = new ArrayList<>();
+                node.forEach(element -> list.add(convertJsonNodeToObject(element)));
+                return list;
+            } else if (node.isObject()) {
+                final Map<String, Object> map = new HashMap<>();
+                node.fields().forEachRemaining(entry ->
+                        map.put(entry.getKey(), convertJsonNodeToObject(entry.getValue()))
+                );
+                return map;
+            } else {
+                // Fallback for any other type and String values
+                return node.asText();
+            }
+        }
     }
 
     /**

dotCMS/src/main/java/com/dotmarketing/beans/MultiTree.java

@@ -7,6 +7,7 @@
 import com.dotmarketing.portlets.containers.model.Container;
 import com.dotmarketing.portlets.contentlet.model.Contentlet;
 import com.dotmarketing.portlets.htmlpageasset.model.HTMLPageAsset;
+import com.dotmarketing.util.UtilMethods;
 import com.fasterxml.jackson.annotation.JsonIgnore;
 import java.util.Map;
 import org.apache.commons.lang.builder.EqualsBuilder;
@@ -75,7 +76,7 @@ public MultiTree(final String htmlPage,
         this.treeOrder    = Math.max(treeOrder, 0);
         this.personalization = personalization;
         this.variantId = variantId;
-        this.styleProperties = styleProperties;
+        this.styleProperties = UtilMethods.isSet(styleProperties) ? Map.copyOf(styleProperties) : Map.of();
     }
 
     /** full constructor */

dotCMS/src/main/java/com/dotmarketing/factories/PersonalizedContentlet.java

@@ -1,5 +1,6 @@
 package com.dotmarketing.factories;
 
+import com.dotmarketing.util.UtilMethods;
 import java.io.Serializable;
 import java.util.Map;
 import java.util.Objects;
@@ -22,14 +23,14 @@ public PersonalizedContentlet(final String contentletId, final String personaliz
         this.contentletId    = contentletId;
         this.personalization = personalization;
         this.treeOrder = treeOrder;
-        this.styleProperties = styleProperties;
+        this.styleProperties = UtilMethods.isSet(styleProperties) ? Map.copyOf(styleProperties) : Map.of();
     }
 
     public PersonalizedContentlet(final String contentletId, final String personalization) {
         this.contentletId    = contentletId;
         this.personalization = personalization;
         this.treeOrder = 0;
-        this.styleProperties = null;
+        this.styleProperties = Map.of();
     }
 
     public String getContentletId() {
@@ -44,6 +45,10 @@ public Object getTreeOrder() {
         return treeOrder;
     }
 
+    public Map<String, Object> getStyleProperties() {
+        return styleProperties;
+    }
+
     @Override
     public boolean equals(Object o) {
         if (this == o) return true;

dotcms-integration/src/test/java/com/dotmarketing/factories/MultiTreeAPITest.java

@@ -4359,8 +4359,7 @@ public void testUpdate_stylePropertiesToNull() throws Exception {
 
         MultiTree retrieved = retrieveMultiTree(0);
 
-        assertNull("NULL style properties should be persisted",
-                retrieved.getStyleProperties());
+        assertEquals("Style properties should be empty", Map.of(), retrieved.getStyleProperties());
     }
 
     /**
@@ -4376,7 +4375,7 @@ public void testCreate_withNullStyleProperties() throws Exception {
         MultiTree retrieved = retrieveMultiTree(0);
 
         assertNotNull("Retrieved multiTree should not be null", retrieved);
-        assertNull("Style properties should be null", retrieved.getStyleProperties());
+        assertEquals("Style properties should be empty", Map.of(), retrieved.getStyleProperties());
     }
 
     /**
@@ -4393,7 +4392,7 @@ public void testCreate_withEmptyStyleProperties() throws Exception {
         MultiTree retrieved = retrieveMultiTree(0);
 
         assertNotNull("Retrieved multiTree should not be null", retrieved);
-        assertNull("Style properties should be null", retrieved.getStyleProperties());
+        assertEquals("Style properties should be empty", Map.of(), retrieved.getStyleProperties());
     }
 
 }