backends: when adding a persona by details, do not assume HashSet
authorSimon McVittie <simon.mcvittie@collabora.co.uk>
Tue, 2 Apr 2013 17:30:13 +0000 (18:30 +0100)
committerSimon McVittie <simon.mcvittie@collabora.co.uk>
Wed, 3 Apr 2013 15:55:20 +0000 (16:55 +0100)
All we need is a Set, so we shouldn't need to cast this strictly.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=687161
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Philip Withnall <philip@tecnocode.co.uk>
backends/eds/lib/edsf-persona-store.vala
backends/tracker/lib/trf-persona-store.vala

index 0a2c292..a2d84d9 100644 (file)
@@ -520,7 +520,7 @@ public class Edsf.PersonaStore : Folks.PersonaStore
             }
           else if (k == Folks.PersonaStore.detail_key (PersonaDetail.NOTES))
             {
-              var notes = (Gee.HashSet<NoteFieldDetails>) v.get_object ();
+              var notes = (Gee.Set<NoteFieldDetails>) v.get_object ();
               this._set_contact_notes (contact, notes);
             }
           else if (k == Folks.PersonaStore.detail_key (PersonaDetail.GENDER))
index 6b97503..25fb8e1 100644 (file)
@@ -587,7 +587,7 @@ public class Trf.PersonaStore : Folks.PersonaStore
             }
           else if (k == Folks.PersonaStore.detail_key (PersonaDetail.NOTES))
             {
-              var notes = (Gee.HashSet<NoteFieldDetails>) v.get_object ();
+              var notes = (Gee.Set<NoteFieldDetails>) v.get_object ();
               foreach (var n in notes)
                 {
                   builder.subject ("_:p");
@@ -605,7 +605,7 @@ public class Trf.PersonaStore : Folks.PersonaStore
             }
           else if (k == Folks.PersonaStore.detail_key (PersonaDetail.ROLES))
             {
-              var roles = (Gee.HashSet<RoleFieldDetails>) v.get_object ();
+              var roles = (Gee.Set<RoleFieldDetails>) v.get_object ();
 
               int roles_cnt = 0;
               foreach (var role_fd in roles)
@@ -697,7 +697,7 @@ public class Trf.PersonaStore : Folks.PersonaStore
             }
           else if (k == Folks.PersonaStore.detail_key (PersonaDetail.LOCAL_IDS))
             {
-              var local_ids = (Gee.HashSet<string>) v.get_object ();
+              var local_ids = (Gee.Set<string>) v.get_object ();
               string ids = Trf.PersonaStore.serialize_local_ids (local_ids);
 
               builder.subject ("_:folks_ids");