Paging: IndexOutOfBounds

[alashow]

When using epoxy with paging, before calling controller.setList() I have to check if the list is not empty and call it only if not empty.
Otherwise app crashes with the following exception:

java.lang.IndexOutOfBoundsException: Index: 0
        at java.util.Collections$EmptyList.get(Collections.java:4500)
        at android.arch.paging.PagedStorage.getFirstLoadedItem(PagedStorage.java:238)
        at android.arch.paging.ContiguousPagedList.schedulePrepend(ContiguousPagedList.java:189)
        at android.arch.paging.ContiguousPagedList.loadAroundInternal(ContiguousPagedList.java:170)
        at android.arch.paging.PagedList.loadAround(PagedList.java:380)
        at com.airbnb.epoxy.paging.PagingEpoxyController.onModelBound(PagingEpoxyController.java:127)
        at com.airbnb.epoxy.EpoxyControllerAdapter.onModelBound(EpoxyControllerAdapter.java:81)
        at com.airbnb.epoxy.BaseEpoxyAdapter.onBindViewHolder(BaseEpoxyAdapter.java:108)
        at com.airbnb.epoxy.EpoxyControllerAdapter.onBindViewHolder(EpoxyControllerAdapter.java:10)
        at com.airbnb.epoxy.BaseEpoxyAdapter.onBindViewHolder(BaseEpoxyAdapter.java:14)

// crash with IOB
viewModel.liveList.behold(this) { controller.setList(it) }

// works
viewModel.liveList.behold(this) {
    if (it?.isNotEmpty() == true) {
       controller.setList(it)
    }
}

// viewModel.liveList
val liveList by lazy(mode = LazyThreadSafetyMode.NONE) {
        LivePagedListBuilder<Int, Model>(
                dao.dataSourceFactory(),
                PagedList.Config.Builder().run {
                    setPageSize(21)
                    setEnablePlaceholders(false)
                    build()
                }
        ).run {
            build()
        }
}

[elihart]

Thanks for reporting! I'll try to have a fix for this soon

[elihart]

Fixed with #421

Also, since you use this would you mind commenting on the other change bundled with that? I think reading the config off the PagedList will be nice to reduce duplication of needing to specify them again on the EpoxyController.

[alashow]

Actually, I wasn't specifying any configurations on my controller. I assumed it was already reading from PagedList. So that's a good change!

Thanks!