1018031 - [Dolphin][Flame] data sim was not refreshed after switching data sim from MMS

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::Settings, defect)

Description

[Peipei Cheng (needinfo if you need my action)]

Steps to reproduce
------------------------------------------------------------------------------
0. Insert two SIM cards to device
1. Launch Settings and go to Settings -> SIM Manager
   The default data SIM is SIM 1
2. Send a MMS to SIM 2
   There is a MMS notification appears. 
3. Tap on the notification and then start download this MMS. 
   Confirm to "Switch data connection"
4. Use Card View to switch to Settings -> SIM Manager
   --> You will see data SIM still shows SIM 1
5. Kill Settings and relaunch
   data SIM already changed to SIM 2.


[Expected] data sim should be updated in Settings after changing even when Settings is not relaunched.

Comment 1

[Jason Smith [:jsmith]]

This is an unrealistic use case that is easy to workaround, so this is not a blocker.

Comment 2

[Wei Gao (Spreadtrum)]

Attachment: mms_switch_data_card.patch

For this bug, my idea is if data is switched in mms, it should be switched back after the mms is sent or received.

This is a patch for it, can you help to review?

Thanks very much.

Comment 3

[Wei Gao (Spreadtrum)]

Hi Jason

This is a patch for it, can you help to review?
Thanks very much.

Comment 4

[Jason Smith [:jsmith]]

I can't help with reviewing a patch here. This appears to be a SMS patch though, so maybe Julien could help you with this.

Comment 5

[Julien Wajsberg [:julienw]]

Comment on attachment 8437472 [details] [diff] [review]
mms_switch_data_card.patch

We discussed about this a lot with UX and decided to not change the configured SIM back.

The main issue is that a MMS can take a long time to be sent and as a result we don't want to make a switch at an inadequate moment.

So my suggestion is to make the Settings app observe this Setting and change the value in the panel instead.

Comment 6

[Evelyn Hung]

Julien reviewed the patch. Thanks.

Comment 7

[Julien Wajsberg [:julienw]]

Was this fixed in bug 1035823?

Comment 8

[Wei Gao (Spreadtrum)]

(In reply to Julien Wajsberg [:julienw] from comment #7)
> Was this fixed in bug 1035823?

Yes, but I hope v1.4 can also be fixed using that solution.

Comment 9

[BMO Automation]

Firefox OS is not being worked on