Merge "Move phone ID to extra for action provision"

This commit is contained in:
Sarah Chin
2020-12-17 06:06:41 +00:00
committed by Gerrit Code Review
2 changed files with 9 additions and 3 deletions

View File

@@ -3154,7 +3154,7 @@ public class ConnectivityManager {
/** /**
* Set sign in error notification to visible or invisible * Set sign in error notification to visible or invisible
* *
* {@hide} * @hide
* @deprecated Doesn't properly deal with multiple connected networks of the same type. * @deprecated Doesn't properly deal with multiple connected networks of the same type.
*/ */
@Deprecated @Deprecated

View File

@@ -334,7 +334,13 @@ public class NetworkNotificationManager {
*/ */
public void setProvNotificationVisible(boolean visible, int id, String action) { public void setProvNotificationVisible(boolean visible, int id, String action) {
if (visible) { if (visible) {
Intent intent = new Intent(action); // For legacy purposes, action is sent as the action + the phone ID from DcTracker.
// Split the string here and send the phone ID as an extra instead.
String[] splitAction = action.split(":");
Intent intent = new Intent(splitAction[0]);
try {
intent.putExtra("provision.phone.id", Integer.parseInt(splitAction[1]));
} catch (NumberFormatException ignored) { }
PendingIntent pendingIntent = PendingIntent.getBroadcast( PendingIntent pendingIntent = PendingIntent.getBroadcast(
mContext, 0 /* requestCode */, intent, PendingIntent.FLAG_IMMUTABLE); mContext, 0 /* requestCode */, intent, PendingIntent.FLAG_IMMUTABLE);
showNotification(id, NotificationType.SIGN_IN, null, null, pendingIntent, false); showNotification(id, NotificationType.SIGN_IN, null, null, pendingIntent, false);