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



Firefox OS
4 years ago
4 years ago


(Reporter: Peipei Cheng (needinfo if you need my action), Unassigned)



Firefox Tracking Flags



(Whiteboard: [sprd312718])


(1 attachment)

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.
Whiteboard: [sprd312718]
blocking-b2g: --- → 1.4?
Whiteboard: [sprd312718] → [partner-blocker][sprd312718]
This is an unrealistic use case that is easy to workaround, so this is not a blocker.
blocking-b2g: 1.4? → -

Comment 2

4 years ago
Created attachment 8437472 [details] [diff] [review]

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.
Flags: needinfo?(ehung)

Comment 3

4 years ago
Hi Jason

This is a patch for it, can you help to review?
Thanks very much.
Flags: needinfo?(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.
Component: Gaia::Settings → Gaia::SMS
Flags: needinfo?(jsmith) → needinfo?(felash)
Comment on attachment 8437472 [details] [diff] [review]

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.
Attachment #8437472 - Flags: review-
Flags: needinfo?(felash)
Component: Gaia::SMS → Gaia::Settings

Comment 6

4 years ago
Julien reviewed the patch. Thanks.
Flags: needinfo?(ehung)
Whiteboard: [partner-blocker][sprd312718] → [sprd312718]
Was this fixed in bug 1035823?

Comment 8

4 years ago
(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.
You need to log in before you can comment on or make changes to this bug.