X-Git-Url: https://git.ktnx.net/?a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2FCurrencySelectorFragment.java;h=4993df37e1a71a4c93cf0b7561c53314da8d5d3b;hb=HEAD;hp=47f2353e6093eaa8de0418ac5545c3eb5197aa86;hpb=5df10dc0b58df4d4be4e9ab34f1e0f477ca46766;p=mobile-ledger.git diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/CurrencySelectorFragment.java b/app/src/main/java/net/ktnx/mobileledger/ui/CurrencySelectorFragment.java index 47f2353e..759cd90e 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/CurrencySelectorFragment.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/CurrencySelectorFragment.java @@ -1,5 +1,5 @@ /* - * Copyright © 2019 Damyan Ivanov. + * Copyright © 2021 Damyan Ivanov. * 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 @@ -130,6 +130,8 @@ public class CurrencySelectorFragment extends AppCompatDialogFragment final TextView tvNoCurrBtn = csd.findViewById(R.id.btn_no_currency); final TextView tvAddCurrOkBtn = csd.findViewById(R.id.btn_add_currency); final TextView tvAddCurrBtn = csd.findViewById(R.id.btn_add_new); + final SwitchMaterial gap = csd.findViewById(R.id.currency_gap); + final RadioGroup rgPosition = csd.findViewById(R.id.position_radio_group); tvNewCurrName.setVisibility(View.GONE); tvAddCurrOkBtn.setVisibility(View.GONE); @@ -149,15 +151,14 @@ public class CurrencySelectorFragment extends AppCompatDialogFragment }); tvAddCurrOkBtn.setOnClickListener(v -> { - - String currName = String.valueOf(tvNewCurrName.getText()); if (!currName.isEmpty()) { DB.get() .getCurrencyDAO() - .insert(new net.ktnx.mobileledger.db.Currency(null, - String.valueOf(tvNewCurrName.getText()), "after", false), null); - // FIXME hardcoded position and gap setting + .insert(new net.ktnx.mobileledger.db.Currency(0, + String.valueOf(tvNewCurrName.getText()), + (rgPosition.getCheckedRadioButtonId() == R.id.currency_position_left) + ? "before" : "after", gap.isChecked())); } tvNewCurrName.setVisibility(View.GONE); @@ -180,7 +181,6 @@ public class CurrencySelectorFragment extends AppCompatDialogFragment else rbPositionRight.toggle(); - RadioGroup rgPosition = csd.findViewById(R.id.position_radio_group); rgPosition.setOnCheckedChangeListener((group, checkedId) -> { if (checkedId == R.id.currency_position_left) Data.currencySymbolPosition.setValue(Currency.Position.before); @@ -188,8 +188,6 @@ public class CurrencySelectorFragment extends AppCompatDialogFragment Data.currencySymbolPosition.setValue(Currency.Position.after); }); - SwitchMaterial gap = csd.findViewById(R.id.currency_gap); - gap.setChecked(Data.currencyGap.getValue()); gap.setOnCheckedChangeListener((v, checked) -> Data.currencyGap.setValue(checked)); @@ -199,8 +197,13 @@ public class CurrencySelectorFragment extends AppCompatDialogFragment visible ? View.VISIBLE : View.GONE)); - if ((savedInstanceState != null) ? savedInstanceState.getBoolean(ARG_SHOW_PARAMS, - DEFAULT_SHOW_PARAMS) : DEFAULT_SHOW_PARAMS) + final boolean showParams; + if (getArguments() == null) + showParams = DEFAULT_SHOW_PARAMS; + else + showParams = getArguments().getBoolean(ARG_SHOW_PARAMS, DEFAULT_SHOW_PARAMS); + + if (showParams) model.showPositionAndPadding(); else model.hidePositionAndPadding();