From addf8d89962bf3de6d70330f9264d0e4d866613e Mon Sep 17 00:00:00 2001
From: cyberta <cyberta@riseup.net>
Date: Sat, 29 Jul 2023 17:15:05 +0200
Subject: update design and UX for provider setup

---
 .../bitmaskclient/providersetup/ProviderManager.java   | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

(limited to 'app/src/main/java/se/leap/bitmaskclient/providersetup/ProviderManager.java')

diff --git a/app/src/main/java/se/leap/bitmaskclient/providersetup/ProviderManager.java b/app/src/main/java/se/leap/bitmaskclient/providersetup/ProviderManager.java
index 775e174a..0f6c5090 100644
--- a/app/src/main/java/se/leap/bitmaskclient/providersetup/ProviderManager.java
+++ b/app/src/main/java/se/leap/bitmaskclient/providersetup/ProviderManager.java
@@ -41,7 +41,7 @@ import se.leap.bitmaskclient.base.models.Provider;
  */
 public class ProviderManager implements AdapteeCollection<Provider> {
 
-    private AssetManager assetsManager;
+    private final AssetManager assetsManager;
     private File externalFilesDir;
     private Set<Provider> defaultProviders;
     private Set<Provider> customProviders;
@@ -49,6 +49,7 @@ public class ProviderManager implements AdapteeCollection<Provider> {
     private Set<String> customProviderURLs;
 
     private static ProviderManager instance;
+    private boolean addDummyEntry = false;
 
     public static ProviderManager getInstance(AssetManager assetsManager, File externalFilesDir) {
         if (instance == null)
@@ -62,6 +63,10 @@ public class ProviderManager implements AdapteeCollection<Provider> {
         instance = null;
     }
 
+    public void setAddDummyEntry(boolean addDummyEntry) {
+        this.addDummyEntry = addDummyEntry;
+    }
+
     private ProviderManager(AssetManager assetManager, File externalFilesDir) {
         this.assetsManager = assetManager;
         addDefaultProviders(assetManager);
@@ -145,13 +150,18 @@ public class ProviderManager implements AdapteeCollection<Provider> {
     }
 
     public List<Provider> providers() {
+       return providers(addDummyEntry);
+    }
+
+    private List<Provider> providers(boolean addEmptyProvider) {
         List<Provider> allProviders = new ArrayList<>();
         allProviders.addAll(defaultProviders);
         if(customProviders != null)
             allProviders.addAll(customProviders);
-        //add an option to add a custom provider
-        //TODO: refactor me?
-        allProviders.add(new Provider());
+        if (addEmptyProvider) {
+            //add an option to add a custom provider
+            allProviders.add(new Provider());
+        }
         return allProviders;
     }
 
-- 
cgit v1.2.3