remove unnecessary asyncMap

vatesfr · Sep 9, 2022 · eca6244 · eca6244
1 parent 0472122
commit eca6244
Showing 1 changed file with 13 additions and 15 deletions.
diff --git a/@xen-orchestra/backups/RemoteAdapter.js b/@xen-orchestra/backups/RemoteAdapter.js
@@ -226,21 +226,19 @@ class RemoteAdapter {
   }
 
   #removeVmBackupsFromCache(backups) {
-    // will not throw
-    asyncMap(
-      Object.entries(
-        groupBy(
-          backups.map(_ => _._filename),
-          dirname
-        )
-      ),
-      ([dir, filenames]) =>
-        this.#updateCache(dir + '/cache.json.gz', backups => {
-          for (const filename of filenames) {
-            delete backups[filename]
-          }
-        })
-    )
+    for (const [dir, filenames] of Object.entries(
+      groupBy(
+        backups.map(_ => _._filename),
+        dirname
+      )
+    )) {
+      // detached async action, will not reject
+      this.#updateCache(dir + '/cache.json.gz', backups => {
+        for (const filename of filenames) {
+          delete backups[filename]
+        }
+      })
+    }
   }
 
   async deleteDeltaVmBackups(backups) {