From 7a6fe4925956e31a270144f6e247c1a962ef9ce0 Mon Sep 17 00:00:00 2001 From: Tim Su Date: Mon, 16 Aug 2010 17:08:32 -0700 Subject: [PATCH] overdue alarm is still scheudled if task ain't due --- .../com/todoroo/astrid/reminders/ReminderService.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/reminders/ReminderService.java b/astrid/plugin-src/com/todoroo/astrid/reminders/ReminderService.java index ca4b04580..183dab987 100644 --- a/astrid/plugin-src/com/todoroo/astrid/reminders/ReminderService.java +++ b/astrid/plugin-src/com/todoroo/astrid/reminders/ReminderService.java @@ -195,9 +195,9 @@ public final class ReminderService { private long calculateNextOverdueReminder(Task task) { if(task.hasDueDate() && task.getFlag(Task.REMINDER_FLAGS, Task.NOTIFY_AFTER_DEADLINE)) { long dueDate = task.getValue(Task.DUE_DATE); - if(dueDate > DateUtilities.now()) - return NO_ALARM; - return DateUtilities.now() + (long)((4 + 30 * random.nextFloat()) * DateUtilities.ONE_HOUR); + if(dueDate < DateUtilities.now()) + dueDate = DateUtilities.now(); + return dueDate + (long)((4 + 30 * random.nextFloat()) * DateUtilities.ONE_HOUR); } return NO_ALARM; }