Fixes #586 : Remove content of AccountsActivity.AccountViewPagerAdapt… #875
Conversation
…PagerAdapter.destroyItem() to avoid app crash when opening another book after switching to Recent or Favorites Accounts Tab
|
@JeanGarf Can you make a build with your improvements? This project looks dead. |
Yes I can. Where can I do that ? |
|
You know what, I'll set up an f-droid repo for your version using github actions. I need a couple of other forked apps built anyway which nobody is managing automated builds for. |
|
I've set up a repo here (https://madushan1000.github.io/fdroid-repo/repo/), I can configure it to do automated builds for each commit to your repo if you can configure a webhook. |
|
Hi madushan1000, Sorry, I didn't saw your post and therefore, I added a page on my web site to allow downloading of the apk with all my contributions (my 10 Pull Requests. See #882 ) If you're interested to test or use it, tell me if it works fine. See you |
The problem appeared when switching to Recent or Favorites Account Tab and then opening another Book.
The problem did not appear when switching to "All" tab, then opening another book, even having open Recent or Favorites tab before "All" tab.
The Exception was thrown in DatabaseAdapter.getID() by the line :
throw new IllegalArgumentException(mTableName + " with GUID " + uid + " does not exist in the db");
because the uid has been built from cursor given in AccountRecyclerAdapter.AccountsListFragment.onBindViewHolderCursor(AccountViewHolder, Cursor), which is a cursor on the previous Book and this uid does not exists in the newly opened Book.
I fixed the issue in (7b6ebb3) by removing the content of AccountsActivity.AccountViewPagerAdapter.destroyItem().
I don't know why this fixed the problem, but I found it by discovering that destroyItem() was not called when switching to "All" Tab, but was when switching to Recent or Favorites Tab.