refactor(helper): Simplify getContainersData logic and improve logging

- Removed redundant comments and streamlined the getContainersData method for clarity.
- Enhanced logging for missing contentlets to provide better context during debugging.
- Updated return structure to directly return container data, improving efficiency.

Author: fmontes

examples/dotcms-laravel/app/Helpers/DotCmsHelpers.php

@@ -16,7 +16,6 @@ class DotCmsHelpers
      */
     public function getContainersData($containers, $container)
     {
-        // First try to get the container data using the SDK helper
         $containerData = DotCmsHelper::getContainerData($containers, $container);
 
         if (!$containerData) {
@@ -28,26 +27,11 @@ public function getContainersData($containers, $container)
             ];
         }
 
-        $identifier = $container['identifier'] ?? '';
-        $uuid = $container['uuid'] ?? '';
-        
-        $structures = $containerData['containerStructures'] ?? [];
-        $container = $containerData['container'] ?? [];
-        
-        $contentlets = $containerData['contentlets']["uuid-$uuid"] 
-            ?? $containerData['contentlets']["uuid-dotParser_$uuid"] 
-            ?? [];
-        
-        if (empty($contentlets)) {
-            Log::warning("No contentlets found for container: $identifier, uuid: $uuid");
+        if (empty($containerData['contentlets'])) {
+            Log::warning("No contentlets found for container: " . ($container['identifier'] ?? 'unknown') . ", uuid: " . ($container['uuid'] ?? 'unknown'));
         }
 
-        return [
-            ...$container,
-            'acceptTypes' => implode(',', array_column($structures, 'contentTypeVar')),
-            'contentlets' => $contentlets,
-            'variantId' => $container['parentPermissionable']['variantId'] ?? null
-        ];
+        return $containerData;
     }
 
     /**

examples/dotcms-symfony/src/Twig/DotCMSExtension.php

@@ -69,32 +69,16 @@ public function generateHtmlBasedOnProperty(array $content): string
 
     public function getContainersData(array $containers, array $containerRef): array 
     {
-        // First try to get the container data using the SDK helper
         $containerData = DotCmsHelper::getContainerData($containers, $containerRef);
 
         if (!$containerData) {
             throw new RuntimeException("Container not found: " . ($containerRef['identifier'] ?? 'unknown'));
         }
 
-        $identifier = $containerRef['identifier'] ?? throw new RuntimeException("Missing container identifier");
-        $uuid = $containerRef['uuid'] ?? throw new RuntimeException("Missing container UUID");
-        
-        $structures = $containerData['containerStructures'] ?? [];
-        $container = $containerData['container'] ?? [];
-        
-        $contentlets = $containerData['contentlets']["uuid-$uuid"] 
-            ?? $containerData['contentlets']["uuid-dotParser_$uuid"] 
-            ?? [];
-        
-        if (empty($contentlets)) {
-            error_log("No contentlets found for container: $identifier, uuid: $uuid");
+        if (empty($containerData['contentlets'])) {
+            error_log("No contentlets found for container: " . ($containerRef['identifier'] ?? 'unknown') . ", uuid: " . ($containerRef['uuid'] ?? 'unknown'));
         }
 
-        return [
-            ...$container,
-            'acceptTypes' => implode(',', array_column($structures, 'contentTypeVar')),
-            'contentlets' => $contentlets,
-            'variantId' => $container['parentPermissionable']['variantId'] ?? null
-        ];
+        return $containerData;
     }
 } 

src/Utils/DotCmsHelper.php

@@ -21,6 +21,7 @@ public static function getContainerData(array $containers, array $container): ?a
         }
 
         $identifier = $container['identifier'] ?? null;
+        $uuid = $container['uuid'] ?? null;
 
         if (! $identifier || ! isset($containers[$identifier])) {
             return null;
@@ -30,7 +31,21 @@ public static function getContainerData(array $containers, array $container): ?a
             return null;
         }
 
-        return $containers[$identifier];
+        $containerData = $containers[$identifier];
+        $structures = $containerData['containerStructures'] ?? [];
+        $container = $containerData['container'] ?? [];
+        
+        $contentlets = $containerData['contentlets']["uuid-$uuid"] 
+            ?? $containerData['contentlets']["uuid-dotParser_$uuid"] 
+            ?? [];
+
+        return [
+            ...$container,
+            'acceptTypes' => implode(',', array_column($structures, 'contentTypeVar')),
+            'contentlets' => $contentlets,
+            'maxContentlets' => $container['maxContentlets'] ?? 0,
+            'variantId' => $container['parentPermissionable']['variantId'] ?? null
+        ];
     }
 
     /**