Mon, 28 Nov 2022 23:48:33 -0600
Split PurpleContactInfo out of PurpleContact
This change separates all of the data away from the runtime information which
makes it easier to store contacts with everything.
Testing Done:
Ran the unit tests and sent a few messages with ircv3 and demo protocols.
Reviewed at https://reviews.imfreedom.org/r/2091/
| 41749 | 1 | /* |
| 2 | * Purple - Internet Messaging Library | |
| 3 | * Copyright (C) Pidgin Developers <devel@pidgin.im> | |
| 4 | * | |
| 5 | * This program is free software; you can redistribute it and/or modify | |
| 6 | * it under the terms of the GNU General Public License as published by | |
| 7 | * the Free Software Foundation; either version 2 of the License, or | |
| 8 | * (at your option) any later version. | |
| 9 | * | |
| 10 | * This program is distributed in the hope that it will be useful, | |
| 11 | * but WITHOUT ANY WARRANTY; without even the implied warranty of | |
| 12 | * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
| 13 | * GNU General Public License for more details. | |
| 14 | * | |
| 15 | * You should have received a copy of the GNU General Public License | |
| 16 | * along with this program; if not, see <https://www.gnu.org/licenses/>. | |
| 17 | */ | |
| 18 | ||
| 19 | #include "purpleperson.h" | |
| 20 | ||
| 21 | struct _PurplePerson { | |
| 22 | GObject parent; | |
| 23 | ||
| 24 | gchar *id; | |
| 25 | ||
| 26 | gchar *alias; | |
| 27 | GdkPixbuf *avatar; | |
| 28 | PurpleTags *tags; | |
| 29 | ||
| 30 | GPtrArray *contacts; | |
| 31 | }; | |
| 32 | ||
| 33 | enum { | |
| 34 | PROP_0, | |
| 35 | PROP_ID, | |
| 36 | PROP_ALIAS, | |
| 37 | PROP_AVATAR, | |
| 38 | PROP_TAGS, | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
39 | PROP_PRIORITY_CONTACT_INFO, |
| 41749 | 40 | N_PROPERTIES |
| 41 | }; | |
| 42 | static GParamSpec *properties[N_PROPERTIES] = {NULL, }; | |
| 43 | ||
| 44 | /****************************************************************************** | |
| 45 | * Helpers | |
| 46 | *****************************************************************************/ | |
| 47 | static void | |
| 48 | purple_person_set_id(PurplePerson *person, const gchar *id) { | |
| 49 | g_return_if_fail(PURPLE_IS_PERSON(person)); | |
| 50 | ||
| 51 | g_free(person->id); | |
| 52 | ||
| 53 | if(id != NULL) { | |
| 54 | person->id = g_strdup(id); | |
| 55 | } else { | |
| 56 | person->id = g_uuid_string_random(); | |
| 57 | } | |
| 58 | ||
| 59 | g_object_notify_by_pspec(G_OBJECT(person), properties[PROP_ID]); | |
| 60 | } | |
| 61 | ||
| 62 | static gint | |
| 63 | purple_person_contact_compare(gconstpointer a, gconstpointer b) { | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
64 | PurpleContactInfo *c1 = *(PurpleContactInfo **)a; |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
65 | PurpleContactInfo *c2 = *(PurpleContactInfo **)b; |
| 41749 | 66 | PurplePresence *p1 = NULL; |
| 67 | PurplePresence *p2 = NULL; | |
| 68 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
69 | p1 = purple_contact_info_get_presence(c1); |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
70 | p2 = purple_contact_info_get_presence(c2); |
| 41749 | 71 | |
| 72 | return purple_presence_compare(p1, p2); | |
| 73 | } | |
| 74 | ||
| 75 | static void | |
| 76 | purple_person_sort_contacts(PurplePerson *person) { | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
77 | PurpleContactInfo *original_priority = NULL; |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
78 | PurpleContactInfo *new_priority = NULL; |
| 41749 | 79 | guint n_items = person->contacts->len; |
| 80 | ||
| 81 | if(n_items <= 1) { | |
| 82 | g_object_notify_by_pspec(G_OBJECT(person), | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
83 | properties[PROP_PRIORITY_CONTACT_INFO]); |
| 41749 | 84 | |
| 85 | g_list_model_items_changed(G_LIST_MODEL(person), 0, n_items, n_items); | |
| 86 | ||
| 87 | return; | |
| 88 | } | |
| 89 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
90 | original_priority = purple_person_get_priority_contact_info(person); |
| 41749 | 91 | |
| 92 | g_ptr_array_sort(person->contacts, purple_person_contact_compare); | |
| 93 | ||
| 94 | /* Tell the list we update our stuff. */ | |
| 95 | g_list_model_items_changed(G_LIST_MODEL(person), 0, n_items, n_items); | |
| 96 | ||
| 97 | /* See if the priority contact changed. */ | |
| 98 | new_priority = g_ptr_array_index(person->contacts, 0); | |
| 99 | if(original_priority != new_priority) { | |
| 100 | g_object_notify_by_pspec(G_OBJECT(person), | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
101 | properties[PROP_PRIORITY_CONTACT_INFO]); |
| 41749 | 102 | } |
| 103 | } | |
| 104 | ||
| 105 | /****************************************************************************** | |
| 106 | * Callbacks | |
| 107 | *****************************************************************************/ | |
| 108 | static void | |
| 109 | purple_person_presence_notify_cb(G_GNUC_UNUSED GObject *obj, | |
| 110 | G_GNUC_UNUSED GParamSpec *pspec, | |
| 111 | gpointer data) | |
| 112 | { | |
| 113 | purple_person_sort_contacts(data); | |
| 114 | } | |
| 115 | ||
| 116 | /****************************************************************************** | |
| 117 | * GListModel Implementation | |
| 118 | *****************************************************************************/ | |
| 119 | static GType | |
| 120 | purple_person_get_item_type(G_GNUC_UNUSED GListModel *list) { | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
121 | return PURPLE_TYPE_CONTACT_INFO; |
| 41749 | 122 | } |
| 123 | ||
| 124 | static guint | |
| 125 | purple_person_get_n_items(GListModel *list) { | |
| 126 | PurplePerson *person = PURPLE_PERSON(list); | |
| 127 | ||
| 128 | return person->contacts->len; | |
| 129 | } | |
| 130 | ||
| 131 | static gpointer | |
| 132 | purple_person_get_item(GListModel *list, guint position) { | |
| 133 | PurplePerson *person = PURPLE_PERSON(list); | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
134 | PurpleContactInfo *info = NULL; |
| 41749 | 135 | |
| 136 | if(position < person->contacts->len) { | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
137 | info = g_ptr_array_index(person->contacts, position); |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
138 | g_object_ref(info); |
| 41749 | 139 | } |
| 140 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
141 | return info; |
| 41749 | 142 | } |
| 143 | ||
| 144 | static void | |
| 145 | purple_person_list_model_init(GListModelInterface *iface) { | |
| 146 | iface->get_item_type = purple_person_get_item_type; | |
| 147 | iface->get_n_items = purple_person_get_n_items; | |
| 148 | iface->get_item = purple_person_get_item; | |
| 149 | } | |
| 150 | ||
| 151 | /****************************************************************************** | |
| 152 | * GObject Implementation | |
| 153 | *****************************************************************************/ | |
| 154 | G_DEFINE_TYPE_EXTENDED(PurplePerson, purple_person, G_TYPE_OBJECT, 0, | |
| 155 | G_IMPLEMENT_INTERFACE(G_TYPE_LIST_MODEL, | |
| 156 | purple_person_list_model_init)) | |
| 157 | ||
| 158 | static void | |
| 159 | purple_person_get_property(GObject *obj, guint param_id, GValue *value, | |
| 160 | GParamSpec *pspec) | |
| 161 | { | |
| 162 | PurplePerson *person = PURPLE_PERSON(obj); | |
| 163 | ||
| 164 | switch(param_id) { | |
| 165 | case PROP_ID: | |
| 166 | g_value_set_string(value, purple_person_get_id(person)); | |
| 167 | break; | |
| 168 | case PROP_ALIAS: | |
| 169 | g_value_set_string(value, purple_person_get_alias(person)); | |
| 170 | break; | |
| 171 | case PROP_AVATAR: | |
| 172 | g_value_set_object(value, purple_person_get_avatar(person)); | |
| 173 | break; | |
| 174 | case PROP_TAGS: | |
| 175 | g_value_set_object(value, purple_person_get_tags(person)); | |
| 176 | break; | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
177 | case PROP_PRIORITY_CONTACT_INFO: |
| 41749 | 178 | g_value_set_object(value, |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
179 | purple_person_get_priority_contact_info(person)); |
| 41749 | 180 | break; |
| 181 | default: | |
| 182 | G_OBJECT_WARN_INVALID_PROPERTY_ID(obj, param_id, pspec); | |
| 183 | break; | |
| 184 | } | |
| 185 | } | |
| 186 | ||
| 187 | static void | |
| 188 | purple_person_set_property(GObject *obj, guint param_id, const GValue *value, | |
| 189 | GParamSpec *pspec) | |
| 190 | { | |
| 191 | PurplePerson *person = PURPLE_PERSON(obj); | |
| 192 | ||
| 193 | switch(param_id) { | |
| 194 | case PROP_ID: | |
| 195 | purple_person_set_id(person, g_value_get_string(value)); | |
| 196 | break; | |
| 197 | case PROP_ALIAS: | |
| 198 | purple_person_set_alias(person, g_value_get_string(value)); | |
| 199 | break; | |
| 200 | case PROP_AVATAR: | |
| 201 | purple_person_set_avatar(person, g_value_get_object(value)); | |
| 202 | break; | |
| 203 | default: | |
| 204 | G_OBJECT_WARN_INVALID_PROPERTY_ID(obj, param_id, pspec); | |
| 205 | break; | |
| 206 | } | |
| 207 | } | |
| 208 | ||
| 209 | static void | |
| 210 | purple_person_dispose(GObject *obj) { | |
| 211 | PurplePerson *person = PURPLE_PERSON(obj); | |
| 212 | ||
| 213 | g_clear_object(&person->avatar); | |
| 214 | g_clear_object(&person->tags); | |
| 215 | ||
| 216 | if(person->contacts != NULL) { | |
| 217 | g_ptr_array_free(person->contacts, TRUE); | |
| 218 | person->contacts = NULL; | |
| 219 | } | |
| 220 | ||
| 221 | G_OBJECT_CLASS(purple_person_parent_class)->dispose(obj); | |
| 222 | } | |
| 223 | ||
| 224 | static void | |
| 225 | purple_person_finalize(GObject *obj) { | |
| 226 | PurplePerson *person = PURPLE_PERSON(obj); | |
| 227 | ||
| 228 | g_clear_pointer(&person->id, g_free); | |
| 229 | g_clear_pointer(&person->alias, g_free); | |
| 230 | ||
| 231 | G_OBJECT_CLASS(purple_person_parent_class)->finalize(obj); | |
| 232 | } | |
| 233 | ||
| 234 | static void | |
| 235 | purple_person_constructed(GObject *obj) { | |
| 236 | PurplePerson *person = NULL; | |
| 237 | ||
| 238 | G_OBJECT_CLASS(purple_person_parent_class)->constructed(obj); | |
| 239 | ||
| 240 | person = PURPLE_PERSON(obj); | |
| 241 | if(person->id == NULL) { | |
| 242 | purple_person_set_id(person, NULL); | |
| 243 | } | |
| 244 | } | |
| 245 | ||
| 246 | static void | |
| 247 | purple_person_init(PurplePerson *person) { | |
| 248 | person->tags = purple_tags_new(); | |
| 249 | person->contacts = g_ptr_array_new_full(0, (GDestroyNotify)g_object_unref); | |
| 250 | } | |
| 251 | ||
| 252 | static void | |
| 253 | purple_person_class_init(PurplePersonClass *klass) { | |
| 254 | GObjectClass *obj_class = G_OBJECT_CLASS(klass); | |
| 255 | ||
| 256 | obj_class->get_property = purple_person_get_property; | |
| 257 | obj_class->set_property = purple_person_set_property; | |
| 258 | obj_class->constructed = purple_person_constructed; | |
| 259 | obj_class->dispose = purple_person_dispose; | |
| 260 | obj_class->finalize = purple_person_finalize; | |
| 261 | ||
| 262 | /** | |
| 263 | * PurplePerson:id: | |
| 264 | * | |
| 265 | * The protocol specific id for the contact. | |
| 266 | * | |
| 267 | * Since: 3.0.0 | |
| 268 | */ | |
| 269 | properties[PROP_ID] = g_param_spec_string( | |
| 270 | "id", "id", | |
| 271 | "The id of this contact", | |
| 272 | NULL, | |
| 273 | G_PARAM_READWRITE | G_PARAM_CONSTRUCT_ONLY | G_PARAM_STATIC_STRINGS); | |
| 274 | ||
| 275 | /** | |
| 276 | * PurplePerson:alias: | |
| 277 | * | |
| 278 | * The alias for this person. This is controlled by the libpurple user. | |
| 279 | * | |
| 280 | * Since: 3.0.0 | |
| 281 | */ | |
| 282 | properties[PROP_ALIAS] = g_param_spec_string( | |
| 283 | "alias", "alias", | |
| 284 | "The alias of this person.", | |
| 285 | NULL, | |
| 286 | G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS); | |
| 287 | ||
| 288 | /** | |
| 289 | * PurplePerson:avatar: | |
| 290 | * | |
| 291 | * The avatar for this person. This is controlled by the libpurple user, | |
| 292 | * which they can use to set a custom avatar. | |
| 293 | * | |
| 294 | * Since: 3.0.0 | |
| 295 | */ | |
| 296 | properties[PROP_AVATAR] = g_param_spec_object( | |
| 297 | "avatar", "avatar", | |
| 298 | "The avatar of this person", | |
| 299 | GDK_TYPE_PIXBUF, | |
| 300 | G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS); | |
| 301 | ||
| 302 | /** | |
| 303 | * PurplePerson:tags: | |
| 304 | * | |
| 305 | * The [class@Purple.Tags] for this person. | |
| 306 | * | |
| 307 | * Since: 3.0.0 | |
| 308 | */ | |
| 309 | properties[PROP_TAGS] = g_param_spec_object( | |
| 310 | "tags", "tags", | |
| 311 | "The tags for this person", | |
| 312 | PURPLE_TYPE_TAGS, | |
| 313 | G_PARAM_READABLE | G_PARAM_STATIC_STRINGS); | |
| 314 | ||
| 315 | /** | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
316 | * PurplePerson:priority-contact-info: |
| 41749 | 317 | * |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
318 | * The [class@Purple.ContactInfo] that currently has the highest priority. |
| 41749 | 319 | * |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
320 | * This is used by user interfaces to determine which |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
321 | * [class@Purple.ContactInfo] to use when messaging and so on. |
| 41749 | 322 | * |
| 323 | * Since: 3.0.0 | |
| 324 | */ | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
325 | properties[PROP_PRIORITY_CONTACT_INFO] = g_param_spec_object( |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
326 | "priority-contact-info", "priority-contact-info", |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
327 | "The priority contact info for the person", |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
328 | PURPLE_TYPE_CONTACT_INFO, |
| 41749 | 329 | G_PARAM_READABLE | G_PARAM_STATIC_STRINGS); |
| 330 | ||
| 331 | g_object_class_install_properties(obj_class, N_PROPERTIES, properties); | |
| 332 | } | |
| 333 | ||
| 334 | /****************************************************************************** | |
| 335 | * Public API | |
| 336 | *****************************************************************************/ | |
| 337 | PurplePerson * | |
| 338 | purple_person_new(void) { | |
| 339 | return g_object_new(PURPLE_TYPE_PERSON, NULL); | |
| 340 | } | |
| 341 | ||
| 342 | const gchar * | |
| 343 | purple_person_get_id(PurplePerson *person) { | |
| 344 | g_return_val_if_fail(PURPLE_IS_PERSON(person), NULL); | |
| 345 | ||
| 346 | return person->id; | |
| 347 | } | |
| 348 | ||
| 349 | const gchar * | |
| 350 | purple_person_get_alias(PurplePerson *person) { | |
| 351 | g_return_val_if_fail(PURPLE_IS_PERSON(person), NULL); | |
| 352 | ||
| 353 | return person->alias; | |
| 354 | } | |
| 355 | ||
| 356 | void | |
| 357 | purple_person_set_alias(PurplePerson *person, const gchar *alias) { | |
| 358 | g_return_if_fail(PURPLE_IS_PERSON(person)); | |
| 359 | ||
| 360 | g_free(person->alias); | |
| 361 | person->alias = g_strdup(alias); | |
| 362 | ||
| 363 | g_object_notify_by_pspec(G_OBJECT(person), properties[PROP_ALIAS]); | |
| 364 | } | |
| 365 | ||
| 366 | GdkPixbuf * | |
| 367 | purple_person_get_avatar(PurplePerson *person) { | |
| 368 | g_return_val_if_fail(PURPLE_IS_PERSON(person), NULL); | |
| 369 | ||
| 370 | return person->avatar; | |
| 371 | } | |
| 372 | ||
| 373 | void | |
| 374 | purple_person_set_avatar(PurplePerson *person, GdkPixbuf *avatar) { | |
| 375 | g_return_if_fail(PURPLE_IS_PERSON(person)); | |
| 376 | ||
| 377 | if(g_set_object(&person->avatar, avatar)) { | |
| 378 | g_object_notify_by_pspec(G_OBJECT(person), properties[PROP_AVATAR]); | |
| 379 | } | |
| 380 | } | |
| 381 | ||
| 382 | PurpleTags * | |
| 383 | purple_person_get_tags(PurplePerson *person) { | |
| 384 | g_return_val_if_fail(PURPLE_IS_PERSON(person), NULL); | |
| 385 | ||
| 386 | return person->tags; | |
| 387 | } | |
| 388 | ||
| 389 | void | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
390 | purple_person_add_contact_info(PurplePerson *person, |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
391 | PurpleContactInfo *info) |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
392 | { |
| 41749 | 393 | PurplePresence *presence = NULL; |
| 394 | ||
| 395 | g_return_if_fail(PURPLE_IS_PERSON(person)); | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
396 | g_return_if_fail(PURPLE_IS_CONTACT_INFO(info)); |
| 41749 | 397 | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
398 | g_ptr_array_add(person->contacts, g_object_ref(info)); |
| 41749 | 399 | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
400 | presence = purple_contact_info_get_presence(info); |
|
41771
c5877e2c93f2
Create and add PurpleContacts to the manager when purple_buddy_new is called
Gary Kramlich <grim@reaperworld.com>
parents:
41761
diff
changeset
|
401 | g_signal_connect_object(presence, "notify", |
|
c5877e2c93f2
Create and add PurpleContacts to the manager when purple_buddy_new is called
Gary Kramlich <grim@reaperworld.com>
parents:
41761
diff
changeset
|
402 | G_CALLBACK(purple_person_presence_notify_cb), |
| 41749 | 403 | person, 0); |
| 404 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
405 | purple_contact_info_set_person(info, person); |
|
41761
9dc5d28fca99
When a Contact is added to a Person set the Contact's Person to that Person.
Gary Kramlich <grim@reaperworld.com>
parents:
41749
diff
changeset
|
406 | |
| 41749 | 407 | purple_person_sort_contacts(person); |
| 408 | } | |
| 409 | ||
| 410 | gboolean | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
411 | purple_person_remove_contact_info(PurplePerson *person, |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
412 | PurpleContactInfo *info) |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
413 | { |
| 41749 | 414 | gboolean removed = FALSE; |
| 415 | ||
| 416 | g_return_val_if_fail(PURPLE_IS_PERSON(person), FALSE); | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
417 | g_return_val_if_fail(PURPLE_IS_CONTACT_INFO(info), FALSE); |
| 41749 | 418 | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
419 | /* Ref the contact info to avoid a use-after free. */ |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
420 | g_object_ref(info); |
| 41749 | 421 | |
| 422 | /* g_ptr_array_remove calls g_object_unref because we passed it in as a | |
| 423 | * GDestroyNotify. | |
| 424 | */ | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
425 | removed = g_ptr_array_remove(person->contacts, info); |
| 41749 | 426 | |
| 427 | if(removed) { | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
428 | PurplePresence *presence = purple_contact_info_get_presence(info); |
| 41749 | 429 | |
|
41771
c5877e2c93f2
Create and add PurpleContacts to the manager when purple_buddy_new is called
Gary Kramlich <grim@reaperworld.com>
parents:
41761
diff
changeset
|
430 | g_signal_handlers_disconnect_by_func(presence, |
|
c5877e2c93f2
Create and add PurpleContacts to the manager when purple_buddy_new is called
Gary Kramlich <grim@reaperworld.com>
parents:
41761
diff
changeset
|
431 | purple_person_presence_notify_cb, |
| 41749 | 432 | person); |
| 433 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
434 | purple_contact_info_set_person(info, NULL); |
|
41761
9dc5d28fca99
When a Contact is added to a Person set the Contact's Person to that Person.
Gary Kramlich <grim@reaperworld.com>
parents:
41749
diff
changeset
|
435 | |
| 41749 | 436 | purple_person_sort_contacts(person); |
| 437 | } | |
| 438 | ||
| 439 | /* Remove our reference. */ | |
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
440 | g_object_unref(info); |
| 41749 | 441 | |
| 442 | return removed; | |
| 443 | } | |
| 444 | ||
|
41948
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
445 | PurpleContactInfo * |
|
6d844d2faff1
Split PurpleContactInfo out of PurpleContact
Gary Kramlich <grim@reaperworld.com>
parents:
41771
diff
changeset
|
446 | purple_person_get_priority_contact_info(PurplePerson *person) { |
| 41749 | 447 | g_return_val_if_fail(PURPLE_IS_PERSON(person), NULL); |
| 448 | ||
| 449 | if(person->contacts->len == 0) { | |
| 450 | return NULL; | |
| 451 | } | |
| 452 | ||
| 453 | return g_ptr_array_index(person->contacts, 0); | |
| 454 | } |