Closed Bug 898297 Opened 9 years ago Closed 9 years ago

[Contacts] Merging process stucks and never ends

Categories

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

x86_64
Windows 7
defect
Not set
normal

Tracking

(blocking-b2g:koi+)

VERIFIED FIXED
blocking-b2g koi+

People

(Reporter: isabelrios, Assigned: salva)

References

Details

(Whiteboard: [u=commsapps-user c=contacts p=0])

Attachments

(2 files)

Attached file merging.txt
Unagi device master brach:
Gecko-11373a6
Gaia-ccf877b

This has been reproduced randomly trying to merge different contacts with different options. 
It always happens though when trying to merge two contacts with:
Same Name
Same Last Name
Same Phone number (but one with prefix and the other without)

Adb logcat attached.
I will try to get more logs under other circunstances in case the root cause of the error is not the same.
Blocks: 895799
Blocks: 895797
No longer blocks: 895799
blocking-b2g: --- → koi?
blocking-b2g: koi? → koi+
Whiteboard: [u=commsapps-user c=contacts p=0]
Assignee: nobody → salva
Find here a reproducible STR:

1- Open contacts and add a new one with name, last name. Save the contact.
2- Add another contact with the same name and last name and a mobile number. Save the contact.
3- When `Duplicates found` window appears, tap on `Merge`

Expected:
The merging process starts and finalizes merging the selected contact and the new one being added into one.

Actual:
The merging process starts but never ends.
May Travis fail on Communications but it is the dialer part and master is currently failing on the same tests.
Attachment #793432 - Flags: review?(francisco.jordano)
Small comment on github.

Thanks!
Attachment #793432 - Flags: review?(francisco.jordano) → review+
Comment on attachment 793432 [details]
Fixing the fake matching result, a convenient renaming and some related documentation added.

Applying KISS ,)

Please, review again and thank you very much for your feedback.
Attachment #793432 - Flags: review+ → review?(francisco.jordano)
Comment on attachment 793432 [details]
Fixing the fake matching result, a convenient renaming and some related documentation added.

Thanks Salva!
Attachment #793432 - Flags: review?(francisco.jordano) → review+
Master: 7e83343a1ea9050216dfa150d0897a7d858225c1
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Verified, master 09/23 build:
Gecko-f156bef
Gaia-3786676
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.