Updated remove() method from Account class.

[python-purple] / account.pyx

diff --git a/account.pyx b/account.pyx
index 407dc4e..cddbba7 100644 (file)
--- a/account.pyx
+++ b/account.pyx
@@ -391,14 +391,25 @@ cdef class Account:
         if self.__exists:
             return False
         else:
-            # FIXME: Using purple_accounts_add(...) to save to xml
-            #   I think we could improve this ..
             account.purple_accounts_add(account.purple_account_new( \
                     self.__username, self.__protocol.id))
 
             self.__exists = True
             return True
 
+    def remove(self):
+        """
+        Removes an existing account.
+
+        @return True if successful, False if account doesn't exists
+        """
+        if self.__exists:
+            account.purple_accounts_delete(self._get_structure())
+            self__exists = False
+            return True
+        else:
+            return False
+
     def connect(self):
         """
         Connects to an account.
@@ -505,13 +516,13 @@ cdef class Account:
                                 blist.purple_buddy_get_presence(c_buddy)):
                     name = <char *> blist.purple_buddy_get_name(c_buddy)
 
+                    new_buddy = Buddy(name, self)
+
                     c_alias = <char *> blist.purple_buddy_get_alias_only(c_buddy)
-                    if c_alias == NULL:
-                        alias = None
-                    else:
-                        alias = c_alias
+                    if c_alias:
+                        new_buddy.set_alias(c_alias)
 
-                    buddies_list.append((name, alias))
+                    buddies_list.append(new_buddy)
                 iter = iter.next
             return buddies_list
         else: