mirror of https://github.com/tasks/tasks
Refactor to allow tag data to experience the joy of draggable lists
Tim Su
14 years ago
parent
51324c9d2f
commit
e530e5f561
@ -0,0 +1,265 @@
|
|||||||
|
package com.todoroo.astrid.subtasks;
|
||||||
|
|
||||||
|
import java.util.ArrayList;
|
||||||
|
import java.util.Arrays;
|
||||||
|
import java.util.Collections;
|
||||||
|
import java.util.HashMap;
|
||||||
|
import java.util.Map;
|
||||||
|
import java.util.concurrent.atomic.AtomicBoolean;
|
||||||
|
import java.util.concurrent.atomic.AtomicInteger;
|
||||||
|
import java.util.concurrent.atomic.AtomicLong;
|
||||||
|
import java.util.concurrent.atomic.AtomicReference;
|
||||||
|
|
||||||
|
import android.app.Activity;
|
||||||
|
import android.database.Cursor;
|
||||||
|
import android.util.DisplayMetrics;
|
||||||
|
import android.util.TypedValue;
|
||||||
|
import android.view.View;
|
||||||
|
import android.widget.ListView;
|
||||||
|
|
||||||
|
import com.commonsware.cwac.tlv.TouchListView.DropListener;
|
||||||
|
import com.commonsware.cwac.tlv.TouchListView.GrabberClickListener;
|
||||||
|
import com.commonsware.cwac.tlv.TouchListView.SwipeListener;
|
||||||
|
import com.timsu.astrid.R;
|
||||||
|
import com.todoroo.andlib.data.Property;
|
||||||
|
import com.todoroo.andlib.data.TodorooCursor;
|
||||||
|
import com.todoroo.andlib.service.Autowired;
|
||||||
|
import com.todoroo.andlib.service.DependencyInjectionService;
|
||||||
|
import com.todoroo.andlib.utility.DateUtilities;
|
||||||
|
import com.todoroo.astrid.activity.TaskListFragment;
|
||||||
|
import com.todoroo.astrid.adapter.TaskAdapter;
|
||||||
|
import com.todoroo.astrid.adapter.TaskAdapter.OnCompletedTaskListener;
|
||||||
|
import com.todoroo.astrid.api.Filter;
|
||||||
|
import com.todoroo.astrid.data.Metadata;
|
||||||
|
import com.todoroo.astrid.data.Task;
|
||||||
|
import com.todoroo.astrid.service.TaskService;
|
||||||
|
import com.todoroo.astrid.subtasks.OrderedListUpdater.OrderedListIterator;
|
||||||
|
import com.todoroo.astrid.ui.DraggableListView;
|
||||||
|
|
||||||
|
public class SubtasksFragmentHelper {
|
||||||
|
|
||||||
|
private final DisplayMetrics metrics = new DisplayMetrics();
|
||||||
|
private final SubtasksUpdater updater = new SubtasksUpdater();
|
||||||
|
private final TaskListFragment fragment;
|
||||||
|
|
||||||
|
@Autowired TaskService taskService;
|
||||||
|
|
||||||
|
private DraggableTaskAdapter taskAdapter;
|
||||||
|
|
||||||
|
private String list;
|
||||||
|
|
||||||
|
public SubtasksFragmentHelper(TaskListFragment fragment) {
|
||||||
|
DependencyInjectionService.getInstance().inject(this);
|
||||||
|
this.fragment = fragment;
|
||||||
|
}
|
||||||
|
|
||||||
|
// --- ui component setup
|
||||||
|
|
||||||
|
private Activity getActivity() {
|
||||||
|
return fragment.getActivity();
|
||||||
|
}
|
||||||
|
|
||||||
|
private ListView getListView() {
|
||||||
|
return fragment.getListView();
|
||||||
|
}
|
||||||
|
|
||||||
|
private Filter getFilter() {
|
||||||
|
return fragment.getFilter();
|
||||||
|
}
|
||||||
|
|
||||||
|
public DraggableListView getTouchListView() {
|
||||||
|
DraggableListView tlv = (DraggableListView) fragment.getListView();
|
||||||
|
return tlv;
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setUpUiComponents() {
|
||||||
|
TypedValue tv = new TypedValue();
|
||||||
|
getActivity().getTheme().resolveAttribute(R.attr.asThemeTextColor, tv, false);
|
||||||
|
getTouchListView().setDragndropBackgroundColor(tv.data);
|
||||||
|
getTouchListView().setDropListener(dropListener);
|
||||||
|
getTouchListView().setClickListener(rowClickListener);
|
||||||
|
getTouchListView().setSwipeListener(swipeListener);
|
||||||
|
getActivity().getWindowManager().getDefaultDisplay().getMetrics(metrics);
|
||||||
|
}
|
||||||
|
|
||||||
|
public void beforeSetUpTaskList(Filter filter) {
|
||||||
|
updater.applySubtasksToFilter(filter, null);
|
||||||
|
updater.sanitizeTaskList(filter, list);
|
||||||
|
}
|
||||||
|
|
||||||
|
public Property<?>[] taskProperties() {
|
||||||
|
ArrayList<Property<?>> properties = new ArrayList<Property<?>>(Arrays.asList(TaskAdapter.PROPERTIES));
|
||||||
|
properties.add(SubtasksMetadata.INDENT);
|
||||||
|
properties.add(SubtasksMetadata.ORDER);
|
||||||
|
return properties.toArray(new Property<?>[properties.size()]);
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
private final DropListener dropListener = new DropListener() {
|
||||||
|
@Override
|
||||||
|
public void drop(int from, int to) {
|
||||||
|
long targetTaskId = taskAdapter.getItemId(from);
|
||||||
|
long destinationTaskId = taskAdapter.getItemId(to);
|
||||||
|
|
||||||
|
if(to == getListView().getCount() - 1)
|
||||||
|
updater.moveTo(getFilter(), list, targetTaskId, -1);
|
||||||
|
else
|
||||||
|
updater.moveTo(getFilter(), list, targetTaskId, destinationTaskId);
|
||||||
|
|
||||||
|
fragment.loadTaskListContent(true);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
private final SwipeListener swipeListener = new SwipeListener() {
|
||||||
|
@Override
|
||||||
|
public void swipeRight(int which) {
|
||||||
|
long targetTaskId = taskAdapter.getItemId(which);
|
||||||
|
updater.indent(getFilter(), list, targetTaskId, 1);
|
||||||
|
fragment.loadTaskListContent(true);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void swipeLeft(int which) {
|
||||||
|
long targetTaskId = taskAdapter.getItemId(which);
|
||||||
|
updater.indent(getFilter(), list, targetTaskId, -1);
|
||||||
|
fragment.loadTaskListContent(true);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
private final GrabberClickListener rowClickListener = new GrabberClickListener() {
|
||||||
|
@Override
|
||||||
|
public void onLongClick(final View v) {
|
||||||
|
fragment.registerForContextMenu(getListView());
|
||||||
|
getListView().showContextMenuForChild(v);
|
||||||
|
fragment.unregisterForContextMenu(getListView());
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public void onClick(View v) {
|
||||||
|
((DraggableTaskAdapter) taskAdapter).getListener().onClick(v);
|
||||||
|
}
|
||||||
|
};
|
||||||
|
|
||||||
|
public TaskAdapter createTaskAdapter(TodorooCursor<Task> cursor,
|
||||||
|
AtomicReference<String> sqlQueryTemplate) {
|
||||||
|
taskAdapter = new DraggableTaskAdapter(fragment, R.layout.task_adapter_row,
|
||||||
|
cursor, sqlQueryTemplate, false, null);
|
||||||
|
|
||||||
|
taskAdapter.addOnCompletedTaskListener(new OnCompletedTaskListener() {
|
||||||
|
@Override
|
||||||
|
public void onCompletedTask(Task item, boolean newState) {
|
||||||
|
setCompletedForItemAndSubtasks(item, newState);
|
||||||
|
}
|
||||||
|
});
|
||||||
|
|
||||||
|
return taskAdapter;
|
||||||
|
}
|
||||||
|
|
||||||
|
private final class DraggableTaskAdapter extends TaskAdapter {
|
||||||
|
|
||||||
|
private DraggableTaskAdapter(TaskListFragment activity, int resource,
|
||||||
|
Cursor c, AtomicReference<String> query, boolean autoRequery,
|
||||||
|
OnCompletedTaskListener onCompletedTaskListener) {
|
||||||
|
super(activity, resource, c, query, autoRequery,
|
||||||
|
onCompletedTaskListener);
|
||||||
|
|
||||||
|
applyListeners = APPLY_LISTENERS_NONE;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected ViewHolder getTagFromCheckBox(View v) {
|
||||||
|
return (ViewHolder)((View)v.getParent()).getTag();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public synchronized void setFieldContentsAndVisibility(View view) {
|
||||||
|
super.setFieldContentsAndVisibility(view);
|
||||||
|
|
||||||
|
view.getLayoutParams().height = Math.round(45 * metrics.density);
|
||||||
|
ViewHolder vh = (ViewHolder) view.getTag();
|
||||||
|
int indent = vh.task.getValue(SubtasksMetadata.INDENT);
|
||||||
|
vh.rowBody.setPadding(Math.round(indent * 20 * metrics.density), 0, 0, 0);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected void addListeners(View container) {
|
||||||
|
super.addListeners(container);
|
||||||
|
}
|
||||||
|
|
||||||
|
public TaskRowListener getListener() {
|
||||||
|
return listener;
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
private void setCompletedForItemAndSubtasks(final Task item, final boolean completedState) {
|
||||||
|
final Map<Long, ArrayList<Long>> chainedCompletions =
|
||||||
|
Collections.synchronizedMap(new HashMap<Long, ArrayList<Long>>());
|
||||||
|
|
||||||
|
final long itemId = item.getId();
|
||||||
|
|
||||||
|
final Task task = new Task();
|
||||||
|
task.setValue(Task.COMPLETION_DATE, completedState ? DateUtilities.now() : 0);
|
||||||
|
|
||||||
|
if(completedState == false) {
|
||||||
|
ArrayList<Long> chained = chainedCompletions.get(itemId);
|
||||||
|
if(chained != null) {
|
||||||
|
for(Long taskId : chained) {
|
||||||
|
taskAdapter.getCompletedItems().put(taskId, false);
|
||||||
|
task.setId(taskId);
|
||||||
|
taskService.save(task);
|
||||||
|
}
|
||||||
|
taskAdapter.notifyDataSetInvalidated();
|
||||||
|
}
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
new Thread() {
|
||||||
|
@Override
|
||||||
|
public void run() {
|
||||||
|
final AtomicInteger startIndent = new AtomicInteger(
|
||||||
|
item.getValue(SubtasksMetadata.INDENT));
|
||||||
|
final AtomicLong startOrder= new AtomicLong(
|
||||||
|
item.getValue(SubtasksMetadata.ORDER));
|
||||||
|
final AtomicBoolean finished = new AtomicBoolean(false);
|
||||||
|
final ArrayList<Long> chained = new ArrayList<Long>();
|
||||||
|
chainedCompletions.put(itemId, chained);
|
||||||
|
|
||||||
|
updater.iterateThroughList(getFilter(), list, new OrderedListIterator() {
|
||||||
|
@Override
|
||||||
|
public void processTask(long taskId, Metadata metadata) {
|
||||||
|
if(finished.get())
|
||||||
|
return;
|
||||||
|
|
||||||
|
long order = metadata.containsNonNullValue(SubtasksMetadata.ORDER) ?
|
||||||
|
metadata.getValue(SubtasksMetadata.ORDER) : 0;
|
||||||
|
int indent = metadata.containsNonNullValue(SubtasksMetadata.INDENT) ?
|
||||||
|
metadata.getValue(SubtasksMetadata.INDENT) : 0;
|
||||||
|
|
||||||
|
if(order < startOrder.get())
|
||||||
|
return;
|
||||||
|
else if(indent == startIndent.get()) {
|
||||||
|
finished.set(true);
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
|
||||||
|
taskAdapter.getCompletedItems().put(taskId, true);
|
||||||
|
task.setId(taskId);
|
||||||
|
taskService.save(task);
|
||||||
|
chained.add(taskId);
|
||||||
|
}
|
||||||
|
});
|
||||||
|
getActivity().runOnUiThread(new Runnable() {
|
||||||
|
public void run() {
|
||||||
|
taskAdapter.notifyDataSetInvalidated();
|
||||||
|
}
|
||||||
|
});
|
||||||
|
}
|
||||||
|
}.start();
|
||||||
|
}
|
||||||
|
|
||||||
|
public void setList(String list) {
|
||||||
|
this.list = list;
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
@ -0,0 +1,77 @@
|
|||||||
|
package com.todoroo.astrid.subtasks;
|
||||||
|
|
||||||
|
import android.content.Intent;
|
||||||
|
import android.view.View;
|
||||||
|
import android.view.ViewGroup;
|
||||||
|
|
||||||
|
import com.timsu.astrid.R;
|
||||||
|
import com.todoroo.andlib.data.Property;
|
||||||
|
import com.todoroo.andlib.data.TodorooCursor;
|
||||||
|
import com.todoroo.astrid.actfm.TagViewFragment;
|
||||||
|
import com.todoroo.astrid.adapter.TaskAdapter;
|
||||||
|
import com.todoroo.astrid.data.Task;
|
||||||
|
|
||||||
|
public class SubtasksTagListFragment extends TagViewFragment {
|
||||||
|
|
||||||
|
private final SubtasksFragmentHelper helper;
|
||||||
|
|
||||||
|
public SubtasksTagListFragment() {
|
||||||
|
super();
|
||||||
|
helper = new SubtasksFragmentHelper(this);
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected void onNewIntent(Intent intent) {
|
||||||
|
super.onNewIntent(intent);
|
||||||
|
String list = "td:" + tagData.getId(); //$NON-NLS-1$
|
||||||
|
helper.setList(list);
|
||||||
|
}
|
||||||
|
|
||||||
|
/* (non-Javadoc)
|
||||||
|
* @see com.todoroo.astrid.activity.TaskListActivity#getListBody(android.view.ViewGroup)
|
||||||
|
*/
|
||||||
|
@Override
|
||||||
|
protected View getListBody(ViewGroup root) {
|
||||||
|
ViewGroup parent = (ViewGroup) getActivity().getLayoutInflater().inflate(
|
||||||
|
R.layout.task_list_body_tag, root, false);
|
||||||
|
|
||||||
|
taskListView =
|
||||||
|
getActivity().getLayoutInflater().inflate(R.layout.task_list_body_subtasks, root, false);
|
||||||
|
parent.addView(taskListView);
|
||||||
|
|
||||||
|
return parent;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected void setUpUiComponents() {
|
||||||
|
super.setUpUiComponents();
|
||||||
|
|
||||||
|
helper.setUpUiComponents();
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected void setUpTaskList() {
|
||||||
|
helper.beforeSetUpTaskList(filter);
|
||||||
|
|
||||||
|
super.setUpTaskList();
|
||||||
|
|
||||||
|
unregisterForContextMenu(getListView());
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
public Property<?>[] taskProperties() {
|
||||||
|
return helper.taskProperties();
|
||||||
|
}
|
||||||
|
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected boolean isDraggable() {
|
||||||
|
return true;
|
||||||
|
}
|
||||||
|
|
||||||
|
@Override
|
||||||
|
protected TaskAdapter createTaskAdapter(TodorooCursor<Task> cursor) {
|
||||||
|
return helper.createTaskAdapter(cursor, sqlQueryTemplate);
|
||||||
|
}
|
||||||
|
|
||||||
|
}
|
||||||
Loading…
Reference in New Issue