From 371dd2bbb83e67bb8e1bcc0473c719557fd84087 Mon Sep 17 00:00:00 2001
From: Tim Su <tim@todoroo.com>
Date: Mon, 1 Aug 2011 22:25:37 -0700
Subject: [PATCH] Don't send before completing repeating task

---
 .../com/todoroo/astrid/actfm/sync/ActFmSyncService.java     | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncService.java b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncService.java
index d376054a7..270f3cd37 100644
--- a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncService.java
+++ b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/ActFmSyncService.java
@@ -94,6 +94,8 @@ public final class ActFmSyncService {
                 final ContentValues setValues = model.getSetValues();
                 if(setValues == null || !checkForToken() || setValues.containsKey(RemoteModel.REMOTE_ID_PROPERTY_NAME))
                     return;
+                if(completedRepeatingTask(model))
+                    return;
 
                 new Thread(new Runnable() {
                     @Override
@@ -104,6 +106,10 @@ public final class ActFmSyncService {
                     }
                 }).start();
             }
+
+            private boolean completedRepeatingTask(Task model) {
+                return !TextUtils.isEmpty(model.getValue(Task.RECURRENCE)) && model.isCompleted();
+            }
         });
 
         updateDao.addListener(new ModelUpdateListener<Update>() {