Revert 21aaeca8.

That change makes no sense latency-wise, iteration will anyway happen
inside "list()". Also, it potentially degrades memory usage.

product/ERP5/bootstrap/erp5_core/SkinTemplateItem/portal_skins/erp5_core/ERP5Site_checkDataWithScript.xml

@@ -108,8 +108,7 @@ if depth == maximum_depth and id_list:\n
 else:\n
   if id_list is None:\n
     if full or not getattr(document, \'isHBTree\', lambda: 0)():\n
-      #Cast in list to avoid accessing ZODB objects for each iteration\n
-      id_list = list(document.objectIds())\n
+      id_list = document.objectIds()\n
       if not depth:\n
         id_list = tuple(x for x in id_list if x.endswith(\'_module\') or x in [\n
           \'portal_alarms\',\n