Closed Bug 1105170 Opened 7 years ago Closed 7 years ago

[FFOS2.0][Woodduck][costcontrol] data usage does not consider dual SIM scenario while switch sim card

Categories

(Firefox OS Graveyard :: Gaia::Cost Control, defect, P2)

defect

Tracking

(blocking-b2g:2.0M+, b2g-v2.0 wontfix, b2g-v2.0M verified, b2g-v2.1 verified, b2g-v2.2 verified)

VERIFIED FIXED
2.2 S2 (19dec)
blocking-b2g 2.0M+
Tracking Status
b2g-v2.0 --- wontfix
b2g-v2.0M --- verified
b2g-v2.1 --- verified
b2g-v2.2 --- verified

People

(Reporter: sync-1, Assigned: mai)

References

Details

Attachments

(9 files)

DEFECT DESCRIPTION:
 >When switching sim card, all founction on data usage settings page doesn't work.
 
  REPRODUCING PROCEDURES:
 >1)Insert two sim card and power on the phone;
 >2)Enter data usage,and click "settings" on top right corner;
 >3)Modify the setings and click "Done".It works--OK
 >4)Press "home" and go to "Settings"->SIM Manager->Data,changing to another SIM card.
 >5)Press "home" and go to "Usage",and click "settings" on top right corner;
 >6)Try to operate the fountion on the page
 find that all founction on data usage settings page doesn't work.--KO
 
  EXPECTED BEHAVIOUR:
 >It should work normally when changing sim card.
Hi Norry,
qawanted for Woodduck 2.0M and Flame 2.0/2.1/2.2. Thanks!
Blocks: Woodduck
blocking-b2g: --- → 2.0M?
Flags: needinfo?(fan.luo)
PS:restart usage will be ok
Attached file logcat.txt
This bug can be repro on Woodduck 2.0M and Flame 2.0/2.1/2.2

Attachment: Verify.mp4 & logcat.txt
Rate: 100%
Found time:14:59

Woodduck build:
Gaia-Rev        d742e375aca6dc1bf3a36638000ad7f5338ef457
Gecko-Rev       d049d4ef127844121c9cf14d2e8ca91fd9045fcb
Build-ID        20141127050313
Version         32.0
Device-Name     jrdhz72_w_ff
FW-Release      4.4.2
FW-Incremental  1417035915
FW-Date         Thu Nov 27 05:05:39 CST 2014
------------------
Flame 2.0:
Gaia-Rev        f9d6e3d83c3922e9399a6c27f5ce4cdd27bdfd05
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g32_v2_0/rev/45112935086f
Build-ID        20141126000203
Version         32.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141126.032754
FW-Date         Wed Nov 26 03:28:05 EST 2014
Bootloader      L1TC00011880
------------------
FLame2.1 build:
Flame 2.1 new build:
Gaia-Rev        db2e84860f5a7cc334464618c6ea9e92ff82e9dd
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/211eae88f119
Build-ID        20141126001202
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141126.033519
FW-Date         Wed Nov 26 03:35:30 EST 2014
Bootloader      L1TC00011880
------------------
FLame2.2 build:
Gaia-Rev        41b7be7c67167f367c3c4982ff08651d55455373
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/ff4a63d66806
Build-ID        20141126160201
Version         36.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141126.193315
FW-Date         Wed Nov 26 19:33:28 EST 2014
Bootloader      L1TC00011880
Flags: needinfo?(fan.luo)
Attached video Verify.mp4
Triage:
This is feature request which we can consider enhance in the future.

