From 38907795de2f2903f6e2349d052a8184b538c53d Mon Sep 17 00:00:00 2001
From: Samuel Holland <samuel@sholland.org>
Date: Tue, 15 Aug 2017 17:36:11 -0500
Subject: ConfigAddActivity: Add an standalone config-creating activity

---
 .../com/wireguard/android/ConfigAddActivity.java   | 37 ++++++++++++++++++++++
 .../com/wireguard/android/ConfigEditFragment.java  | 13 ++++++--
 2 files changed, 48 insertions(+), 2 deletions(-)
 create mode 100644 app/src/main/java/com/wireguard/android/ConfigAddActivity.java

(limited to 'app/src/main/java/com/wireguard')

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);
     }
 }
-- 
cgit v1.2.3