Rename ViewHolder to TaskViewHolder

pull/996/head
Alex Baker 6 years ago
parent 0f55cacf10
commit 681d3aab64

@ -78,10 +78,10 @@ class DragAndDropRecyclerAdapter(
super.onSelectedChanged(viewHolder, actionState) super.onSelectedChanged(viewHolder, actionState)
if (actionState == ItemTouchHelper.ACTION_STATE_DRAG) { if (actionState == ItemTouchHelper.ACTION_STATE_DRAG) {
taskList.startActionMode() taskList.startActionMode()
(viewHolder as ViewHolder?)!!.isMoving = true (viewHolder as TaskViewHolder?)!!.isMoving = true
dragging = true dragging = true
val position = viewHolder!!.adapterPosition val position = viewHolder!!.adapterPosition
updateIndents(viewHolder as ViewHolder?, position, position) updateIndents(viewHolder as TaskViewHolder?, position, position)
} }
} }
@ -98,8 +98,8 @@ class DragAndDropRecyclerAdapter(
taskList.finishActionMode() taskList.finishActionMode()
val fromPosition = src.adapterPosition val fromPosition = src.adapterPosition
val toPosition = target.adapterPosition val toPosition = target.adapterPosition
val source = src as ViewHolder val source = src as TaskViewHolder
if (!adapter.canMove(source.task, fromPosition, (target as ViewHolder).task, toPosition)) { if (!adapter.canMove(source.task, fromPosition, (target as TaskViewHolder).task, toPosition)) {
return false return false
} }
if (from == -1) { if (from == -1) {
@ -112,7 +112,7 @@ class DragAndDropRecyclerAdapter(
return true return true
} }
private fun updateIndents(source: ViewHolder?, from: Int, to: Int) { private fun updateIndents(source: TaskViewHolder?, from: Int, to: Int) {
val task = source!!.task val task = source!!.task
source.minIndent = if (to == 0 || to == itemCount - 1) 0 else adapter.minIndent(if (from <= to) to + 1 else to, task) source.minIndent = if (to == 0 || to == itemCount - 1) 0 else adapter.minIndent(if (from <= to) to + 1 else to, task)
source.maxIndent = if (to == 0) 0 else adapter.maxIndent(if (from >= to) to - 1 else to, task) source.maxIndent = if (to == 0) 0 else adapter.maxIndent(if (from >= to) to - 1 else to, task)
@ -127,7 +127,7 @@ class DragAndDropRecyclerAdapter(
actionState: Int, actionState: Int,
isCurrentlyActive: Boolean) { isCurrentlyActive: Boolean) {
var dX = dXOrig var dX = dXOrig
val vh = viewHolder as ViewHolder val vh = viewHolder as TaskViewHolder
val task = vh.task val task = vh.task
val shiftSize = vh.shiftSize val shiftSize = vh.shiftSize
if (actionState == ItemTouchHelper.ACTION_STATE_DRAG) { if (actionState == ItemTouchHelper.ACTION_STATE_DRAG) {
@ -159,7 +159,7 @@ class DragAndDropRecyclerAdapter(
override fun clearView( override fun clearView(
recyclerView: RecyclerView, viewHolder: RecyclerView.ViewHolder) { recyclerView: RecyclerView, viewHolder: RecyclerView.ViewHolder) {
super.clearView(recyclerView, viewHolder) super.clearView(recyclerView, viewHolder)
val vh = viewHolder as ViewHolder val vh = viewHolder as TaskViewHolder
vh.isMoving = false vh.isMoving = false
dragging = false dragging = false
drainQueue() drainQueue()

@ -10,19 +10,19 @@ import com.todoroo.astrid.api.Filter
import com.todoroo.astrid.dao.TaskDao import com.todoroo.astrid.dao.TaskDao
import org.tasks.data.TaskContainer import org.tasks.data.TaskContainer
import org.tasks.intents.TaskIntents import org.tasks.intents.TaskIntents
import org.tasks.tasklist.ViewHolder.ViewHolderCallbacks import org.tasks.tasklist.TaskViewHolder.ViewHolderCallbacks
abstract class TaskListRecyclerAdapter internal constructor( abstract class TaskListRecyclerAdapter internal constructor(
private val adapter: TaskAdapter, private val adapter: TaskAdapter,
private val viewHolderFactory: ViewHolderFactory, private val viewHolderFactory: ViewHolderFactory,
private val taskList: TaskListFragment, private val taskList: TaskListFragment,
private val taskDao: TaskDao) : RecyclerView.Adapter<ViewHolder>(), ViewHolderCallbacks, ListUpdateCallback, TaskAdapterDataSource { private val taskDao: TaskDao) : RecyclerView.Adapter<TaskViewHolder>(), ViewHolderCallbacks, ListUpdateCallback, TaskAdapterDataSource {
override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder { override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): TaskViewHolder {
return viewHolderFactory.newViewHolder(parent, this) return viewHolderFactory.newViewHolder(parent, this)
} }
override fun onBindViewHolder(holder: ViewHolder, position: Int) { override fun onBindViewHolder(holder: TaskViewHolder, position: Int) {
val task = getItem(position) val task = getItem(position)
if (task != null) { if (task != null) {
holder.bindView(task, taskList.getFilter()) holder.bindView(task, taskList.getFilter())
@ -39,11 +39,11 @@ abstract class TaskListRecyclerAdapter internal constructor(
taskList.loadTaskListContent() taskList.loadTaskListContent()
} }
override fun onClick(viewHolder: ViewHolder) { override fun onClick(taskViewHolder: TaskViewHolder) {
if (taskList.isActionModeActive) { if (taskList.isActionModeActive) {
toggle(viewHolder) toggle(taskViewHolder)
} else { } else {
taskList.onTaskListItemClicked(viewHolder.task.getTask()) taskList.onTaskListItemClicked(taskViewHolder.task.getTask())
} }
} }
@ -54,12 +54,12 @@ abstract class TaskListRecyclerAdapter internal constructor(
} }
} }
override fun onLongPress(viewHolder: ViewHolder): Boolean { override fun onLongPress(taskViewHolder: TaskViewHolder): Boolean {
if (!dragAndDropEnabled()) { if (!dragAndDropEnabled()) {
taskList.startActionMode() taskList.startActionMode()
} }
if (taskList.isActionModeActive && !viewHolder.isMoving) { if (taskList.isActionModeActive && !taskViewHolder.isMoving) {
toggle(viewHolder) toggle(taskViewHolder)
} }
return true return true
} }
@ -73,9 +73,9 @@ abstract class TaskListRecyclerAdapter internal constructor(
taskList.broadcastRefresh() taskList.broadcastRefresh()
} }
fun toggle(viewHolder: ViewHolder) { fun toggle(taskViewHolder: TaskViewHolder) {
adapter.toggleSelection(viewHolder.task) adapter.toggleSelection(taskViewHolder.task)
notifyItemChanged(viewHolder.adapterPosition) notifyItemChanged(taskViewHolder.adapterPosition)
if (adapter.getSelected().isEmpty()) { if (adapter.getSelected().isEmpty()) {
taskList.finishActionMode() taskList.finishActionMode()
} else { } else {

@ -30,7 +30,7 @@ import org.tasks.ui.CheckBoxProvider;
import org.tasks.ui.ChipProvider; import org.tasks.ui.ChipProvider;
import org.threeten.bp.format.FormatStyle; import org.threeten.bp.format.FormatStyle;
public class ViewHolder extends RecyclerView.ViewHolder { public class TaskViewHolder extends RecyclerView.ViewHolder {
private final Activity context; private final Activity context;
private final Preferences preferences; private final Preferences preferences;
@ -78,7 +78,7 @@ public class ViewHolder extends RecyclerView.ViewHolder {
private int minIndent; private int minIndent;
private int maxIndent; private int maxIndent;
ViewHolder( TaskViewHolder(
Activity context, Activity context,
ViewGroup view, ViewGroup view,
Preferences preferences, Preferences preferences,
@ -336,13 +336,13 @@ public class ViewHolder extends RecyclerView.ViewHolder {
void onCompletedTask(TaskContainer task, boolean newState); void onCompletedTask(TaskContainer task, boolean newState);
void onClick(ViewHolder viewHolder); void onClick(TaskViewHolder taskViewHolder);
void onClick(Filter filter); void onClick(Filter filter);
void toggleSubtasks(TaskContainer task, boolean collapsed); void toggleSubtasks(TaskContainer task, boolean collapsed);
boolean onLongPress(ViewHolder viewHolder); boolean onLongPress(TaskViewHolder taskViewHolder);
void onChangeDueDate(TaskContainer task); void onChangeDueDate(TaskContainer task);
} }

@ -57,8 +57,8 @@ public class ViewHolderFactory {
rowPadding = convertDpToPixels(metrics, preferences.getInt(R.string.p_rowPadding, 16)); rowPadding = convertDpToPixels(metrics, preferences.getInt(R.string.p_rowPadding, 16));
} }
ViewHolder newViewHolder(ViewGroup parent, ViewHolder.ViewHolderCallbacks callbacks) { TaskViewHolder newViewHolder(ViewGroup parent, TaskViewHolder.ViewHolderCallbacks callbacks) {
return new ViewHolder( return new TaskViewHolder(
(Activity) context, (Activity) context,
(ViewGroup) (ViewGroup)
LayoutInflater.from(context).inflate(R.layout.task_adapter_row, parent, false), LayoutInflater.from(context).inflate(R.layout.task_adapter_row, parent, false),

Loading…
Cancel
Save