Commit cf255636 authored by Alexander Larsson's avatar Alexander Larsson Committed by Raul Gutierrez Segales
Browse files

Correctly update email based im addresses during contact changes

When we have already read a contact, but are updating it e.g. due
to a property change there is a problem in
Edsf.Persons._update_im_addressess(). Its looking at the existing
personas im_addresses for already existing im addresses before adding
new ones. However, they were already added there based on the email
addresses last time, which means we don't add them now.

The fix is simple, just look in new_im_addressess for existing
im_addresses.

https://bugzilla.gnome.org/show_bug.cgi?id=662285
parent 965b4b47
Overview of changes from libfolks 0.6.4.1 to libfolks 0.6.5
=============================================================
Bugs fixed:
* Bug 662285 — Error with email -> im_addresses when updating a contact
Overview of changes from libfolks 0.6.4 to libfolks 0.6.4.1
=============================================================
......
......@@ -1278,7 +1278,7 @@ public class Edsf.Persona : Folks.Persona,
{
/* Has this already been added? */
var exists = false;
var current_im_addrs = this.im_addresses.get (proto);
var current_im_addrs = new_im_addresses.get (proto);
if (current_im_addrs != null)
{
foreach (var cur_im in current_im_addrs)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment