Fixing UnicodeDecodeError in buddy's alias
[python-purple] / account.pyx
index fe27e58..96fc918 100644 (file)
 
 cimport purple
 
 
 cimport purple
 
-cdef extern from *:
-    ctypedef char const_char "const char"
-
 cdef class Account:
 cdef class Account:
-    """ Account class """
-    cdef account.PurpleAccount *c_account
-    cdef plugin.PurplePlugin *c_plugin
-    cdef prpl.PurplePluginProtocolInfo *c_prpl_info
-    cdef plugin.PurplePluginInfo *c_plugin_info
-    cdef savedstatuses.PurpleSavedStatus *__sstatus
-    cdef ProxyInfo __proxy
-
-    def __init__(self, char *username, char *protocol_id):
-        cdef proxy.PurpleProxyInfo *c_proxyinfo
-        cdef account.PurpleAccount *acc = NULL
-
-        acc = account.c_purple_accounts_find(username, protocol_id)
-        if acc:
-            self.c_account = acc
-            c_proxyinfo = account.c_purple_account_get_proxy_info(self.c_account)
+    """
+    Account class
+    @param username
+    @param protocol Protocol class instance
+    @param core Purple class instance
+    """
+
+    cdef object __username
+    cdef object __protocol
+    cdef object __core
+    cdef object __exists
+
+    def __init__(self, username, protocol, core):
+        self.__username = username
+        self.__protocol = protocol
+        self.__core = core
+
+        if protocol.exists and self._get_structure() != NULL:
+            self.__exists = True
         else:
         else:
-            self.c_account = account.c_purple_account_new(username, protocol_id)
-            c_proxyinfo = account.c_purple_account_get_proxy_info(self.c_account)
-            if c_proxyinfo == NULL:
-                c_proxyinfo = proxy.c_purple_proxy_info_new()
-                proxy.c_purple_proxy_info_set_type(c_proxyinfo, proxy.PURPLE_PROXY_NONE)
-                account.c_purple_account_set_proxy_info(self.c_account, c_proxyinfo)
-        self.__proxy = ProxyInfo()
-        self.__proxy.c_proxyinfo = c_proxyinfo
-        acc = NULL
-
-        self.c_plugin = plugin.c_purple_plugins_find_with_id(protocol_id)
-        self.c_prpl_info = plugin.c_PURPLE_PLUGIN_PROTOCOL_INFO(self.c_plugin)
+            self.__exists = False
 
 
-    def __get_username(self):
-        if self.c_account:
-            return account.c_purple_account_get_username(self.c_account)
-        else:
-            return None
-    def __set_username(self, username):
-        if self.c_account:
-            account.c_purple_account_set_username(self.c_account, username)
-    username = property(__get_password, __set_username)
+    cdef account.PurpleAccount *_get_structure(self):
+        return account.purple_accounts_find(self.__username, \
+                self.__protocol.id)
 
 
-    def __get_password(self):
-        if self.c_account:
-            return account.c_purple_account_get_password(self.c_account)
+    def __is_connected(self):
+        if self.__exists:
+            return account.purple_account_is_connected(self._get_structure())
         else:
             return None
         else:
             return None
-    def __set_password(self, password):
-        if self.c_account:
-            account.c_purple_account_set_password(self.c_account, password)
-    password = property(__get_password, __set_password)
+    is_connected = property(__is_connected)
 
 
-    def __get_alias(self):
-        if self.c_account:
-            return account.c_purple_account_get_alias(self.c_account)
+    def __is_connecting(self):
+        if self.__exists:
+            return account.purple_account_is_connecting(self._get_structure())
         else:
             return None
         else:
             return None
-    def __set_alias(self, alias):
-        if self.c_account:
-            account.c_purple_account_set_alias(self.c_account, alias)
-    alias = property(__get_alias, __set_alias)
+    is_connecting = property(__is_connecting)
 
 
-    def __get_user_info(self):
-        if self.c_account:
-            return account.c_purple_account_get_user_info(self.c_account)
-        else:
-            return None
-    def __set_user_info(self, user_info):
-        if self.c_account:
-            account.c_purple_account_set_user_info(self.c_account, user_info)
-    user_info = property(__get_user_info, __set_user_info)
-
-    def __get_protocol_id(self):
-        if self.c_account:
-            return account.c_purple_account_get_protocol_id(self.c_account)
+    def __is_disconnected(self):
+        if self.__exists:
+            return account.purple_account_is_disconnected( \
+                    self._get_structure())
         else:
             return None
         else:
             return None
