Closed Bug 944575 Opened 12 years ago Closed 12 years ago

When exporting a local contact to SIM save the iccContactId in order to align with bug 943234

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:koi+, b2g-v1.2 fixed)

VERIFIED FIXED
1.2 C6(Dec6)
blocking-b2g koi+
Tracking Status
b2g-v1.2 --- fixed

People

(Reporter: jmcf, Assigned: jmcf)

References

Details

Attachments

(2 files)

We need to store the iccContactId provided in a local contact (not previously imported from the SIM) - export flow in order to reuse it and not to duplicate the contact if it is exported two times.
Attached file 14198.html —
for testing it is needed the corresponding Gecko patch
Attachment #8340199 - Flags: review?(francisco.jordano)
Comment on attachment 8340199 [details] 14198.html Looking good to me, r+, is there any chance to add a couple of extra unit test for the new methods? Thanks!
Attachment #8340199 - Flags: review?(francisco.jordano) → review+
Blocks: 922988
landed in master, including the tests requested by Francisco https://github.com/mozilla-b2g/gaia/commit/a9468b0ad5923badb7130a60b639914d9b227985
for the same reasons as bug 943234 this should be also koi+
blocking-b2g: --- → koi?
koi+ for broken feature similar to 943234.https://bugzilla.mozilla.org/show_bug.cgi?id=943234
blocking-b2g: koi? → koi+
Attached file 14412.html —
patch for v1.2
Attachment #8343075 - Flags: review?(francisco.jordano)
Comment on attachment 8343075 [details] 14412.html Looking good to me, tested on the phone and working fine. Please merge once travis is green :) Thanks Jose!
Attachment #8343075 - Flags: review?(francisco.jordano) → review+
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Keywords: verifyme
Target Milestone: --- → 1.2 C6(Dec6)
Verified with 12/16 v1.2 build: Gecko-f6f325b Gaia-fcf1c2f If the same contact manually created in the contact list is exported twice to the SIM Card, it is not duplicated in the SIM Card.
Status: RESOLVED → VERIFIED
Keywords: verifyme
removing status-b2g-v1.3 flag since the patch was landed before v1.3 branch creation (https://bugzilla.mozilla.org/show_bug.cgi?id=944575#c3)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: