From 6baf8163deb045f8a31fea7ad853b156502f018e Mon Sep 17 00:00:00 2001
From: Adithya Abraham Philip <adithyaphilip@gmail.com>
Date: Tue, 7 Jul 2015 23:31:34 +0530
Subject: linked proxy settings to other proxy checkbox

---
 .../keychain/ui/SettingsActivity.java              | 29 ++++++++++++++++------
 1 file changed, 22 insertions(+), 7 deletions(-)

(limited to 'OpenKeychain/src')

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SettingsActivity.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SettingsActivity.java
index f72a552d5..bcf42ba74 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SettingsActivity.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/SettingsActivity.java
@@ -277,8 +277,14 @@ public class SettingsActivity extends AppCompatPreferenceActivity {
                 initializeEditTextPreferences();
                 initializeProxyTypePreference();
 
-                if (mUseTor.isChecked()) disableNormalProxyPrefs();
-                else if (mUseNormalProxy.isChecked()) disableUseTorPrefs();
+                if (mUseTor.isChecked()) {
+                    disableNormalProxyPrefs();
+                }
+                else if (mUseNormalProxy.isChecked()) {
+                    disableUseTorPrefs();
+                } else {
+                    disableNormalProxySettings();
+                }
             }
 
             private void initializeUseTorPref() {
@@ -301,7 +307,7 @@ public class SettingsActivity extends AppCompatPreferenceActivity {
                             }
                         } else {
                             // we're unchecking Tor, so enable other proxy
-                            enableNormalProxyPrefs();
+                            enableNormalProxyCheckbox();
                             return true;
                         }
                     }
@@ -314,8 +320,10 @@ public class SettingsActivity extends AppCompatPreferenceActivity {
                     public boolean onPreferenceChange(Preference preference, Object newValue) {
                         if ((Boolean) newValue) {
                             disableUseTorPrefs();
+                            enableNormalProxySettings();
                         } else {
                             enableUseTorPrefs();
+                            disableNormalProxySettings();
                         }
                         return true;
                     }
@@ -389,18 +397,25 @@ public class SettingsActivity extends AppCompatPreferenceActivity {
             private void disableNormalProxyPrefs() {
                 mUseNormalProxy.setChecked(false);
                 mUseNormalProxy.setEnabled(false);
-                mProxyHost.setEnabled(false);
-                mProxyPort.setEnabled(false);
-                mProxyType.setEnabled(false);
+                disableNormalProxySettings();
             }
 
-            private void enableNormalProxyPrefs() {
+            private void enableNormalProxyCheckbox() {
                 mUseNormalProxy.setEnabled(true);
+            }
+
+            private void enableNormalProxySettings() {
                 mProxyHost.setEnabled(true);
                 mProxyPort.setEnabled(true);
                 mProxyType.setEnabled(true);
             }
 
+            private void disableNormalProxySettings() {
+                mProxyHost.setEnabled(false);
+                mProxyPort.setEnabled(false);
+                mProxyType.setEnabled(false);
+            }
+
             private void disableUseTorPrefs() {
                 mUseTor.setChecked(false);
                 mUseTor.setEnabled(false);
-- 
cgit v1.2.3