From 504cb473fe18ea33507e39d839ba5ce248c00b59 Mon Sep 17 00:00:00 2001
From: Sam Bosley <sboz88@gmail.com>
Date: Tue, 9 Apr 2013 11:05:08 -0700
Subject: [PATCH] Partial solution to sync failure when adding blank
 task--delete invalid outstanding entries. Need a solution that won't send the
 other changes until full message is ready though

---
 .../todoroo/astrid/actfm/sync/messages/ChangesHappened.java   | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java
index 8b46796f3..f5540f64c 100644
--- a/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java
+++ b/astrid/plugin-src/com/todoroo/astrid/actfm/sync/messages/ChangesHappened.java
@@ -153,8 +153,10 @@ public class ChangesHappened<TYPE extends RemoteModel, OE extends OutstandingEnt
                         throw new RuntimeException("No server column found for local column " + localColumn + " in table " + table);
 
                     Object value = localProperty.accept(visitor, change);
-                    if (!validateValue(localProperty, value))
+                    if (!validateValue(localProperty, value)) {
+                        outstandingDao.delete(change.getId());
                         return null;
+                    }
 
                     if (value == null)
                         changeJson.put("value", JSONObject.NULL);