Closed
Bug 936309
Opened 11 years ago
Closed 11 years ago
[FDN] "Add contact" is broken
Categories
(Firefox OS Graveyard :: Gaia::Settings, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kaze, Assigned: allstars.chh)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
allstars.chh
:
review+
|
Details | Diff | Splinter Review |
STR:
• Settings > Call Settings > Fixed dialing numbers
• make sure FDN is enabled
• Authorized numbers
• add a new FDN contact, tap [OK]
• enter the correct PIN2
Expected:
• contact is added to the FDN contact list
Actual:
• the FDN contact list is not changed
Tested with Gecko 28 (20131106164831) and Gaia master (ac2c668).
This might be a Gecko regression: it looks like the `navigator.mozIccManager.updateContact' method works fine when updating a contact but does not allow to create a new contact any more.
Reporter | ||
Comment 2•11 years ago
|
||
Thank *you*! :-)
Assignee | ||
Comment 3•11 years ago
|
||
Just flashed latest gaia, seems met Bug 936324 now.
Kaze, I cannot find the gaia commit in Comment 0, (ac2c668)
$git checkout ac2c668
error: pathspec 'ac2c668' did not match any file(s) known to git.
Reporter | ||
Comment 4•11 years ago
|
||
Ugh, 'ac2c668' is my local branch. :-$
I meant “latest Gaia”. E.g. this bug still happens with Gaia 275ca2c.
https://github.com/mozilla-b2g/gaia/commit/275ca2c
Assignee | ||
Comment 5•11 years ago
|
||
Kaze, can you show me your gecko commit as well? seems the keyboard is broken now and it's caused by gecko. (From the comment in Bug 936324) :(
Assignee | ||
Comment 6•11 years ago
|
||
(In reply to Yoshi Huang[:allstars.chh][:yoshi] from comment #5)
> Kaze, can you show me your gecko commit as well? seems the keyboard is
> broken now and it's caused by gecko. (From the comment in Bug 936324) :(
Never mind, I got it from Bug 936324 now.
Assignee | ||
Comment 7•11 years ago
|
||
The problem is in ril_worker.js, contact.id is undefined. So calling startsWith will cause TypeError.
Previous in Bug 915987 could work because in RILContentHelper.js, mozContact.id is "undefined", (typeof(contact.id) = "string"), now since MozContact has been WebIDLized, so RILContentHelper.js receives undefined (typeof(contact.id) = "undefined"), so causes this bug. However even mozContact is WebIDLized, when running Marionette test, RILContentHelper still gets "undefined" (string) when reading contact.id.
Will write a simple patch do check contact.id.
Assignee | ||
Comment 8•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Attachment #830127 -
Flags: review?(vyang)
Comment 9•11 years ago
|
||
Comment on attachment 830127 [details] [diff] [review]
Patch
Review of attachment 830127 [details] [diff] [review]:
-----------------------------------------------------------------
::: dom/system/gonk/ril_worker.js
@@ +918,5 @@
> }
>
> let contact = options.contact;
> let iccid = RIL.iccInfo.iccid;
> + if (typeof (contact.contactId) === "string" &&
nit: 'typeof' is an operator, just have |typeof foo === "string"|.
Attachment #830127 -
Flags: review?(vyang) → review+
Assignee | ||
Comment 10•11 years ago
|
||
Filed the Marionette problem in Bug 937440
Assignee | ||
Comment 11•11 years ago
|
||
using typeof foo
Attachment #830127 -
Attachment is obsolete: true
Attachment #830611 -
Flags: review+
Assignee | ||
Comment 12•11 years ago
|
||
Assignee | ||
Comment 13•11 years ago
|
||
Comment 14•11 years ago
|
||
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•