fix transaction account filter selection

[mobile-ledger.git] / app / src / main / java / net / ktnx / mobileledger / ui / transaction_list / TransactionListFragment.java

diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/transaction_list/TransactionListFragment.java b/app/src/main/java/net/ktnx/mobileledger/ui/transaction_list/TransactionListFragment.java
index 505c4063855eb9719a99f5947827c61b21f7fad7..3625ea40a64f3412adc99b7efb6573846e8d1de9 100644 (file)
--- a/app/src/main/java/net/ktnx/mobileledger/ui/transaction_list/TransactionListFragment.java
+++ b/app/src/main/java/net/ktnx/mobileledger/ui/transaction_list/TransactionListFragment.java
@@ -18,7 +18,7 @@
 package net.ktnx.mobileledger.ui.transaction_list;
 
 import android.content.Context;
-import android.database.MatrixCursor;
+import android.database.Cursor;
 import android.os.Bundle;
 import android.view.LayoutInflater;
 import android.view.Menu;
@@ -28,13 +28,14 @@ import android.view.View;
 import android.view.ViewGroup;
 import android.view.inputmethod.InputMethodManager;
 import android.widget.AutoCompleteTextView;
-import android.widget.Toast;
 
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.recyclerview.widget.LinearLayoutManager;
 import androidx.recyclerview.widget.RecyclerView;
 
+import com.google.android.material.snackbar.Snackbar;
+
 import net.ktnx.mobileledger.R;
 import net.ktnx.mobileledger.model.Data;
 import net.ktnx.mobileledger.ui.MobileLedgerListFragment;
@@ -48,6 +49,8 @@ import org.jetbrains.annotations.NotNull;
 import static android.content.Context.INPUT_METHOD_SERVICE;
 import static net.ktnx.mobileledger.utils.Logger.debug;
 
+// TODO: support transaction-level comment
+
 public class TransactionListFragment extends MobileLedgerListFragment {
     private MenuItem menuTransactionListFilter;
     private View vAccountFilter;
@@ -56,8 +59,6 @@ public class TransactionListFragment extends MobileLedgerListFragment {
     public void onCreate(@Nullable Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
         setHasOptionsMenu(true);
-        Data.backgroundTasksRunning.observe(getViewLifecycleOwner(),
-                this::onBackgroundTaskRunningChanged);
     }
     @Override
     public void onAttach(@NotNull Context context) {
@@ -90,6 +91,9 @@ public class TransactionListFragment extends MobileLedgerListFragment {
         debug("flow", "TransactionListFragment.onActivityCreated called");
         super.onActivityCreated(savedInstanceState);
 
+        Data.backgroundTasksRunning.observe(getViewLifecycleOwner(),
+                this::onBackgroundTaskRunningChanged);
+
         swiper = mActivity.findViewById(R.id.transaction_swipe);
         if (swiper == null)
             throw new RuntimeException("Can't get hold on the swipe layout");
@@ -121,8 +125,8 @@ public class TransactionListFragment extends MobileLedgerListFragment {
         MLDB.hookAutocompletionAdapter(mActivity, accNameFilter, "accounts", "name");
         accNameFilter.setOnItemClickListener((parent, view, position, id) -> {
 //                debug("tmp", "direct onItemClick");
-            MatrixCursor mc = (MatrixCursor) parent.getItemAtPosition(position);
-            Data.accountFilter.setValue(mc.getString(1));
+            Cursor c = (Cursor) parent.getItemAtPosition(position);
+            Data.accountFilter.setValue(c.getString(1));
             Globals.hideSoftKeyboard(mActivity);
         });
 
@@ -135,8 +139,8 @@ public class TransactionListFragment extends MobileLedgerListFragment {
             if (err == null)
                 return;
 
-            Toast.makeText(mActivity, err, Toast.LENGTH_SHORT)
-                 .show();
+            Snackbar.make(this.root, err, Snackbar.LENGTH_LONG)
+                    .show();
             TransactionListViewModel.updateError.set(null);
         });
         Data.transactions.addObserver(
@@ -197,4 +201,4 @@ public class TransactionListFragment extends MobileLedgerListFragment {
             return true;
         });
     }
-}
\ No newline at end of file
+}