X-Git-Url: https://git.ktnx.net/?p=mobile-ledger.git;a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2Faccount_summary%2FAccountSummaryFragment.java;h=7f1c8f3a27aff856611b52eae8453226b2b44715;hp=82709138207aaf553b51f29b4f893a8d1c409895;hb=e1a130f8b260df37640540c0098859d957761643;hpb=198f2762d2236a57cec413322c0bde6985ead88a diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryFragment.java b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryFragment.java index 82709138..7f1c8f3a 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryFragment.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/account_summary/AccountSummaryFragment.java @@ -50,10 +50,7 @@ import static net.ktnx.mobileledger.ui.activity.SettingsActivity.PREF_KEY_SHOW_O public class AccountSummaryFragment extends MobileLedgerListFragment { - private static long account_list_last_updated; - private static boolean account_list_needs_update = true; - MenuItem mShowHiddenAccounts; - SharedPreferences.OnSharedPreferenceChangeListener sBindPreferenceSummaryToValueListener; + MenuItem mShowOnlyStarred; private AccountSummaryViewModel model; private AccountSummaryAdapter modelAdapter; private Menu optMenu; @@ -61,7 +58,7 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { private Observer backgroundTaskCountObserver; @Override public void onDestroy() { - if(backgroundTaskCountObserver!= null) { + if (backgroundTaskCountObserver != null) { Log.d("acc", "destroying background task count observer"); Data.backgroundTaskCount.deleteObserver(backgroundTaskCountObserver); } @@ -95,24 +92,24 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { @Override public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { + Log.d("flow", "AccountSummaryFragment.onCreateView()"); return inflater.inflate(R.layout.account_summary_fragment, container, false); } @Override + public void onActivityCreated(@Nullable Bundle savedInstanceState) { + Log.d("flow", "AccountSummaryFragment.onActivityCreated()"); super.onActivityCreated(savedInstanceState); - mActivity.markDrawerItemCurrent(R.id.nav_account_summary); - model = ViewModelProviders.of(this).get(AccountSummaryViewModel.class); modelAdapter = new AccountSummaryAdapter(); - RecyclerView root = mActivity.findViewById(R.id.account_root); - root.setAdapter(modelAdapter); - + root = mActivity.findViewById(R.id.account_root); LinearLayoutManager llm = new LinearLayoutManager(mActivity); llm.setOrientation(LinearLayoutManager.VERTICAL); root.setLayoutManager(llm); + root.setAdapter(modelAdapter); fab = mActivity.findViewById(R.id.btn_add_transaction); @@ -174,25 +171,19 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { mActivity.runOnUiThread(() -> modelAdapter.notifyDataSetChanged()); } }); + Data.profile.addObserver(new Observer() { + @Override + public void update(Observable o, Object arg) { + mActivity.runOnUiThread(() -> model.scheduleAccountListReload()); + } + }); update_account_table(); } private void update_account_table() { if (this.getContext() == null) return; - model.scheduleAccountListReload(this.getContext()); - } - public void onShowOnlyStarredClicked(MenuItem mi) { - SharedPreferences pref = PreferenceManager.getDefaultSharedPreferences(mActivity); - boolean flag = pref.getBoolean(PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS, false); - - SharedPreferences.Editor editor = pref.edit(); - editor.putBoolean(PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS, !flag); - Log.d("pref", "Setting show only starred accounts pref to " + (flag ? "false" : "true")); - editor.apply(); - - update_account_table(); + model.scheduleAccountListReload(); } - void stopSelection() { modelAdapter.stopSelection(); if (optMenu != null) { @@ -217,16 +208,46 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { inflater.inflate(R.menu.account_summary, menu); optMenu = menu; - mShowHiddenAccounts = menu.findItem(R.id.menu_acc_summary_only_starred); - if (mShowHiddenAccounts == null) throw new AssertionError(); + mShowOnlyStarred = menu.findItem(R.id.menu_acc_summary_only_starred); + if (mShowOnlyStarred == null) throw new AssertionError(); + MenuItem mCancelSelection = menu.findItem(R.id.menu_acc_summary_cancel_selection); + if (mCancelSelection == null) throw new AssertionError(); + MenuItem mConfirmSelection = menu.findItem(R.id.menu_acc_summary_confirm_selection); + if (mConfirmSelection == null) throw new AssertionError(); + + Data.optShowOnlyStarred.addObserver((o, arg) -> { + boolean newValue = Data.optShowOnlyStarred.get(); + Log.d("pref", String.format("pref change came (%s)", newValue ? "true" : "false")); + mShowOnlyStarred.setChecked(newValue); + update_account_table(); + }); + + mShowOnlyStarred.setChecked(Data.optShowOnlyStarred.get()); - sBindPreferenceSummaryToValueListener = (preference, value) -> mShowHiddenAccounts - .setChecked(preference.getBoolean(PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS, false)); - SharedPreferences pref = PreferenceManager.getDefaultSharedPreferences(mActivity); - pref.registerOnSharedPreferenceChangeListener(sBindPreferenceSummaryToValueListener); + Log.d("menu", "Accounts: onCreateOptionsMenu called"); - mShowHiddenAccounts.setChecked(pref.getBoolean(PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS, false)); + mShowOnlyStarred.setOnMenuItemClickListener(item -> { + SharedPreferences pref = PreferenceManager.getDefaultSharedPreferences(mActivity); + SharedPreferences.Editor editor = pref.edit(); + boolean flag = item.isChecked(); + editor.putBoolean(PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS, !flag); + Log.d("pref", + "Setting show only starred accounts pref to " + (flag ? "false" : "true")); + editor.apply(); - Log.d("menu", "MainActivity: onCreateOptionsMenu called"); + return true; + }); + + mCancelSelection.setOnMenuItemClickListener(item -> { + stopSelection(); + return true; + }); + + mConfirmSelection.setOnMenuItemClickListener(item -> { + AccountSummaryViewModel.commitSelections(mActivity); + stopSelection(); + + return true; + }); } }