886692 - [Settings]CB service enable/disable in Settings

Status: RESOLVED DUPLICATE of bug 882175

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

Description

[Leo]

1. Title : CB servoce enable/disable in Settings
2. Precondition : 
3. Tester's Action : Trun on/off CB service
4. Detailed Symptom : 
When turn on, CB message can't receive
When turn off, CB message can receive
5. Expected : When turn on, CB message should be received. Please check.
6. Reproducibility: Y
   - Frequency Rate : 100%
7.Gaia Master/v1-train : 
8. Version Info
Build ID : 20130616070209
GAIA : f2d6ed54a236e6e3b94f0abad9f0dacb8a1cc7b3
GECKO : be276cf55ce160bca09f36d9ca679a2ae20ea7cc

Comment 1

[Anthony Ricaud (:rik)]

This already exists in Settings > Call Settings > Cell broadcast service.

Comment 2

[Leo]

The problem is that it displays wrong icon on the menu all the time. 
When the Cell broadcast service is disabled, the icon is show "ON" instead of "OFF and vise versa. 
Please check it again.

Comment 3

[(Inactive after June) George Duan [:gduan] [:喬智]]

Attachment: https://github.com/mozilla-b2g/gaia/pull/10667

Comment 4

[Dale Harvey (:daleharvey)]

Comment on attachment 768250 [details]
https://github.com/mozilla-b2g/gaia/pull/10667

Stealing the review, this is either a diagnosing error or one with the settings value, I dont think the settings code is wrong here

Comment 5

[hyuna.cho]

The patch of Bug 882175 is working well in leo.
Please check.

Comment 6

[(Inactive after June) George Duan [:gduan] [:喬智]]

When tracing code,
the initial value for ril.cellbroadcast.disabled is set to false , which equals to enable cellbraodcast , the icon shows correctly. And also, after switching , the ril.cellbroadcast.disabled is set to true . I will track whether backend is also set correctly or not , however the original code works fine as expected.

Comment 7

[Leo]

Bug 882175 patch was not included in leo when CB service enable/disable is strange.
After applying the patch, it works well in leo.

Comment 8

[Dale Harvey (:daleharvey)]

Comment on attachment 768250 [details]
https://github.com/mozilla-b2g/gaia/pull/10667

Clearing the review, figuring out the root cause of this would be useful as we just blocking+'d and almost committed a patch that regressed working functionality.

Also tests (my bad).