temporarily disable un-setting due date, squash another source of duplicate tasks - remotely new and completed tasks

astrid/plugin-src/com/todoroo/astrid/common/SyncProvider.java

@@ -304,6 +304,12 @@ public abstract class SyncProvider<TYPE extends TaskContainer> {
         length = data.remoteUpdated.size();
         for(int i = 0; i < length; i++) {
             TYPE remote = data.remoteUpdated.get(i);
+
+            // don't synchronize new & deleted / completed tasks
+            if(!remote.task.isSaved() && (remote.task.isDeleted() ||
+                    remote.task.isCompleted()))
+                continue;
+
             try {
                 write(remote);
             } catch (Exception e) {

astrid/plugin-src/com/todoroo/astrid/producteev/sync/ProducteevSyncProvider.java

@@ -518,7 +518,7 @@ public class ProducteevSyncProvider extends SyncProvider<ProducteevTaskContainer
             invoker.tasksSetTitle(idTask, local.task.getValue(Task.TITLE));
         if(shouldTransmit(local, Task.IMPORTANCE, remote))
             invoker.tasksSetStar(idTask, createStars(local.task));
-        if(shouldTransmit(local, Task.DUE_DATE, remote))
+        if(shouldTransmit(local, Task.DUE_DATE, remote) && local.task.hasDueDate()) // temporary can't unset deadline
             invoker.tasksSetDeadline(idTask, createDeadline(local.task), local.task.hasDueTime() ? 0 : 1);
         if(shouldTransmit(local, Task.COMPLETION_DATE, remote))
             invoker.tasksSetStatus(idTask, local.task.isCompleted() ? 2 : 1);
@@ -662,10 +662,7 @@ public class ProducteevSyncProvider extends SyncProvider<ProducteevTaskContainer
     private String createDeadline(Task task) {
         if(!task.hasDueDate())
             return "";
-        if(!task.hasDueTime())
+        return ApiUtilities.unixTimeToProducteev(task.getValue(Task.DUE_DATE));
-            return ApiUtilities.unixDateToProducteev(task.getValue(Task.DUE_DATE));
-        String time = ApiUtilities.unixTimeToProducteev(task.getValue(Task.DUE_DATE));
-        return time.substring(0, time.lastIndexOf(' '));
     }
 
     /**