]> git.ktnx.net Git - mobile-ledger.git/commit
always work with a clone of the live list when checking transaction
authorDamyan Ivanov <dam+mobileledger@ktnx.net>
Sun, 7 Mar 2021 09:40:43 +0000 (11:40 +0200)
committerDamyan Ivanov <dam+mobileledger@ktnx.net>
Sun, 7 Mar 2021 09:40:43 +0000 (11:40 +0200)
commitfc6385026ff7f7fb74c7e21f6243a1fa90eb352e
tree2f8374660f38d798f08c7b1a13043946f6128ebd
parentb36e2e3152f57528aacf94f31985952cf80406a4
always work with a clone of the live list when checking transaction

shortcuts bite back. the codes makes several lists with items from the
checked list and this may mess with the "get-copy-modify-set" actions
when some item in the working list needs to be changed

instead, make a clone (new list, cloned items; except if the input list
is not the live one, i.e. it is a working copy already) and freely
modify its items directly

if the "always clone" approach gets slow, this whole process may be
moved to a thread

also, note focus has moved when a item above the focused item is removed
app/src/main/java/net/ktnx/mobileledger/ui/new_transaction/NewTransactionModel.java