Hi Wesley,
Can you take a look at this bug as feature enhancement in future release?
Thanks!
blocking-b2g: 2.0M? → backlog
feature-b2g: --- → 2.2?
Flags: needinfo?(whuang)
The initiative "Smart Data" might cover this.
NI? to Peter for confirmation of plan.
Flags: needinfo?(whuang) → needinfo?(pdolanjski)
hi guys,
what's the latest status about this PR
(In reply to wei.deng from comment #7)
> hi guys,
> what's the latest status about this PR

Hi DengWei,
This is feature request which we can consider enhance in the future but will not be able to fix in 2.0M Woodduck. Can you postpone this TC?
Flags: needinfo?(wei.deng)
Blocks: Woodduck_P2
dear mozilla,
 
      This is the basic function issue and have bad effect on user expierence.we hope it can be fixed in 2.0M Woodduck if it's not difficult to solve it.
blocking-b2g: backlog → 2.0M?
Summary: [FFOS2.0][Woodduck][costcontrol]data usage works abnormally after switch sim card → [FFOS2.0][Woodduck][costcontrol] data usage does not consider dual SIM scenario while switch sim card
Hi Evelyn,
This is feature request per bug 1035568. 
Partner request to fix this in 2.0M. Could you help to assess whether gaia team can implement this at minimum effort?
Flags: needinfo?(wei.deng) → needinfo?(ehung)
See Also: → 1035568
dear mozilla,
 
      i have found the bug reason.in costcontrol/js/settings/settings.js,the variable ConfigManager.configuration is null when switch sim card.i try to fix it by add a condition statement,it can roughly works now.thanks.
Ethan, please check this bug and see if you need Gaia dev's help. Thanks.
Assignee: nobody → ettseng
feature-b2g: 2.2? → 2.2+
I can reproduce this bug on Flame v2.2.

When the SIM card for data network is switched, the value of the mozSettings
|ril.data.defaultServiceId| will be changed. I think usage app should refresh its UI when this change
is observed.
Group: woodduck-confidential
Hi Salva,

You are much more familiar with the Gaia code in costcontrol app than I am.
Could you help to see if we can have a quick fix for this bug?
Flags: needinfo?(salva)
[Tracking Requested - why for this release]:
feature-b2g: 2.2+ → ---
tracking-b2g: --- → ?
Tzu-lin, could you check comment 13 and see if there is something you can help? Thanks.
Flags: needinfo?(ehung) → needinfo?(tzhuang)
Attached file patch v1.0
Hi Salva,
would you mind reviewing the patch?
Regards
Flags: needinfo?(salva)
Attachment #8534955 - Flags: review?(salva)
Assignee: ettseng → marina.rodriguez.iglesias
Clear NI since :mai has patch already.
Flags: needinfo?(tzhuang)
blocking-b2g: 2.0M? → 2.0M+
Dear Salva,
Partner is requesting this fix urgently. Could you kindly help to review in your earliest possible? Thanks!
Flags: needinfo?(pdolanjski)
Comment on attachment 8534955 [details] [review]
patch v1.0

LGTM, but please, read the comments on GitHub and add a regression test case for this.
Attachment #8534955 - Flags: review?(salva)
Comment on attachment 8534955 [details] [review]
patch v1.0

updated the pr with your comments.
Attachment #8534955 - Flags: review?(salva)
Comment on attachment 8534955 [details] [review]
patch v1.0

The tests you added cover `requestAll()` but they are not regression tests. What it is observed is that after changing the SIM to another from the same carrier, the configuration is not prooperly set. That is what we must test in a regression test. Don't remove the tests from the source code but please, add the regression test.
Attachment #8534955 - Flags: review?(salva)
Comment on attachment 8534955 [details] [review]
patch v1.0

Hi Salva,
updated the pr with your comments
Attachment #8534955 - Flags: review?(salva)
Comment on attachment 8534955 [details] [review]
patch v1.0

You have a little comment on GitHub. You have the r+ after making the pointed changes. Thank you Mai.
Attachment #8534955 - Flags: review?(salva) → review+
Master: a143f9e7045f56212137c1dfb0832fe0cc980b28
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Marina, Uplift to v2.0m branch got conflicts in the following files. Could you provide a patch for v2.0m? Thanks!
--
apps/costcontrol/js/app.js
apps/costcontrol/js/config/config_manager.js
Flags: needinfo?(marina.rodriguez.iglesias)
Attached file patch for v2.0m
Flags: needinfo?(marina.rodriguez.iglesias)
Marina, Thanks! If v2.1 also need this patch, please request for approval.
Comment on attachment 8534955 [details] [review]
patch v1.0

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): 
[User impact] if declined: after changing the SIM to another from the same carrier, the configuration is not prooperly set
[Testing completed]: yes
[Risk to taking this patch] (and alternatives if risky): Low risk, testing on device and working currently on master
[String changes made]: No
Attachment #8534955 - Flags: approval-gaia-v2.1?(release-mgmt)
This issue has been failed verified on Flame v2.2.
See attachments: verify_v2.2.MP4 and logcat_v2.2.txt.
Reproduce rate: 4/4.

Repro STR:
1.Power on device with two SIM cards and enable data connection.
2.Enter data usage,and click "Settings" on top right corner.
**User can modify the setings successfully.And "Data Limit(10.00MB)" prompt displays as same as Usage app.
3.Go to "Settings"->SIM Manager->Data,change to SIM2 card.
**But SIM2 card has no data connection.(at 16:41)
4.Tap on "New SIM inserted" after sliding down the notifications bar,and then click "Next".
5.Then tap "Let's go!".
**"Let's go!" button has no response(at 16:42).But user can set "Data use alert"(such as "31MB").
6.Long tap Home button and close it.
7.Open Usage from homescreen.
**SIM2 card's data usage info is shown and usage settings page works normally.
8.Slide down notifications bar.
**It displays "Data Limit(10.00MB)" prompt and inconsistent with Usage(31MB).

