Syncer: make sure collections which are deleted are not synced

bitfireAT · Oct 9, 2024 · ba0db9f · ba0db9f
1 parent 26a670c
commit ba0db9f
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/app/src/main/kotlin/at/bitfire/davdroid/sync/Syncer.kt b/app/src/main/kotlin/at/bitfire/davdroid/sync/Syncer.kt
@@ -137,16 +137,17 @@ abstract class Syncer<CollectionType: LocalCollection<*>>(
  */
  @VisibleForTesting
  internal fun updateCollections(
- localCollections: List<CollectionType>,
+ localCollections: MutableList<CollectionType>,
  dbCollections: Map<HttpUrl, Collection>
  ): HashMap<HttpUrl, Collection> {
  val newDbCollections = HashMap(dbCollections) // create a copy
  for (localCollection in localCollections) {
  val dbCollection = dbCollections[localCollection.collectionUrl?.toHttpUrlOrNull()]
- if (dbCollection == null)
+ if (dbCollection == null) {
  // Collection not available in db = on server (anymore), delete obsolete local collection
  localCollection.deleteCollection()
- else {
+ localCollections -= localCollection
+ } else {
  // Collection exists locally, update local collection and remove it from "to be created" map
  update(localCollection, dbCollection)
  newDbCollections -= dbCollection.url