Don't allow inactive sync providers (like gtasks) to sync in background

12 years ago · 922db53862
parent 33a93a40ab
commit 922db53862
1 changed files with 9 additions and 7 deletions
--- a/api/src/com/todoroo/astrid/sync/SyncV2BackgroundService.java
+++ b/api/src/com/todoroo/astrid/sync/SyncV2BackgroundService.java
@ -77,13 +77,15 @@ abstract public class SyncV2BackgroundService extends Service {
        if(!getSyncUtilities().isLoggedIn())
            return;

-        getSyncProvider().synchronizeActiveTasks(false, new SyncResultCallbackAdapter() {
-            @Override
-            public void finished() {
-                getSyncUtilities().recordSuccessfulSync();
-                context.sendBroadcast(new Intent(AstridApiConstants.BROADCAST_EVENT_REFRESH));
-            }
-        });
+        SyncV2Provider provider = getSyncProvider();
+        if (provider.isActive())
+            provider.synchronizeActiveTasks(false, new SyncResultCallbackAdapter() {
+                @Override
+                public void finished() {
+                    getSyncUtilities().recordSuccessfulSync();
+                    context.sendBroadcast(new Intent(AstridApiConstants.BROADCAST_EVENT_REFRESH));
+                }
+            });
    }

    @Override