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=82709138207aaf553b51f29b4f893a8d1c409895;hp=d1241601809c09f5699e74924db6363b0f1cef63;hb=198f2762d2236a57cec413322c0bde6985ead88a;hpb=a1d55154e1c9fb72fcd60de31d6e64e8d046f96d 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 d1241601..82709138 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 @@ -36,16 +36,15 @@ import android.view.View; import android.view.ViewGroup; import net.ktnx.mobileledger.R; -import net.ktnx.mobileledger.async.RetrieveAccountsTask; +import net.ktnx.mobileledger.model.Data; import net.ktnx.mobileledger.model.LedgerAccount; import net.ktnx.mobileledger.ui.MobileLedgerListFragment; import net.ktnx.mobileledger.ui.RecyclerItemListener; import net.ktnx.mobileledger.ui.activity.MainActivity; -import net.ktnx.mobileledger.utils.MLDB; -import java.lang.ref.WeakReference; -import java.util.Date; import java.util.List; +import java.util.Observable; +import java.util.Observer; import static net.ktnx.mobileledger.ui.activity.SettingsActivity.PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS; @@ -59,10 +58,35 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { private AccountSummaryAdapter modelAdapter; private Menu optMenu; private FloatingActionButton fab; + private Observer backgroundTaskCountObserver; + @Override + public void onDestroy() { + if(backgroundTaskCountObserver!= null) { + Log.d("acc", "destroying background task count observer"); + Data.backgroundTaskCount.deleteObserver(backgroundTaskCountObserver); + } + super.onDestroy(); + } @Override public void onCreate(@Nullable Bundle savedInstanceState) { super.onCreate(savedInstanceState); setHasOptionsMenu(true); + + if (backgroundTaskCountObserver == null) { + Log.d("acc", "creating background task count observer"); + Data.backgroundTaskCount.addObserver(backgroundTaskCountObserver = new Observer() { + @Override + public void update(Observable o, Object arg) { + if (mActivity == null) return; + if (swiper == null) return; + mActivity.runOnUiThread(() -> { + int cnt = Data.backgroundTaskCount.get(); + Log.d("acc", String.format("background task count changed to %d", cnt)); + swiper.setRefreshing(cnt > 0); + }); + } + }); + } } public void onAttach(Context context) { super.onAttach(context); @@ -81,8 +105,7 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { mActivity.markDrawerItemCurrent(R.id.nav_account_summary); model = ViewModelProviders.of(this).get(AccountSummaryViewModel.class); - List accounts = model.getAccounts(this.getContext()); - modelAdapter = new AccountSummaryAdapter(accounts); + modelAdapter = new AccountSummaryAdapter(); RecyclerView root = mActivity.findViewById(R.id.account_root); root.setAdapter(modelAdapter); @@ -102,7 +125,7 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { modelAdapter.selectItem(position); } else { - List accounts = model.getAccounts(mActivity); + List accounts = Data.accounts.get(); if (accounts != null) { LedgerAccount account = accounts.get(position); @@ -142,39 +165,21 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { swiper.setColorSchemeResources(R.color.colorPrimary, R.color.colorAccent); swiper.setOnRefreshListener(() -> { Log.d("ui", "refreshing accounts via swipe"); - update_accounts(true); + mActivity.scheduleTransactionListRetrieval(); }); - prepare_db(); -// update_account_table(); - update_accounts(false); - - } - private void prepare_db() { - account_list_last_updated = MLDB.get_option_value(MLDB.OPT_LAST_REFRESH, (long) 0); - } - - private void update_accounts(boolean force) { - long now = new Date().getTime(); - if ((now > (account_list_last_updated + (24 * 3600 * 1000))) || force) { - Log.d("db", - "accounts last updated at " + account_list_last_updated + " and now is " + now + - ". re-fetching"); - update_accounts(); - } - } - - private void update_accounts() { - RetrieveAccountsTask task = new RetrieveAccountsTask(new WeakReference<>(mActivity)); - - task.setPref(PreferenceManager.getDefaultSharedPreferences(mActivity)); - task.execute(); + Data.accounts.addObserver(new Observer() { + @Override + public void update(Observable o, Object arg) { + mActivity.runOnUiThread(() -> modelAdapter.notifyDataSetChanged()); + } + }); + update_account_table(); } private void update_account_table() { if (this.getContext() == null) return; - model.reloadAccounts(this.getContext()); - modelAdapter.notifyDataSetChanged(); + model.scheduleAccountListReload(this.getContext()); } public void onShowOnlyStarredClicked(MenuItem mi) { SharedPreferences pref = PreferenceManager.getDefaultSharedPreferences(mActivity); @@ -203,7 +208,7 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { stopSelection(); } public void onConfirmAccSelection(MenuItem item) { - model.commitSelections(mActivity); + AccountSummaryViewModel.commitSelections(mActivity); stopSelection(); } @Override