# HG changeset patch
# User Goffi <goffi@goffi.org>
# Date 1423245284 -3600
# Node ID ba73798317a753b15896105be0e77d1a8afc7f87
# Parent  3360074a2f001fae5ea04bf5ef7a5aa928f844b3
core: fixed items registration in roster management

diff -r 3360074a2f00 -r ba73798317a7 src/core/xmpp.py
--- a/src/core/xmpp.py	Fri Feb 06 18:53:07 2015 +0100
+++ b/src/core/xmpp.py	Fri Feb 06 18:54:44 2015 +0100
@@ -180,6 +180,8 @@
     def rosterCb(self, roster):
         assert roster is not None # FIXME: must be managed with roster versioning
         self._jids = roster
+        for roster_item in roster.itervalues():
+            self._registerItem(roster_item)
 
     def _registerItem(self, item):
         """Register item in local cache
@@ -265,13 +267,13 @@
         """Return a list of groups"""
         return self._groups.keys()
 
-    def getItem(self, jid):
+    def getItem(self, entity_jid):
         """Return RosterItem for a given jid
 
-        @param jid: jid of the contact
+        @param entity_jid: jid of the contact
         @return: RosterItem or None if contact is not in cache
         """
-        return self._jids.get(jid, None)
+        return self._jids.get(entity_jid, None)
 
     def getJids(self):
         """Return all jids of the roster"""
@@ -289,7 +291,7 @@
         try:
             return self._groups[group]
         except KeyError:
-            raise exceptions.UnknownGroupError
+            raise exceptions.UnknownGroupError(group)
 
 
 class SatPresenceProtocol(xmppim.PresenceClientProtocol):