Closed
Bug 901887
Opened 9 years ago
Closed 9 years ago
[RIL][Contacts] Saving to SIM doesn't work
Categories
(Firefox OS Graveyard :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: arcturus, Assigned: airpingu)
References
Details
Attachments
(1 file, 1 obsolete file)
1.19 KB,
patch
|
airpingu
:
review+
|
Details | Diff | Splinter Review |
+++ This bug was initially created as a clone of Bug #895886 +++ |var request = icc.updateContact('adn', aContact);| And setting the |onsuccess| and |onerror| on the DOMRequest, we don't go through any of the callbacks. When enabling the RIL logs we don't get much information, while performing the icc.updateContact we just get: D/memalloc( 108): /dev/pmem: Allocated buffer base:0x48b00000 size:614400 offset:6144000 fd:80 D/memalloc( 487): /dev/pmem: Mapped buffer base:0x46100000 size:6758400 offset:6144000 fd:34 D/memalloc( 108): /dev/pmem: Allocated buffer base:0x48b00000 size:614400 offset:6758400 fd:104 D/memalloc( 487): /dev/pmem: Mapped buffer base:0x46864000 size:7372800 offset:6758400 fd:43 D/memalloc( 108): /dev/pmem: Allocated buffer base:0x48b00000 size:8192 offset:7372800 fd:111 D/memalloc( 487): /dev/pmem: Mapped buffer base:0x46f86000 size:7380992 offset:7372800 fd:47 D/memalloc( 108): /dev/pmem: Allocated buffer base:0x48b00000 size:8192 offset:7380992 fd:118 D/memalloc( 487): /dev/pmem: Mapped buffer base:0x476a0000 size:7389184 offset:7380992 fd:50
Comment 1•9 years ago
|
||
Hi, Francisco Can you show me your gaia repository and branch should I can reproduce by myself?
Reporter | ||
Comment 2•9 years ago
|
||
Right, perhaps I'm doing something terrible wrong there :) Here is how I'm testing: https://github.com/arcturus/gaia/tree/contacts-export Go to settings and choose the first export option. Thanks!
Comment 3•9 years ago
|
||
Hi, Francisco I've tried your branch, but seems the error is from your gaia app, "contact.email is undefined", I try to fix the error, then exporting is fine, except it seems the Gaia app doesn't export the correct contact. (for example, I select contact A, and from ril log, it seems contact B is exported).
Assignee | ||
Comment 4•9 years ago
|
||
I can reproduce this issue. Let me take this. :)
Assignee: nobody → gene.lian
blocking-b2g: --- → koi?
OS: Mac OS X → Gonk (Firefox OS)
Hardware: x86 → ARM
Assignee | ||
Comment 5•9 years ago
|
||
This bug is really hard to catch because the console cannot display any error message when contact.anr is not available. Anyway, we need to add this check to ensure it can be returned eventually because we don't want to update the anr for sure.
Attachment #786980 -
Flags: review?(allstars.chh)
Comment 6•9 years ago
|
||
Comment on attachment 786980 [details] [diff] [review] Patch Review of attachment 786980 [details] [diff] [review]: ----------------------------------------------------------------- ::: dom/system/gonk/ril_worker.js @@ +12593,5 @@ > > // Check if contact has additional properties (email, anr, ...etc) need > // to be updated as well. > if ((field === USIM_PBR_EMAIL && !contact.email) || > + (field === USIM_PBR_ANR0 && (!contact.anr || !contact.anr[0]))) { I think using Array.isArray to check anr is better.
Attachment #786980 -
Flags: review?(allstars.chh) → review+
Comment 7•9 years ago
|
||
I think we also need to update xpcshell test case to test this. I'll file another bug to test this. Bug 902788
Assignee | ||
Comment 8•9 years ago
|
||
Thanks Yoshi for the review and providing the test! Addressing comments. Ready to land.
Attachment #786980 -
Attachment is obsolete: true
Attachment #787373 -
Flags: review+
Assignee | ||
Comment 9•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/86091b087cb6
Comment 10•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/86091b087cb6
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
blocking-b2g: koi? → ---
You need to log in
before you can comment on or make changes to this bug.
Description
•