summaryrefslogtreecommitdiff
path: root/packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java
diff options
context:
space:
mode:
authorSudheer Shanka <sudheersai@google.com>2016-03-14 14:59:52 -0700
committerSudheer Shanka <sudheersai@google.com>2016-03-14 14:59:52 -0700
commit2893302afc4137e63c9174a5d471d1f18b8b5d87 (patch)
treed83acf504b60dba50645df7f4e24b45b4a1bc8f3 /packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java
parent76001741814f16522c321108f7ef89965389cbe0 (diff)
Call preference.setEnabled only after updating disabledByAdmin state.
We check the value of disabledByAdmin state in setEnabled, so update it first before calling setEnabled. Bug: 27642236 Change-Id: Ie6c805b85a3afb87ffdaad0b80dbadc172b62d49
Diffstat (limited to 'packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java')
-rw-r--r--packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java7
1 files changed, 4 insertions, 3 deletions
diff --git a/packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java b/packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java
index 47023c108eab..d0aba227baa5 100644
--- a/packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java
+++ b/packages/SettingsLib/src/com/android/settingslib/RestrictedPreferenceHelper.java
@@ -146,12 +146,13 @@ public class RestrictedPreferenceHelper {
public boolean setDisabledByAdmin(EnforcedAdmin admin) {
final boolean disabled = (admin != null ? true : false);
mEnforcedAdmin = admin;
- mPreference.setEnabled(!disabled);
+ boolean changed = false;
if (mDisabledByAdmin != disabled) {
mDisabledByAdmin = disabled;
- return true;
+ changed = true;
}
- return false;
+ mPreference.setEnabled(!disabled);
+ return changed;
}
public boolean isDisabledByAdmin() {