mirror of
https://github.com/wismna/DoNext.git
synced 2025-10-03 23:50:13 -04:00
Corrected crash when deleting a task list after changing its name
New tasks are now on top instead of on bottom
This commit is contained in:
@@ -78,6 +78,9 @@ public class TaskListRecyclerViewAdapter extends RecyclerView.Adapter<TaskListRe
|
|||||||
holder.mTaskDeleteButton.setOnClickListener(new View.OnClickListener() {
|
holder.mTaskDeleteButton.setOnClickListener(new View.OnClickListener() {
|
||||||
@Override
|
@Override
|
||||||
public void onClick(View v) {
|
public void onClick(View v) {
|
||||||
|
// Disable the OnFocusChanged listener as it is pointless now
|
||||||
|
holder.mTaskNameView.setOnFocusChangeListener(null);
|
||||||
|
|
||||||
TaskListDataAccess taskListDataAccess = new TaskListDataAccess(mContext);
|
TaskListDataAccess taskListDataAccess = new TaskListDataAccess(mContext);
|
||||||
taskListDataAccess.open();
|
taskListDataAccess.open();
|
||||||
|
|
||||||
|
@@ -120,7 +120,7 @@ public class TaskDataAccess {
|
|||||||
" AND " + DatabaseHelper.TASKS_COLUMN_DONE + " = " + 0 +
|
" AND " + DatabaseHelper.TASKS_COLUMN_DONE + " = " + 0 +
|
||||||
" AND " + DatabaseHelper.TASKS_COLUMN_DELETED + " = " + 0,
|
" AND " + DatabaseHelper.TASKS_COLUMN_DELETED + " = " + 0,
|
||||||
null, null, null,
|
null, null, null,
|
||||||
DatabaseHelper.TASKS_COLUMN_CYCLE + ", " + DatabaseHelper.COLUMN_ID + " ASC");
|
DatabaseHelper.TASKS_COLUMN_CYCLE + ", " + DatabaseHelper.COLUMN_ID + " DESC");
|
||||||
}
|
}
|
||||||
|
|
||||||
public int setDone(long id) {
|
public int setDone(long id) {
|
||||||
|
@@ -90,13 +90,6 @@ public class TaskListsFragment extends Fragment implements TaskListRecyclerViewA
|
|||||||
toggleVisibleCreateNewTaskListLayout(mView);
|
toggleVisibleCreateNewTaskListLayout(mView);
|
||||||
}
|
}
|
||||||
|
|
||||||
@Override
|
|
||||||
public void onPause() {
|
|
||||||
// TODO: persist changes in DB here
|
|
||||||
|
|
||||||
super.onPause();
|
|
||||||
}
|
|
||||||
|
|
||||||
public class GetTaskListsTask extends AsyncTask<TaskListDataAccess, Void, List<TaskList>> {
|
public class GetTaskListsTask extends AsyncTask<TaskListDataAccess, Void, List<TaskList>> {
|
||||||
@Override
|
@Override
|
||||||
protected List<TaskList> doInBackground(TaskListDataAccess... params) {
|
protected List<TaskList> doInBackground(TaskListDataAccess... params) {
|
||||||
|
@@ -295,7 +295,7 @@ public class TasksFragment extends Fragment implements
|
|||||||
|
|
||||||
// Add the task
|
// Add the task
|
||||||
if (task == null)
|
if (task == null)
|
||||||
taskRecyclerViewAdapter.add(newTask, taskRecyclerViewAdapter.getItemCount());
|
taskRecyclerViewAdapter.add(newTask, 0);
|
||||||
// Update the task
|
// Update the task
|
||||||
else {
|
else {
|
||||||
int position = args.getInt("position");
|
int position = args.getInt("position");
|
||||||
|
Reference in New Issue
Block a user