diff options
author | Samuel Holland <samuel@sholland.org> | 2017-08-15 17:36:11 -0500 |
---|---|---|
committer | Samuel Holland <samuel@sholland.org> | 2017-08-15 17:36:11 -0500 |
commit | 61d6b89bbeeefd56af4886579db741ece0d88dc8 (patch) | |
tree | fad8e18ef9848df131c87261e72f8c6a3f761581 /app/src/main/java/com/wireguard | |
parent | e9de916d69acf4b9242d3561403bb25e995cc4b7 (diff) |
ConfigAddActivity: Add an standalone config-creating activity
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
Diffstat (limited to 'app/src/main/java/com/wireguard')
-rw-r--r-- | app/src/main/java/com/wireguard/android/ConfigAddActivity.java | 37 | ||||
-rw-r--r-- | app/src/main/java/com/wireguard/android/ConfigEditFragment.java | 13 |
2 files changed, 48 insertions, 2 deletions
diff --git a/app/src/main/java/com/wireguard/android/ConfigAddActivity.java b/app/src/main/java/com/wireguard/android/ConfigAddActivity.java new file mode 100644 index 00000000..f299a67c --- /dev/null +++ b/app/src/main/java/com/wireguard/android/ConfigAddActivity.java @@ -0,0 +1,37 @@ +package com.wireguard.android; + +import android.app.FragmentManager; +import android.app.FragmentTransaction; +import android.os.Bundle; + +import com.wireguard.config.Config; + +/** + * Standalone activity for creating configurations. + */ + +public class ConfigAddActivity extends BaseConfigActivity { + @Override + public void onCreate(final Bundle savedInstanceState) { + super.onCreate(savedInstanceState); + setContentView(R.layout.config_add_activity); + } + + @Override + protected void onCurrentConfigChanged(final Config config) { + // This is the result of ConfigEditFragment signalling that a configuration was created. + if (config != null) + finish(); + } + + @Override + protected void onServiceAvailable() { + super.onServiceAvailable(); + final FragmentManager fm = getFragmentManager(); + if (fm.findFragmentById(R.id.master_fragment) == null) { + final FragmentTransaction transaction = fm.beginTransaction(); + transaction.add(R.id.master_fragment, new ConfigEditFragment()); + transaction.commit(); + } + } +} diff --git a/app/src/main/java/com/wireguard/android/ConfigEditFragment.java b/app/src/main/java/com/wireguard/android/ConfigEditFragment.java index 488d7915..7cbcfddd 100644 --- a/app/src/main/java/com/wireguard/android/ConfigEditFragment.java +++ b/app/src/main/java/com/wireguard/android/ConfigEditFragment.java @@ -12,6 +12,7 @@ import android.view.View; import android.view.ViewGroup; import android.view.inputmethod.InputMethodManager; import android.widget.EditText; +import android.widget.Toast; import com.wireguard.android.databinding.ConfigEditFragmentBinding; import com.wireguard.config.Config; @@ -73,7 +74,15 @@ public class ConfigEditFragment extends BaseConfigFragment { private void saveConfig() { // FIXME: validate input - VpnService.getInstance().update(getCurrentConfig().getName(), localConfig); + try { + if (getCurrentConfig() != null) + VpnService.getInstance().update(getCurrentConfig().getName(), localConfig); + else + VpnService.getInstance().add(localConfig); + } catch (final IllegalStateException e) { + Toast.makeText(getActivity(), e.getMessage(), Toast.LENGTH_SHORT).show(); + return; + } // Hide the keyboard; it rarely goes away on its own. final BaseConfigActivity activity = (BaseConfigActivity) getActivity(); final View focusedView = activity.getCurrentFocus(); @@ -83,7 +92,7 @@ public class ConfigEditFragment extends BaseConfigFragment { inputManager.hideSoftInputFromWindow(focusedView.getWindowToken(), InputMethodManager.HIDE_NOT_ALWAYS); } - // Tell the activity to go back to the detail view. + // Tell the activity to finish itself or go back to the detail view. activity.setCurrentConfig(localConfig); } } |