Show default data SIM in Wi-Fi details when dual SIMs have the same carrier ID
- Because the wifiConfiguration is only save the carrier ID for EAP-SIM authentication - If multiple SIMs have the same carrier ID, the Wi-Fi framework will use the default data SIM for EAP-SIM authentication - Show default data SIM in Wi-Fi details settings, when dual SIMs have the same carrier ID Bug: 233765468 Test: manual test make RunSettingsRoboTests ROBOTEST_FILTER=WifiDetailPreferenceController2Test Change-Id: I350fe637f506134770ccf316e47c0e225661bb6a
This commit is contained in:
@@ -19,6 +19,7 @@ import static android.net.NetworkCapabilities.NET_CAPABILITY_CAPTIVE_PORTAL;
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_PARTIAL_CONNECTIVITY;
|
||||
import static android.net.NetworkCapabilities.NET_CAPABILITY_VALIDATED;
|
||||
import static android.net.NetworkCapabilities.TRANSPORT_WIFI;
|
||||
import static android.telephony.TelephonyManager.UNKNOWN_CARRIER_ID;
|
||||
|
||||
import android.app.Activity;
|
||||
import android.app.AlertDialog;
|
||||
@@ -49,7 +50,6 @@ import android.os.Handler;
|
||||
import android.provider.Telephony.CarrierId;
|
||||
import android.telephony.SubscriptionInfo;
|
||||
import android.telephony.SubscriptionManager;
|
||||
import android.telephony.TelephonyManager;
|
||||
import android.text.TextUtils;
|
||||
import android.util.FeatureFlagUtils;
|
||||
import android.util.Log;
|
||||
@@ -690,34 +690,17 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
|
||||
// Checks if the SIM subscription is active.
|
||||
final List<SubscriptionInfo> activeSubscriptionInfos = mContext
|
||||
.getSystemService(SubscriptionManager.class).getActiveSubscriptionInfoList();
|
||||
final int defaultDataSubscriptionId = SubscriptionManager.getDefaultDataSubscriptionId();
|
||||
if (activeSubscriptionInfos != null) {
|
||||
CharSequence firstCarrierIdMatchedDisplayName = null;
|
||||
for (SubscriptionInfo subscriptionInfo : activeSubscriptionInfos) {
|
||||
final CharSequence displayName = SubscriptionUtil.getUniqueSubscriptionDisplayName(
|
||||
subscriptionInfo, mContext);
|
||||
if (firstCarrierIdMatchedDisplayName == null
|
||||
&& config.carrierId == subscriptionInfo.getCarrierId()) {
|
||||
firstCarrierIdMatchedDisplayName = displayName;
|
||||
}
|
||||
|
||||
// When it's UNKNOWN_CARRIER_ID or matched with configured CarrierId,
|
||||
// devices connects it with the SIM subscription of defaultDataSubscriptionId.
|
||||
if (defaultDataSubscriptionId == subscriptionInfo.getSubscriptionId()
|
||||
&& (config.carrierId == subscriptionInfo.getCarrierId()
|
||||
|| config.carrierId == TelephonyManager.UNKNOWN_CARRIER_ID)) {
|
||||
mEapSimSubscriptionPref.setSummary(displayName);
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
if (firstCarrierIdMatchedDisplayName != null) {
|
||||
mEapSimSubscriptionPref.setSummary(firstCarrierIdMatchedDisplayName);
|
||||
SubscriptionInfo info = fineSubscriptionInfo(config.carrierId, activeSubscriptionInfos,
|
||||
SubscriptionManager.getDefaultDataSubscriptionId());
|
||||
if (info != null) {
|
||||
mEapSimSubscriptionPref.setSummary(
|
||||
SubscriptionUtil.getUniqueSubscriptionDisplayName(info, mContext));
|
||||
return;
|
||||
}
|
||||
}
|
||||
|
||||
if (config.carrierId == TelephonyManager.UNKNOWN_CARRIER_ID) {
|
||||
if (config.carrierId == UNKNOWN_CARRIER_ID) {
|
||||
mEapSimSubscriptionPref.setSummary(R.string.wifi_no_related_sim_card);
|
||||
return;
|
||||
}
|
||||
@@ -736,6 +719,25 @@ public class WifiDetailPreferenceController2 extends AbstractPreferenceControlle
|
||||
null /* orderBy */);
|
||||
}
|
||||
|
||||
@VisibleForTesting
|
||||
SubscriptionInfo fineSubscriptionInfo(int carrierId,
|
||||
List<SubscriptionInfo> activeSubscriptionInfos, int defaultDataSubscriptionId) {
|
||||
SubscriptionInfo firstMatchedInfo = null;
|
||||
for (SubscriptionInfo info : activeSubscriptionInfos) {
|
||||
// When it's UNKNOWN_CARRIER_ID or matched with configured CarrierId,
|
||||
// devices connects it with the SIM subscription of defaultDataSubscriptionId.
|
||||
if (defaultDataSubscriptionId == info.getSubscriptionId()
|
||||
&& (carrierId == info.getCarrierId() || carrierId == UNKNOWN_CARRIER_ID)) {
|
||||
return info;
|
||||
}
|
||||
|
||||
if (firstMatchedInfo == null && carrierId == info.getCarrierId()) {
|
||||
firstMatchedInfo = info;
|
||||
}
|
||||
}
|
||||
return firstMatchedInfo;
|
||||
}
|
||||
|
||||
private void refreshMacAddress() {
|
||||
final String macAddress = mWifiEntry.getMacAddress();
|
||||
if (TextUtils.isEmpty(macAddress)) {
|
||||
|
Reference in New Issue
Block a user