From d507084ddce8493a2b51e31079673f6e78896632 Mon Sep 17 00:00:00 2001 From: Tim Su Date: Fri, 13 Aug 2010 03:06:27 -0700 Subject: [PATCH] Alarms in the past should not be scheduled --- astrid/plugin-src/com/todoroo/astrid/alarms/AlarmService.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/alarms/AlarmService.java b/astrid/plugin-src/com/todoroo/astrid/alarms/AlarmService.java index a1fa25b84..4cdef4b35 100644 --- a/astrid/plugin-src/com/todoroo/astrid/alarms/AlarmService.java +++ b/astrid/plugin-src/com/todoroo/astrid/alarms/AlarmService.java @@ -15,6 +15,7 @@ import com.todoroo.andlib.sql.Criterion; import com.todoroo.andlib.sql.Join; import com.todoroo.andlib.sql.Order; import com.todoroo.andlib.sql.Query; +import com.todoroo.andlib.utility.DateUtilities; import com.todoroo.astrid.core.PluginServices; import com.todoroo.astrid.dao.MetadataDao.MetadataCriteria; import com.todoroo.astrid.dao.TaskDao.TaskCriteria; @@ -192,7 +193,7 @@ public class AlarmService { long time = alarm.getValue(Alarm.TIME); if(time == 0 || time == NO_ALARM) am.cancel(pendingIntent); - else { + else if(time > DateUtilities.now()) { Log.e("Astrid", "Alarm (" + taskId + ", " + type + ") set for " + new Date(time)); am.set(AlarmManager.RTC_WAKEUP, time, pendingIntent);