-    def __set_protocol_id(self, protocol_id):
-        if self.c_account:
-            account.c_purple_account_set_protocol_id(self.c_account, protocol_id)
-    protocol_id = property(__get_protocol_id, __set_protocol_id)
+    is_disconnected = property(__is_disconnected)
 
 
-    def __get_remember_password(self):
-        if self.c_account:
-            return account.c_purple_account_get_remember_password(self.c_account)
+    def __get_core(self):
+        return self.__core
+    core = property(__get_core)
+
+    def __get_exists(self):
+        return self.__exists
+    exists = property(__get_exists)
+
+    def __get_username(self):
+        cdef char *username = NULL
+        if self.__exists:
+            username = <char *> account.purple_account_get_username( \
+                    self._get_structure())
+            if username:
+                return username
+            else:
+                return None
         else:
         else:
-            return None
-    def __set_remember_password(self, value):
-        if self.c_account:
-            account.c_purple_account_set_remember_password(self.c_account, value)
-    remember_password = property(__get_remember_password, __set_remember_password)
+            return self.__username
+    username = property(__get_username)
+
+    def __get_protocol(self):
+        return self.__protocol
+    protocol = property(__get_protocol)
 
     def _get_protocol_options(self):
 
     def _get_protocol_options(self):
+        """
+        @return Dictionary {'setting': value, ...} 
+        """
         cdef glib.GList *iter
         cdef glib.GList *iter
+        cdef account.PurpleAccount *c_account
+        cdef plugin.PurplePlugin *c_plugin
+        cdef prpl.PurplePluginProtocolInfo *prpl_info
         cdef accountopt.PurpleAccountOption *option
         cdef prefs.PurplePrefType type
         cdef accountopt.PurpleAccountOption *option
         cdef prefs.PurplePrefType type
-        cdef const_char *label_name
-        cdef const_char *str_value
-        cdef const_char *setting
+        cdef char *label_name
+        cdef char *str_value
+        cdef char *setting
         cdef int int_value
         cdef glib.gboolean bool_value
 
         cdef int int_value
         cdef glib.gboolean bool_value
 
-        if self.c_account == NULL:
+        c_account = self._get_structure()
+
+        if c_account == NULL:
             return None
 
         po = {}
 
             return None
 
         po = {}
 
-        iter = self.c_prpl_info.protocol_options
+        c_plugin = plugin.purple_plugins_find_with_id(self.__protocol.id)
+        prpl_info = plugin.PURPLE_PLUGIN_PROTOCOL_INFO(c_plugin)
+        iter = prpl_info.protocol_options
 
         while iter:
 
             option = <accountopt.PurpleAccountOption *> iter.data
 
         while iter:
 
             option = <accountopt.PurpleAccountOption *> iter.data
-            type = accountopt.c_purple_account_option_get_type(option)
-            label_name = accountopt.c_purple_account_option_get_text(option)
-            setting = accountopt.c_purple_account_option_get_setting(option)
+            type = accountopt.purple_account_option_get_type(option)
+            label_name = <char *> accountopt.purple_account_option_get_text(option)
+            setting = <char *> accountopt.purple_account_option_get_setting(option)
 
             sett = str(<char *> setting)
 
             if type == prefs.PURPLE_PREF_STRING:
 
 
             sett = str(<char *> setting)
 
             if type == prefs.PURPLE_PREF_STRING:
 
-                str_value = accountopt.c_purple_account_option_get_default_string(option)
+                str_value = <char *> accountopt.purple_account_option_get_default_string(option)
 
                 # Google Talk default domain hackery!
                 if str_value == NULL and str(<char *> label_name) == "Connect server":
                     str_value = "talk.google.com"
 
                 # Google Talk default domain hackery!
                 if str_value == NULL and str(<char *> label_name) == "Connect server":
                     str_value = "talk.google.com"
