Closed Bug 953150 Opened 11 years ago Closed 11 years ago

[DSDS] Unable to Reset PIN2 from FDN menu

Categories

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

Product:
Firefox OS Graveyard
Component:
Gaia::Settings
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(blocking-b2g:1.3+, b2g-v1.3 fixed, b2g-v1.3T fixed, b2g-v1.4 fixed)

Status:
RESOLVED FIXED
Milestone:
1.3 C2/1.4 S2(17jan)
Project Flags:
blocking-b2g 1.3+
Tracking Flags:
Tracking Status
b2g-v1.3 --- fixed
b2g-v1.3T --- fixed
b2g-v1.4 --- fixed

People

(Reporter: poojas, Assigned: kaze)

References

(
URL
)

Details

(Keywords: regression, Whiteboard: [CR 592872])

Attachments

(2 files)

pr.html
191 bytes, text/html
kaze
: review-
Details
patch
46 bytes, text/x-github-pull-request
arthurcc
: review+
Details | Review
User Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.24) Gecko/20111107 Ubuntu/10.04 (lucid) Firefox/3.6.24 (Nightly/Aurora) Build ID: 20111107172717 Steps to reproduce: Go to Settings -> Call Settings -> Select a SIM Card -> Fixed dialing number -> Reset PIN2 Actual results: No activity after clicking "Reset PIN2" button In adb logs, it shows below error E/GeckoConsole( 1150): [JavaScript Error: "TypeError: options is undefined" {file: "app://settings.gaiamobile.org/js/simcard_dialog.js" line: 402}] Expected results: After clicking reset button, it should have shown "Reset PIN2" window. And take input to change PIN2. In bug 921971 and bug 947907, its mentioned that fdn pin2 will work for both the sims appropriately from 1.4. But due to this bug, reset pin2 is not working even for single sim.
Attached file pr.html —
Attached patch fixes this issue. Please review
Attachment #8351482 - Flags: review?(kaze)
Attachment #8351482 - Flags: review?(ehung)
Whiteboard: [CR 592872]
Status: UNCONFIRMED → ASSIGNED
blocking-b2g: --- → 1.3+
Ever confirmed: true
No longer depends on: 953159
Hi kaze and ehung, Please provide feedback on this.
Keywords: regression
Keywords: regression
Fabien, did you get a chance to review the fix for this bug?
Flags: needinfo?(kaze)
Comment on attachment 8351482 [details] pr.html Redirect to Arthur, since he knows more on SIM2 and FDN.
Attachment #8351482 - Flags: review?(ehung) → review?(arthur.chen)
Assignee: nobody → poojas
Comment on attachment 8351482 [details] pr.html OMG, good catch!
Attachment #8351482 - Flags: review?(kaze)
Attachment #8351482 - Flags: review?(arthur.chen)
Attachment #8351482 - Flags: review+
Flags: needinfo?(kaze)
Hmm, looks like I’ve merged this too fast. It’s a regression of bug 928295: the `options' argument of the `show()' method was optional, but the patch provided for bug 928295 made it mandatory.
Back-out: https://github.com/mozilla-b2g/gaia/commit/beb7180 Stealing.
Assignee: poojas → kaze
Depends on: 928295
Keywords: regression
Attachment #8351482 - Flags: review+ → review-
Attached file patch —
Attachment #8357151 - Flags: review?(arthur.chen)
Arthur, Please help provide your input on the review at the earliest.
Flags: needinfo?(arthur.chen)
Comment on attachment 8357151 [details] [review] patch r=me. Thanks!
Attachment #8357151 - Flags: review?(arthur.chen) → review+
Flags: needinfo?(arthur.chen)
master: 8abf6df115305809c8b1e79e62834fb0a5b4c3aa
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
status-b2g-v1.3: --- → affected
status-firefox29: --- → fixed
Resolution: --- → FIXED
Uplifted 8abf6df115305809c8b1e79e62834fb0a5b4c3aa to: v1.3: dc36d5dbf966891bfdbbfd9821f07aafbf484a38
status-b2g-v1.3: affected → fixed
Flags: in-testsuite?
status-b2g-v1.3T: --- → fixed
status-b2g-v1.4: --- → fixed
status-firefox29: fixed → ---
Target Milestone: --- → 1.3 C2/1.4 S2(17jan)
Flags: in-moztrap?
URL: https://moztrap.mozilla.org/manage/ca...
Flags: in-moztrap? → in-moztrap+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: