aboutsummaryrefslogtreecommitdiffstatshomepage
path: root/app/src/main/java/com/wireguard/android/ConfigEditFragment.java
diff options
context:
space:
mode:
authorSamuel Holland <samuel@sholland.org>2017-08-24 22:13:46 -0500
committerSamuel Holland <samuel@sholland.org>2017-08-24 22:13:46 -0500
commita828e83399a1ea29341e10d498002216be54ca8c (patch)
tree1b6cebc7d02a84c9f9d5b25610ebbd8d702e8ade /app/src/main/java/com/wireguard/android/ConfigEditFragment.java
parentConfigActivity: Invert isSplitLayout→isSingleLayout for clarity (diff)
downloadwireguard-android-a828e83399a1ea29341e10d498002216be54ca8c.tar.xz
wireguard-android-a828e83399a1ea29341e10d498002216be54ca8c.zip
Centralize/unify validation of configurations
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'app/src/main/java/com/wireguard/android/ConfigEditFragment.java')
-rw-r--r--app/src/main/java/com/wireguard/android/ConfigEditFragment.java7
1 files changed, 1 insertions, 6 deletions
diff --git a/app/src/main/java/com/wireguard/android/ConfigEditFragment.java b/app/src/main/java/com/wireguard/android/ConfigEditFragment.java
index ebebef42..1edae9cc 100644
--- a/app/src/main/java/com/wireguard/android/ConfigEditFragment.java
+++ b/app/src/main/java/com/wireguard/android/ConfigEditFragment.java
@@ -113,18 +113,13 @@ public class ConfigEditFragment extends BaseConfigFragment {
}
private void saveConfig() {
- final String errorMessage = localConfig.validate();
final VpnService service = VpnService.getInstance();
- if (errorMessage != null) {
- Toast.makeText(getActivity(), errorMessage, Toast.LENGTH_SHORT).show();
- return;
- }
try {
if (getCurrentConfig() != null)
service.update(getCurrentConfig().getName(), localConfig);
else
service.add(localConfig);
- } catch (final IllegalStateException e) {
+ } catch (final IllegalArgumentException | IllegalStateException e) {
Toast.makeText(getActivity(), e.getMessage(), Toast.LENGTH_SHORT).show();
return;
}