Don't try to get item at NO_POSITION.

This commit is contained in:
akwizgran
2019-03-28 11:06:13 +00:00
parent bc8bb08853
commit 3d94ffb714
2 changed files with 6 additions and 4 deletions

View File

@@ -23,7 +23,7 @@ class ThreadScrollListener<I extends ThreadItem>
private final ThreadListController<?, I> controller;
private final UnreadMessageButton upButton, downButton;
protected ThreadScrollListener(ThreadItemAdapter<I> adapter,
ThreadScrollListener(ThreadItemAdapter<I> adapter,
ThreadListController<?, I> controller,
UnreadMessageButton upButton,
UnreadMessageButton downButton) {
@@ -57,7 +57,7 @@ class ThreadScrollListener<I extends ThreadItem>
private void updateUnreadButtons(int firstVisible, int lastVisible,
int count) {
if (firstVisible == NO_POSITION && lastVisible == NO_POSITION) {
if (firstVisible == NO_POSITION || lastVisible == NO_POSITION) {
setUnreadButtons(0, 0);
return;
}

View File

@@ -46,8 +46,10 @@ public abstract class BriarRecyclerViewScrollListener<A extends ItemReturningAda
@CallSuper
protected void onItemsVisible(int firstVisible, int lastVisible,
int itemCount) {
for (int i = firstVisible; i <= lastVisible; i++) {
onItemVisible(i);
if (firstVisible != NO_POSITION && lastVisible != NO_POSITION) {
for (int i = firstVisible; i <= lastVisible; i++) {
onItemVisible(i);
}
}
}