-                str_value = account.c_purple_account_get_string(self.c_account, setting, str_value)
+                str_value = <char *> account.purple_account_get_string(c_account, setting, str_value)
 
                 val = str(<char *> str_value)
 
             elif type == prefs.PURPLE_PREF_INT:
 
 
                 val = str(<char *> str_value)
 
             elif type == prefs.PURPLE_PREF_INT:
 
-                int_value = accountopt.c_purple_account_option_get_default_int(option)
-                int_value = account.c_purple_account_get_int(self.c_account, setting, int_value)
+                int_value = accountopt.purple_account_option_get_default_int(option)
+                int_value = account.purple_account_get_int(c_account, setting, int_value)
 
                 val = int(int_value)
 
             elif type == prefs.PURPLE_PREF_BOOLEAN:
 
 
                 val = int(int_value)
 
             elif type == prefs.PURPLE_PREF_BOOLEAN:
 
-                bool_value = accountopt.c_purple_account_option_get_default_bool(option)
-                bool_value = account.c_purple_account_get_bool(self.c_account, setting, bool_value)
+                bool_value = accountopt.purple_account_option_get_default_bool(option)
+                bool_value = account.purple_account_get_bool(c_account, setting, bool_value)
 
                 val = bool(bool_value)
 
             elif type == prefs.PURPLE_PREF_STRING_LIST:
 
 
                 val = bool(bool_value)
 
             elif type == prefs.PURPLE_PREF_STRING_LIST:
 
-                str_value = accountopt.c_purple_account_option_get_default_list_value(option)
-                str_value = account.c_purple_account_get_string(self.c_account, setting, str_value)
+                str_value = <char *> accountopt.purple_account_option_get_default_list_value(option)
+                str_value = <char *> account.purple_account_get_string(c_account, setting, str_value)
 
                 val = str(<char *> str_value)
 
 
                 val = str(<char *> str_value)
 
@@ -176,178 +166,365 @@ cdef class Account:
             po[sett] = val
 
         return po
             po[sett] = val
 
         return po
