Fix for 13186759, note-ordering and pref-setting show all notes. couldnt reproduce disappearing notes.

pull/14/head
Arne Jans 14 years ago committed by Tim Su
parent dc7f2f5d5e
commit 74203d1c2c

@ -15,6 +15,7 @@ import android.widget.TextView;
import com.timsu.astrid.R; import com.timsu.astrid.R;
import com.todoroo.andlib.data.TodorooCursor; import com.todoroo.andlib.data.TodorooCursor;
import com.todoroo.andlib.sql.Order;
import com.todoroo.andlib.sql.Query; import com.todoroo.andlib.sql.Query;
import com.todoroo.astrid.core.PluginServices; import com.todoroo.astrid.core.PluginServices;
import com.todoroo.astrid.dao.MetadataDao.MetadataCriteria; import com.todoroo.astrid.dao.MetadataDao.MetadataCriteria;
@ -56,7 +57,7 @@ public class NoteViewingActivity extends Activity {
TodorooCursor<Metadata> cursor = PluginServices.getMetadataService().query( TodorooCursor<Metadata> cursor = PluginServices.getMetadataService().query(
Query.select(Metadata.PROPERTIES).where( Query.select(Metadata.PROPERTIES).where(
MetadataCriteria.byTaskAndwithKey(task.getId(), MetadataCriteria.byTaskAndwithKey(task.getId(),
NoteMetadata.METADATA_KEY))); NoteMetadata.METADATA_KEY)).orderBy(Order.desc(Metadata.CREATION_DATE)));
Metadata metadata = new Metadata(); Metadata metadata = new Metadata();
try { try {
for(cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) { for(cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) {

@ -11,6 +11,7 @@ import android.text.TextUtils;
import com.timsu.astrid.R; import com.timsu.astrid.R;
import com.todoroo.andlib.data.TodorooCursor; import com.todoroo.andlib.data.TodorooCursor;
import com.todoroo.andlib.sql.Order;
import com.todoroo.andlib.sql.Query; import com.todoroo.andlib.sql.Query;
import com.todoroo.andlib.utility.Preferences; import com.todoroo.andlib.utility.Preferences;
import com.todoroo.astrid.api.AstridApiConstants; import com.todoroo.astrid.api.AstridApiConstants;
@ -51,7 +52,7 @@ public class NotesDetailExposer extends BroadcastReceiver {
if(!Preferences.getBoolean(R.string.p_showNotes, false)) if(!Preferences.getBoolean(R.string.p_showNotes, false))
return null; return null;
Task task = PluginServices.getTaskService().fetchById(id, Task.NOTES); Task task = PluginServices.getTaskService().fetchById(id, Task.ID, Task.NOTES);
if(task == null) if(task == null)
return null; return null;
@ -64,7 +65,7 @@ public class NotesDetailExposer extends BroadcastReceiver {
TodorooCursor<Metadata> cursor = PluginServices.getMetadataService().query( TodorooCursor<Metadata> cursor = PluginServices.getMetadataService().query(
Query.select(Metadata.PROPERTIES).where( Query.select(Metadata.PROPERTIES).where(
MetadataCriteria.byTaskAndwithKey(task.getId(), MetadataCriteria.byTaskAndwithKey(task.getId(),
NoteMetadata.METADATA_KEY))); NoteMetadata.METADATA_KEY)).orderBy(Order.asc(Metadata.CREATION_DATE)));
Metadata metadata = new Metadata(); Metadata metadata = new Metadata();
try { try {
for(cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) { for(cursor.moveToFirst(); !cursor.isAfterLast(); cursor.moveToNext()) {

Loading…
Cancel
Save