X-Git-Url: https://git.ktnx.net/?p=mobile-ledger.git;a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2Factivity%2FNewTransactionFragment.java;h=5cbbab71f5cb86c71ec23073437d2723e191972d;hp=5b197b0024adfc9a6c9805bccc3c0373cea48f30;hb=bd79894e6d9c996cebd017d7d5e226aae9568821;hpb=b78d85afef509ea029aefe10e179f87a60108218 diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionFragment.java b/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionFragment.java index 5b197b00..5cbbab71 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionFragment.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/activity/NewTransactionFragment.java @@ -17,7 +17,6 @@ package net.ktnx.mobileledger.ui.activity; -import android.app.Activity; import android.content.Context; import android.os.Bundle; import android.renderscript.RSInvalidStateException; @@ -30,7 +29,8 @@ import android.view.ViewGroup; import androidx.annotation.NonNull; import androidx.annotation.Nullable; import androidx.fragment.app.Fragment; -import androidx.lifecycle.ViewModelProviders; +import androidx.fragment.app.FragmentActivity; +import androidx.lifecycle.ViewModelProvider; import androidx.recyclerview.widget.ItemTouchHelper; import androidx.recyclerview.widget.LinearLayoutManager; import androidx.recyclerview.widget.RecyclerView; @@ -43,6 +43,7 @@ import net.ktnx.mobileledger.model.Data; import net.ktnx.mobileledger.model.LedgerTransaction; import net.ktnx.mobileledger.model.LedgerTransactionAccount; import net.ktnx.mobileledger.model.MobileLedgerProfile; +import net.ktnx.mobileledger.utils.Logger; import net.ktnx.mobileledger.utils.Misc; import org.jetbrains.annotations.NotNull; @@ -92,8 +93,7 @@ public class NewTransactionFragment extends Fragment { "getActivity() returned null within onActivityCreated()"); list = activity.findViewById(R.id.new_transaction_accounts); - viewModel = ViewModelProviders.of(activity) - .get(NewTransactionModel.class); + viewModel = new ViewModelProvider(activity).get(NewTransactionModel.class); mProfile = Data.profile.getValue(); listAdapter = new NewTransactionItemsAdapter(viewModel, mProfile); list.setAdapter(listAdapter); @@ -160,24 +160,30 @@ public class NewTransactionFragment extends Fragment { fab = activity.findViewById(R.id.fab); fab.setOnClickListener(v -> onFabPressed()); + boolean keep = false; + Bundle args = getArguments(); if (args != null) { String error = args.getString("error"); if (error != null) { // TODO display error - } - else { + Logger.debug("new-trans-f", String.format("Got error: %s", error)); + Snackbar.make(list, error, Snackbar.LENGTH_LONG) + .show(); + keep = true; } } + int focused = 0; if (savedInstanceState != null) { - boolean keep = savedInstanceState.getBoolean("keep", true); - if (!keep) - viewModel.reset(); - else { - final int focused = savedInstanceState.getInt("focused", 0); - viewModel.setFocusedItem(focused); - } + keep |= savedInstanceState.getBoolean("keep", true); + focused = savedInstanceState.getInt("focused", 0); + } + + if (!keep) + viewModel.reset(); + else { + viewModel.setFocusedItem(focused); } } @Override @@ -198,7 +204,8 @@ public class NewTransactionFragment extends Fragment { LedgerTransactionAccount emptyAmountAccount = null; float emptyAmountAccountBalance = 0; for (int i = 0; i < viewModel.getAccountCount(); i++) { - LedgerTransactionAccount acc = viewModel.getAccount(i); + LedgerTransactionAccount acc = + new LedgerTransactionAccount(viewModel.getAccount(i)); if (acc.getAccountName() .trim() .isEmpty())