From 7839e67f1f91ec40a54b6ac992a36024510dc488 Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Fri, 12 Apr 2013 13:29:44 -0700 Subject: [PATCH] Don't show sync notification if user logged out but still has pending messages (edge case) --- .../com/todoroo/astrid/actfm/sync/ActFmSyncThread.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncThread.java b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncThread.java index 1029d5a5d..75d5563e8 100644 --- a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncThread.java +++ b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncThread.java @@ -253,7 +253,7 @@ public class ActFmSyncThread { synchronized(monitor) { while ((pendingMessages.isEmpty() && !timeForBackgroundSync()) || !actFmPreferenceService.isLoggedIn() || !syncMigration) { try { - if (pendingMessages.isEmpty() && notificationId >= 0) { + if ((pendingMessages.isEmpty() || !actFmPreferenceService.isLoggedIn()) && notificationId >= 0) { notificationManager.cancel(notificationId); notificationId = -1; }