X-Git-Url: https://git.ktnx.net/?p=mobile-ledger.git;a=blobdiff_plain;f=app%2Fsrc%2Fmain%2Fjava%2Fnet%2Fktnx%2Fmobileledger%2Fui%2Fprofiles%2FProfileDetailFragment.java;h=672acf0ec4a58cb37d7ba0a897c8065b9433feae;hp=6d26e4ba71efeb84d8ef03a69e1e871084248319;hb=b296570e4c84379133927f7f35e1220b9b1d6862;hpb=09e26d2279484b4dfe0de218b05f075362fff4b5 diff --git a/app/src/main/java/net/ktnx/mobileledger/ui/profiles/ProfileDetailFragment.java b/app/src/main/java/net/ktnx/mobileledger/ui/profiles/ProfileDetailFragment.java index 6d26e4ba..672acf0e 100644 --- a/app/src/main/java/net/ktnx/mobileledger/ui/profiles/ProfileDetailFragment.java +++ b/app/src/main/java/net/ktnx/mobileledger/ui/profiles/ProfileDetailFragment.java @@ -1,5 +1,5 @@ /* - * Copyright © 2019 Damyan Ivanov. + * Copyright © 2020 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 @@ -18,47 +18,57 @@ package net.ktnx.mobileledger.ui.profiles; import android.app.Activity; -import android.content.Context; -import android.os.Build; +import android.app.AlertDialog; +import android.graphics.Typeface; import android.os.Bundle; -import androidx.annotation.ColorInt; -import androidx.annotation.NonNull; -import androidx.annotation.Nullable; -import com.google.android.material.appbar.CollapsingToolbarLayout; -import com.google.android.material.floatingactionbutton.FloatingActionButton; -import com.google.android.material.textfield.TextInputLayout; -import androidx.fragment.app.Fragment; import android.text.Editable; import android.text.TextWatcher; -import android.util.Log; -import android.view.LayoutInflater; import android.view.Menu; import android.view.MenuInflater; import android.view.MenuItem; import android.view.View; -import android.view.ViewGroup; -import android.widget.AdapterView; -import android.widget.ArrayAdapter; import android.widget.LinearLayout; -import android.widget.Spinner; -import android.widget.Switch; +import android.widget.PopupMenu; import android.widget.TextView; +import androidx.annotation.NonNull; +import androidx.annotation.Nullable; +import androidx.appcompat.app.AppCompatActivity; +import androidx.fragment.app.Fragment; +import androidx.fragment.app.FragmentActivity; +import androidx.lifecycle.LifecycleOwner; +import androidx.lifecycle.ViewModelProvider; + +import com.google.android.material.appbar.CollapsingToolbarLayout; +import com.google.android.material.floatingactionbutton.FloatingActionButton; +import com.google.android.material.switchmaterial.SwitchMaterial; +import com.google.android.material.textfield.TextInputLayout; + +import net.ktnx.mobileledger.BuildConfig; import net.ktnx.mobileledger.R; +import net.ktnx.mobileledger.async.SendTransactionTask; import net.ktnx.mobileledger.model.Data; import net.ktnx.mobileledger.model.MobileLedgerProfile; +import net.ktnx.mobileledger.ui.CurrencySelectorFragment; +import net.ktnx.mobileledger.ui.HueRingDialog; import net.ktnx.mobileledger.ui.activity.ProfileDetailActivity; -import net.ktnx.mobileledger.ui.activity.ProfileListActivity; import net.ktnx.mobileledger.utils.Colors; +import net.ktnx.mobileledger.utils.Misc; +import org.jetbrains.annotations.NonNls; import org.jetbrains.annotations.NotNull; -import java.util.List; +import java.net.MalformedURLException; +import java.net.URL; +import java.util.ArrayList; +import java.util.Objects; +import java.util.UUID; + +import static net.ktnx.mobileledger.utils.Logger.debug; /** * A fragment representing a single Profile detail screen. - * This fragment is either contained in a {@link ProfileListActivity} - * in two-pane mode (on tablets) or a {@link ProfileDetailActivity} + * a {@link ProfileDetailActivity} * on handsets. */ public class ProfileDetailFragment extends Fragment { @@ -67,225 +77,462 @@ public class ProfileDetailFragment extends Fragment { * represents. */ public static final String ARG_ITEM_ID = "item_id"; + public static final String ARG_HUE = "hue"; + @NonNls - /** - * The dummy content this fragment is presenting. - */ private MobileLedgerProfile mProfile; private TextView url; - private Switch postingPermitted; + private TextView defaultCommodity; + private boolean defaultCommoditySet; private TextInputLayout urlLayout; private LinearLayout authParams; - private Switch useAuthentication; + private SwitchMaterial useAuthentication; private TextView userName; private TextInputLayout userNameLayout; private TextView password; private TextInputLayout passwordLayout; private TextView profileName; private TextInputLayout profileNameLayout; - private FloatingActionButton fab; - private Spinner colorSpinner; - + private TextView preferredAccountsFilter; + private View huePickerView; + private View insecureWarningText; + private TextView futureDatesText; + private TextView apiVersionText; + private boolean syncingModelFromUI = false; /** * Mandatory empty constructor for the fragment manager to instantiate the * fragment (e.g. upon screen orientation changes). */ public ProfileDetailFragment() { + super(R.layout.profile_detail); } @Override - public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { - Log.d("profiles", "[fragment] Creating profile details options menu"); + public void onCreateOptionsMenu(@NotNull Menu menu, @NotNull MenuInflater inflater) { + debug("profiles", "[fragment] Creating profile details options menu"); super.onCreateOptionsMenu(menu, inflater); inflater.inflate(R.menu.profile_details, menu); final MenuItem menuDeleteProfile = menu.findItem(R.id.menuDelete); - menuDeleteProfile.setOnMenuItemClickListener(item -> { - Log.d("profiles", String.format("[fragment] removing profile %s", mProfile.getUuid())); + menuDeleteProfile.setOnMenuItemClickListener(item -> onDeleteProfile()); + final ArrayList profiles = Data.profiles.getValue(); + + if (BuildConfig.DEBUG) { + final MenuItem menuWipeProfileData = menu.findItem(R.id.menuWipeData); + menuWipeProfileData.setOnMenuItemClickListener(ignored -> onWipeDataMenuClicked()); + menuWipeProfileData.setVisible(mProfile != null); + } + } + private boolean onDeleteProfile() { + AlertDialog.Builder builder = new AlertDialog.Builder(getContext()); + builder.setTitle(mProfile.getName()); + builder.setMessage(R.string.remove_profile_dialog_message); + builder.setPositiveButton(R.string.Remove, (dialog, which) -> { + debug("profiles", String.format("[fragment] removing profile %s", mProfile.getUuid())); mProfile.removeFromDB(); - Data.profiles.remove(mProfile); - if (Data.profile.get().equals(mProfile)) { - Log.d("profiles", "[fragment] setting current profile to 0"); - Data.setCurrentProfile(Data.profiles.get(0)); + ArrayList oldList = Data.profiles.getValue(); + if (oldList == null) + throw new AssertionError(); + ArrayList newList = new ArrayList<>(oldList); + newList.remove(mProfile); + Data.profiles.setValue(newList); + if (mProfile.equals(Data.getProfile())) { + debug("profiles", "[fragment] setting current profile to 0"); + Data.setCurrentProfile(newList.get(0)); } - return false; + + final FragmentActivity activity = getActivity(); + if (activity != null) + activity.finish(); + }); + builder.show(); + return false; + } + private boolean onWipeDataMenuClicked() { + // this is a development option, so no confirmation + mProfile.wipeAllData(); + if (mProfile.equals(Data.getProfile())) + triggerProfileChange(); + return true; + } + private void triggerProfileChange() { + int index = Data.getProfileIndex(mProfile); + MobileLedgerProfile newProfile = new MobileLedgerProfile(mProfile); + final ArrayList profiles = + Objects.requireNonNull(Data.profiles.getValue()); + profiles.set(index, newProfile); + + ProfilesRecyclerViewAdapter viewAdapter = ProfilesRecyclerViewAdapter.getInstance(); + if (viewAdapter != null) + viewAdapter.notifyItemChanged(index); + + if (mProfile.equals(Data.getProfile())) + Data.setCurrentProfile(newProfile); + } + private void hookTextChangeSyncRoutine(TextView view, TextChangeSyncRoutine syncRoutine) { + view.addTextChangedListener(new TextWatcher() { + @Override + public void beforeTextChanged(CharSequence s, int start, int count, int after) {} + @Override + public void onTextChanged(CharSequence s, int start, int before, int count) {} + @Override + public void afterTextChanged(Editable s) { syncRoutine.onTextChanged(s.toString());} }); - menuDeleteProfile.setVisible((mProfile != null) && (Data.profiles.size() > 1)); } @Override - public void onCreate(Bundle savedInstanceState) { - super.onCreate(savedInstanceState); + public void onViewCreated(@NotNull View view, @Nullable Bundle savedInstanceState) { + super.onViewCreated(view, savedInstanceState); + Activity context = getActivity(); + if (context == null) + return; if ((getArguments() != null) && getArguments().containsKey(ARG_ITEM_ID)) { int index = getArguments().getInt(ARG_ITEM_ID, -1); - if (index != -1) mProfile = Data.profiles.get(index); + ArrayList profiles = Data.profiles.getValue(); + if ((profiles != null) && (index != -1) && (index < profiles.size())) + mProfile = profiles.get(index); Activity activity = this.getActivity(); - if (activity == null) throw new AssertionError(); + if (activity == null) + throw new AssertionError(); CollapsingToolbarLayout appBarLayout = activity.findViewById(R.id.toolbar_layout); if (appBarLayout != null) { - if (mProfile != null) appBarLayout.setTitle(mProfile.getName()); - else appBarLayout.setTitle(getResources().getString(R.string.new_profile_title)); + if (mProfile != null) + appBarLayout.setTitle(mProfile.getName()); + else + appBarLayout.setTitle(getResources().getString(R.string.new_profile_title)); } } - } - @Override - public void onActivityCreated(@Nullable Bundle savedInstanceState) { - super.onActivityCreated(savedInstanceState); - Activity context = getActivity(); - if (context == null) return; - - fab = context.findViewById(R.id.fab); - fab.setOnClickListener(v -> { - if (!checkValidity()) return; - - if (mProfile != null) { - mProfile.setName(profileName.getText()); - mProfile.setUrl(url.getText()); - mProfile.setPostingPermitted(postingPermitted.isChecked()); - mProfile.setAuthEnabled(useAuthentication.isChecked()); - mProfile.setAuthUserName(userName.getText()); - mProfile.setAuthPassword(password.getText()); - mProfile.setThemeId(colorSpinner.getSelectedItem()); -// Log.d("profiles", String.format("Selected item is %d", mProfile.getThemeId())); - mProfile.storeInDB(); - Log.d("profiles", "profile stored in DB"); - Data.profiles.triggerItemChangedNotification(mProfile); - - - if (mProfile.getUuid().equals(Data.profile.get().getUuid())) { - // dummy update to notify the observers of the possibly new name/URL - Data.profile.set(mProfile); - } - } - else { - mProfile = - new MobileLedgerProfile(profileName.getText(), postingPermitted.isChecked(), - url.getText(), useAuthentication.isChecked(), userName.getText(), - password.getText(), - Integer.valueOf((String) colorSpinner.getSelectedItem())); - mProfile.storeInDB(); - Data.profiles.add(mProfile); - MobileLedgerProfile.storeProfilesOrder(); - - // first profile ever? - if (Data.profiles.getList().size() == 1) Data.profile.set(mProfile); - } - Activity activity = getActivity(); - if (activity != null) activity.finish(); + final LifecycleOwner viewLifecycleOwner = getViewLifecycleOwner(); + final ProfileDetailModel model = getModel(); + + model.observeDefaultCommodity(viewLifecycleOwner, c -> { + if (c != null) + setDefaultCommodity(c.getName()); + else + resetDefaultCommodity(); }); - profileName.requestFocus(); - } - @Override - public View onCreateView(@NonNull LayoutInflater inflater, ViewGroup container, - Bundle savedInstanceState) { - View rootView = inflater.inflate(R.layout.profile_detail, container, false); - - profileName = rootView.findViewById(R.id.profile_name); - profileNameLayout = rootView.findViewById(R.id.profile_name_layout); - url = rootView.findViewById(R.id.url); - urlLayout = rootView.findViewById(R.id.url_layout); - postingPermitted = rootView.findViewById(R.id.profile_permit_posting); - authParams = rootView.findViewById(R.id.auth_params); - useAuthentication = rootView.findViewById(R.id.enable_http_auth); - userName = rootView.findViewById(R.id.auth_user_name); - userNameLayout = rootView.findViewById(R.id.auth_user_name_layout); - password = rootView.findViewById(R.id.password); - passwordLayout = rootView.findViewById(R.id.password_layout); - colorSpinner = rootView.findViewById(R.id.colorSpinner); - - ArrayAdapter adapter = ColorListAdapter - .createFromResource(rootView.getContext(), R.array.profile_colors, - R.layout.color_selector_item); -// Log.d("profiles", String.format("color count: %s", adapter.getCount())); - adapter.setDropDownViewResource(android.R.layout.simple_spinner_dropdown_item); - colorSpinner.setAdapter(adapter); - colorSpinner.setOnItemSelectedListener(new AdapterView.OnItemSelectedListener() { - @Override - public void onNothingSelected(AdapterView parent) { + FloatingActionButton fab = context.findViewById(R.id.fab); + fab.setOnClickListener(v -> onSaveFabClicked()); - } - @Override - public void onItemSelected(AdapterView parent, View view, int position, long id) { - final int primaryColor; - final int degrees = - Integer.valueOf((String) (parent.getAdapter().getItem(position))); - if (degrees < 0) { - if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.M) { - if (getActivity() != null) primaryColor = getResources() - .getColor(R.color.colorPrimary, getActivity().getTheme()); - else primaryColor = Colors.getPrimaryColorForHue(Colors.DEFAULT_HUE_DEG); - } - else { - primaryColor = getResources().getColor(R.color.colorPrimary); - } - } - else primaryColor = Colors.getPrimaryColorForHue(degrees); - - if (colorSpinner != null) { - colorSpinner.setBackgroundColor(primaryColor); -// for (int i = 0; i < colorSpinner.getChildCount(); i++) { -// View v = colorSpinner.getChildAt(i); -// -// if (v instanceof TextView) { -// ((TextView) v).setTextColor(Color.TRANSPARENT); -// } -// } - } - } + profileName = context.findViewById(R.id.profile_name); + hookTextChangeSyncRoutine(profileName, model::setProfileName); + model.observeProfileName(viewLifecycleOwner, pn -> { + if (!Misc.equalStrings(pn, profileName.getText())) + profileName.setText(pn); + }); + + profileNameLayout = context.findViewById(R.id.profile_name_layout); + + url = context.findViewById(R.id.url); + hookTextChangeSyncRoutine(url, model::setUrl); + model.observeUrl(viewLifecycleOwner, u -> { + if (!Misc.equalStrings(u, url.getText())) + url.setText(u); }); + urlLayout = context.findViewById(R.id.url_layout); + + context.findViewById(R.id.default_commodity_layout) + .setOnClickListener(v -> { + CurrencySelectorFragment cpf = CurrencySelectorFragment.newInstance( + CurrencySelectorFragment.DEFAULT_COLUMN_COUNT, false); + cpf.setOnCurrencySelectedListener(model::setDefaultCommodity); + final AppCompatActivity activity = (AppCompatActivity) v.getContext(); + cpf.show(activity.getSupportFragmentManager(), "currency-selector"); + }); + + SwitchMaterial showCommodityByDefault = context.findViewById(R.id.profile_show_commodity); + showCommodityByDefault.setOnCheckedChangeListener( + (buttonView, isChecked) -> model.setShowCommodityByDefault(isChecked)); + model.observeShowCommodityByDefault(viewLifecycleOwner, showCommodityByDefault::setChecked); + + View postingSubItems = context.findViewById(R.id.posting_sub_items); + + SwitchMaterial postingPermitted = context.findViewById(R.id.profile_permit_posting); + model.observePostingPermitted(viewLifecycleOwner, isChecked -> { + postingPermitted.setChecked(isChecked); + postingSubItems.setVisibility(isChecked ? View.VISIBLE : View.GONE); + }); + postingPermitted.setOnCheckedChangeListener( + ((buttonView, isChecked) -> model.setPostingPermitted(isChecked))); + + SwitchMaterial showCommentsByDefault = context.findViewById(R.id.profile_show_comments); + model.observeShowCommentsByDefault(viewLifecycleOwner, showCommentsByDefault::setChecked); + showCommentsByDefault.setOnCheckedChangeListener( + ((buttonView, isChecked) -> model.setShowCommentsByDefault(isChecked))); + + defaultCommodity = context.findViewById(R.id.default_commodity_text); + + futureDatesText = context.findViewById(R.id.future_dates_text); + context.findViewById(R.id.future_dates_layout) + .setOnClickListener(v -> { + MenuInflater mi = new MenuInflater(context); + PopupMenu menu = new PopupMenu(context, v); + menu.inflate(R.menu.future_dates); + menu.setOnMenuItemClickListener(item -> { + model.setFutureDates(futureDatesSettingFromMenuItemId(item.getItemId())); + return true; + }); + menu.show(); + }); + model.observeFutureDates(viewLifecycleOwner, + v -> futureDatesText.setText(v.getText(getResources()))); + + apiVersionText = context.findViewById(R.id.api_version_text); + model.observeApiVersion(viewLifecycleOwner, + apiVer -> apiVersionText.setText(apiVer.getDescription(getResources()))); + context.findViewById(R.id.api_version_label) + .setOnClickListener(this::chooseAPIVersion); + context.findViewById(R.id.api_version_text) + .setOnClickListener(this::chooseAPIVersion); + + TextView detectedApiVersion = context.findViewById(R.id.detected_version_text); + model.observeDetectedVersion(viewLifecycleOwner, ver -> { + if (ver == null) + detectedApiVersion.setText(context.getResources() + .getString(R.string.api_version_unknown_label)); + else if (ver.isPre_1_20()) + detectedApiVersion.setText(context.getResources() + .getString(R.string.api_pre_1_19)); + else + detectedApiVersion.setText(ver.toString()); + }); + detectedApiVersion.setOnClickListener(v -> model.triggerVersionDetection()); + final View detectButton = context.findViewById(R.id.api_version_detect_button); + detectButton.setOnClickListener(v -> model.triggerVersionDetection()); + model.observeDetectingHledgerVersion(viewLifecycleOwner, running -> { + detectButton.setVisibility(running ? View.VISIBLE : View.INVISIBLE); + }); + + authParams = context.findViewById(R.id.auth_params); + + useAuthentication = context.findViewById(R.id.enable_http_auth); useAuthentication.setOnCheckedChangeListener((buttonView, isChecked) -> { - Log.d("profiles", isChecked ? "auth enabled " : "auth disabled"); + model.setUseAuthentication(isChecked); + if (isChecked) + userName.requestFocus(); + }); + model.observeUseAuthentication(viewLifecycleOwner, isChecked -> { + useAuthentication.setChecked(isChecked); authParams.setVisibility(isChecked ? View.VISIBLE : View.GONE); - if (isChecked) userName.requestFocus(); + checkInsecureSchemeWithAuth(); + }); + + userName = context.findViewById(R.id.auth_user_name); + model.observeUserName(viewLifecycleOwner, text -> { + if (!Misc.equalStrings(text, userName.getText())) + userName.setText(text); + }); + hookTextChangeSyncRoutine(userName, model::setAuthUserName); + userNameLayout = context.findViewById(R.id.auth_user_name_layout); + + password = context.findViewById(R.id.password); + model.observePassword(viewLifecycleOwner, text -> { + if (!Misc.equalStrings(text, password.getText())) + password.setText(text); + }); + hookTextChangeSyncRoutine(password, model::setAuthPassword); + passwordLayout = context.findViewById(R.id.password_layout); + + huePickerView = context.findViewById(R.id.btn_pick_ring_color); + model.observeThemeId(viewLifecycleOwner, themeId -> { + final int hue = (themeId == -1) ? Colors.DEFAULT_HUE_DEG : themeId; + final int profileColor = Colors.getPrimaryColorForHue(hue); + huePickerView.setBackgroundColor(profileColor); + huePickerView.setTag(hue); }); + preferredAccountsFilter = context.findViewById(R.id.preferred_accounts_filter_filter); + model.observePreferredAccountsFilter(viewLifecycleOwner, text -> { + if (!Misc.equalStrings(text, preferredAccountsFilter.getText())) + preferredAccountsFilter.setText(text); + }); + hookTextChangeSyncRoutine(preferredAccountsFilter, model::setPreferredAccountsFilter); + + insecureWarningText = context.findViewById(R.id.insecure_scheme_text); + hookClearErrorOnFocusListener(profileName, profileNameLayout); hookClearErrorOnFocusListener(url, urlLayout); hookClearErrorOnFocusListener(userName, userNameLayout); hookClearErrorOnFocusListener(password, passwordLayout); - if (mProfile != null) { - profileName.setText(mProfile.getName()); - postingPermitted.setChecked(mProfile.isPostingPermitted()); - url.setText(mProfile.getUrl()); - useAuthentication.setChecked(mProfile.isAuthEnabled()); - authParams.setVisibility(mProfile.isAuthEnabled() ? View.VISIBLE : View.GONE); - userName.setText(mProfile.isAuthEnabled() ? mProfile.getAuthUserName() : ""); - password.setText(mProfile.isAuthEnabled() ? mProfile.getAuthPassword() : ""); - - colorSpinner.setSelection(0); - int i = 0; - int sought = mProfile.getThemeId(); -// Log.d("profiles", String.format("Looking for %d",sought)); - while (i < adapter.getCount()) { - int item = Integer.valueOf(String.valueOf(adapter.getItem(i))); -// Log.d("profiles", String.format("Item %d is %d", i, item)); - if (item == sought) { - colorSpinner.setSelection(i); + if (savedInstanceState == null) { + model.setValuesFromProfile(mProfile, getArguments().getInt(ARG_HUE, -1)); + } + checkInsecureSchemeWithAuth(); + + url.addTextChangedListener(new TextWatcher() { + @Override + public void beforeTextChanged(CharSequence s, int start, int count, int after) {} + @Override + public void onTextChanged(CharSequence s, int start, int before, int count) {} + @Override + public void afterTextChanged(Editable s) { + checkInsecureSchemeWithAuth(); + } + }); + + huePickerView.setOnClickListener(v -> { + HueRingDialog d = new HueRingDialog(ProfileDetailFragment.this.requireContext(), + model.initialThemeHue, (Integer) v.getTag()); + d.show(); + d.setColorSelectedListener(model::setThemeId); + }); + + profileName.requestFocus(); + } + private void chooseAPIVersion(View v) { + Activity context = getActivity(); + ProfileDetailModel model = getModel(); + MenuInflater mi = new MenuInflater(context); + PopupMenu menu = new PopupMenu(context, v); + menu.inflate(R.menu.api_version); + menu.setOnMenuItemClickListener(item -> { + SendTransactionTask.API apiVer; + switch (item.getItemId()) { + case R.id.api_version_menu_html: + apiVer = SendTransactionTask.API.html; + break; + case R.id.api_version_menu_post_1_14: + apiVer = SendTransactionTask.API.post_1_14; break; - } + case R.id.api_version_menu_pre_1_15: + apiVer = SendTransactionTask.API.pre_1_15; + break; + case R.id.api_version_menu_auto: + default: + apiVer = SendTransactionTask.API.auto; + } + model.setApiVersion(apiVer); + apiVersionText.setText(apiVer.getDescription(getResources())); + return true; + }); + menu.show(); + } + private MobileLedgerProfile.FutureDates futureDatesSettingFromMenuItemId(int itemId) { + switch (itemId) { + case R.id.menu_future_dates_7: + return MobileLedgerProfile.FutureDates.OneWeek; + case R.id.menu_future_dates_14: + return MobileLedgerProfile.FutureDates.TwoWeeks; + case R.id.menu_future_dates_30: + return MobileLedgerProfile.FutureDates.OneMonth; + case R.id.menu_future_dates_60: + return MobileLedgerProfile.FutureDates.TwoMonths; + case R.id.menu_future_dates_90: + return MobileLedgerProfile.FutureDates.ThreeMonths; + case R.id.menu_future_dates_180: + return MobileLedgerProfile.FutureDates.SixMonths; + case R.id.menu_future_dates_365: + return MobileLedgerProfile.FutureDates.OneYear; + case R.id.menu_future_dates_all: + return MobileLedgerProfile.FutureDates.All; + default: + return MobileLedgerProfile.FutureDates.None; + } + } + @NotNull + private ProfileDetailModel getModel() { + return new ViewModelProvider(requireActivity()).get(ProfileDetailModel.class); + } + private void onSaveFabClicked() { + if (!checkValidity()) + return; + + ProfileDetailModel model = getModel(); + final ArrayList profiles = + Objects.requireNonNull(Data.profiles.getValue()); - i++; + if (mProfile != null) { + int pos = Data.profiles.getValue() + .indexOf(mProfile); + mProfile = new MobileLedgerProfile(mProfile); + model.updateProfile(mProfile); + mProfile.storeInDB(); + debug("profiles", "profile stored in DB"); + profiles.set(pos, mProfile); +// debug("profiles", String.format("Selected item is %d", mProfile.getThemeHue())); + + final MobileLedgerProfile currentProfile = Data.getProfile(); + if (mProfile.getUuid() + .equals(currentProfile.getUuid())) + { + Data.setCurrentProfile(mProfile); } + + ProfilesRecyclerViewAdapter viewAdapter = ProfilesRecyclerViewAdapter.getInstance(); + if (viewAdapter != null) + viewAdapter.notifyItemChanged(pos); } else { - profileName.setText(""); - url.setText(""); - postingPermitted.setChecked(true); - useAuthentication.setChecked(false); - authParams.setVisibility(View.GONE); - userName.setText(""); - password.setText(""); - colorSpinner.setSelection(0); + mProfile = new MobileLedgerProfile(String.valueOf(UUID.randomUUID())); + model.updateProfile(mProfile); + mProfile.storeInDB(); + final ArrayList newList = new ArrayList<>(profiles); + newList.add(mProfile); + Data.profiles.setValue(newList); + MobileLedgerProfile.storeProfilesOrder(); + + // first profile ever? + if (newList.size() == 1) + Data.setCurrentProfile(mProfile); } - return rootView; + Activity activity = getActivity(); + if (activity != null) + activity.finish(); + } + private boolean checkUrlValidity() { + boolean valid = true; + + ProfileDetailModel model = getModel(); + + String val = model.getUrl() + .trim(); + if (val.isEmpty()) { + valid = false; + urlLayout.setError(getResources().getText(R.string.err_profile_url_empty)); + } + try { + URL url = new URL(val); + String host = url.getHost(); + if (host == null || host.isEmpty()) + throw new MalformedURLException("Missing host"); + String protocol = url.getProtocol() + .toUpperCase(); + if (!protocol.equals("HTTP") && !protocol.equals("HTTPS")) { + valid = false; + urlLayout.setError(getResources().getText(R.string.err_invalid_url)); + } + } + catch (MalformedURLException e) { + valid = false; + urlLayout.setError(getResources().getText(R.string.err_invalid_url)); + } + + return valid; + } + private void checkInsecureSchemeWithAuth() { + boolean showWarning = false; + + final ProfileDetailModel model = getModel(); + + if (model.getUseAuthentication()) { + String urlText = model.getUrl(); + if (urlText.startsWith("http") && !urlText.startsWith("https")) + showWarning = true; + } + + if (showWarning) + insecureWarningText.setVisibility(View.VISIBLE); + else + insecureWarningText.setVisibility(View.GONE); } private void hookClearErrorOnFocusListener(TextView view, TextInputLayout layout) { view.setOnFocusChangeListener((v, hasFocus) -> { - if (hasFocus) layout.setError(null); + if (hasFocus) + layout.setError(null); }); view.addTextChangedListener(new TextWatcher() { @Override @@ -300,74 +547,72 @@ public class ProfileDetailFragment extends Fragment { } }); } - boolean checkValidity() { + private void syncModelFromUI() { + if (syncingModelFromUI) + return; + + syncingModelFromUI = true; + + try { + ProfileDetailModel model = getModel(); + + model.setProfileName(profileName.getText()); + model.setUrl(url.getText()); + model.setPreferredAccountsFilter(preferredAccountsFilter.getText()); + model.setAuthUserName(userName.getText()); + model.setAuthPassword(password.getText()); + } + finally { + syncingModelFromUI = false; + } + } + private boolean checkValidity() { boolean valid = true; String val = String.valueOf(profileName.getText()); - if (val.trim().isEmpty()) { + if (val.trim() + .isEmpty()) + { valid = false; profileNameLayout.setError(getResources().getText(R.string.err_profile_name_empty)); } - val = String.valueOf(url.getText()); - if (val.trim().isEmpty()) { + if (!checkUrlValidity()) valid = false; - urlLayout.setError(getResources().getText(R.string.err_profile_url_empty)); - } + if (useAuthentication.isChecked()) { val = String.valueOf(userName.getText()); - if (val.trim().isEmpty()) { + if (val.trim() + .isEmpty()) + { valid = false; - userNameLayout - .setError(getResources().getText(R.string.err_profile_user_name_empty)); + userNameLayout.setError( + getResources().getText(R.string.err_profile_user_name_empty)); } val = String.valueOf(password.getText()); - if (val.trim().isEmpty()) { + if (val.trim() + .isEmpty()) + { valid = false; - passwordLayout - .setError(getResources().getText(R.string.err_profile_password_empty)); + passwordLayout.setError( + getResources().getText(R.string.err_profile_password_empty)); } } return valid; } - private class ColorListAdapter extends ArrayAdapter { - public ColorListAdapter(@NonNull Context context, int resource) { - super(context, resource); - } - public ColorListAdapter(@NonNull Context context, int resource, int textViewResourceId) { - super(context, resource, textViewResourceId); - } - public ColorListAdapter(@NonNull Context context, int resource, @NonNull String[] objects) { - super(context, resource, objects); - } - public ColorListAdapter(@NonNull Context context, int resource, int textViewResourceId, - @NonNull String[] objects) { - super(context, resource, textViewResourceId, objects); - } - public ColorListAdapter(@NonNull Context context, int resource, - @NonNull List objects) { - super(context, resource, objects); - } - public ColorListAdapter(@NonNull Context context, int resource, int textViewResourceId, - @NonNull List objects) { - super(context, resource, textViewResourceId, objects); - } - @NotNull - @Override - public View getView(int position, View convertView, @NotNull ViewGroup parent) { - String hueStr = getItem(position); - int hue = (hueStr == null) ? -1 : Integer.valueOf(hueStr); - @ColorInt int primaryColor = Colors.getPrimaryColorForHue(hue); - - View view = convertView; - if (convertView == null) { - view = getLayoutInflater().inflate(R.layout.color_selector_item, parent); - } - - view.setBackgroundColor(primaryColor); - return view; - } + private void resetDefaultCommodity() { + defaultCommoditySet = false; + defaultCommodity.setText(R.string.btn_no_currency); + defaultCommodity.setTypeface(defaultCommodity.getTypeface(), Typeface.ITALIC); + } + private void setDefaultCommodity(@NonNull @NotNull String name) { + defaultCommoditySet = true; + defaultCommodity.setText(name); + defaultCommodity.setTypeface(Typeface.DEFAULT); + } + interface TextChangeSyncRoutine { + void onTextChanged(String text); } }