+    protocol_options = property(_get_protocol_options)
+
+    def __get_password(self):
+        cdef char *password = NULL
+        if self.__exists:
+            password = <char *> account.purple_account_get_password( \
+                    self._get_structure())
+            if password:
+                return password
+            else:
+                return None
+        else:
+            return None
+    password = property(__get_password)
 
 
-    def _set_protocol_options(self, po):
+    def __get_alias(self):
+        cdef char *alias = NULL
+        if self.__exists:
+            alias = <char *> account.purple_account_get_alias(self._get_structure())
+            if alias:
+                return alias
+            else:
+                return None
+        else:
+            return None
+    alias = property(__get_alias)
+
+    def __get_user_info(self):
+        cdef char *user_info = NULL
+        if self.__exists:
+            user_info = <char *> account.purple_account_get_user_info(self._get_structure())
+            if user_info:
+                return user_info
+            else:
+                return None
+        else:
+            return None
+    user_info = property(__get_user_info)
+
+    def __get_remember_password(self):
+        if self.__exists:
+            return account.purple_account_get_remember_password( \
+                    self._get_structure())
+        else:
+            return None
+    remember_password = property(__get_remember_password)
+
+    def __get_enabled(self):
+        if self.__exists:
+            return account.purple_account_get_enabled(self._get_structure(), \
+                    self.__core.ui_name)
+        else:
+            return None
+    enabled = property(__get_enabled)
+
+    def set_username(self, username):
+        """
+        Sets the account's username.
+
+        @param username The username
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_username(self._get_structure(), \
+                    username)
+            return True
+        else:
+            return False
+
+    def set_protocol(self, protocol):
+        """
+        Sets the account's protocol.
+
+        @param protocol A Protocol class instance
+        @return True if successful, False if account doesn't exists
+        """
+        if protocol.exists and self.__exists:
+            account.purple_account_set_protocol_id(self._get_structure(), \
+                        protocol.id)
+            self.__protocol = protocol
+            return True
+        else:
+            return False
+
+    def set_protocol_options(self, po):
+        """
+        @param po Dictionary {'setting': value, ...} options to be updated
+        @return True to success or False to failure
+        """
         cdef glib.GList *iter
         cdef glib.GList *iter
+        cdef account.PurpleAccount *c_account
+        cdef plugin.PurplePlugin *c_plugin
+        cdef prpl.PurplePluginProtocolInfo *prpl_info
         cdef accountopt.PurpleAccountOption *option
         cdef prefs.PurplePrefType type
         cdef accountopt.PurpleAccountOption *option
         cdef prefs.PurplePrefType type
-        cdef const_char *str_value
-        cdef const_char *setting
+        cdef char *str_value
+        cdef char *setting
         cdef int int_value
         cdef glib.gboolean bool_value
 
         cdef int int_value
         cdef glib.gboolean bool_value
 
-        if self.c_account == NULL:
-            return
+        c_account = self._get_structure()
 
 
-        po = {}
+        if c_account == NULL:
+            return False
 
 
-        iter = self.c_prpl_info.protocol_options
+        c_plugin = plugin.purple_plugins_find_with_id(self.__protocol.id)
+        prpl_info = plugin.PURPLE_PLUGIN_PROTOCOL_INFO(c_plugin)
+        iter = prpl_info.protocol_options
 
         while iter:
 
             option = <accountopt.PurpleAccountOption *> iter.data
 
         while iter:
 
             option = <accountopt.PurpleAccountOption *> iter.data
-            type = accountopt.c_purple_account_option_get_type(option)
-            setting = accountopt.c_purple_account_option_get_setting(option)
+            type = accountopt.purple_account_option_get_type(option)
+            setting = <char *> accountopt.purple_account_option_get_setting(option)
 
             sett = str(<char *> setting)
 
 
             sett = str(<char *> setting)
 
+            if not po.has_key(sett):
+                iter = iter.next
+                continue
+
             if type == prefs.PURPLE_PREF_STRING:
 
                 str_value = <char *> po[sett]
             if type == prefs.PURPLE_PREF_STRING:
 
                 str_value = <char *> po[sett]
-                account.c_purple_account_set_string(self.c_account, setting, str_value)
+                account.purple_account_set_string(c_account, setting, str_value)
 
             elif type == prefs.PURPLE_PREF_INT:
 
                 int_value = int(po[sett])
 
             elif type == prefs.PURPLE_PREF_INT:
 
                 int_value = int(po[sett])
-                account.c_purple_account_set_int(self.c_account, setting, int_value)
+                account.purple_account_set_int(c_account, setting, int_value)
 
             elif type == prefs.PURPLE_PREF_BOOLEAN:
 
                 bool_value = bool(po[sett])
 
             elif type == prefs.PURPLE_PREF_BOOLEAN:
 
                 bool_value = bool(po[sett])
-                account.c_purple_account_set_bool(self.c_account, setting, bool_value)
+                account.purple_account_set_bool(c_account, setting, bool_value)
 
             elif type == prefs.PURPLE_PREF_STRING_LIST:
 
                 str_value = <char *> po[sett]
 
             elif type == prefs.PURPLE_PREF_STRING_LIST:
 
                 str_value = <char *> po[sett]
-                account.c_purple_account_set_string(self.c_account, setting, str_value)
+                account.purple_account_set_string(c_account, setting, str_value)
 
             iter = iter.next
 
 
             iter = iter.next
 
-    protocol_options = property(_get_protocol_options, _set_protocol_options)
-
-    def _get_protocol_labels(self):
-        cdef glib.GList *iter
-        cdef accountopt.PurpleAccountOption *option
-        cdef const_char *label_name
-        cdef const_char *setting
-
-        if self.c_account == NULL:
-            return None
-
-        po = {}
+        return True
 
 
-        iter = self.c_prpl_info.protocol_options
+    def set_password(self, password):
+        """
+        Sets the account's password.
 
 
-        while iter:
+        @param password The password
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_password(self._get_structure(), \
+                    password)
+            return True
+        else:
+            return False
+
+    def set_alias(self, alias):
+        """
+        Sets the account's alias
+
+        @param alias The alias
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_alias(self._get_structure(), \
+                    alias)
+            return True
+        else:
+            return False
+
+    def set_user_info(self, user_info):
+        """
+        Sets the account's user information
+
+        @param user_info The user information
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_user_info(self._get_structure(), \
+                    user_info)
+            return True
+        else:
+            return False
+
+    def set_remember_password(self, remember_password):
+        """
+        Sets whether or not this account should save its password.
+
+        @param remember_password True if should remember the password,
+                                 or False otherwise
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_remember_password( \
+                self._get_structure(), remember_password)
+            return True
+        else:
+            return False
+
+    def set_enabled(self, value):
+        """
+        Sets wheter or not this account is enabled.
+
+        @param value True if it is enabled, or False otherwise
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_set_enabled(self._get_structure(), \
+                    self.__core.ui_name, bool(value))
+            return True
+        else:
+            return False
 
 
-            option = <accountopt.PurpleAccountOption *> iter.data
-            label_name = accountopt.c_purple_account_option_get_text(option)
-            setting = accountopt.c_purple_account_option_get_setting(option)
+    def new(self):
+        """
+        Creates a new account.
 
 
-            sett = str(<char *> setting)
-            label = str(<char *> label_name)
+        @return True if successful, False if account already exists
+        """
+        if self.__exists:
+            return False
+        else:
+            account.purple_accounts_add(account.purple_account_new( \
+                    self.__username, self.__protocol.id))
+
+            self.__exists = True
+            return True
+    
+    def remove(self):
+        """
+        Removes as existing account.
+
+        @return True if successful, False if account doesnt exists
+        """
+        if self.__exists:
+            account.purple_accounts_remove(self._get_structure())
+        
+            self__exists = False
+            return True
+        else:
+            return False
 
 
-            po[sett] = label
+    def connect(self):
+        """
+        Connects to an account.
 
 
-        return po
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_connect(self._get_structure())
+            return True
+        else:
+            return False
 
 
-    protocol_labels = property(_get_protocol_labels)
+    def disconnect(self):
+        """
+        Disconnects from an account.
 
 
-    def get_protocol_name(self):
-        if self.c_account:
-            return account.c_purple_account_get_protocol_name(self.c_account)
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_account_disconnect(self._get_structure())
+            return True
         else:
         else:
