X-Git-Url: https://git.ktnx.net/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2Ftemplates%2FTemplateDetailsViewModel.java;h=8d0e29e64792eee5548e1072883e3b26dbe7cae1;hb=7d66376e2470788c6b1ad5df18603c814aa389ea;hp=5132af264025056cb95a9ab9fde9271aadeea702;hpb=4b16829ace7160afba02d7f9bba516bd40f1fcbd;p=mobile-ledger.git diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/templates/TemplateDetailsViewModel.java b/app/src/main/java/net/ktnx/mobileledger/ui/templates/TemplateDetailsViewModel.java index 5132af26..8d0e29e6 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/templates/TemplateDetailsViewModel.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/templates/TemplateDetailsViewModel.java @@ -24,6 +24,7 @@ import androidx.lifecycle.MutableLiveData; import androidx.lifecycle.Observer; import androidx.lifecycle.ViewModel; +import net.ktnx.mobileledger.BuildConfig; import net.ktnx.mobileledger.dao.TemplateAccountDAO; import net.ktnx.mobileledger.dao.TemplateHeaderDAO; import net.ktnx.mobileledger.db.DB; @@ -32,6 +33,7 @@ import net.ktnx.mobileledger.db.TemplateHeader; import net.ktnx.mobileledger.db.TemplateWithAccounts; import net.ktnx.mobileledger.model.TemplateDetailsItem; import net.ktnx.mobileledger.utils.Logger; +import net.ktnx.mobileledger.utils.Misc; import java.util.ArrayList; import java.util.Collections; @@ -43,44 +45,105 @@ import java.util.concurrent.atomic.AtomicInteger; public class TemplateDetailsViewModel extends ViewModel { private final MutableLiveData> items = new MutableLiveData<>(Collections.emptyList()); + private final AtomicInteger syntheticItemId = new AtomicInteger(0); private Long mPatternId; - private String mDefaultPatternName; + private String mDefaultTemplateName; private boolean itemsLoaded = false; - private final AtomicInteger syntheticItemId = new AtomicInteger(0); - - public String getDefaultPatternName() { - return mDefaultPatternName; + public String getDefaultTemplateName() { + return mDefaultTemplateName; } - public void setDefaultPatternName(String name) { - mDefaultPatternName = name; + public void setDefaultTemplateName(String name) { + mDefaultTemplateName = name; } public void resetItems() { - checkItemConsistency(new ArrayList<>()); + applyList(new ArrayList<>()); + } + public void applyList(List srcList) { + applyList(srcList, false); } - public void checkItemConsistency(List list) { - if (list == null) - list = new ArrayList<>(items.getValue()); + public void applyList(List srcList, boolean async) { + boolean changes; + if (srcList == null) { + srcList = new ArrayList<>(items.getValue()); + changes = false; + } + else + changes = true; + + srcList = Collections.unmodifiableList(srcList); - boolean changes = false; - if (list.size() < 1) { + { + Logger.debug("tmpl", "Considering old list"); + for (TemplateDetailsItem item : srcList) + Logger.debug("tmpl", String.format(Locale.US, " id %d pos %d", item.getId(), + item.getPosition())); + } + + ArrayList newList = new ArrayList<>(); + + boolean hasEmptyItem = false; + + if (srcList.size() < 1) { final TemplateDetailsItem.Header header = TemplateDetailsItem.createHeader(); - header.setName(mDefaultPatternName); - header.setId(genItemId()); - list.add(header); + header.setId(0); + newList.add(header); + changes = true; + } + else { + newList.add(srcList.get(0)); + } + + for (int i = 1; i < srcList.size(); i++) { + final TemplateDetailsItem.AccountRow accRow = srcList.get(i) + .asAccountRowItem(); + if (accRow.isEmpty()) { + // it is normal to have two empty rows if they are at the + // top (position 1 and 2) + if (!hasEmptyItem || i < 3) { + accRow.setPosition(newList.size()); + newList.add(accRow); + } + else + changes = true; // row skipped + + hasEmptyItem = true; + } + else { + accRow.setPosition(newList.size()); + newList.add(accRow); + } + } + + while (newList.size() < 3) { + final TemplateDetailsItem.AccountRow accountRow = + TemplateDetailsItem.createAccountRow(); + accountRow.setId(genItemId()); + accountRow.setPosition(newList.size()); + newList.add(accountRow); changes = true; + hasEmptyItem = true; } - while (list.size() < 3) { + if (!hasEmptyItem) { final TemplateDetailsItem.AccountRow accountRow = TemplateDetailsItem.createAccountRow(); accountRow.setId(genItemId()); - list.add(accountRow); + accountRow.setPosition(newList.size()); + newList.add(accountRow); changes = true; } - if (changes) - items.setValue(list); + if (changes) { + Logger.debug("tmpl", "Changes detected, applying new list"); + + if (async) + items.postValue(newList); + else + items.setValue(newList); + } + else + Logger.debug("tmpl", "No changes, ignoring new list"); } public int genItemId() { return syntheticItemId.decrementAndGet(); @@ -109,15 +172,18 @@ public class TemplateDetailsViewModel extends ViewModel { ArrayList l = new ArrayList<>(); TemplateDetailsItem header = TemplateDetailsItem.fromRoomObject(src.header); + Logger.debug("tmpl-db", "Got header template item with id of " + header.getId()); l.add(header); + Collections.sort(src.accounts, + (o1, o2) -> Long.compare(o1.getPosition(), o2.getPosition())); for (TemplateAccount acc : src.accounts) { l.add(TemplateDetailsItem.fromRoomObject(acc)); } for (TemplateDetailsItem i : l) { - Logger.debug("patterns-db", "Loaded pattern item " + i); + Logger.debug("tmpl-db", "Loaded pattern item " + i); } - items.postValue(l); + applyList(l, true); itemsLoaded = true; dbList.removeObserver(this); @@ -145,6 +211,9 @@ public class TemplateDetailsViewModel extends ViewModel { TemplateDetailsItem.Header modelHeader = list.get(0) .asHeaderItem(); + modelHeader.setName(Misc.trim(modelHeader.getName())); + if (modelHeader.getName().isEmpty()) + modelHeader.setName(getDefaultTemplateName()); TemplateHeaderDAO headerDAO = DB.get() .getTemplateDAO(); TemplateHeader dbHeader = modelHeader.toDBO(); @@ -184,16 +253,78 @@ public class TemplateDetailsViewModel extends ViewModel { taDAO.finishSave(mPatternId); }); } + private ArrayList copyItems() { + List oldList = items.getValue(); + ArrayList result = new ArrayList<>(oldList.size()); + + for (TemplateDetailsItem item : oldList) { + if (item instanceof TemplateDetailsItem.Header) + result.add(new TemplateDetailsItem.Header(item.asHeaderItem())); + else if (item instanceof TemplateDetailsItem.AccountRow) + result.add(new TemplateDetailsItem.AccountRow(item.asAccountRowItem())); + else + throw new RuntimeException("Unexpected item " + item); + } + + return result; + } public void moveItem(int sourcePos, int targetPos) { - ArrayList newList = new ArrayList<>(items.getValue()); - TemplateDetailsItem item = newList.remove(sourcePos); - newList.add(targetPos, item); + final List newList = copyItems(); + + if (BuildConfig.DEBUG) { + Logger.debug("drag", "Before move:"); + for (int i = 1; i < newList.size(); i++) { + final TemplateDetailsItem item = newList.get(i); + Logger.debug("drag", + String.format(Locale.US, " %d: id %d, pos %d", i, item.getId(), + item.getPosition())); + } + } + + { + TemplateDetailsItem item = newList.remove(sourcePos); + newList.add(targetPos, item); + } + + // adjust affected items' positions + { + int startPos, endPos; + if (sourcePos < targetPos) { + // moved down + startPos = sourcePos; + endPos = targetPos; + } + else { + // moved up + startPos = targetPos; + endPos = sourcePos; + } + + for (int i = startPos; i <= endPos; i++) { + newList.get(i) + .setPosition(i); + } + } + + if (BuildConfig.DEBUG) { + Logger.debug("drag", "After move:"); + for (int i = 1; i < newList.size(); i++) { + final TemplateDetailsItem item = newList.get(i); + Logger.debug("drag", + String.format(Locale.US, " %d: id %d, pos %d", i, item.getId(), + item.getPosition())); + } + } + items.setValue(newList); } public void removeItem(int position) { - ArrayList newList = new ArrayList<>(items.getValue()); + Logger.debug("tmpl", "Removing item at position " + position); + ArrayList newList = copyItems(); newList.remove(position); - checkItemConsistency(newList); - items.setValue(newList); + for (int i = position; i < newList.size(); i++) + newList.get(i) + .setPosition(i); + applyList(newList); } } \ No newline at end of file