draft gateway selection UI according to simlpy secure's proposals

1 files changed, 61 insertions, 97 deletions

diff --git a/app/src/main/java/se/leap/bitmaskclient/base/fragments/GatewaySelectionFragment.java b/app/src/main/java/se/leap/bitmaskclient/base/fragments/GatewaySelectionFragment.java
index e4d8ca8b..ee4aea74 100644
--- a/app/src/main/java/se/leap/bitmaskclient/base/fragments/GatewaySelectionFragment.java
+++ b/app/src/main/java/se/leap/bitmaskclient/base/fragments/GatewaySelectionFragment.java
@@ -19,7 +19,6 @@ package se.leap.bitmaskclient.base.fragments;
 import android.app.Activity;
 import android.content.Intent;
 import android.content.SharedPreferences;
-import android.graphics.drawable.Drawable;
 import android.os.Bundle;
 import android.util.Log;
 import android.view.LayoutInflater;
@@ -29,23 +28,21 @@ import android.view.ViewGroup;
 import androidx.annotation.NonNull;
 import androidx.annotation.Nullable;
 import androidx.appcompat.widget.AppCompatButton;
-import androidx.appcompat.widget.AppCompatImageView;
 import androidx.appcompat.widget.AppCompatTextView;
-import androidx.core.content.ContextCompat;
-import androidx.core.graphics.drawable.DrawableCompat;
 import androidx.fragment.app.Fragment;
 import androidx.recyclerview.widget.LinearLayoutManager;
 import androidx.recyclerview.widget.RecyclerView;
 
+import java.lang.ref.WeakReference;
 import java.util.List;
 import java.util.Observable;
 import java.util.Observer;
 
+import de.blinkt.openvpn.core.VpnStatus;
 import se.leap.bitmaskclient.R;
 import se.leap.bitmaskclient.base.MainActivity;
 import se.leap.bitmaskclient.base.models.Location;
 import se.leap.bitmaskclient.base.utils.PreferenceHelper;
-import se.leap.bitmaskclient.base.views.IconSwitchEntry;
 import se.leap.bitmaskclient.base.views.LocationIndicator;
 import se.leap.bitmaskclient.eip.EipCommand;
 import se.leap.bitmaskclient.eip.EipStatus;
@@ -55,21 +52,23 @@ import static android.content.Context.MODE_PRIVATE;
 import static android.view.View.INVISIBLE;
 import static android.view.View.VISIBLE;
 import static se.leap.bitmaskclient.base.MainActivity.ACTION_SHOW_VPN_FRAGMENT;
-import static se.leap.bitmaskclient.base.models.Constants.PREFERRED_CITY;
 import static se.leap.bitmaskclient.base.models.Constants.SHARED_PREFERENCES;
 import static se.leap.bitmaskclient.base.models.Constants.USE_BRIDGES;
-import static se.leap.bitmaskclient.base.utils.PreferenceHelper.getPreferredCity;
-import static se.leap.bitmaskclient.base.utils.PreferenceHelper.setPreferredCity;
 
