Use disabled print services instead of enabled print services in print settings.

Change-Id: I32bc8a8dda86f92b180f043a98b304fe676db98c
This commit is contained in:
Philip P. Moltmann
2015-12-22 13:43:14 -08:00
parent 738e6db7b8
commit fae4a88530
3 changed files with 28 additions and 27 deletions

View File

@@ -191,13 +191,13 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment
private void onPreferenceToggled(String preferenceKey, boolean enabled) { private void onPreferenceToggled(String preferenceKey, boolean enabled) {
ComponentName service = ComponentName.unflattenFromString(preferenceKey); ComponentName service = ComponentName.unflattenFromString(preferenceKey);
List<ComponentName> services = PrintSettingsUtils.readEnabledPrintServices(getActivity()); List<ComponentName> services = PrintSettingsUtils.readDisabledPrintServices(getActivity());
if (enabled) { if (enabled) {
services.add(service);
} else {
services.remove(service); services.remove(service);
} else {
services.add(service);
} }
PrintSettingsUtils.writeEnabledPrintServices(getActivity(), services); PrintSettingsUtils.writeDisabledPrintServices(getActivity(), services);
} }
private ListView getBackupListView() { private ListView getBackupListView() {
@@ -253,8 +253,9 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment
} }
private void updateUiForServiceState() { private void updateUiForServiceState() {
List<ComponentName> services = PrintSettingsUtils.readEnabledPrintServices(getActivity()); List<ComponentName> disabledServices = PrintSettingsUtils
mServiceEnabled = services.contains(mComponentName); .readDisabledPrintServices(getActivity());
mServiceEnabled = !disabledServices.contains(mComponentName);
if (mServiceEnabled) { if (mServiceEnabled) {
mSwitchBar.setCheckedInternal(true); mSwitchBar.setCheckedInternal(true);
mPrintersAdapter.enable(); mPrintersAdapter.enable();
@@ -431,7 +432,7 @@ public class PrintServiceSettingsFragment extends SettingsPreferenceFragment
public void register(ContentResolver contentResolver) { public void register(ContentResolver contentResolver) {
contentResolver.registerContentObserver(android.provider.Settings.Secure.getUriFor( contentResolver.registerContentObserver(android.provider.Settings.Secure.getUriFor(
android.provider.Settings.Secure.ENABLED_PRINT_SERVICES), false, this); android.provider.Settings.Secure.DISABLED_PRINT_SERVICES), false, this);
} }
public void unregister(ContentResolver contentResolver) { public void unregister(ContentResolver contentResolver) {

View File

@@ -207,8 +207,8 @@ public class PrintSettingsFragment extends SettingsPreferenceFragment
mPrintServicesCategory.removeAll(); mPrintServicesCategory.removeAll();
} }
List<ComponentName> enabledServices = PrintSettingsUtils List<ComponentName> disabledServices = PrintSettingsUtils
.readEnabledPrintServices(getActivity()); .readDisabledPrintServices(getActivity());
final PackageManager pm = getActivity().getPackageManager(); final PackageManager pm = getActivity().getPackageManager();
@@ -236,7 +236,7 @@ public class PrintSettingsFragment extends SettingsPreferenceFragment
preference.setFragment(PrintServiceSettingsFragment.class.getName()); preference.setFragment(PrintServiceSettingsFragment.class.getName());
preference.setPersistent(false); preference.setPersistent(false);
final boolean serviceEnabled = enabledServices.contains(componentName); final boolean serviceEnabled = !disabledServices.contains(componentName);
if (serviceEnabled) { if (serviceEnabled) {
preference.setSummary(getString(R.string.print_feature_state_on)); preference.setSummary(getString(R.string.print_feature_state_on));
} else { } else {
@@ -388,7 +388,7 @@ public class PrintSettingsFragment extends SettingsPreferenceFragment
public void register(ContentResolver contentResolver) { public void register(ContentResolver contentResolver) {
contentResolver.registerContentObserver(Settings.Secure.getUriFor( contentResolver.registerContentObserver(Settings.Secure.getUriFor(
Settings.Secure.ENABLED_PRINT_SERVICES), false, this); Settings.Secure.DISABLED_PRINT_SERVICES), false, this);
} }
public void unregister(ContentResolver contentResolver) { public void unregister(ContentResolver contentResolver) {

View File

@@ -30,50 +30,50 @@ import java.util.List;
*/ */
public class PrintSettingsUtils { public class PrintSettingsUtils {
private static final char ENABLED_PRINT_SERVICES_SEPARATOR = ':'; private static final char DISABLED_PRINT_SERVICES_SEPARATOR = ':';
private PrintSettingsUtils() { private PrintSettingsUtils() {
/* do nothing */ /* do nothing */
} }
public static List<ComponentName> readEnabledPrintServices(Context context) { public static List<ComponentName> readDisabledPrintServices(Context context) {
List<ComponentName> enabledServices = new ArrayList<ComponentName>(); List<ComponentName> disabledServices = new ArrayList<ComponentName>();
String enabledServicesSetting = Settings.Secure.getString(context String disabledServicesSetting = Settings.Secure.getString(context
.getContentResolver(), Settings.Secure.ENABLED_PRINT_SERVICES); .getContentResolver(), Settings.Secure.DISABLED_PRINT_SERVICES);
if (TextUtils.isEmpty(enabledServicesSetting)) { if (TextUtils.isEmpty(disabledServicesSetting)) {
return enabledServices; return disabledServices;
} }
SimpleStringSplitter colonSplitter = new SimpleStringSplitter( SimpleStringSplitter colonSplitter = new SimpleStringSplitter(
ENABLED_PRINT_SERVICES_SEPARATOR); DISABLED_PRINT_SERVICES_SEPARATOR);
colonSplitter.setString(enabledServicesSetting); colonSplitter.setString(disabledServicesSetting);
while (colonSplitter.hasNext()) { while (colonSplitter.hasNext()) {
String componentNameString = colonSplitter.next(); String componentNameString = colonSplitter.next();
ComponentName enabledService = ComponentName.unflattenFromString( ComponentName disabledService = ComponentName.unflattenFromString(
componentNameString); componentNameString);
if (enabledService != null) { if (disabledService != null) {
enabledServices.add(enabledService); disabledServices.add(disabledService);
} }
} }
return enabledServices; return disabledServices;
} }
public static void writeEnabledPrintServices(Context context, public static void writeDisabledPrintServices(Context context,
List<ComponentName> services) { List<ComponentName> services) {
StringBuilder builder = new StringBuilder(); StringBuilder builder = new StringBuilder();
final int serviceCount = services.size(); final int serviceCount = services.size();
for (int i = 0; i < serviceCount; i++) { for (int i = 0; i < serviceCount; i++) {
ComponentName service = services.get(i); ComponentName service = services.get(i);
if (builder.length() > 0) { if (builder.length() > 0) {
builder.append(ENABLED_PRINT_SERVICES_SEPARATOR); builder.append(DISABLED_PRINT_SERVICES_SEPARATOR);
} }
builder.append(service.flattenToString()); builder.append(service.flattenToString());
} }
Settings.Secure.putString(context.getContentResolver(), Settings.Secure.putString(context.getContentResolver(),
Settings.Secure.ENABLED_PRINT_SERVICES, Settings.Secure.DISABLED_PRINT_SERVICES,
builder.toString()); builder.toString());
} }
} }