Fixed a bug where reminders for tasks without due times wouldn't show at the default time

14 years ago · d1a398a74d
parent eef8dcefbb
commit d1a398a74d
1 changed files with 3 additions and 1 deletions
--- a/astrid/plugin-src/com/todoroo/astrid/reminders/Notifications.java
+++ b/astrid/plugin-src/com/todoroo/astrid/reminders/Notifications.java
@ -140,8 +140,10 @@ public class Notifications extends BroadcastReceiver {
            return true;

        // task due date was changed, but alarm wasn't rescheduled
+        boolean dueInFuture = task.hasDueTime() && task.getValue(Task.DUE_DATE) > DateUtilities.now() ||
+            !task.hasDueTime() && task.getValue(Task.DUE_DATE) - DateUtilities.now() > DateUtilities.ONE_DAY;
        if((type == ReminderService.TYPE_DUE || type == ReminderService.TYPE_OVERDUE) &&
-                (!task.hasDueDate() || task.getValue(Task.DUE_DATE) > DateUtilities.now()))
+                (!task.hasDueDate() || dueInFuture))
            return true;

        // read properties