From 6cd3f712d88c04381308992e5d305d4f12f2a173 Mon Sep 17 00:00:00 2001
From: Bradlee Speice <bspeice.nc@gmail.com>
Date: Sat, 10 May 2014 19:19:23 -0400
Subject: [PATCH] Sort downloadable books

---
 .../activities/downloader/BookListFragment.java    | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
index fe3bcd0..6b11fd5 100644
--- a/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
+++ b/MinimalBible/src/org/bspeice/minimalbible/activities/downloader/BookListFragment.java
@@ -23,9 +23,12 @@ import org.bspeice.minimalbible.activities.downloader.manager.DownloadManager;
 import org.bspeice.minimalbible.activities.downloader.manager.EventBookList;
 import org.crosswire.jsword.book.Book;
 import org.crosswire.jsword.book.BookCategory;
+import org.crosswire.jsword.book.BookComparators;
 import org.crosswire.jsword.book.BookFilter;
 import org.crosswire.jsword.book.FilterUtil;
 
+import java.util.Collection;
+import java.util.Collections;
 import java.util.List;
 
 import javax.inject.Inject;
@@ -69,8 +72,6 @@ public class BookListFragment extends Fragment {
     @Override
     public void onCreate(Bundle state) {
         super.onCreate(state);
-        //TODO: Figure out why this doesn't work. Best guess is because the context from
-        //getApplication(getActivity()) isn't actually MinimalBible.getAppContext()
         MinimalBible.getApplication().inject(this);
     }
 
@@ -141,10 +142,15 @@ public class BookListFragment extends Fragment {
     public void displayBooks(List<Book> bookList) {
         try {
             // TODO: Should the filter be applied earlier in the process?
+            // TODO: Sort books by name?
+            List<Book> displayList;
+
             BookCategory c = BookCategory.fromString(getArguments().getString(ARG_BOOK_CATEGORY));
             BookFilter f = FilterUtil.filterFromCategory(c);
-            List<Book> filteredBooks = FilterUtil.applyFilter(bookList, f);
-            downloadsAvailable.setAdapter(new BookListAdapter(this.getActivity(), filteredBooks));
+            displayList = FilterUtil.applyFilter(bookList, f);
+            Collections.sort(displayList, BookComparators.getInitialComparator());
+
+            downloadsAvailable.setAdapter(new BookListAdapter(this.getActivity(), displayList));
             setInsets(getActivity(), downloadsAvailable);
         } catch (FilterUtil.InvalidFilterCategoryMappingException e) {
             // To be honest, there should be no reason you end up here.