From 6dded6fb062bd1db773732ec5662e028d2db8ecb Mon Sep 17 00:00:00 2001 From: Sam Bosley Date: Thu, 28 Feb 2013 16:51:21 -0800 Subject: [PATCH] In subtasks mode, new tasks should be added at the top of the list like on the web --- .../todoroo/astrid/subtasks/AstridOrderedListUpdater.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/astrid/plugin-src/com/todoroo/astrid/subtasks/AstridOrderedListUpdater.java b/astrid/plugin-src/com/todoroo/astrid/subtasks/AstridOrderedListUpdater.java index 9c713340c..655e911dd 100644 --- a/astrid/plugin-src/com/todoroo/astrid/subtasks/AstridOrderedListUpdater.java +++ b/astrid/plugin-src/com/todoroo/astrid/subtasks/AstridOrderedListUpdater.java @@ -83,7 +83,9 @@ public abstract class AstridOrderedListUpdater { currentIds.add(id); } Set idsInQuery = new HashSet(); - TodorooCursor tasks = taskService.fetchFiltered(filter.getSqlQuery(), null, Task.UUID); + String sql = filter.getSqlQuery().replaceAll("ORDER BY .*", ""); //$NON-NLS-1$//$NON-NLS-2$ + sql = sql + String.format(" ORDER BY %s", Task.CREATION_DATE); //$NON-NLS-1$ + TodorooCursor tasks = taskService.fetchFiltered(sql, null, Task.UUID); try { for (tasks.moveToFirst(); !tasks.isAfterLast(); tasks.moveToNext()) { String id = tasks.getString(0); @@ -93,7 +95,7 @@ public abstract class AstridOrderedListUpdater { changedThings = true; Node newNode = new Node(id, treeRoot, 0); - treeRoot.children.add(newNode); + treeRoot.children.add(0, newNode); idToNode.put(id, newNode); } @@ -331,7 +333,7 @@ public abstract class AstridOrderedListUpdater { return; Node newNode = new Node(uuid, treeRoot, 0); - treeRoot.children.add(newNode); + treeRoot.children.add(0, newNode); idToNode.put(uuid, newNode); writeSerialization(list, serializeTree(), true); applyToFilter(filter);