]> git.ktnx.net Git - mobile-ledger.git/blobdiff - app/src/main/java/net/ktnx/mobileledger/ui/profiles/ProfileDetailFragment.java
migrate a bunch of globals to LiveData
[mobile-ledger.git] / app / src / main / java / net / ktnx / mobileledger / ui / profiles / ProfileDetailFragment.java
index 91780c71b4dbec9a29d1e0f115a45c1c117e3053..0843dc29eb56ba145e46a7a93bb1b34348a33af3 100644 (file)
@@ -23,7 +23,6 @@ import android.content.DialogInterface;
 import android.os.Bundle;
 import android.text.Editable;
 import android.text.TextWatcher;
-import android.util.Log;
 import android.view.LayoutInflater;
 import android.view.Menu;
 import android.view.MenuInflater;
@@ -46,11 +45,17 @@ import net.ktnx.mobileledger.ui.HueRingDialog;
 import net.ktnx.mobileledger.ui.activity.ProfileDetailActivity;
 import net.ktnx.mobileledger.utils.Colors;
 
+import org.jetbrains.annotations.NotNull;
+
+import java.util.ArrayList;
 import java.util.Objects;
 
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.fragment.app.Fragment;
+import androidx.fragment.app.FragmentActivity;
+
+import static net.ktnx.mobileledger.utils.Logger.debug;
 
 /**
  * A fragment representing a single Profile detail screen.
@@ -90,8 +95,8 @@ public class ProfileDetailFragment extends Fragment implements HueRingDialog.Hue
     public ProfileDetailFragment() {
     }
     @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);
@@ -99,44 +104,59 @@ public class ProfileDetailFragment extends Fragment implements HueRingDialog.Hue
             AlertDialog.Builder builder = new AlertDialog.Builder(getContext());
             builder.setTitle(mProfile.getName());
             builder.setMessage(R.string.remove_profile_dialog_message);
-            builder.setPositiveButton(R.string.Remove, new DialogInterface.OnClickListener() {
-                @Override
-                public void onClick(DialogInterface dialog, int which) {
-                    Log.d("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));
-                    }
-                    getActivity().finish();
+            builder.setPositiveButton(R.string.Remove, (dialog, which) -> {
+                debug("profiles",
+                        String.format("[fragment] removing profile %s", mProfile.getUuid()));
+                mProfile.removeFromDB();
+                ArrayList<MobileLedgerProfile> oldList = Data.profiles.getValue();
+                assert oldList != null;
+                ArrayList<MobileLedgerProfile> newList =
+                        (ArrayList<MobileLedgerProfile>) oldList.clone();
+                newList.remove(mProfile);
+                Data.profiles.setValue(newList);
+                if (mProfile.equals(Data.profile.getValue())) {
+                    debug("profiles", "[fragment] setting current profile to 0");
+                    Data.setCurrentProfile(newList.get(0));
+                    final FragmentActivity activity = getActivity();
+                    if (activity != null) activity.finish();
                 }
             });
             builder.show();
             return false;
         });
-        menuDeleteProfile.setVisible((mProfile != null) && (Data.profiles.size() > 1));
+        final ArrayList<MobileLedgerProfile> profiles = Data.profiles.getValue();
+        menuDeleteProfile
+                .setVisible((mProfile != null) && (profiles != null) && (profiles.size() > 1));
 
         if (BuildConfig.DEBUG) {
             final MenuItem menuWipeProfileData = menu.findItem(R.id.menuWipeData);
-            menuWipeProfileData.setOnMenuItemClickListener(this::onWipeDataMenuClicked);
+            menuWipeProfileData.setOnMenuItemClickListener(ignored -> onWipeDataMenuClicked());
             menuWipeProfileData.setVisible(mProfile != null);
         }
     }
-    private boolean onWipeDataMenuClicked(MenuItem item) {
+    private boolean onWipeDataMenuClicked() {
         // this is a development option, so no confirmation
         mProfile.wipeAllData();
-        Data.profile.forceNotifyObservers();
+        if (mProfile.equals(Data.profile.getValue())) triggerProfileChange();
         return true;
     }
+    private void triggerProfileChange() {
+        int index = Data.getProfileIndex(mProfile);
+        MobileLedgerProfile newProfile = new MobileLedgerProfile(mProfile);
+        final ArrayList<MobileLedgerProfile> profiles = Data.profiles.getValue();
+        assert profiles != null;
+        profiles.set(index, newProfile);
+        if (mProfile.equals(Data.profile.getValue())) Data.profile.setValue(newProfile);
+    }
     @Override
     public void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
 
         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<MobileLedgerProfile> 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();
@@ -154,38 +174,38 @@ public class ProfileDetailFragment extends Fragment implements HueRingDialog.Hue
         if (context == null) return;
 
         FloatingActionButton fab = context.findViewById(R.id.fab);
-        fab.setOnClickListener(v -> {
-            if (!checkValidity()) return;
-
-            if (mProfile != null) {
-                updateProfileFromUI();
-//                Log.d("profiles", String.format("Selected item is %d", mProfile.getThemeId()));
-                mProfile.storeInDB();
-                Log.d("profiles", "profile stored in DB");
-                Data.profiles.triggerItemChangedNotification(mProfile);
+        fab.setOnClickListener(v -> onSaveFabClicked());
 
+        profileName.requestFocus();
+    }
+    private void onSaveFabClicked() {
+        if (!checkValidity()) return;
 
-                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();
-                updateProfileFromUI();
-                mProfile.storeInDB();
-                Data.profiles.add(mProfile);
-                MobileLedgerProfile.storeProfilesOrder();
-
-                // first profile ever?
-                if (Data.profiles.size() == 1) Data.profile.set(mProfile);
-            }
+        if (mProfile != null) {
+            updateProfileFromUI();
+//                debug("profiles", String.format("Selected item is %d", mProfile.getThemeId()));
+            mProfile.storeInDB();
+            debug("profiles", "profile stored in DB");
+            triggerProfileChange();
+        }
+        else {
+            mProfile = new MobileLedgerProfile();
+            updateProfileFromUI();
+            mProfile.storeInDB();
+            final ArrayList<MobileLedgerProfile> profiles = Data.profiles.getValue();
+            assert profiles != null;
+            ArrayList<MobileLedgerProfile> newList =
+                    (ArrayList<MobileLedgerProfile>) profiles.clone();
+            newList.add(mProfile);
+            Data.profiles.setValue(newList);
+            MobileLedgerProfile.storeProfilesOrder();
 
-            Activity activity = getActivity();
-            if (activity != null) activity.finish();
-        });
+            // first profile ever?
+            if (newList.size() == 1) Data.profile.setValue(mProfile);
+        }
 
-        profileName.requestFocus();
+        Activity activity = getActivity();
+        if (activity != null) activity.finish();
     }
     private void updateProfileFromUI() {
         mProfile.setName(profileName.getText());
@@ -219,7 +239,7 @@ public class ProfileDetailFragment extends Fragment implements HueRingDialog.Hue
                 rootView.findViewById(R.id.preferred_accounts_accounts_filter_layout);
 
         useAuthentication.setOnCheckedChangeListener((buttonView, isChecked) -> {
-            Log.d("profiles", isChecked ? "auth enabled " : "auth disabled");
+            debug("profiles", isChecked ? "auth enabled " : "auth disabled");
             authParams.setVisibility(isChecked ? View.VISIBLE : View.GONE);
             if (isChecked) userName.requestFocus();
         });