fix memory issue when dealing with large asset and document lists

Author: benwalch

UPGRADE.md

@@ -1,7 +1,7 @@
 # Upgrade Notes
 
 ## 3.0.3
-- [IMPROVEMENT] fix memory issue when dealing with large object lists (#41)
+- [IMPROVEMENT] fix memory issue when dealing with large lists (assets, documents, objects) (#41)
 ## 3.0.2
 - [LICENSE] Dual-License with GPL and Dachcom Commercial License (DCL) added
 ## 3.0.1

src/Service/Builder/AssetListBuilder.php

@@ -32,10 +32,14 @@ public function buildByList(array $options): \Generator
     {
         $list = $this->getList($options);
 
-        foreach ($list->loadIdList() as $id) {
-            if ($asset = Asset::getById($id)) {
+        $idList = $list->loadIdList();
+        for ($i = 0; $i < count($idList); $i++) {
+            if ($asset = Asset::getById($idList[$i])) {
                 yield $asset;
             }
+            if ($i % 400 === 0) {
+                \Pimcore::collectGarbage();
+            }
         }
     }
 

src/Service/Builder/DocumentListBuilder.php

@@ -32,10 +32,14 @@ public function buildByList(array $options): \Generator
     {
         $list = $this->getList($options);
 
-        foreach ($list->loadIdList() as $id) {
-            if ($doc = Document::getById($id)) {
+        $idList = $list->loadIdList();
+        for ($i = 0; $i < count($idList); $i++) {
+            if ($doc = Document::getById($idList[$i])) {
                 yield $doc;
             }
+            if ($i % 400 === 0) {
+                \Pimcore::collectGarbage();
+            }
         }
     }