-            return None
+            return False
+
+    def add_buddy(self, name, alias=None, group=None):
+        """
+        Adds a buddy to account's buddy list.
+
+        @param name  Buddy name
+        @param alias Buddy alias (optional)
+        @return True if successfull, False otherwise
+        """
+        cdef blist.PurpleBuddy *c_buddy = NULL
+        cdef blist.PurpleGroup *c_group = NULL
+        cdef char *c_alias = NULL
 
 
-    def get_enabled(self, ui):
-        if self.c_account:
-            return account.c_purple_account_get_enabled(self.c_account, ui)
+        if alias:
+            c_alias = alias
         else:
         else:
-            return None
-
-    def set_enabled(self, ui, value):
-        if self.c_account:
-            account.c_purple_account_set_enabled(self.c_account, ui, value)
+            c_alias = NULL
 
 
-    def set_status(self):
-        self.__sstatus = savedstatuses.c_purple_savedstatus_new(NULL, status.PURPLE_STATUS_AVAILABLE)
-        savedstatuses.c_purple_savedstatus_activate(self.__sstatus)
+        if self.__exists and \
+                account.purple_account_is_connected(self._get_structure()):
+            if blist.purple_find_buddy(self._get_structure(), name):
+                return False
 
 
-    def __get_proxy(self):
-        return self.__proxy
-    proxy = property(__get_proxy)
+            if group:
+                c_group = blist.purple_find_group(group)
+                if c_group == NULL:
+                    c_group = blist.purple_group_new(group)
 
 
-    def get_buddies_online(self):
-        cdef glib.GSList *iter
-        cdef blist.PurpleBuddy *buddy
-        cdef char *c_name = NULL
-        cdef char *c_alias = NULL
-        buddies = []
-        iter = blist.c_purple_find_buddies(self.c_account, NULL)
-        while iter:
-            c_name = NULL
-            c_alias = NULL
-            buddy = <blist.PurpleBuddy *> iter.data
-            if <blist.PurpleBuddy *>buddy and \
-                account.c_purple_account_is_connected(blist.c_purple_buddy_get_account(buddy)) and \
-                status.c_purple_presence_is_online(blist.c_purple_buddy_get_presence(buddy)):
-                c_name = <char *> blist.c_purple_buddy_get_name(buddy)
-                if c_name == NULL:
-                    name = None
-                else:
-                    name = c_name
-                c_alias = <char *> blist.c_purple_buddy_get_alias_only(buddy)
-                if c_alias == NULL:
-                    alias = None
-                else:
-                    alias = c_alias
-                buddies.append((name, alias))
-            iter = iter.next
-        return buddies
+            c_buddy = blist.purple_buddy_new(self._get_structure(), \
+                    name, c_alias)
+            if c_buddy == NULL:
+                return False
 
 
-    def new(self, username, protocol_id):
-        cdef account.PurpleAccount *c_account
-        c_account = account.c_purple_account_new(username, protocol_id)
+            blist.purple_blist_add_buddy(c_buddy, NULL, c_group, NULL)
+            account.purple_account_add_buddy(self._get_structure(), c_buddy)
+            return True
 
 
-        if c_account == NULL:
+        else:
             return None
 
             return None
 
