From d1a398a74db45199e40053b49c1fb4e545e04eeb Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Mon, 29 Aug 2011 17:49:22 -0700 Subject: [PATCH] Fixed a bug where reminders for tasks without due times wouldn't show at the default time --- .../com/todoroo/astrid/reminders/Notifications.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/reminders/Notifications.java b/astrid/plugin-src/com/todoroo/astrid/reminders/Notifications.java index ad6786198..abc494172 100644 --- 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