fix(SyncProcess#removeMapping): Only wait for GC to remove mappings with atomic adapters

Signed-off-by: Marcel Klehr <[email protected]>

Author: marcelklehr

src/lib/Account.ts

@@ -313,7 +313,7 @@ export default class Account {
 
       if (mappings) {
         // Remove superfluous items from mappings
-        // as we don't remove items immediately anymore, due to possible interrupts
+        // as we don't remove items immediately for anymore (for Atomic adapters), due to possible interrupts
         await mappings.gc(cache)
         // store mappings
         await mappings.persist()

src/lib/strategies/Default.ts

@@ -1381,7 +1381,15 @@ export default class SyncProcess {
     } else {
       localId = item.id
     }
-    // We don't remove from mappings immediately anymore, but wait for GC
+    if (!this.server.isAtomic()) {
+      if (item.type === 'folder') {
+        await this.mappings.removeFolder({ localId, remoteId })
+      } else {
+        await this.mappings.removeBookmark({ localId, remoteId })
+      }
+    } else {
+      // We don't remove from mappings immediately anymore, but wait for GC
+    }
   }
 
   async loadChildren(