]> git.ktnx.net Git - mobile-ledger.git/blobdiff - app/src/main/java/net/ktnx/mobileledger/ui/transaction_list/TransactionListAdapter.java
fix transaction colouring to ignore delimiter items
[mobile-ledger.git] / app / src / main / java / net / ktnx / mobileledger / ui / transaction_list / TransactionListAdapter.java
index 5b1e46f4b5ea3e05e3737c2a37457dca063c9e27..6c245644d55db2742ca11d5ef7796c31a75d8461 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * Copyright © 2018 Damyan Ivanov.
+ * Copyright © 2019 Damyan Ivanov.
  * This file is part of Mobile-Ledger.
  * Mobile-Ledger is free software: you can distribute it and/or modify it
  * under the term of the GNU General Public License as published by
@@ -20,11 +20,10 @@ package net.ktnx.mobileledger.ui.transaction_list;
 import android.content.Context;
 import android.database.sqlite.SQLiteDatabase;
 import android.graphics.Typeface;
+import android.os.AsyncTask;
 import android.support.annotation.NonNull;
-import android.support.constraint.ConstraintLayout;
 import android.support.v7.widget.AppCompatTextView;
 import android.support.v7.widget.RecyclerView;
-import android.util.Log;
 import android.view.Gravity;
 import android.view.LayoutInflater;
 import android.view.View;
@@ -35,103 +34,66 @@ import android.widget.TextView;
 import net.ktnx.mobileledger.R;
 import net.ktnx.mobileledger.model.LedgerTransaction;
 import net.ktnx.mobileledger.model.LedgerTransactionAccount;
+import net.ktnx.mobileledger.model.TransactionListItem;
 import net.ktnx.mobileledger.utils.Globals;
 import net.ktnx.mobileledger.utils.MLDB;
 
+import java.text.DateFormat;
+import java.util.Date;
+
 import static net.ktnx.mobileledger.utils.DimensionUtils.dp2px;
 
