Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 0056bbf6 authored by Automerger Merge Worker's avatar Automerger Merge Worker
Browse files

Fixed that SIM name not updated issue am: c07d64ca

Change-Id: I16fc4ee521a0b19afbc9abc24100284a6f73cb42
parents ae2177c1 c07d64ca
Loading
Loading
Loading
Loading
+11 −8
Original line number Diff line number Diff line
@@ -1507,14 +1507,13 @@ public class SubscriptionController extends ISub.Stub {
     * @return int representing the priority. Higher value means higher priority.
     */
    public static int getNameSourcePriority(@SimDisplayNameSource int nameSource) {
        int index = Arrays.binarySearch(
                new int[] {
        int index = Arrays.asList(
                SubscriptionManager.NAME_SOURCE_DEFAULT_SOURCE,
                SubscriptionManager.NAME_SOURCE_SIM_PNN,
                SubscriptionManager.NAME_SOURCE_SIM_SPN,
                SubscriptionManager.NAME_SOURCE_CARRIER,
                SubscriptionManager.NAME_SOURCE_USER_INPUT // user has highest priority.
                }, nameSource);
        ).indexOf(nameSource);
        return (index < 0) ? 0 : index;
    }

@@ -1547,6 +1546,10 @@ public class SubscriptionController extends ISub.Stub {
                        && (getNameSourcePriority(subInfo.getNameSource())
                                > getNameSourcePriority(nameSource)
                        || (displayName != null && displayName.equals(subInfo.getDisplayName())))) {
                    logd("Name source " + subInfo.getNameSource() + "'s priority "
                            + getNameSourcePriority(subInfo.getNameSource()) + " is greater than "
                            + "name source " + nameSource + "'s priority "
                            + getNameSourcePriority(nameSource) + ", return now.");
                    return 0;
                }
            }
+24 −0
Original line number Diff line number Diff line
@@ -1004,4 +1004,28 @@ public class SubscriptionControllerTest extends TelephonyTest {
        mSubscriptionControllerUT.setAlwaysAllowMmsData(0, false);
        verify(mDataEnabledSettings).setAlwaysAllowMmsData(eq(false));
    }

    @Test
    @SmallTest
    public void testNameSourcePriority() throws Exception {
        assertTrue(mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_USER_INPUT)
                > mSubscriptionControllerUT.getNameSourcePriority(
                        SubscriptionManager.NAME_SOURCE_CARRIER));

        assertTrue(mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_CARRIER)
                > mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_SIM_SPN));

        assertTrue(mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_SIM_SPN)
                > mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_SIM_PNN));

        assertTrue(mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_SIM_PNN)
                > mSubscriptionControllerUT.getNameSourcePriority(
                SubscriptionManager.NAME_SOURCE_DEFAULT_SOURCE));
    }
}