1163126 - [Contacts][Facebook] Facebook Contacts can be linked across multiple Phone Contacts; Phone Contacts cannot be linked to more than one Facebook C

Status: RESOLVED WONTFIX

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

Description

[Oliver Nelson [:oliverthor]]

Attachment: logcat_facebookcontacts.txt

Description:
When a user imports Facebook Contacts, they will observe the ability to 'link' a facebook contact to a phone contact they had previously. Currently, a phone contact may only be linked to one facebook contact, and they are considered the same contact. Odd behavior appears when the user links another contact to the same facebook contact. This can produce duplicate appearing entries in a user's Contacts app. Contacts and Facebook contacts appear to have a 1-1 relation, so allowing Facebook Accounts to be linked across multiple different contacts creates odd behavior.

PreReq:
* phone contacts
* facebook account to import contacts
Repro Steps:
1) Update a Flame to 20150508010203
2) Open the Contacts app
3) Create/Import-from-SIM phone contacts
4) Sync Facebook contacts ** contact list should now have Phone and Facebook contacts
5) Select a Contact-A and link to FB-Contact-A (names don't matter, just remember which contacts)
6) Select a Contact-B and link to FB-Contact-A
7) Observe Contacts list

Actual:
Linking the same Facebook Contact to multiple contacts creates duplicate appearing entries in contacts list

Expected:
Phone Contacts and Facebook Contacts maintain a 1-to-1 connection


Environmental Variables:
----------------------------------

Device: Flame 3.0
Build ID: 20150508010203
Gaia: bc5bfa18f795919b56b952bbf3637c235d0e13dc
Gecko: 356e735fa908
Gonk: a9f3f8fb8b0844724de32426b7bcc4e6dc4fa2ed
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0

Device: Flame 2.2
BuildID: 20150508002501
Gaia: 88d3ac2721a5484495c2ed60e4a068945f0de5aa
Gecko: 8ad16ebe659d
Gonk: ab265fb203390c70b8f2a054f38cf4b2f2dad70a
Version: 37.0 (2.2) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0

Device: Flame 2.1
BuildID: 20150508001200
Gaia: 3e7bd686ecd852f4dfa4605b45f558e6bd34f02a
Gecko: d85173eb5bf4
Gonk: ab265fb203390c70b8f2a054f38cf4b2f2dad70a
Version: 34.0 (2.1) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:34.0) Gecko/34.0 Firefox/34.0

Device: Flame 2.0
BuildID: 20150508000202
Gaia: 84898cadf28b1a1fcd03b726cff658de470282f0
Gecko: e67ed29e8ad0
Gonk: ab265fb203390c70b8f2a054f38cf4b2f2dad70a
Version: 32.0 (2.0) 
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:32.0) Gecko/32.0 Firefox/32.0
----------------------------------

Repro frequency: 5/5
See attached: 
video- https://youtu.be/3Lhnmz0s6MQ
screenshot
logcat

Comment 1

[Oliver Nelson [:oliverthor]]

Attachment: 2015-05-08-12-46-12.png

Comment 2

[Johan Lorenzo [:jlorenzo]]

This is a problem on Facebook's side. They use a text input instead of an email input like on the Google sign-in page. I think they won't change it, because they need to accept phone numbers too.

Moving this bug to tech evangelism.

Comment 3

[Johan Lorenzo [:jlorenzo]]

[Tracking Requested - why for this release]: Edge case

Comment 4

[Johan Lorenzo [:jlorenzo]]

This is an edge case, but this doesn't require a graphical change[1], in my opinion.

[1] https://bugzilla.mozilla.org/describekeywords.cgi

Comment 5

[Johan Lorenzo [:jlorenzo]]

Facebook was dropped in bug 1178891.