store: Fix ignoring newly added individuals

Apparently, Vala isn't smart enough to be able to deal with a `foreach`
statement and remove elements at the same time, leading to some
unexpected results.

This commit fixes it by just using a classic index-based `for`-loop.

1 files changed, 4 insertions, 2 deletions

diff --git a/src/contacts-store.vala b/src/contacts-store.vala
index 6a39240..7a70cad 100644
--- a/src/contacts-store.vala
+++ b/src/contacts-store.vala
@@ -207,9 +207,11 @@ public class Contacts.Store : GLib.Object {
     }
 
     // Add new individuals
-    foreach (unowned var indiv in to_add) {
+    for (uint i = 0; i < to_add.length; i++) {
+      unowned var indiv = to_add[i];
       if (indiv.personas.size == 0 || Utils.is_ignorable (indiv)) {
-        to_add.remove_fast (indiv);
+        to_add.remove_index_fast (i);
+        i--;
       } else {
         // We want to make sure that changes in the Individual triggers changes
         // in the list model if it affects sorting and/or filtering. Atm, the