DO NOT MERGE: Fix NPE in ConfigureWifiSettings.
NPE was introduced as part of recent changes for Carrier Wifi networks. This CL adds a null check. Bug: 34050661 Change-Id: I585f6499b1ca06d9bd25ad3449f035c5a616a140
This commit is contained in:
@@ -101,6 +101,13 @@ public class ConfigureWifiSettings extends SettingsPreferenceFragment
|
|||||||
|
|
||||||
if (!mWifiManager.hasCarrierConfiguredNetworks()){
|
if (!mWifiManager.hasCarrierConfiguredNetworks()){
|
||||||
removePreference(KEY_CONNECT_CARRIER_NETWORKS);
|
removePreference(KEY_CONNECT_CARRIER_NETWORKS);
|
||||||
|
} else {
|
||||||
|
SwitchPreference connectToCarrierNetworks =
|
||||||
|
(SwitchPreference) findPreference(KEY_CONNECT_CARRIER_NETWORKS);
|
||||||
|
if (connectToCarrierNetworks != null) {
|
||||||
|
connectToCarrierNetworks.setChecked(Settings.Global.getInt(getContentResolver(),
|
||||||
|
Settings.Global.WIFI_CONNECT_CARRIER_NETWORKS, 0) == 1);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
SwitchPreference notifyOpenNetworks =
|
SwitchPreference notifyOpenNetworks =
|
||||||
@@ -109,11 +116,6 @@ public class ConfigureWifiSettings extends SettingsPreferenceFragment
|
|||||||
Settings.Global.WIFI_NETWORKS_AVAILABLE_NOTIFICATION_ON, 0) == 1);
|
Settings.Global.WIFI_NETWORKS_AVAILABLE_NOTIFICATION_ON, 0) == 1);
|
||||||
notifyOpenNetworks.setEnabled(mWifiManager.isWifiEnabled());
|
notifyOpenNetworks.setEnabled(mWifiManager.isWifiEnabled());
|
||||||
|
|
||||||
SwitchPreference connectToCarrierNetworks =
|
|
||||||
(SwitchPreference) findPreference(KEY_CONNECT_CARRIER_NETWORKS);
|
|
||||||
connectToCarrierNetworks.setChecked(Settings.Global.getInt(getContentResolver(),
|
|
||||||
Settings.Global.WIFI_CONNECT_CARRIER_NETWORKS, 0) == 1);
|
|
||||||
|
|
||||||
final Context context = getActivity();
|
final Context context = getActivity();
|
||||||
if (avoidBadWifiConfig()) {
|
if (avoidBadWifiConfig()) {
|
||||||
// Hide preference toggle, always avoid bad wifi networks.
|
// Hide preference toggle, always avoid bad wifi networks.
|
||||||
|
Reference in New Issue
Block a user