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=362dadb9658c1d8a841c9ba55bf2d6bc49f01083;hp=b6943adc41ae8d89d994d25a1c28c91f79665c31;hb=bd5da50ef980c0c9657ec1e9c3e681ab5092f438;hpb=9f35f59fcffd6b9778b36bf31f8b94d91411664f 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 b6943adc..362dadb9 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 @@ -1,18 +1,18 @@ /* * Copyright © 2019 Damyan Ivanov. - * This file is part of Mobile-Ledger. - * Mobile-Ledger is free software: you can distribute it and/or modify it + * This file is part of MoLe. + * MoLe is free software: you can distribute it and/or modify it * under the term of the GNU General Public License as published by * the Free Software Foundation, either version 3 of the License, or * (at your opinion), any later version. * - * Mobile-Ledger is distributed in the hope that it will be useful, + * MoLe is distributed in the hope that it will be useful, * but WITHOUT ANY WARRANTY; without even the implied warranty of * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the * GNU General Public License terms for details. * * You should have received a copy of the GNU General Public License - * along with Mobile-Ledger. If not, see . + * along with MoLe. If not, see . */ package net.ktnx.mobileledger.ui.account_summary; @@ -21,12 +21,6 @@ import android.content.Context; import android.content.SharedPreferences; import android.os.Bundle; import android.preference.PreferenceManager; -import android.support.annotation.NonNull; -import android.support.annotation.Nullable; -import android.support.design.widget.FloatingActionButton; -import android.support.v7.widget.LinearLayoutManager; -import android.support.v7.widget.RecyclerView; -import android.util.Log; import android.view.LayoutInflater; import android.view.Menu; import android.view.MenuInflater; @@ -34,112 +28,104 @@ import android.view.MenuItem; import android.view.View; import android.view.ViewGroup; +import com.google.android.material.floatingactionbutton.FloatingActionButton; + import net.ktnx.mobileledger.R; 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.Colors; + +import org.jetbrains.annotations.NotNull; -import java.util.List; -import java.util.Observer; +import androidx.annotation.NonNull; +import androidx.annotation.Nullable; +import androidx.recyclerview.widget.DividerItemDecoration; +import androidx.recyclerview.widget.LinearLayoutManager; +import androidx.recyclerview.widget.RecyclerView; import static net.ktnx.mobileledger.ui.activity.SettingsActivity.PREF_KEY_SHOW_ONLY_STARRED_ACCOUNTS; +import static net.ktnx.mobileledger.utils.Logger.debug; public class AccountSummaryFragment extends MobileLedgerListFragment { - MenuItem mShowOnlyStarred; - private AccountSummaryAdapter modelAdapter; + private MenuItem mShowOnlyStarred; + public 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); + debug("flow", "AccountSummaryFragment.onCreate()"); setHasOptionsMenu(true); - if (backgroundTaskCountObserver == null) { - Log.d("acc", "creating background task count observer"); - Data.backgroundTaskCount.addObserver(backgroundTaskCountObserver = (o, 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); - }); - }); - } + Data.backgroundTasksRunning.observe(this, this::onBackgroundTaskRunningChanged); } - public void onAttach(Context context) { + public void onAttach(@NotNull Context context) { super.onAttach(context); + debug("flow", "AccountSummaryFragment.onAttach()"); mActivity = (MainActivity) context; } @Override public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { - Log.d("flow", "AccountSummaryFragment.onCreateView()"); + debug("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()"); + debug("flow", "AccountSummaryFragment.onActivityCreated()"); super.onActivityCreated(savedInstanceState); modelAdapter = new AccountSummaryAdapter(); + mActivity.mAccountSummaryFragment = this; root = mActivity.findViewById(R.id.account_root); LinearLayoutManager llm = new LinearLayoutManager(mActivity); - llm.setOrientation(LinearLayoutManager.VERTICAL); + llm.setOrientation(RecyclerView.VERTICAL); root.setLayoutManager(llm); root.setAdapter(modelAdapter); + DividerItemDecoration did = new DividerItemDecoration(mActivity, DividerItemDecoration.VERTICAL); + root.addItemDecoration(did); fab = mActivity.findViewById(R.id.btn_add_transaction); - root.addOnItemTouchListener(new RecyclerItemListener(mActivity, root, - new RecyclerItemListener.RecyclerTouchListener() { - @Override - public void onClickItem(View v, int position) { - Log.d("value", String.format("item %d clicked", position)); - if (modelAdapter.isSelectionActive()) { - modelAdapter.selectItem(position); - } - else { - List accounts = Data.accounts.get(); - if (accounts != null) { - LedgerAccount account = accounts.get(position); - - mActivity.showAccountTransactions(account); - } - } - } - - @Override - public void onLongClickItem(View v, int position) { - Log.d("value", String.format("item %d long-clicked", position)); - modelAdapter.startSelection(); - if (optMenu != null) { - optMenu.findItem(R.id.menu_acc_summary_cancel_selection) - .setVisible(true); - optMenu.findItem(R.id.menu_acc_summary_confirm_selection) - .setVisible(true); - optMenu.findItem(R.id.menu_acc_summary_only_starred).setVisible(false); - } - { - if (fab != null) fab.hide(); - } - } - })); +// root.addOnItemTouchListener(new RecyclerItemListener(mActivity, root, +// new RecyclerItemListener.RecyclerTouchListener() { +// @Override +// public void onClickItem(View v, int position) { +// debug("value", String.format("item %d clicked", position)); +// if (modelAdapter.isSelectionActive()) { +// modelAdapter.selectItem(position); +// } +// else { +// List accounts = Data.accounts.get(); +// if (accounts != null) { +// LedgerAccount account = accounts.get(position); +// +// mActivity.showAccountTransactions(account); +// } +// } +// } +// +// @Override +// public void onLongClickItem(View v, int position) { +// debug("value", String.format("item %d long-clicked", position)); +// modelAdapter.startSelection(); +// if (optMenu != null) { +// optMenu.findItem(R.id.menu_acc_summary_cancel_selection) +// .setVisible(true); +// optMenu.findItem(R.id.menu_acc_summary_confirm_selection) +// .setVisible(true); +// optMenu.findItem(R.id.menu_acc_summary_only_starred).setVisible(false); +// } +// { +// if (fab != null) fab.hide(); +// } +// } +// })); mActivity.fabShouldShow(); root.addOnScrollListener(new RecyclerView.OnScrollListener() { @@ -152,17 +138,14 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { } }); swiper = mActivity.findViewById(R.id.account_swiper); - swiper.setColorSchemeResources(R.color.colorPrimary, R.color.colorAccent); + Colors.themeWatch.observe(this, this::themeChanged); swiper.setOnRefreshListener(() -> { - Log.d("ui", "refreshing accounts via swipe"); + debug("ui", "refreshing accounts via swipe"); mActivity.scheduleTransactionListRetrieval(); }); Data.accounts.addObserver( (o, arg) -> mActivity.runOnUiThread(() -> modelAdapter.notifyDataSetChanged())); - Data.profile.addObserver( - (o, arg) -> mActivity.runOnUiThread(() -> AccountSummaryViewModel.scheduleAccountListReload())); - update_account_table(); } private void update_account_table() { if (this.getContext() == null) return; @@ -188,7 +171,7 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { stopSelection(); } @Override - public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { + public void onCreateOptionsMenu(@NotNull Menu menu, @NotNull MenuInflater inflater) { // Inflate the menu; this adds items to the action bar if it is present. inflater.inflate(R.menu.account_summary, menu); optMenu = menu; @@ -202,21 +185,21 @@ public class AccountSummaryFragment extends MobileLedgerListFragment { Data.optShowOnlyStarred.addObserver((o, arg) -> { boolean newValue = Data.optShowOnlyStarred.get(); - Log.d("pref", String.format("pref change came (%s)", newValue ? "true" : "false")); + debug("pref", String.format("pref change came (%s)", newValue ? "true" : "false")); mShowOnlyStarred.setChecked(newValue); update_account_table(); }); mShowOnlyStarred.setChecked(Data.optShowOnlyStarred.get()); - Log.d("menu", "Accounts: onCreateOptionsMenu called"); + debug("menu", "Accounts: onCreateOptionsMenu called"); 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", + debug("pref", "Setting show only starred accounts pref to " + (flag ? "false" : "true")); editor.apply();