PawletOS/app_Settings
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "[Settings] mSubId should not be overridden in TelephonyBasePreferenceController" am: 12677c26f4

Browse Source 
Change-Id: I673a260ee1f1175198090c0ec87e989b34bd3635
This commit is contained in:
Bonian Chen
2020-04-07 04:39:57 +00:00
committed by Automerger Merge Worker
parent 7ca7469230 12677c26f4
commit 3b7041d9f3
2 changed files with 13 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

18
src/com/android/settings/network/telephony/DisabledSubscriptionController.java
View File

@@ -28,16 +28,21 @@ import androidx.lifecycle.OnLifecycleEvent;
import androidx.preference.PreferenceCategory;
import androidx.preference.PreferenceScreen;
import com.android.settings.core.BasePreferenceController;
import com.android.settings.network.SubscriptionsChangeListener;
public class DisabledSubscriptionController extends BasePreferenceController implements
/**
* Preference controller group which can hide UI option from visible when SIM is no longer in
* active.
*/
public class DisabledSubscriptionController extends TelephonyBasePreferenceController implements
SubscriptionsChangeListener.SubscriptionsChangeListenerClient, LifecycleObserver {
private PreferenceCategory mCategory;
private int mSubId;
private SubscriptionsChangeListener mChangeListener;
private SubscriptionManager mSubscriptionManager;
/**
* Constructor of preference controller
*/
public DisabledSubscriptionController(Context context, String preferenceKey) {
super(context, preferenceKey);
mSubId = SubscriptionManager.INVALID_SUBSCRIPTION_ID;
@@ -45,6 +50,9 @@ public class DisabledSubscriptionController extends BasePreferenceController imp
mChangeListener = new SubscriptionsChangeListener(context, this);
}
/**
* Re-initialize the configuration based on subscription id provided
*/
public void init(Lifecycle lifecycle, int subId) {
lifecycle.addObserver(this);
mSubId = subId;
@@ -69,7 +77,7 @@ public class DisabledSubscriptionController extends BasePreferenceController imp
}
private void update() {
if (mCategory == null || mSubId == SubscriptionManager.INVALID_SUBSCRIPTION_ID) {
if (mCategory == null || !SubscriptionManager.isValidSubscriptionId(mSubId)) {
return;
}
// TODO b/135222940: re-evaluate whether to use mSubscriptionManager#isSubscriptionEnabled
@@ -77,7 +85,7 @@ public class DisabledSubscriptionController extends BasePreferenceController imp
}
@Override
public int getAvailabilityStatus() {
public int getAvailabilityStatus(int subId) {
return AVAILABLE_UNSEARCHABLE;
}

1
src/com/android/settings/network/telephony/cdma/CdmaBasePreferenceController.java
View File

@@ -44,7 +44,6 @@ public abstract class CdmaBasePreferenceController extends TelephonyBasePreferen
protected Preference mPreference;
protected TelephonyManager mTelephonyManager;
protected PreferenceManager mPreferenceManager;
protected int mSubId;
private DataContentObserver mDataContentObserver;
public CdmaBasePreferenceController(Context context, String key) {