diff options
author | Fup Duck <fupduck@sacknagel.com> | 2018-01-27 16:09:38 +0100 |
---|---|---|
committer | Fup Duck <fupduck@sacknagel.com> | 2018-01-27 16:09:38 +0100 |
commit | 3eb5405141d14a0c75c669c168b272878346cc38 (patch) | |
tree | f4c8f90a4cfea9c7518d57d2e45282e3fc9ec8d5 /app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java | |
parent | d8aaf3ca143a5afb71fb74675deb5be60a4204d9 (diff) |
fix comments for mergerequest
Diffstat (limited to 'app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java')
-rw-r--r-- | app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java b/app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java index f93fb36c..3713e198 100644 --- a/app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java +++ b/app/src/main/java/se/leap/bitmaskclient/ConfigHelper.java @@ -285,12 +285,12 @@ public class ConfigHelper { return getProviderName(null, provider); } - public static String getProviderName(SharedPreferences preferences) { + public static String getProviderName(@Nullable SharedPreferences preferences) { return getProviderName(preferences,null); } - public static String getProviderName(SharedPreferences preferences, @Nullable String provider) { - if (provider == null) { + public static String getProviderName(@Nullable SharedPreferences preferences, @Nullable String provider) { + if (provider == null && preferences != null) { provider = preferences.getString(Provider.KEY, ""); } try { @@ -315,8 +315,8 @@ public class ConfigHelper { return getProviderDomain(null, provider); } - public static String getProviderDomain(SharedPreferences preferences, @Nullable String provider) { - if (provider == null) { + public static String getProviderDomain(@Nullable SharedPreferences preferences, @Nullable String provider) { + if (provider == null && preferences != null) { provider = preferences.getString(Provider.KEY, ""); } try { |