fix synchronizing backups

NeoApplications · Oct 26, 2024 · f5bbde9 · f5bbde9
1 parent 0ae2261
commit f5bbde9
Showing 1 changed file with 17 additions and 11 deletions.
diff --git a/src/main/java/com/machiav3lli/backup/OABX.kt b/src/main/java/com/machiav3lli/backup/OABX.kt
@@ -681,13 +681,15 @@ class OABX : Application() {
             }
         }
 
-        fun setBackups(backups: Map<String, List<Backup>>) {
-            backups.forEach {
-                putBackups(it.key, it.value)
-            }
-            // clear no more existing packages
-            (theBackupsMap.keys - backups.keys).forEach {
-                removeBackups(it)
+        fun setBackups(backupsMap: Map<String, List<Backup>>) {
+            synchronized(theBackupsMap) {
+                backupsMap.forEach { (packageName, backups) ->
+                    theBackupsMap.put(packageName, backups)
+                }
+                // clear no more existing packages
+                (theBackupsMap.keys - backupsMap.keys).forEach {
+                    theBackupsMap.remove(it)
+                }
             }
         }
 
@@ -719,14 +721,18 @@ class OABX : Application() {
         }
 
         fun emptyBackupsForMissingPackages(packageNames: List<String>) {
-            (packageNames - theBackupsMap.keys).forEach {
-                putBackups(it, emptyList())
+            synchronized(theBackupsMap) {
+                (packageNames - theBackupsMap.keys).forEach {
+                    theBackupsMap.put(it, emptyList())
+                }
             }
         }
 
         fun emptyBackupsForAllPackages(packageNames: List<String>) {
-            packageNames.forEach {
-                putBackups(it, emptyList())
+            synchronized(theBackupsMap) {
+                packageNames.forEach {
+                    theBackupsMap.put(it, emptyList())
+                }
             }
         }