Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Resolved - IndexOutOfBoundsException in Favorite view #11112

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ayushbhanu
Copy link

  • Tests written, or not not needed

@nextcloud-android-bot
Copy link
Collaborator

@nextcloud-android-bot
Copy link
Collaborator

@AlvaroBrey AlvaroBrey removed their assignment Nov 30, 2022
@AlvaroBrey
Copy link
Member

I don't think it's a good idea to just swallow exceptions instead of finding their cause and adressing it. In #10725 you mention this is caused by "a bug in recycler view which is still not resolved", do you have any references for that?

You also say that RecyclerView Data modified in different thread, do you have any idea which part of the Nextcloud code is changing RV data in background? That should be easy to fix if it's located.

Additionally, clearing the RV pool is just not a good idea. The RV will handle that more efficiently.

Can you provide a way to reproduce the crash (without this patch)?

@ayushbhanu
Copy link
Author

I don't think it's a good idea to just swallow exceptions instead of finding their cause and adressing it. In #10725 you mention this is caused by "a bug in recycler view which is still not resolved", do you have any references for that?

You also say that RecyclerView Data modified in different thread, do you have any idea which part of the Nextcloud code is changing RV data in background? That should be easy to fix if it's located.

Additionally, clearing the RV pool is just not a good idea. The RV will handle that more efficiently.

Can you provide a way to reproduce the crash (without this patch)?

For the first questions you can also have a look on below stackoverflow link , its is a popular issue.
link - https://stackoverflow.com/questions/31759171/recyclerview-and-java-lang-indexoutofboundsexception-inconsistency-detected-in?page=1&tab=scoredesc#tab-top

Regarding code for thread i am not sure about it but what I understood is that this crash is caused by RV internal threads and above link helps in catching that.

This crash is occuring sometimes when a user scrolls the screen , there is not an exact scenerio to reproduce it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

IndexOutOfBoundsException in Favorite view having duplicate entries: EmptyRecyclerView
4 participants