e-d-s: we should use the Persona's iid as local-ids
authorRaul Gutierrez Segales <rgs@collabora.co.uk>
Mon, 29 Aug 2011 17:25:00 +0000 (18:25 +0100)
committerRaul Gutierrez Segales <rgs@collabora.co.uk>
Wed, 31 Aug 2011 20:32:13 +0000 (21:32 +0100)
If we don't and use the contact_id as before, linking Personas
from different PersonaStores won't work.

Fixes: https://bugzilla.gnome.org/show_bug.cgi?id=657635

NEWS
backends/eds/lib/edsf-persona.vala

diff --git a/NEWS b/NEWS
index 602f4ee..7a401c0 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -5,6 +5,7 @@ Bugs fixed:
 * Bug 645056 — TpLowlevel library should have only static public functions
 * Bug 653777 — Would be nice to have a helper function to create a writable
   persona
+* Bug 657635 — Linking personas from different (e-d-s) stores is not working
 
 API changes:
 * Add PersonaStore:always-writeable-properties property
index 9f48a32..4dc8043 100644 (file)
@@ -161,9 +161,9 @@ public class Edsf.Persona : Folks.Persona,
     {
       get
         {
-          if (this._local_ids.contains (this.contact_id) == false)
+          if (this._local_ids.contains (this.iid) == false)
             {
-              this._local_ids.add (this.contact_id);
+              this._local_ids.add (this.iid);
             }
           return this._local_ids_ro;
         }