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 653602 - Gnome Contacts shouldn't assume TpfPersonas contain valid TpContacts
Gnome Contacts shouldn't assume TpfPersonas contain valid TpContacts
Status: RESOLVED FIXED
Product: gnome-contacts
Classification: Core
Component: general
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Alexander Larsson
Alexander Larsson
Depends on:
Blocks: 652643
 
 
Reported: 2011-06-28 22:35 UTC by Travis Reitter
Modified: 2011-09-01 19:02 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Un-break gnome-contacts with the Tpf.Persona cache (4.15 KB, patch)
2011-06-28 22:39 UTC, Travis Reitter
none Details | Review

Description Travis Reitter 2011-06-28 22:35:46 UTC
It's been fair to assume this historically, but that will no longer be true
once we've merged the fix for bug #652643. Cached Tpf.Personas will contain a
null TpContact (ie, they will be for display purposes only, not for actions).
Comment 1 Travis Reitter 2011-06-28 22:39:22 UTC
Created attachment 190895 [details] [review]
Un-break gnome-contacts with the Tpf.Persona cache

Patches from branch:

http://cgit.collabora.com/git/user/treitter/gnome-contacts.git/log/?h=bgo653602-tpf-persona-cache
Comment 2 Alexander Larsson 2011-06-29 07:16:53 UTC
I picked up patch 2 and 4 from that series. But Patch 1 and 3 are about Contacts.Contact, not TpContact, which seems unrelated. Any reason these should be affected by a lacking TpContact, or was this just a mistake?
Comment 3 Travis Reitter 2011-06-29 16:15:26 UTC
(In reply to comment #2)
> I picked up patch 2 [details] and 4 from that series. But Patch 1 [details] and 3 are about
> Contacts.Contact, not TpContact, which seems unrelated. Any reason these should
> be affected by a lacking TpContact, or was this just a mistake?

Those two aren't strictly required for this current bug but came up in the process. Would you like me to file a bug for each of them?
Comment 4 Alexander Larsson 2011-06-29 18:45:00 UTC
I don't see how there could ever come up an Individual that doesn't have a Contact, because all the Individual we handle are from a single aggregator, and in as soon as individuals are added to that we create a Contacts.Contact for it. 

But, if you ever find g-c looking up non-existing Contacts from individual, file a new bug with the details.