From: Travis Reitter <travis.reitter@collabora.co.uk>
Date: Wed, 20 Oct 2010 19:10:26 +0000 (-0700)
Subject: Ensure Individuals replaced due to re-linking emit removed().
X-Git-Tag: FOLKS_0_3_2~73
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=725ff1499e90e1d7f78fbf01195545db5d0af2d3;p=platform%2Fupstream%2Ffolks.git

Ensure Individuals replaced due to re-linking emit removed().

It turns out the optimization we're removing, which was meant to coalesce
IndividualAggregator:personas-changed signal emissions, was a little
over-reaching.

Fixes bgo#632564.
---

diff --git a/folks/individual-aggregator.vala b/folks/individual-aggregator.vala
index 23096b6..04a972a 100644
--- a/folks/individual-aggregator.vala
+++ b/folks/individual-aggregator.vala
@@ -642,11 +642,6 @@ public class Folks.IndividualAggregator : Object
 
   private void individual_removed_cb (Individual i, Individual? replacement)
     {
-      /* Only signal if the individual is still in this.individuals. This allows
-       * us to group removals together in, e.g., personas_changed_cb(). */
-      if (this.individuals.lookup (i.id) == null)
-        return;
-
       if (this.user == i)
         this.user = null;