fix #3513: do not clean offline logs during background cleanup

1 files changed, 29 insertions, 0 deletions

diff --git a/main/src/cgeo/geocaching/DataStore.java b/main/src/cgeo/geocaching/DataStore.java
index 4543451..14be1f1 100644
--- a/main/src/cgeo/geocaching/DataStore.java
+++ b/main/src/cgeo/geocaching/DataStore.java
@@ -2333,6 +2333,8 @@ public class DataStore {
 
             cursor.close();
 
+            geocodes = exceptCachesWithOfflineLog(geocodes);
+
             if (!geocodes.isEmpty()) {
                 Log.d("Database clean: removing " + geocodes.size() + " geocaches from listId=0");
                 removeCaches(geocodes, LoadFlags.REMOVE_ALL);
@@ -2345,6 +2347,33 @@ public class DataStore {
         databaseCleaned = true;
     }
 
+    /**
+     * remove all geocodes from the given list of geocodes where an offline log exists
+     *
+     * @param geocodes
+     * @return
+     */
+    private static Set<String> exceptCachesWithOfflineLog(Set<String> geocodes) {
+        if (geocodes.isEmpty()) {
+            return geocodes;
+        }
+
+        init();
+        final Cursor cursor = database.query(
+                dbTableLogsOffline,
+                new String[] { "geocode" },
+                null,
+                null,
+                null,
+                null,
+                null);
+
+        final List<String> geocodesWithOfflineLog = Arrays.asList(getFirstColumn(cursor));
+
+        geocodes.removeAll(geocodesWithOfflineLog);
+        return geocodes;
+    }
+
     public static void removeAllFromCache() {
         // clean up CacheCache
         cacheCache.removeAllFromCache();