-public class GatewaySelectionFragment extends Fragment implements SharedPreferences.OnSharedPreferenceChangeListener, Observer {
+interface LocationListSelectionListener {
+    void onLocationSelected(String name);
+}
+
+public class GatewaySelectionFragment extends Fragment implements SharedPreferences.OnSharedPreferenceChangeListener, Observer, LocationListSelectionListener {
 
     private static final String TAG = GatewaySelectionFragment.class.getSimpleName();
 
 
     private RecyclerView recyclerView;
     private LocationListAdapter locationListAdapter;
-    private IconSwitchEntry autoSelectionSwitch;
-    private AppCompatButton vpnButton;
+    private AppCompatTextView currentLocationDescription;
+    private AppCompatTextView currentLocation;
+    private AppCompatButton autoSelectionButton;
     private GatewaysManager gatewaysManager;
     private SharedPreferences preferences;
     private EipStatus eipStatus;
@@ -97,8 +96,8 @@ public class GatewaySelectionFragment extends Fragment implements SharedPreferen
     public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
         super.onViewCreated(view, savedInstanceState);
         initRecyclerView();
-        initAutoSelectionSwitch();
-        initVpnButton();
+        initAutoSelectionButton();
+        initCurrentLocationInfoPanel();
         eipStatus.addObserver(this);
         preferences.registerOnSharedPreferenceChangeListener(this);
     }
@@ -113,93 +112,94 @@ public class GatewaySelectionFragment extends Fragment implements SharedPreferen
 
 
     private void initRecyclerView() {
-        recyclerView = (RecyclerView) getActivity().findViewById(R.id.gatewaySelection_list);
+        recyclerView = getActivity().findViewById(R.id.gatewaySelection_list);
         recyclerView.setHasFixedSize(true);
         LinearLayoutManager layoutManager = new LinearLayoutManager(this.getContext());
         recyclerView.setLayoutManager(layoutManager);
-        locationListAdapter = new LocationListAdapter(gatewaysManager.getGatewayLocations());
+        locationListAdapter = new LocationListAdapter(gatewaysManager.getGatewayLocations(), this);
         recyclerView.setAdapter(locationListAdapter);
-        recyclerView.setVisibility(getPreferredCity(getContext()) == null ? INVISIBLE : VISIBLE);
+        recyclerView.setVisibility(VISIBLE);
     }
 
-    private void initAutoSelectionSwitch() {
-        autoSelectionSwitch = getActivity().findViewById(R.id.automatic_gateway_switch);
-        autoSelectionSwitch.setSingleLine(false);
-        autoSelectionSwitch.setSubtitle(getString(R.string.gateway_selection_warning, getString(R.string.app_name)));
-        autoSelectionSwitch.setChecked(getPreferredCity(getContext()) == null);
-        autoSelectionSwitch.setOnCheckedChangeListener((buttonView, isChecked) -> {
-            recyclerView.setVisibility(!isChecked ? VISIBLE : View.GONE);
-            Log.d(TAG, "autoselection enabled: " + isChecked);
-            if (isChecked) {
-                PreferenceHelper.setPreferredCity(getContext(), null);
-                locationListAdapter.resetSelection();
-            }
-            setVpnButtonState();
+    private void initAutoSelectionButton() {
+        autoSelectionButton = getActivity().findViewById(R.id.automatic_gateway_selection_btn);
+        autoSelectionButton.setOnClickListener(v -> {
+            startEipService(null);
         });
     }
 
-    private void initVpnButton() {
-        vpnButton = getActivity().findViewById(R.id.vpn_button);
-        setVpnButtonState();
-        vpnButton.setOnClickListener(v -> {
+    private void initCurrentLocationInfoPanel() {
+        currentLocationDescription = getActivity().findViewById(R.id.current_location_description);
+        currentLocation = getActivity().findViewById(R.id.current_location);
+        setLocationDescription(EipStatus.getInstance(), PreferenceHelper.getPreferredCity(getContext()));
+    }
+
+    private void setLocationDescription(EipStatus eipStatus, String preferredCity) {
+        if (eipStatus.isConnected()) {
+            currentLocationDescription.setText(preferredCity == null ?
+                    R.string.gateway_selection_automatic_location :
+                    R.string.gateway_selection_manual_location);
+            currentLocation.setText(VpnStatus.getLastConnectedVpnName());
+            currentLocation.setVisibility(VISIBLE);
+        } else if (preferredCity == null) {
+            currentLocationDescription.setText(R.string.gateway_selection_automatic_not_connected);
+            currentLocation.setVisibility(INVISIBLE);
+        } else {
+            currentLocationDescription.setText(R.string.gateway_selection_manual_not_connected);
+            currentLocation.setText(preferredCity);
+            currentLocation.setVisibility(VISIBLE);
+        }
+    }
+
+    protected void startEipService(String preferredCity) {
+        new Thread(() -> {
+            PreferenceHelper.setPreferredCity(getContext(), preferredCity);
             EipCommand.startVPN(getContext(), false);
             Intent intent = new Intent(getContext(), MainActivity.class);
             intent.setFlags(Intent.FLAG_ACTIVITY_SINGLE_TOP | Intent.FLAG_ACTIVITY_CLEAR_TOP);
             intent.setAction(ACTION_SHOW_VPN_FRAGMENT);
             startActivity(intent);
-        });
-    }
-
-    private void setVpnButtonState() {
-        if (eipStatus.isDisconnected()) {
-            vpnButton.setText(R.string.vpn_button_turn_on);
-        } else {
-            vpnButton.setText(R.string.reconnect);
-        }
-        vpnButton.setEnabled(
-                (locationListAdapter.selectedLocation != null && locationListAdapter.selectedLocation.selected) ||
-                        autoSelectionSwitch.isChecked());
+        }).start();
     }
 
     @Override
     public void onSharedPreferenceChanged(SharedPreferences sharedPreferences, String key) {
         if (USE_BRIDGES.equals(key)) {
             locationListAdapter.updateData(gatewaysManager.getGatewayLocations());
-            setVpnButtonState();
-        } else if (PREFERRED_CITY.equals(key)) {
-            setVpnButtonState();
         }
     }
 
     @Override
+    public void onLocationSelected(String name) {
+        startEipService(name);
+    }
+
+    @Override
     public void update(Observable o, Object arg) {
         if (o instanceof EipStatus) {
-            eipStatus = (EipStatus) o;
             Activity activity = getActivity();
             if (activity != null) {
-                activity.runOnUiThread(this::setVpnButtonState);
+                activity.runOnUiThread(() -> setLocationDescription((EipStatus) o, PreferenceHelper.getPreferredCity(getContext())));
             }
         }
-
     }
 
+
     static class LocationListAdapter extends RecyclerView.Adapter<LocationListAdapter.ViewHolder> {
         private static final String TAG = LocationListAdapter.class.getSimpleName();
         private List<Location> values;
-        private Location selectedLocation = null;
+        private final WeakReference<LocationListSelectionListener> callback;
 
         static class ViewHolder extends RecyclerView.ViewHolder {
             public AppCompatTextView locationLabel;
             public LocationIndicator locationIndicator;
-            public AppCompatImageView checkedIcon;
             public View layout;
 
             public ViewHolder(View v) {
                 super(v);
                 layout = v;
-                locationLabel = (AppCompatTextView) v.findViewById(R.id.location);
-                locationIndicator = (LocationIndicator) v.findViewById(R.id.quality);
-                checkedIcon = (AppCompatImageView) v.findViewById(R.id.checked_icon);
+                locationLabel = v.findViewById(R.id.location);
+                locationIndicator = v.findViewById(R.id.quality);
             }
         }
 
@@ -213,20 +213,14 @@ public class GatewaySelectionFragment extends Fragment implements SharedPreferen
             notifyItemRemoved(position);
         }
 
-        public void resetSelection() {
-            if (selectedLocation != null) {
-                selectedLocation.selected = false;
-                notifyDataSetChanged();
-            }
-        }
-
         public void updateData(List<Location> data) {
             values = data;
             notifyDataSetChanged();
         }
 
-        public LocationListAdapter(List<Location> data) {
+        public LocationListAdapter(List<Location> data, LocationListSelectionListener selectionListener) {
             values = data;
+            callback = new WeakReference<>(selectionListener);
         }
 
         @NonNull
@@ -246,42 +240,12 @@ public class GatewaySelectionFragment extends Fragment implements SharedPreferen
             holder.locationLabel.setText(location.name);
             holder.layout.setOnClickListener(v -> {
                 Log.d(TAG, "view at position clicked: " + position);
-                if (selectedLocation == null) {
-                    selectedLocation = location;
-                    selectedLocation.selected = true;
-                } else if (selectedLocation.name.equals(location.name)){
-                    selectedLocation.selected = !selectedLocation.selected;
-                } else {
-                    selectedLocation.selected = false;
-                    selectedLocation = location;
-                    selectedLocation.selected = true;
+                LocationListSelectionListener listener = callback.get();
+                if (listener != null) {
+                    listener.onLocationSelected(location.name);
                 }
-                setPreferredCity(holder.layout.getContext(), selectedLocation.selected ? selectedLocation.name : null);
-                holder.checkedIcon.setVisibility(selectedLocation.selected ? VISIBLE : INVISIBLE);
-                notifyDataSetChanged();
             });
-            Drawable checkIcon = DrawableCompat.wrap(holder.layout.getContext().getResources().getDrawable(R.drawable.ic_check_bold)).mutate();
-            DrawableCompat.setTint(checkIcon, ContextCompat.getColor(holder.layout.getContext(), R.color.colorSuccess));
-            holder.checkedIcon.setImageDrawable(checkIcon);
-            holder.checkedIcon.setVisibility(location.selected ? VISIBLE : INVISIBLE);
             holder.locationIndicator.setLoad(GatewaysManager.Load.getLoadByValue(location.averageLoad));
-            if (location.selected) {
-                selectedLocation = location;
-            }
-        }
-
-        public String getQualityString(double quality) {
-            if (quality == 0) {
-                return "";
-            } else if (quality < 0.25) {
-                return "BAD";
-            } else if (quality < 0.6) {
-                return "AVERAGE";
-            } else if (quality < 0.8){
-                return "GOOD";
-            } else {
-                return "EXCELLENT";
-            }
         }