From 8e5249205498b4389c6cf5001cedde6c362a81e4 Mon Sep 17 00:00:00 2001
From: Tim Su <tim@todoroo.com>
Date: Mon, 23 Aug 2010 18:09:04 -0700
Subject: [PATCH] AST-295 - if a task is overdue upon sync, turn off
 notify-after-deadline flag

---
 .../producteev/sync/ProducteevSyncProvider.java       |  3 +++
 .../todoroo/astrid/rmilk/sync/RTMSyncProvider.java    | 11 ++++++++---
 2 files changed, 11 insertions(+), 3 deletions(-)

diff --git a/astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevSyncProvider.java b/astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevSyncProvider.java
index 950c28d78..f2898bf1b 100644
--- a/astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevSyncProvider.java
+++ b/astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevSyncProvider.java
@@ -228,6 +228,9 @@ public class ProducteevSyncProvider extends SyncProvider<ProducteevTaskContainer
                 JSONArray tasks = invoker.tasksShowList(dashboardId, lastServerSync);
                 for(int i = 0; i < tasks.length(); i++) {
                     ProducteevTaskContainer remote = parseRemoteTask(tasks.getJSONObject(i));
+                    // update reminder flags for incoming remote tasks to prevent annoying
+                    if(remote.task.hasDueDate() && remote.task.getValue(Task.DUE_DATE) < DateUtilities.now())
+                        remote.task.setFlag(Task.REMINDER_FLAGS, Task.NOTIFY_AFTER_DEADLINE, false);
                     boolean foundLocal = dataService.findLocalMatch(remote);
 
                     // if creator & responsible != current user, skip / delete it
diff --git a/astrid/plugin-src/com/todoroo/astrid/rmilk/sync/RTMSyncProvider.java b/astrid/plugin-src/com/todoroo/astrid/rmilk/sync/RTMSyncProvider.java
index e1584cbcf..103c701db 100644
--- a/astrid/plugin-src/com/todoroo/astrid/rmilk/sync/RTMSyncProvider.java
+++ b/astrid/plugin-src/com/todoroo/astrid/rmilk/sync/RTMSyncProvider.java
@@ -357,9 +357,14 @@ public class RTMSyncProvider extends SyncProvider<RTMTaskContainer> {
     private void addTasksToList(RtmTasks tasks, ArrayList<RTMTaskContainer> list) {
         for (RtmTaskList taskList : tasks.getLists()) {
             for (RtmTaskSeries taskSeries : taskList.getSeries()) {
-                RTMTaskContainer remoteTask = parseRemoteTask(taskSeries);
-                dataService.findLocalMatch(remoteTask);
-                list.add(remoteTask);
+                RTMTaskContainer remote = parseRemoteTask(taskSeries);
+
+                // update reminder flags for incoming remote tasks to prevent annoying
+                if(remote.task.hasDueDate() && remote.task.getValue(Task.DUE_DATE) < DateUtilities.now())
+                    remote.task.setFlag(Task.REMINDER_FLAGS, Task.NOTIFY_AFTER_DEADLINE, false);
+
+                dataService.findLocalMatch(remote);
+                list.add(remote);
             }
         }
     }