Skip to content
Snippets Groups Projects
Commit 7c4ee6ef authored by Geoffrey Métais's avatar Geoffrey Métais
Browse files

Release cover on view recycled

parent fe31b5a5
No related branches found
No related tags found
No related merge requests found
......@@ -79,15 +79,22 @@ public class AudioBrowserAdapter extends RecyclerView.Adapter<AudioBrowserAdapte
}
}
@Override
public void onViewRecycled(ViewHolder holder) {
holder.vdb.setVariable(BR.cover, null);
}
@Override
public int getItemCount() {
return mDataList == null ? 0 : mDataList.size();
}
public MediaLibraryItem getItem(int position) {
if (position < 0 || position >= mDataList.size())
return null;
return mDataList.get(position);
return isPositionValid(position) ? mDataList.get(position) : null;
}
private boolean isPositionValid(int position) {
return position >= 0 || position < mDataList.size();
}
public ArrayList<MediaLibraryItem> getAll() {
......@@ -109,16 +116,13 @@ public class AudioBrowserAdapter extends RecyclerView.Adapter<AudioBrowserAdapte
if (i < position)
++offset;
} else
list.add(mDataList.get(i));
list.add(mDataList.get(i));
return position-offset;
}
@Override
public long getItemId(int position) {
if (position < 0 || position >= mDataList.size())
return super.getItemId(position);
else
return mDataList.get(position).getId();
return isPositionValid(position) ? mDataList.get(position).getId() : -1;
}
@Override
......
......@@ -120,6 +120,7 @@ public class VideoListAdapter extends RecyclerView.Adapter<VideoListAdapter.View
@Override
public void onViewRecycled(ViewHolder holder) {
mHolders.remove(holder.getAdapterPosition());
holder.binding.setVariable(BR.cover, null);
}
@MainThread
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment