X-Git-Url: https://git.ktnx.net/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2Faccount_summary%2FAccountSummaryAdapter.java;h=30025c2876d7cded8cd12cd9034718bb086e6eb6;hb=20c03b7a5eb152d42fbbe9ecbaae27530563b398;hp=c0ed9bd7a4f3421c28b7df2bfb86323d03e9f26a;hpb=1935a9be236159b58ba1d1fab44e8ac76783ed7c;p=mobile-ledger.git diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java index c0ed9bd7..30025c28 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryAdapter.java @@ -20,7 +20,6 @@ package net.ktnx.mobileledger.ui.account_summary; import android.content.Context; import android.content.res.Resources; import android.text.TextUtils; -import android.util.Log; import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; @@ -38,20 +37,26 @@ import net.ktnx.mobileledger.R; import net.ktnx.mobileledger.async.DbOpQueue; import net.ktnx.mobileledger.model.LedgerAccount; import net.ktnx.mobileledger.model.MobileLedgerProfile; +import net.ktnx.mobileledger.ui.MainModel; import net.ktnx.mobileledger.ui.activity.MainActivity; +import net.ktnx.mobileledger.utils.Locker; +import net.ktnx.mobileledger.utils.Logger; import org.jetbrains.annotations.NotNull; -import java.util.ArrayList; +import java.util.List; +import java.util.Locale; import static net.ktnx.mobileledger.utils.Logger.debug; public class AccountSummaryAdapter extends RecyclerView.Adapter { public static final int AMOUNT_LIMIT = 3; - private MobileLedgerProfile profile; - private AsyncListDiffer listDiffer; - AccountSummaryAdapter() { + private final AsyncListDiffer listDiffer; + private final MainModel model; + AccountSummaryAdapter(MainModel model) { + this.model = model; + listDiffer = new AsyncListDiffer<>(this, new DiffUtil.ItemCallback() { @Override public boolean areItemsTheSame(@NotNull LedgerAccount oldItem, @@ -61,9 +66,7 @@ public class AccountSummaryAdapter @Override public boolean areContentsTheSame(@NotNull LedgerAccount oldItem, @NotNull LedgerAccount newItem) { - return (oldItem.isExpanded() == newItem.isExpanded()) && - (oldItem.amountsExpanded() == newItem.amountsExpanded() && - TextUtils.equals(oldItem.getAmountsString(), newItem.getAmountsString())); + return oldItem.equals(newItem); } }); } @@ -86,18 +89,19 @@ public class AccountSummaryAdapter return listDiffer.getCurrentList() .size(); } - public void setAccounts(MobileLedgerProfile profile, ArrayList newList) { - this.profile = profile; + public void setAccounts(List newList) { listDiffer.submitList(newList); } class LedgerRowHolder extends RecyclerView.ViewHolder { - TextView tvAccountName, tvAccountAmounts; - ConstraintLayout row; - View expanderContainer; - ImageView expander; - View accountExpanderContainer; + final TextView tvAccountName, tvAccountAmounts; + final ConstraintLayout row; + final View expanderContainer; + final ImageView expander; + final View accountExpanderContainer; + LedgerAccount mAccount; public LedgerRowHolder(@NonNull View itemView) { super(itemView); + row = itemView.findViewById(R.id.account_summary_row); tvAccountName = itemView.findViewById(R.id.account_row_acc_name); tvAccountAmounts = itemView.findViewById(R.id.account_row_acc_amounts); @@ -118,94 +122,67 @@ public class AccountSummaryAdapter expander.setOnClickListener(v -> toggleAccountExpanded()); tvAccountAmounts.setOnClickListener(v -> toggleAmountsExpanded()); } - private @NonNull - LedgerAccount getAccount() { - final ArrayList accountList = profile.getAccounts() - .getValue(); - if (accountList == null) - throw new IllegalStateException("No account list"); - - return accountList.get(getAdapterPosition()); - } private void toggleAccountExpanded() { - LedgerAccount acc = getAccount(); - if (!acc.hasSubAccounts()) + if (!mAccount.hasSubAccounts()) return; debug("accounts", "Account expander clicked"); - acc.toggleExpanded(); - expanderContainer.animate() - .rotation(acc.isExpanded() ? 0 : 180); - - MobileLedgerProfile profile = acc.getProfile(); - if (profile == null) + // make sure we use the same object as the one in the allAccounts list + MobileLedgerProfile profile = mAccount.getProfile(); + if (profile == null) { return; + } + try (Locker ignored = model.lockAccountsForWriting()) { + LedgerAccount realAccount = model.locateAccount(mAccount.getName()); + if (realAccount == null) + return; + + mAccount = realAccount; + mAccount.toggleExpanded(); + } + expanderContainer.animate() + .rotation(mAccount.isExpanded() ? 0 : 180); + model.updateDisplayedAccounts(); DbOpQueue.add("update accounts set expanded=? where name=? and profile=?", - new Object[]{acc.isExpanded(), acc.getName(), profile.getUuid() - }, profile::scheduleAccountListReload); + new Object[]{mAccount.isExpanded(), mAccount.getName(), profile.getUuid() + }); } private void toggleAmountsExpanded() { - LedgerAccount acc = getAccount(); - if (acc.getAmountCount() <= AMOUNT_LIMIT) + if (mAccount.getAmountCount() <= AMOUNT_LIMIT) return; - acc.toggleAmountsExpanded(); - if (acc.amountsExpanded()) { - tvAccountAmounts.setText(acc.getAmountsString()); + mAccount.toggleAmountsExpanded(); + if (mAccount.amountsExpanded()) { + tvAccountAmounts.setText(mAccount.getAmountsString()); accountExpanderContainer.setVisibility(View.GONE); } else { - tvAccountAmounts.setText(acc.getAmountsString(AMOUNT_LIMIT)); + tvAccountAmounts.setText(mAccount.getAmountsString(AMOUNT_LIMIT)); accountExpanderContainer.setVisibility(View.VISIBLE); } - MobileLedgerProfile profile = acc.getProfile(); + MobileLedgerProfile profile = mAccount.getProfile(); if (profile == null) return; DbOpQueue.add("update accounts set amounts_expanded=? where name=? and profile=?", - new Object[]{acc.amountsExpanded(), acc.getName(), profile.getUuid() + new Object[]{mAccount.amountsExpanded(), mAccount.getName(), profile.getUuid() }); } private boolean onItemLongClick(View v) { MainActivity activity = (MainActivity) v.getContext(); AlertDialog.Builder builder = new AlertDialog.Builder(activity); - View row; - int id = v.getId(); - switch (id) { - case R.id.account_summary_row: - row = v; - break; - case R.id.account_row_acc_amounts: - case R.id.account_row_amounts_expander_container: - row = (View) v.getParent(); - break; - case R.id.account_row_acc_name: - case R.id.account_expander_container: - row = (View) v.getParent() - .getParent(); - break; - case R.id.account_expander: - row = (View) v.getParent() - .getParent() - .getParent(); - break; - default: - Log.e("error", - String.format("Don't know how to handle long click on id %d", id)); - return false; - } - LedgerAccount acc = getAccount(); - builder.setTitle(acc.getName()); + final String accountName = mAccount.getName(); + builder.setTitle(accountName); builder.setItems(R.array.acc_ctx_menu, (dialog, which) -> { - switch (which) { - case 0: - // show transactions - activity.showAccountTransactions(acc.getName()); - break; + if (which == 0) {// show transactions + activity.showAccountTransactions(accountName); + } + else { + throw new RuntimeException(String.format("Unknown menu item id (%d)", which)); } dialog.dismiss(); }); @@ -213,8 +190,10 @@ public class AccountSummaryAdapter return true; } public void bindToAccount(LedgerAccount acc) { + Logger.debug("accounts", String.format(Locale.US, "Binding to '%s'", acc.getName())); Context ctx = row.getContext(); Resources rm = ctx.getResources(); + mAccount = acc; row.setTag(acc);