Don't show sync notification if user logged out but still has pending messages (edge case)

12 years ago · 7839e67f1f
parent a0626ddfc2
commit 7839e67f1f
1 changed files with 1 additions and 1 deletions
--- 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;
                            }