diff libpurple/protocols/myspace/myspace.c @ 25057:4278d13b5471

Wait to set the account online until after we've fetched our buddy list. I think it's a good idea for protocols to do this, in general.
author Mark Doliner <mark@kingant.net>
date Wed, 04 Feb 2009 18:59:11 +0000
parents b15168f2b7ee
children c02da09b567f 27e6ab4ff554 9fcff08ce726
line wrap: on
line diff
--- a/libpurple/protocols/myspace/myspace.c	Wed Feb 04 05:15:49 2009 +0000
+++ b/libpurple/protocols/myspace/myspace.c	Wed Feb 04 18:59:11 2009 +0000
@@ -1146,7 +1146,11 @@
 			break;
 
 		case MSIM_CONTACT_LIST_INITIAL_FRIENDS:
-			/* Nothing */
+			/* The session is now set up, ready to be connected. This emits the
+			 * signedOn signal, so clients can now do anything with msimprpl, and
+			 * we're ready for it (session key, userid, username all setup). */
+			purple_connection_update_progress(session->gc, _("Connected"), 3, 4);
+			purple_connection_set_state(session->gc, PURPLE_CONNECTED);
 			break;
 	}
 
@@ -1185,12 +1189,6 @@
 	/* Set display name to username (otherwise will show email address) */
 	purple_connection_set_display_name(session->gc, session->username);
 
-	/* The session is now set up, ready to be connected. This emits the
-	 * signedOn signal, so clients can now do anything with msimprpl, and
-	 * we're ready for it (session key, userid, username all setup). */
-	purple_connection_update_progress(session->gc, _("Connected"), 3, 4);
-	purple_connection_set_state(session->gc, PURPLE_CONNECTED);
-
 	body = msim_msg_new(
 			"UserID", MSIM_TYPE_INTEGER, session->userid,
 			NULL);