Fixed order of playback history. closes issue #325

2 files changed, 6 insertions, 4 deletions

diff --git a/src/de/danoeh/antennapod/storage/DBReader.java b/src/de/danoeh/antennapod/storage/DBReader.java
index 8aa93d7ed..d950747e6 100644
--- a/src/de/danoeh/antennapod/storage/DBReader.java
+++ b/src/de/danoeh/antennapod/storage/DBReader.java
@@ -15,6 +15,7 @@ import de.danoeh.antennapod.service.download.*;
 import de.danoeh.antennapod.util.DownloadError;
 import de.danoeh.antennapod.util.comparator.DownloadStatusComparator;
 import de.danoeh.antennapod.util.comparator.FeedItemPubdateComparator;
+import de.danoeh.antennapod.util.comparator.PlaybackCompletionDateComparator;
 
 /**
  * Provides methods for reading data from the AntennaPod database.
@@ -515,8 +516,9 @@ public final class DBReader {
         List<FeedItem> items = extractItemlistFromCursor(adapter, itemCursor);
         loadFeedDataOfFeedItemlist(context, items);
         itemCursor.close();
-
         adapter.close();
+
+        Collections.sort(items, new PlaybackCompletionDateComparator());
         return items;
     }
 
diff --git a/src/de/danoeh/antennapod/storage/PodDBAdapter.java b/src/de/danoeh/antennapod/storage/PodDBAdapter.java
index 068f80ded..2da835511 100644
--- a/src/de/danoeh/antennapod/storage/PodDBAdapter.java
+++ b/src/de/danoeh/antennapod/storage/PodDBAdapter.java
@@ -849,7 +849,7 @@ public class PodDBAdapter {
 
     /**
      * Returns a cursor which contains feed media objects with a playback
-     * completion date in descending order.
+     * completion date in ascending order.
      *
      * @param limit The maximum row count of the returned cursor. Must be an
      *              integer >= 0.
@@ -860,8 +860,8 @@ public class PodDBAdapter {
             throw new IllegalArgumentException("Limit must be >= 0");
         }
         Cursor c = db.query(TABLE_NAME_FEED_MEDIA, null,
-                KEY_PLAYBACK_COMPLETION_DATE + " > 0", null, null,
-                null, KEY_PLAYBACK_COMPLETION_DATE + " DESC LIMIT " + limit);
+                KEY_PLAYBACK_COMPLETION_DATE + " > 0 LIMIT " + limit, null, null,
+                null, null);
         return c;
     }