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

Commit a00890ec authored by Amit Mahajan's avatar Amit Mahajan Committed by Sarah Chin
Browse files

Update getVoiceMailNumber to match set logic. 

Test: unit test and manual testing
Bug: 139318902
Change-Id: I3dbcd0bfcdaafa815c3b78d9b87af282bb361091
Merged-In: I3dbcd0bfcdaafa815c3b78d9b87af282bb361091
Merged-In: I6d5950cca950b61897a7987031e4705e54592e1b

(cherry picked from commit e278101b)
parent f7995dda

src/java/com/android/internal/telephony/GsmCdmaPhone.java

+17 −7
Original line number Diff line number Diff line
@@ -1498,10 +1498,12 @@ public class GsmCdmaPhone extends Phone { 
    private void storeVoiceMailNumber(String number) {

        SharedPreferences sp = PreferenceManager.getDefaultSharedPreferences(getContext());

        SharedPreferences.Editor editor = sp.edit();

        setVmSimImsi(getSubscriberId());

        logd("storeVoiceMailNumber: mPrecisePhoneType=" + mPrecisePhoneType + " vmNumber="

                + number);

        if (isPhoneTypeGsm()) {

            editor.putString(VM_NUMBER + getPhoneId(), number);

            editor.apply();

            setVmSimImsi(getSubscriberId());

        } else {

            editor.putString(VM_NUMBER_CDMA + getPhoneId(), number);

            editor.apply();
@@ -1511,17 +1513,23 @@ public class GsmCdmaPhone extends Phone { 
    @Override

    public String getVoiceMailNumber() {

        String number = null;

        if (isPhoneTypeGsm()) {

        if (isPhoneTypeGsm() || mSimRecords != null) {

            // Read from the SIM. If its null, try reading from the shared preference area.

            IccRecords r = mIccRecords.get();

            IccRecords r = isPhoneTypeGsm() ? mIccRecords.get() : mSimRecords;

            number = (r != null) ? r.getVoiceMailNumber() : "";

            if (TextUtils.isEmpty(number)) {

                SharedPreferences sp = PreferenceManager.getDefaultSharedPreferences(getContext());

                number = sp.getString(VM_NUMBER + getPhoneId(), null);

            }

                String spName = isPhoneTypeGsm() ? VM_NUMBER : VM_NUMBER_CDMA;

                number = sp.getString(spName + getPhoneId(), null);

                logd("getVoiceMailNumber: from " + spName + " number=" + number);

            } else {

                logd("getVoiceMailNumber: from IccRecords number=" + number);

            }

        }

        if (!isPhoneTypeGsm() && TextUtils.isEmpty(number)) {

            SharedPreferences sp = PreferenceManager.getDefaultSharedPreferences(getContext());

            number = sp.getString(VM_NUMBER_CDMA + getPhoneId(), null);

            logd("getVoiceMailNumber: from VM_NUMBER_CDMA number=" + number);

        }



        if (TextUtils.isEmpty(number)) {
@@ -2674,8 +2682,10 @@ public class GsmCdmaPhone extends Phone { 


            case EVENT_SET_VM_NUMBER_DONE:

                ar = (AsyncResult)msg.obj;

                if ((isPhoneTypeGsm() && IccVmNotSupportedException.class.isInstance(ar.exception)) ||

                        (!isPhoneTypeGsm() && IccException.class.isInstance(ar.exception))){

                if (((isPhoneTypeGsm() || mSimRecords != null)

                        && IccVmNotSupportedException.class.isInstance(ar.exception))

                        || (!isPhoneTypeGsm() && mSimRecords == null

                        && IccException.class.isInstance(ar.exception))) {

                    storeVoiceMailNumber(mVmNumber);

                    ar.exception = null;

                }

tests/telephonytests/src/com/android/internal/telephony/GsmCdmaPhoneTest.java

+2 −2
Original line number Diff line number Diff line
@@ -65,8 +65,8 @@ import androidx.test.filters.FlakyTest; 


import com.android.internal.telephony.test.SimulatedCommands;

import com.android.internal.telephony.uicc.IccCardApplicationStatus;

import com.android.internal.telephony.uicc.IccException;

import com.android.internal.telephony.uicc.IccRecords;

import com.android.internal.telephony.uicc.IccVmNotSupportedException;

import com.android.internal.telephony.uicc.UiccController;

import com.android.internal.telephony.uicc.UiccProfile;

import com.android.internal.telephony.uicc.UiccSlot;
@@ -561,7 +561,7 @@ public class GsmCdmaPhoneTest extends TelephonyTest { 
        // shared pref

        Message msg = messageArgumentCaptor.getValue();

        AsyncResult.forMessage(msg).exception =

                new IccException("setVoiceMailNumber not implemented");

                new IccVmNotSupportedException("setVoiceMailNumber not implemented");

        msg.sendToTarget();

        processAllMessages();