summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Meng Wang <mewan@google.com> 2022-04-14 17:38:01 +0000
committer Meng Wang <mewan@google.com> 2022-04-18 11:54:05 -0700
commitfbbecbd49bf35c669c68eeda2fb8eef094c37a60 (patch)
tree6986ef3d7cc3b99c6bfe874c2c6d9bd7d154fd17
parent41eed224187e2c9cc6f602d41150148f36818caf (diff)
Revert "Use getPhoneNumber API to get phone number on T"
This reverts commit 35b45a774df3efb818cd88e3ed1870792b333cac. Reason for revert: Some carrier require to see SIM phone number in Settings UI. Revert for now and need to think of a better UI. Bug: 228424584 Bug: 159459637 Test: build and manual test Change-Id: I9381f7cdc6343a2e3b886d58f732587a2c8d8b7e
-rw-r--r--packages/SettingsLib/src/com/android/settingslib/DeviceInfoUtils.java33
1 files changed, 8 insertions, 25 deletions
diff --git a/packages/SettingsLib/src/com/android/settingslib/DeviceInfoUtils.java b/packages/SettingsLib/src/com/android/settingslib/DeviceInfoUtils.java
index f627bcb5db95..ff00fb3282b1 100644
--- a/packages/SettingsLib/src/com/android/settingslib/DeviceInfoUtils.java
+++ b/packages/SettingsLib/src/com/android/settingslib/DeviceInfoUtils.java
@@ -26,7 +26,6 @@ import android.system.Os;
import android.system.StructUtsname;
import android.telephony.PhoneNumberUtils;
import android.telephony.SubscriptionInfo;
-import android.telephony.SubscriptionManager;
import android.telephony.TelephonyManager;
import android.text.BidiFormatter;
import android.text.TextDirectionHeuristics;
@@ -34,11 +33,8 @@ import android.text.TextUtils;
import android.text.format.DateFormat;
import android.util.Log;
-import androidx.annotation.RequiresApi;
import androidx.annotation.VisibleForTesting;
-import com.android.settingslib.utils.BuildCompatUtils;
-
import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
@@ -183,8 +179,10 @@ public class DeviceInfoUtils {
SubscriptionInfo subscriptionInfo) {
String formattedNumber = null;
if (subscriptionInfo != null) {
- final String rawNumber = getRawPhoneNumber(
- context, subscriptionInfo.getSubscriptionId());
+ final TelephonyManager telephonyManager = context.getSystemService(
+ TelephonyManager.class);
+ final String rawNumber = telephonyManager.createForSubscriptionId(
+ subscriptionInfo.getSubscriptionId()).getLine1Number();
if (!TextUtils.isEmpty(rawNumber)) {
formattedNumber = PhoneNumberUtils.formatNumber(rawNumber);
}
@@ -196,10 +194,12 @@ public class DeviceInfoUtils {
List<SubscriptionInfo> subscriptionInfoList) {
StringBuilder sb = new StringBuilder();
if (subscriptionInfoList != null) {
+ final TelephonyManager telephonyManager = context.getSystemService(
+ TelephonyManager.class);
final int count = subscriptionInfoList.size();
for (SubscriptionInfo subscriptionInfo : subscriptionInfoList) {
- final String rawNumber = getRawPhoneNumber(
- context, subscriptionInfo.getSubscriptionId());
+ final String rawNumber = telephonyManager.createForSubscriptionId(
+ subscriptionInfo.getSubscriptionId()).getLine1Number();
if (!TextUtils.isEmpty(rawNumber)) {
sb.append(PhoneNumberUtils.formatNumber(rawNumber)).append("\n");
}
@@ -219,21 +219,4 @@ public class DeviceInfoUtils {
final String phoneNumber = getFormattedPhoneNumber(context, subscriptionInfo);
return BidiFormatter.getInstance().unicodeWrap(phoneNumber, TextDirectionHeuristics.LTR);
}
-
- private static String getRawPhoneNumber(Context context, int subscriptionId) {
- if (BuildCompatUtils.isAtLeastT()) {
- return getRawPhoneNumberFromT(context, subscriptionId);
- } else {
- final TelephonyManager telephonyManager = context.getSystemService(
- TelephonyManager.class);
- return telephonyManager.createForSubscriptionId(subscriptionId).getLine1Number();
- }
- }
-
- @RequiresApi(Build.VERSION_CODES.TIRAMISU)
- private static String getRawPhoneNumberFromT(Context context, int subscriptionId) {
- final SubscriptionManager subscriptionManager = context.getSystemService(
- SubscriptionManager.class);
- return subscriptionManager.getPhoneNumber(subscriptionId);
- }
}