Clear listmodel in load_id_cards(); use remove instead of delete

Fixes bogus stale identities bug.

diff --git a/src/moonshot-identity-management-view.vala b/src/moonshot-identity-management-view.vala
index 4e3c462..faa6caf 100644 (file)
--- a/src/moonshot-identity-management-view.vala
+++ b/src/moonshot-identity-management-view.vala
@@ -188,7 +188,7 @@ public class IdentityManagerView : Window {
         foreach (var id_card_widget in children) {
         remove_id_card_widget((IdCardWidget)id_card_widget);
         }   
-
+        this.listmodel->clear();
         LinkedList<IdCard> card_list = identities_manager.get_card_list() ;
         if (card_list == null) {
             return;
@@ -208,7 +208,7 @@ public class IdentityManagerView : Window {
 
        var children = this.services_internal_vbox.get_children ();
        foreach (var hbox in children)
-           hbox.destroy();
+           services_internal_vbox.remove(hbox);
        fill_services_vbox (id_card_widget.id_card);
 //       identities_manager.store_id_cards();
     }
@@ -561,7 +561,7 @@ public class IdentityManagerView : Window {
                 
                 var children = services_internal_vbox.get_children ();
                 foreach (var hbox in children)
-                  hbox.destroy();
+                  services_internal_vbox.remove(hbox);
                 
                 fill_services_vbox (idcard);
                 custom_vbox.current_idcard.update_id_card_label ();