-        return (username, protocol_id)
+    def remove_buddy(self, name):
+        """
+        Removes a buddy from account's buddy list.
 
 
-    def get_all(self):
-        cdef glib.GList *iter
-        cdef account.PurpleAccount *acc
+        @param name Buddy name
+        @return True if successful, False otherwise
+        """
+        cdef blist.PurpleBuddy *c_buddy = NULL
+        cdef blist.PurpleGroup *c_group = NULL
 
 
-        accounts = []
+        if self.__exists and \
+                account.purple_account_is_connected(self._get_structure()):
+            c_buddy = blist.purple_find_buddy(self._get_structure(), name)
+            if c_buddy == NULL:
+                return False
 
 
-        iter = account.c_purple_accounts_get_all()
-        while iter:
-            acc = <account.PurpleAccount *> iter.data
-            if <account.PurpleAccount *>acc:
-                username = account.c_purple_account_get_username(acc)
-                protocol_id = account.c_purple_account_get_protocol_id(acc)
+            c_group = blist.purple_buddy_get_group(c_buddy)
 
 
-                accounts.append((username, protocol_id))
-            iter = iter.next
+            account.purple_account_remove_buddy(self._get_structure(), \
+                    c_buddy, c_group)
+            blist.purple_blist_remove_buddy(c_buddy)
+            return True
+        else:
+            return None
 
 
-        return accounts
+    def get_buddies_online(self):
+        cdef glib.GSList *iter = NULL
+        cdef blist.PurpleBuddy *c_buddy = NULL
+        cdef char *c_alias = NULL
 
 
-    def get_password(self, acc):
-        ''' @param acc Tuple (username, protocol id) '''
-        cdef account.PurpleAccount *c_account
+        if self.__exists and \
+                account.purple_account_is_connected(self._get_structure()):
+            iter = blist.purple_find_buddies(self._get_structure(), NULL)
 
 
-        c_account = account.c_purple_accounts_find(acc[0], acc[1])
-        if c_account:
-            return account.c_purple_account_get_password(c_account)
-        else:
-            return None
+            buddies_list = []
+            while iter:
+                c_alias = NULL
+                c_buddy = <blist.PurpleBuddy *> iter.data
+                if <blist.PurpleBuddy *> c_buddy and \
+                        status.purple_presence_is_online( \
+                                blist.purple_buddy_get_presence(c_buddy)):
+                    name = <char *> blist.purple_buddy_get_name(c_buddy)
 
 
-    def set_password(self, acc, password):
-        ''' @param acc Tuple (username, protocol id) '''
-        ''' @param password The account's password '''
-        cdef account.PurpleAccount *c_account
+                    new_buddy = Buddy(name, self)
 
 
-        c_account = account.c_purple_accounts_find(acc[0], acc[1])
-        if c_account:
-            account.c_purple_account_set_password(c_account, password)
+                    c_alias = <char *> blist.purple_buddy_get_alias_only(c_buddy)
+                    if c_alias:
+                        new_buddy.set_alias(c_alias)
 
 
+                    buddies_list.append(new_buddy)
+                iter = iter.next
+            return buddies_list
+        else:
+            return None