New provider dialog remembers previous data.

It also (supposedly, I didn't test because Bitmask is working ok from our client point of view) distinguishes between cancelling a new custom provider and cancelling a preseeded provider.

1 files changed, 27 insertions, 23 deletions

diff --git a/src/se/leap/bitmaskclient/DownloadFailedDialog.java b/src/se/leap/bitmaskclient/DownloadFailedDialog.java
index 4930da95..f78002b0 100644
--- a/src/se/leap/bitmaskclient/DownloadFailedDialog.java
+++ b/src/se/leap/bitmaskclient/DownloadFailedDialog.java
@@ -14,7 +14,7 @@
  * You should have received a copy of the GNU General Public License
  * along with this program. If not, see <http://www.gnu.org/licenses/>.
  */
- package se.leap.bitmaskclient;
+package se.leap.bitmaskclient;
 
 import se.leap.bitmaskclient.R;
 import se.leap.bitmaskclient.NewProviderDialog.NewProviderDialogInterface;
@@ -45,46 +45,50 @@ public class DownloadFailedDialog extends DialogFragment {
 		return dialog_fragment;
 	}
 
-    @Override
+	@Override
 	public Dialog onCreateDialog(Bundle savedInstanceState) {
 		AlertDialog.Builder builder = new AlertDialog.Builder(getActivity());
-		
+
 		builder.setMessage(reason_to_fail)
 		.setPositiveButton(R.string.retry, new DialogInterface.OnClickListener() {
 			public void onClick(DialogInterface dialog, int id) {
-				dialog.dismiss();
+				dismiss();
 				interface_with_ConfigurationWizard.retrySetUpProvider();
 			}
 		})
 		.setNegativeButton(R.string.cancel, new DialogInterface.OnClickListener() {
 			public void onClick(DialogInterface dialog, int id) {
-				ConfigHelper.removeFromSharedPref(Provider.KEY);
-				ConfigHelper.removeFromSharedPref(ProviderItem.DANGER_ON);
-				ConfigHelper.removeFromSharedPref(EIP.ALLOWED_ANON);
-				ConfigHelper.removeFromSharedPref(EIP.KEY);
-				interface_with_ConfigurationWizard.retrySetUpProvider();
+				interface_with_ConfigurationWizard.cancelSettingUpProvider();
 				dialog.dismiss();
 			}
 		});
-			
+
 		// Create the AlertDialog object and return it
 		return builder.create();
 	}
-    
+
 	public interface DownloadFailedDialogInterface {
-        public void retrySetUpProvider();
-    }
+		public void retrySetUpProvider();
+		public void cancelSettingUpProvider();
+	}
 
 	DownloadFailedDialogInterface interface_with_ConfigurationWizard;
 
-    @Override
-    public void onAttach(Activity activity) {
-        super.onAttach(activity);
-        try {
-        	interface_with_ConfigurationWizard = (DownloadFailedDialogInterface) activity;
-        } catch (ClassCastException e) {
-            throw new ClassCastException(activity.toString()
-                    + " must implement NoticeDialogListener");
-        }
-    }
+	@Override
+	public void onAttach(Activity activity) {
+		super.onAttach(activity);
+		try {
+			interface_with_ConfigurationWizard = (DownloadFailedDialogInterface) activity;
+		} catch (ClassCastException e) {
+			throw new ClassCastException(activity.toString()
+					+ " must implement NoticeDialogListener");
+		}
+	}
+
+	@Override
+	public void onCancel(DialogInterface dialog) {
+		interface_with_ConfigurationWizard.cancelSettingUpProvider();
+		dialog.dismiss();
+	}
+
 }