Assets and market orders fail to display the 2nd time or after phone wake up

Issue #4 resolved
ccleva created an issue

Same for corporation and character assets: click on the asset button, assets display ok, hit back, click again on asset button -> nothing is displayed, no updating happens. Going to the wallet activity restores the correct display (once), going to mail or training queue doesn't (haven't tested all cases).

Comments (14)

  1. ccleva reporter

    Further testing: - Not always reproducible: sometimes the assets are displayed fine on the second access to the activity (typically, 2dn time does not work, 3rd time works, 4th time works, 5th does not, 6th works, etc.). This behavior seems to depend on the number of assets to display: with little to display assets never come back, with lots of assets it works sometimes. - When on the asset activity with assets displayed fine, if phone goes to sleep waking it up leads to no asset displayed. Looks like a problem with onResume() and/or a timer issue/race condition (because of the relation to number of assets)

  2. ccleva reporter

    Same problem with market orders (when going to sleep and accessing a 2dn time) - tested only on characters, no corporation market order available for testing Skills, Mails, Wallet, Contracts and industry are ok when going to sleep or accessing a 2nd time

  3. ccleva reporter

    I found the problem:

    The cached item collection is cleared at some point so it is empty on all further access, correction coming in next pull request

    jeeves could be changed so that cached collections are immutable to avoid similar problems in other fragments

  4. Evanova Android repo owner

    The cached item collection is cleared at some point so it is empty on all further access, correction coming in next pull request

    Excellent! Great news.

    jeeves could be changed so that cached collections are immutable

    Yes, please go ahead with this change.

  5. Log in to comment