Flame 2.2 build:
Gaia-Rev        c2da2bafd4e809317e2ca70c9bf5c11136a32818
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/0532f2509f3f
Build-ID        20141223010202
Version         37.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141223.043429
FW-Date         Tue Dec 23 04:34:39 EST 2014
Bootloader      L1TC00011880


This issue has been successfully verified on Woodduck v2.0.
See attachment: verified_v2.0m.mp4.
Reproduce rate: 0/4

Woodduck 2.0 build:
Gaia-Rev        afa87cffbd3cd9e2070b26d45dd556a9324bd4d5
Gecko-Rev       911e6cd6aecf8d37d42c203e162847b78a68a8d8
Build-ID        20141224050313
Version         32.0
Device-Name     jrdhz72_w_ff
FW-Release      4.4.2
FW-Incremental  1419368730
FW-Date         Wed Dec 24 05:05:52 CST 2014
Flags: needinfo?(echang)
Sorry to update woodduck's result.
Sometimes,in the step 5,when tap "Let's go!","Let's go!" button has no response(at 18:59) on woodduck 2.0(Rate:2/4).

Please see attachments:woodduck_logcat_1859.txt and verify_2.0m.MP4
Hi shally,  
The behavior of the Comment 31 is a different bug. Please, would you mind addressing a new bug for the "let's go" button.
Regards
(In reply to Marina Rodríguez [:mai] from comment #35)
> Hi shally,  
> The behavior of the Comment 31 is a different bug. Please, would you mind
> addressing a new bug for the "let's go" button.
> Regards

ok,thanks!
Hi Sally, Please cc or ni me of the new bug, thank you.
Flags: needinfo?(echang) → needinfo?(lixia)
Hi Eric,
  The new bug about "Let's go!" button is 1115509.
Flags: needinfo?(lixia) → needinfo?(echang)
Thanks!
Flags: needinfo?(echang)
Attachment #8534955 - Flags: approval-gaia-v2.1?(release-mgmt) → approval-gaia-v2.1+
This issue has been successfully verified on Flame v2.1&2.2.
See attachment: verified_v2.1.mp4.
Reproduce rate: 0/4.

Flame 2.1 build:
Gaia-Rev        73be51f998031f06db0cd660c0e388fa621c9f4c
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/ea426e47bfc4
Build-ID        20141228001253
Version         34.0
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141228.034827
FW-Date         Sun Dec 28 03:48:38 EST 2014

Flame 2.2 build:
Gaia-Rev        3554ea9504046646b4679e3a61317c49fc55ca87
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/67c42c076393
Build-ID        20141228010205
Version         37.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20141228.045435
FW-Date         Sun Dec 28 04:54:46 EST 2014
Bootloader      L1TC00011880

Note:The new bug about "Let's go" button still exists.
Status: RESOLVED → VERIFIED
Hi Marina,
Per https://bugzilla.mozilla.org/show_bug.cgi?id=1035568#c2 Currently it reports sum up for both cards. Partner feedback that user will not able to identify which card already over his/her budget. Should we revise the behavior to clean old usage data when user switch to new SIM card?
Thanks!
Flags: needinfo?(marina.rodrigueziglesias)
Hi Josh, 
I think your question is for UX.
NI to Juwei
Flags: needinfo?(marina.rodrigueziglesias) → needinfo?(jhuang)
NI myself for tracking
Flags: needinfo?(jocheng)
Dear Juwei,
Could you please provide spec for what is the correct behavior for usage app?
Thanks!
Hi Josh,
I used to memorize that when people switch SIM card, the app should report the data usage of one SIM only instead of a sum up for all SIM cards?
As you said,revise the behavior to clean old usage data would be make more sense.
Flags: needinfo?(jhuang)
Dear Marina,
Can you confirm this is the current behavior as described by JuWei?
Thank you very much!
Flags: needinfo?(jocheng) → needinfo?(marina.rodrigueziglesias)
Hi Josh,
Currently costcontrol shows always the traffic of the selected data sim and the wifi traffic. I thought that your question is about how identify which card is the active. If the user see the sum of both simcards, it's a bug.

Regards
Flags: needinfo?(marina.rodrigueziglesias)
Hi Marina,
According to https://bugzilla.mozilla.org/show_bug.cgi?id=1035568#c2
The current behavior does show sum of both SIM cards as you mention should be a bug. Can you fix this on bug 1117454? Thanks!
Flags: needinfo?(marina.rodrigueziglesias)
tracking-b2g: ? → ---
Flags: needinfo?(marina.rodrigueziglesias)
You need to log in before you can comment on or make changes to this bug.