commit 9974654c0888694ca187209da782ffc83f479c49
Author: Nathan Freitas <[email protected]>
Date: Thu Apr 9 01:28:38 2015 -0400
fix for setting root and transproxy preferences
---
res/values/strings.xml | 2 +-
res/xml/preferences.xml | 4 ++++
.../android/settings/SettingsPreferences.java | 24 +++++---------------
3 files changed, 11 insertions(+), 19 deletions(-)
diff --git a/res/values/strings.xml b/res/values/strings.xml
index ede9174..0318cac 100644
--- a/res/values/strings.xml
+++ b/res/values/strings.xml
@@ -107,7 +107,7 @@
<string name="wizard_permissions_title">Permissions</string>
<string name="wizard_permissions_root_msg1">You can optionally grant Orbot
\'Superuser\' access to enable advanced features, such as Transparent
Proxying.</string>
<string name="wizard_permissions_root_msg2">If you do not want to do this,
please make sure to use apps made to work with Orbot</string>
- <string name="wizard_permissions_no_root_msg">Your device does not appear to
be rooted or provide \'Superuser\' access.\n\nIn order to you to benefit from
Tor, you will need to use apps built to work with Orbot, or that support HTTP
or SOCKS proxy settings.\n\n</string>
+ <string name="wizard_permissions_no_root_msg">Your device does not appear to
be rooted or provide root or \'Superuser\' access.\n\nPlease try the \'Apps\'
mode on the main screen instead.</string>
<!--TipsAndTricks screen-->
<string name="wizard_tips_title">Orbot-Enabled Apps</string>
<string name="wizard_tips_gibberbot">ChatSecure: Secure chat app with
Off-the-Record Encryption</string>
diff --git a/res/xml/preferences.xml b/res/xml/preferences.xml
index eabc9b3..d5a8a75 100644
--- a/res/xml/preferences.xml
+++ b/res/xml/preferences.xml
@@ -47,6 +47,7 @@ android:defaultValue="false"
android:key="pref_transparent"
android:title="@string/pref_trans_proxy_title"
android:summary="@string/pref_trans_proxy_summary"
+android:dependency="has_root"
android:enabled="true"/>
<CheckBoxPreference
@@ -54,6 +55,7 @@ android:defaultValue="false"
android:key="pref_transparent_all"
android:summary="@string/pref_transparent_all_summary"
android:enabled="true"
+android:dependency="has_root"
android:title="@string/pref_transparent_all_title"/>
<Preference
@@ -61,6 +63,7 @@ android:defaultValue=""
android:key="pref_transparent_app_list"
android:title="@string/pref_select_apps"
android:summary="@string/pref_select_apps_summary"
+android:dependency="has_root"
android:enabled="true"/>
<CheckBoxPreference
@@ -68,6 +71,7 @@ android:defaultValue="false"
android:key="pref_transparent_tethering"
android:summary="@string/pref_transparent_tethering_summary"
android:enabled="true"
+android:dependency="has_root"
android:title="@string/pref_transparent_tethering_title"/>
diff --git a/src/org/torproject/android/settings/SettingsPreferences.java
b/src/org/torproject/android/settings/SettingsPreferences.java
index 0e7f837..147f84d 100644
--- a/src/org/torproject/android/settings/SettingsPreferences.java
+++ b/src/org/torproject/android/settings/SettingsPreferences.java
@@ -64,19 +64,18 @@ public class SettingsPreferences
private void init ()
{
- int REQUEST_ROOT_IDX = 1;
int SET_LOCALE_IDX = 3;
int GENERAL_GROUP_IDX = 0;
- prefRequestRoot =
((CheckBoxPreference)((PreferenceCategory)getPreferenceScreen().getPreference(GENERAL_GROUP_IDX)).getPreference(REQUEST_ROOT_IDX));
+ prefRequestRoot =
((CheckBoxPreference)((PreferenceCategory)getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(0));
prefRequestRoot.setOnPreferenceClickListener(this);
prefLocale =
(((PreferenceCategory)getPreferenceScreen().getPreference(GENERAL_GROUP_IDX)).getPreference(SET_LOCALE_IDX));
prefLocale.setOnPreferenceClickListener(this);
- prefCBTransProxy =
((CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(0));
- prefcBTransProxyAll =
(CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(1);
+ prefCBTransProxy =
((CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(1));
+ prefcBTransProxyAll =
(CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(2);
prefTransProxyFlush =
(Preference)((PreferenceCategory)this.getPreferenceScreen().getPreference(DEBUG_GROUP_IDX)).getPreference(8);
prefTransProxyFlush.setOnPreferenceClickListener(new
OnPreferenceClickListener() {
@@ -94,24 +93,15 @@ public class SettingsPreferences
});
- prefTransProxyApps =
((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(2);
+ prefTransProxyApps =
((PreferenceCategory)this.getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX)).getPreference(3);
prefCBTransProxy.setOnPreferenceClickListener(this);
prefcBTransProxyAll.setOnPreferenceClickListener(this);
prefTransProxyApps.setOnPreferenceClickListener(this);
- if (!hasRoot)
- {
-
getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX).setEnabled(false);
- }
- else
- {
-
-
prefcBTransProxyAll.setEnabled(prefCBTransProxy.isChecked());
-
prefTransProxyApps.setEnabled(prefCBTransProxy.isChecked() &&
(!prefcBTransProxyAll.isChecked()));
-
- }
+ prefcBTransProxyAll.setEnabled(prefCBTransProxy.isChecked());
+ prefTransProxyApps.setEnabled(prefCBTransProxy.isChecked() &&
(!prefcBTransProxyAll.isChecked()));
prefHiddenServices =
((CheckBoxPreference)((PreferenceCategory)this.getPreferenceScreen().getPreference(HIDDEN_SERVICE_PREF_IDX)).getPreference(0));
@@ -154,12 +144,10 @@ public class SettingsPreferences
if (preference == prefRequestRoot)
{
-
if (prefRequestRoot.isChecked())
{
boolean canRoot =
RootCommands.rootAccessGiven();
-
getPreferenceScreen().getPreference(TRANSPROXY_GROUP_IDX).setEnabled(canRoot);
prefRequestRoot.setChecked(canRoot);
if (!canRoot)
_______________________________________________
tor-commits mailing list
[email protected]
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits
