Kiwix-android: Suggestion search fails on cutting-edge Wikispecies custom app

Created on 15 Jan 2021  路  10Comments  路  Source: kiwix/kiwix-android

A few hours ago I have released a new Wikispecies custom app:
https://play.google.com/store/apps/details?id=org.kiwix.kiwixcustomwikispecies

Yesterday I failed because of #2578 but this has been fixed and I change the CI of the Kiwix Android Custom Apps to deal with the develop branch instead of the usual official release branch master. See:
https://github.com/kiwix/kiwix-android-custom/commit/b73f0b5c07b8775d84238b8c217e7b267a83362e

First all seemed first OK and I have put the Wikispecies app in production. Unfortunately I missed an important regression. The problem is that what ever the suggestion I choose in the proposed suggestions, it always load the welcome page. Which make the search/suggestion system useless.

Would be great to identify quickly the problem so I can fix the production release of the Wikispecies custom app (and later updated other app with 3.4.3+ code base).

bug question

All 10 comments

I think this is a flaw in PipedInputStream and PipedOutputStream. Once it closes one of the two, the other one is immediately useless. I would have preferred at least that the PipedInputStream would still be able to read what was written up to the point of closing the PipedOutputStream.

@gouri-panda Thank you for having a look to that. What is really strange is why this impact only the custom app?

@kelson42 I just digged the surface. It closes the inputStrreamReader.If one closes the other one will close while throwing exception. I will definitely check out why this is happening what's the root cause of this problem.

@gouri-panda Hi, do you still plan to work on this ticket?

Yes! @kelson42 Now I'm fully available. Thank you!

Since the commit is reverted now. It is no longer issue now.

@gouri-panda Great news, when was this reverted?

Argh.... Two days after the Wikispecies app release... Will make a new release and check.

Was this page helpful?
0 / 5 - 0 ratings