After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 687049 - Add to My Contacts creates duplicate contacts
Add to My Contacts creates duplicate contacts
Status: RESOLVED OBSOLETE
Product: gnome-contacts
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: GNOME Contacts maintainer(s)
GNOME Contacts maintainer(s)
Depends on: 687050
Blocks:
 
 
Reported: 2012-10-28 17:30 UTC by Giovanni Campagna
Modified: 2013-03-21 10:44 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Contacts: fix changing a Google Other contact into a My Contact (5.87 KB, patch)
2012-10-28 17:39 UTC, Giovanni Campagna
committed Details | Review
Squash a critical warning when the application is closed. (978 bytes, patch)
2012-10-28 17:40 UTC, Giovanni Campagna
committed Details | Review

Description Giovanni Campagna 2012-10-28 17:30:56 UTC
If you click on Add to My Contacts from a Google Other contact, you end up with a duplicate "holder" contact, whose only purpose is to keep linkable properties.
What's worse, the other contact is still a separate persona, so the view just changes from "Add to My Contacts" to "Unlink", with all the details still under the Other Contacts.
While this makes sense once you understand the folks implementation details, it is definitely weird from the user POV. Instead, I think gnome-contacts should change the appropriate details in the Google persona directly.
Comment 1 Giovanni Campagna 2012-10-28 17:39:59 UTC
Created attachment 227465 [details] [review]
Contacts: fix changing a Google Other contact into a My Contact

Previously, clicking on the Add to My Contacts button for a Google
Other contact would attempt a link operation with nothing, that
would end up creating a new scratch contact to link into, which
by chance would also have the magic in-google-personal-group bit set.
Instead, use the new API in Folks to change it directly, to avoid
creating a duplicate contact.
Comment 2 Giovanni Campagna 2012-10-28 17:40:06 UTC
Created attachment 227466 [details] [review]
Squash a critical warning when the application is closed.

Destroying the EggListBox will emit child-selected with a null child.
Comment 3 Alexander Larsson 2012-11-12 11:32:00 UTC
Attachment 227465 [details] pushed as 1b03e01 - Contacts: fix changing a Google Other contact into a My Contact
Attachment 227466 [details] pushed as 4b5ffd3 - Squash a critical warning when the application is closed.
Comment 4 Alexander Larsson 2012-11-12 11:47:29 UTC
reverted this and applied it again on master after having branched for 3.6 first, as we need a new eds for it to work.
Comment 5 Allan Day 2013-03-21 10:44:50 UTC
We don't have the concept of "My Contacts" any more. Closing as obsolete.