diff options
author | Samuel Holland <samuel@sholland.org> | 2017-08-24 22:13:46 -0500 |
---|---|---|
committer | Samuel Holland <samuel@sholland.org> | 2017-08-24 22:13:46 -0500 |
commit | a828e83399a1ea29341e10d498002216be54ca8c (patch) | |
tree | 1b6cebc7d02a84c9f9d5b25610ebbd8d702e8ade /app/src/main/java/com/wireguard/android/ConfigEditFragment.java | |
parent | ConfigActivity: Invert isSplitLayout→isSingleLayout for clarity (diff) | |
download | wireguard-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.java | 7 |
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; } |