only remove the buddy if we don't see it using all CM

services/presence2/buddy.py

@@ -48,7 +48,7 @@ class Buddy(dbus.service.Object):
 
         self._icon_cache = icon_cache
 
-        self._handle = handle
+        self.handles = {} # tp client -> handle
 
         self._nick_name = None
         self._color = None

services/presence2/presenceservice.py

@@ -80,6 +80,7 @@ class PresenceService(dbus.service.Object):
         pass
 
     def _contact_online(self, tp, handle, key):
+        new_buddy = False
         buddy = self._buddies.get(key)
 
         if not buddy:
@@ -89,20 +90,29 @@ class PresenceService(dbus.service.Object):
             buddy.set_key(key)
             print "create buddy"
             self._buddies[key] = buddy
+            new_buddy = True
 
         buddies = self._handles[tp]
         buddies[handle] = buddy
 
+        # store the handle of the buddy for this CM
+        buddy.handles[tp] = handle
+
+        if new_buddy:
             self.BuddyAppeared(buddy.object_path())
         
     def _contact_offline(self, tp, handle):
         buddy = self._handles[tp].pop(handle)
         key = buddy.get_key()
 
-        # TODO: check if we don't see this buddy using the other CM
+        # the handle of the buddy for this CM is not valid anymore
+        buddy.handles.pop(tp)
+
+        if not buddy.handles:
+            # we remove the last handle of the buddy, so we don't see
+            # it anymore.
             self._buddies.pop(key)
             print "remove buddy"
-
             self.BuddyDisappeared(buddy.object_path())
 
     def _get_next_object_id(self):

services/presence2/server_plugin.py

@@ -190,7 +190,6 @@ class ServerPlugin(gobject.GObject):
                     not self._ever_connected:
                 # Hmm; probably aren't registered on the server, try reconnecting
                 # and registering
-                self.disconnect()
                 del self._conn
                 self._conn = self._init_connection(register=True)
                 self.start()