-public class TransactionListAdapter
-        extends RecyclerView.Adapter<TransactionListAdapter.TransactionRowHolder> {
-    TransactionListViewModel model;
+public class TransactionListAdapter extends RecyclerView.Adapter<TransactionRowHolder> {
     private String boldAccountName;
-    public TransactionListAdapter(TransactionListViewModel model) {
-        this.model = model;
-    }
     public void onBindViewHolder(@NonNull TransactionRowHolder holder, int position) {
-        LedgerTransaction tr = model.getTransaction(position);
-        // in a race when transaction list is reduced, but the model hasn't been notified yet
+        TransactionListItem item = TransactionListViewModel.getTransactionListItem(position);
+
+        // in a race when transaction value is reduced, but the model hasn't been notified yet
         // the view will disappear when the notifications reaches the model, so by simply omitting
         // the out-of-range get() call nothing bad happens - just a to-be-deleted view remains
         // a bit longer
-        if (tr == null) return;
+        if (item == null) return;
 
-        Context ctx = holder.row.getContext();
+        if (item.getType() == TransactionListItem.Type.TRANSACTION) {
+            holder.vTransaction.setVisibility(View.VISIBLE);
+            holder.vDelimiter.setVisibility(View.GONE);
+            LedgerTransaction tr = item.getTransaction();
 
-        try (SQLiteDatabase db = MLDB.getReadableDatabase()) {
-            tr.loadData(db);
-            holder.tvDescription.setText(tr.getDescription());
-            holder.tvDate.setText(tr.getDate());
+//        Log.d("transactions", String.format("Filling position %d with %d accounts", position,
+//                tr.getAccounts().size()));
 
-            int rowIndex = 0;
-            for (LedgerTransactionAccount acc : tr.getAccounts()) {
-                LinearLayout row = (LinearLayout) holder.tableAccounts.getChildAt(rowIndex++);
-                TextView accName, accAmount;
-                if (row == null) {
-                    row = new LinearLayout(ctx);
-                    row.setLayoutParams(
-                            new LinearLayout.LayoutParams(LinearLayout.LayoutParams.MATCH_PARENT,
-                                    LinearLayout.LayoutParams.WRAP_CONTENT));
-                    row.setGravity(Gravity.CENTER_VERTICAL);
-                    row.setOrientation(LinearLayout.HORIZONTAL);
-                    row.setPaddingRelative(dp2px(ctx, 8), 0, 0, 0);
-                    accName = new AppCompatTextView(ctx);
-                    accName.setLayoutParams(
-                            new LinearLayout.LayoutParams(0, LinearLayout.LayoutParams.WRAP_CONTENT,
-                                    5f));
-                    accName.setTextAlignment(View.TEXT_ALIGNMENT_VIEW_START);
-                    row.addView(accName);
-                    accAmount = new AppCompatTextView(ctx);
-                    LinearLayout.LayoutParams llp =
-                            new LinearLayout.LayoutParams(LinearLayout.LayoutParams.WRAP_CONTENT,
-                                    LinearLayout.LayoutParams.WRAP_CONTENT);
-                    llp.setMarginEnd(0);
-                    accAmount.setLayoutParams(llp);
-                    accAmount.setTextAlignment(View.TEXT_ALIGNMENT_VIEW_END);
-                    accAmount.setMinWidth(dp2px(ctx, 60));
-                    row.addView(accAmount);
-                    holder.tableAccounts.addView(row);
-                }
-                else {
-                    accName = (TextView) row.getChildAt(0);
-                    accAmount = (TextView) row.getChildAt(1);
-                }
-                accName.setText(acc.getAccountName());
-                accAmount.setText(acc.toString());
-
-                if ((boldAccountName != null) && boldAccountName.equals(acc.getAccountName())) {
-                    accName.setTypeface(null, Typeface.BOLD);
-                    accAmount.setTypeface(null, Typeface.BOLD);
-                    accName.setTextColor(Globals.primaryDark);
-                    accAmount.setTextColor(Globals.primaryDark);
-                }
-                else {
-                    accName.setTypeface(null, Typeface.NORMAL);
-                    accAmount.setTypeface(null, Typeface.NORMAL);
-                    accName.setTextColor(Globals.defaultTextColor);
-                    accAmount.setTextColor(Globals.defaultTextColor);
-                }
-
-            }
-            if (holder.tableAccounts.getChildCount() > rowIndex) {
-                holder.tableAccounts
-                        .removeViews(rowIndex, holder.tableAccounts.getChildCount() - rowIndex);
-            }
+            TransactionLoader loader = new TransactionLoader();
+            loader.execute(new TransactionLoaderParams(tr, holder, position, boldAccountName,
+                    item.isOdd()));
 
-            if (position % 2 == 0) {
-                holder.row.setBackgroundColor(Globals.table_row_even_bg);
+            // WORKAROUND what seems to be a bug in CardHolder somewhere
+            // when a view that was previously holding a delimiter is re-purposed
+            // occasionally it stays too short (not high enough)
+            holder.vTransaction.measure(View.MeasureSpec
+                            .makeMeasureSpec(holder.itemView.getWidth(), View.MeasureSpec.EXACTLY),
+                    View.MeasureSpec.makeMeasureSpec(0, View.MeasureSpec.UNSPECIFIED));
+        }
+        else {
+            Date date = item.getDate();
+            holder.vTransaction.setVisibility(View.GONE);
+            holder.vDelimiter.setVisibility(View.VISIBLE);
+            holder.tvDelimiterDate.setText(DateFormat.getDateInstance().format(date));
+            if (item.isMonthShown()) {
+                holder.tvDelimiterMonth.setText(Globals.monthNames[date.getMonth()]);
+                holder.tvDelimiterMonth.setVisibility(View.VISIBLE);
+                holder.vDelimiterLine.setBackgroundResource(R.drawable.dashed_border_8dp);
             }
             else {
-                holder.row.setBackgroundColor(Globals.table_row_odd_bg);
+                holder.tvDelimiterMonth.setVisibility(View.GONE);
+                holder.vDelimiterLine.setBackgroundResource(R.drawable.dashed_border_1dp);
             }
-
-            Log.d("transactions", String.format("Filled position %d", position));
         }
     }
 
     @NonNull
     @Override
     public TransactionRowHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
-        Log.d("perf", "onCreateViewHolder called");
+//        Log.d("perf", "onCreateViewHolder called");
         View row = LayoutInflater.from(parent.getContext())
                 .inflate(R.layout.transaction_list_row, parent, false);
         return new TransactionRowHolder(row);
@@ -139,7 +101,7 @@ public class TransactionListAdapter
 
     @Override
     public int getItemCount() {
-        return model.getTransactionCount();
+        return TransactionListViewModel.getTransactionCount();
     }
     public void setBoldAccountName(String boldAccountName) {
         this.boldAccountName = boldAccountName;
@@ -147,16 +109,128 @@ public class TransactionListAdapter
     public void resetBoldAccountName() {
         this.boldAccountName = null;
     }
-    class TransactionRowHolder extends RecyclerView.ViewHolder {
-        TextView tvDescription, tvDate;
-        LinearLayout tableAccounts;
-        ConstraintLayout row;
-        public TransactionRowHolder(@NonNull View itemView) {
-            super(itemView);
-            this.row = itemView.findViewById(R.id.transaction_row);
-            this.tvDescription = itemView.findViewById(R.id.transaction_row_description);
-            this.tvDate = itemView.findViewById(R.id.transaction_row_date);
-            this.tableAccounts = itemView.findViewById(R.id.transaction_row_acc_amounts);
+
+    enum LoaderStep {HEAD, ACCOUNTS, DONE}
+
+    private static class TransactionLoader
+            extends AsyncTask<TransactionLoaderParams, TransactionLoaderStep, Void> {
+        @Override
+        protected Void doInBackground(TransactionLoaderParams... p) {
+            LedgerTransaction tr = p[0].transaction;
+            boolean odd = p[0].odd;
+
+            SQLiteDatabase db = MLDB.getReadableDatabase();
+            tr.loadData(db);
+
+            publishProgress(new TransactionLoaderStep(p[0].holder, p[0].position, tr, odd));
+
+            int rowIndex = 0;
+            for (LedgerTransactionAccount acc : tr.getAccounts()) {
+//                Log.d(c.getAccountName(), acc.getAmount()));
+                publishProgress(new TransactionLoaderStep(p[0].holder, acc, rowIndex++,
+                        p[0].boldAccountName));
+            }
+
+            publishProgress(new TransactionLoaderStep(p[0].holder, p[0].position, rowIndex));
+
+            return null;
+        }
+        @Override
+        protected void onProgressUpdate(TransactionLoaderStep... values) {
+            super.onProgressUpdate(values);
+            TransactionLoaderStep step = values[0];
+            TransactionRowHolder holder = step.getHolder();
+
+            switch (step.getStep()) {
+                case HEAD:
+                    holder.tvDescription.setText(step.getTransaction().getDescription());
+
+                    if (step.isOdd()) holder.row.setBackgroundColor(Globals.tableRowDarkBG);
+                    else holder.row.setBackgroundColor(Globals.tableRowLightBG);
+
+                    break;
+                case ACCOUNTS:
+                    int rowIndex = step.getAccountPosition();
+                    Context ctx = holder.row.getContext();
+                    LinearLayout row = (LinearLayout) holder.tableAccounts.getChildAt(rowIndex);
+                    TextView accName, accAmount;
+                    if (row == null) {
+                        row = new LinearLayout(ctx);
+                        row.setLayoutParams(new LinearLayout.LayoutParams(
+                                LinearLayout.LayoutParams.MATCH_PARENT,
+                                LinearLayout.LayoutParams.WRAP_CONTENT));
+                        row.setGravity(Gravity.CENTER_VERTICAL);
+                        row.setOrientation(LinearLayout.HORIZONTAL);
+                        row.setPaddingRelative(dp2px(ctx, 8), 0, 0, 0);
+                        accName = new AppCompatTextView(ctx);
+                        accName.setLayoutParams(new LinearLayout.LayoutParams(0,
+                                LinearLayout.LayoutParams.WRAP_CONTENT, 5f));
+                        accName.setTextAlignment(View.TEXT_ALIGNMENT_VIEW_START);
+                        row.addView(accName);
+                        accAmount = new AppCompatTextView(ctx);
+                        LinearLayout.LayoutParams llp = new LinearLayout.LayoutParams(
+                                LinearLayout.LayoutParams.WRAP_CONTENT,
+                                LinearLayout.LayoutParams.WRAP_CONTENT);
+                        llp.setMarginEnd(0);
+                        accAmount.setLayoutParams(llp);
+                        accAmount.setTextAlignment(View.TEXT_ALIGNMENT_VIEW_END);
+                        accAmount.setMinWidth(dp2px(ctx, 60));
+                        row.addView(accAmount);
+                        holder.tableAccounts.addView(row);
+                    }
+                    else {
+                        accName = (TextView) row.getChildAt(0);
+                        accAmount = (TextView) row.getChildAt(1);
+                    }
+                    LedgerTransactionAccount acc = step.getAccount();
+
+                    accName.setText(acc.getAccountName());
+                    accAmount.setText(acc.toString());
+
+//                    Log.d("tmp", String.format("showing acc row %d: %s %1.2f", rowIndex,
+//                            acc.getAccountName(), acc.getAmount()));
+
+                    String boldAccountName = step.getBoldAccountName();
+                    if ((boldAccountName != null) && boldAccountName.equals(acc.getAccountName())) {
+                        accName.setTypeface(null, Typeface.BOLD);
+                        accAmount.setTypeface(null, Typeface.BOLD);
+                        accName.setTextColor(Globals.primaryDark);
+                        accAmount.setTextColor(Globals.primaryDark);
+                    }
+                    else {
+                        accName.setTypeface(null, Typeface.NORMAL);
+                        accAmount.setTypeface(null, Typeface.NORMAL);
+                        accName.setTextColor(Globals.defaultTextColor);
+                        accAmount.setTextColor(Globals.defaultTextColor);
+                    }
+
+                    break;
+                case DONE:
+                    int accCount = step.getAccountCount();
+                    if (holder.tableAccounts.getChildCount() > accCount) {
+                        holder.tableAccounts.removeViews(accCount,
+                                holder.tableAccounts.getChildCount() - accCount);
+                    }
+
+//                    Log.d("transactions",
+//                            String.format("Position %d fill done", step.getPosition()));
+            }
+        }
+    }
+
+    private class TransactionLoaderParams {
+        LedgerTransaction transaction;
+        TransactionRowHolder holder;
+        int position;
+        String boldAccountName;
+        boolean odd;
+        TransactionLoaderParams(LedgerTransaction transaction, TransactionRowHolder holder,
+                                int position, String boldAccountName, boolean odd) {
+            this.transaction = transaction;
+            this.holder = holder;
+            this.position = position;
+            this.boldAccountName = boldAccountName;
+            this.odd = odd;
         }
     }
 }
\ No newline at end of file