From 8266170fd5e31eaa0a055649ba9bb29c00c25cca Mon Sep 17 00:00:00 2001
From: geoffroy <geogeob@hotmail.com>
Date: Tue, 5 Jan 2016 10:42:01 -0500
Subject: [PATCH] Corrected crash when deleting a task list after changing its
 name New tasks are now on top instead of on bottom

---
 .../donext/adapters/TaskListRecyclerViewAdapter.java       | 3 +++
 .../wismna/geoffroy/donext/database/TaskDataAccess.java    | 2 +-
 .../geoffroy/donext/fragments/TaskListsFragment.java       | 7 -------
 .../wismna/geoffroy/donext/fragments/TasksFragment.java    | 2 +-
 4 files changed, 5 insertions(+), 9 deletions(-)

diff --git a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/adapters/TaskListRecyclerViewAdapter.java b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/adapters/TaskListRecyclerViewAdapter.java
index 8fa13f4..df6c172 100644
--- a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/adapters/TaskListRecyclerViewAdapter.java
+++ b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/adapters/TaskListRecyclerViewAdapter.java
@@ -78,6 +78,9 @@ public class TaskListRecyclerViewAdapter extends RecyclerView.Adapter<TaskListRe
         holder.mTaskDeleteButton.setOnClickListener(new View.OnClickListener() {
             @Override
             public void onClick(View v) {
+                // Disable the OnFocusChanged listener as it is pointless now
+                holder.mTaskNameView.setOnFocusChangeListener(null);
+
                 TaskListDataAccess taskListDataAccess = new TaskListDataAccess(mContext);
                 taskListDataAccess.open();
 
diff --git a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/database/TaskDataAccess.java b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/database/TaskDataAccess.java
index 24d4942..b63b2a4 100644
--- a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/database/TaskDataAccess.java
+++ b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/database/TaskDataAccess.java
@@ -120,7 +120,7 @@ public class TaskDataAccess {
                     " AND " + DatabaseHelper.TASKS_COLUMN_DONE + " = " + 0 +
                     " AND " + DatabaseHelper.TASKS_COLUMN_DELETED + " = " + 0,
                 null, null, null,
-                DatabaseHelper.TASKS_COLUMN_CYCLE + ", " + DatabaseHelper.COLUMN_ID + " ASC");
+                DatabaseHelper.TASKS_COLUMN_CYCLE + ", " + DatabaseHelper.COLUMN_ID + " DESC");
     }
 
     public int setDone(long id) {
diff --git a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TaskListsFragment.java b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TaskListsFragment.java
index d5c1a2c..55992f2 100644
--- a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TaskListsFragment.java
+++ b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TaskListsFragment.java
@@ -90,13 +90,6 @@ public class TaskListsFragment extends Fragment implements TaskListRecyclerViewA
         toggleVisibleCreateNewTaskListLayout(mView);
     }
 
-    @Override
-    public void onPause() {
-        // TODO: persist changes in DB here
-
-        super.onPause();
-    }
-
     public class GetTaskListsTask extends AsyncTask<TaskListDataAccess, Void, List<TaskList>> {
         @Override
         protected List<TaskList> doInBackground(TaskListDataAccess... params) {
diff --git a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TasksFragment.java b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TasksFragment.java
index 51e0f3b..a5f9625 100644
--- a/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TasksFragment.java
+++ b/DoNExt/app/src/main/java/com/wismna/geoffroy/donext/fragments/TasksFragment.java
@@ -295,7 +295,7 @@ public class TasksFragment extends Fragment implements
 
         // Add the task
         if (task == null)
-            taskRecyclerViewAdapter.add(newTask, taskRecyclerViewAdapter.getItemCount());
+            taskRecyclerViewAdapter.add(newTask, 0);
             // Update the task
         else {
             int position = args.getInt("position");