View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0006559||Taler||wallet (TS core)||public||2020-09-03 13:43||2021-08-24 16:23|
|Priority||normal||Severity||minor||Reproducibility||have not tried|
|Target Version||0.8||Fixed in Version||0.8|
|Summary||0006559: Transactions not sorted as documented|
|Description||The transactions currently returned by wallet core are not sorted as documented. The Android wallet currently needs to re-sort them.|
|Tags||No tags attached.|
The wallet currently sorts transactions like this:
[ non-pending, ascending by time ]
[ pending, ascending by time ]
For the UI to show them in descending order, it just needs to reverse the list. No re-sorting required.
It's better to return the transactions in ascending order, since "ascending" is usually the default sorting order, and it is absolutely trivial for the UI to reverse the list before rendering.
Of course the wallet-core docs must be adjusted then.
> For the UI to show them in descending order, it just needs to reverse the list. No re-sorting required.
Last time I checked, I needed to do a complete re-sort, as otherwise pending ended up at the bottom.
> It's better to return the transactions in ascending order, since "ascending" is usually the default sorting order
When is it the default sorting order? AFAIK, the wallet API should sort transactions how wallets would show them.
Can you point out an example where the sorting is wrong (not just reversed)? We should have an integration test for that! Though the current integration test already checks the order for transactions. Maybe the pending transactions are sometimes not ordered correctly?
Actually, the order of pending and non-pending transactions was reversed. It was
[p1 p2] [f1 f2]
[f1 f2] [p1 p2]
Now by just calling .reverse, the transactions will be sorted in "UI view order".
|2020-09-03 13:43||grote||New Issue|
|2020-09-03 13:43||grote||Status||new => assigned|
|2020-09-03 13:43||grote||Assigned To||=> Florian Dold|
|2020-09-04 08:24||Florian Dold||Note Added: 0016833|
|2020-09-04 08:25||Florian Dold||Assigned To||Florian Dold => grote|
|2020-09-04 08:25||Florian Dold||Status||assigned => feedback|
|2020-09-04 14:02||grote||Note Added: 0016844|
|2020-09-04 14:19||Florian Dold||Note Added: 0016846|
|2020-09-09 09:40||Florian Dold||Status||feedback => resolved|
|2020-09-09 09:40||Florian Dold||Resolution||open => fixed|
|2020-09-09 09:40||Florian Dold||Note Added: 0016941|
|2020-10-03 14:08||Christian Grothoff||Fixed in Version||=> 0.8|
|2020-10-03 14:10||Christian Grothoff||Target Version||=> 0.8|
|2021-08-24 16:23||Christian Grothoff||Status||resolved => closed|