pidgin.2.5.7: fccaf233: *** Plucked rev cdbf018a5ef98397e81d5412...
rekkanoryo at pidgin.im
rekkanoryo at pidgin.im
Fri Jun 19 17:40:53 EDT 2009
-----------------------------------------------------------------
Revision: fccaf2332ee694d9be818a7ca0e79f0d7ccde977
Ancestor: fde7624001600a2ba013c3195bba2474dc71e7dd
Author: rekkanoryo at pidgin.im
Date: 2009-06-19T21:10:02
Branch: im.pidgin.pidgin.2.5.7
URL: http://d.pidgin.im/viewmtn/revision/info/fccaf2332ee694d9be818a7ca0e79f0d7ccde977
Modified files:
libpurple/protocols/yahoo/yahoo.c
ChangeLog:
*** Plucked rev cdbf018a5ef98397e81d541269550d9c6fded457 (sulabh at soc.pidgin.im):
Set yahoo to connected after processing the buddy list, should help get rid of the bug reported by the adium user
*** Plucked rev 3952ed728efd23f39d502201b828c86acf6dffd2 (sulabh at soc.pidgin.im):
yahoo login ver 16: Add another case of error reporting. Ask user to visit yahoo website in case of locked account.
-------------- next part --------------
============================================================
--- libpurple/protocols/yahoo/yahoo.c 78215c774b53dc25f2178b131c8fe6a8f8ff4096
+++ libpurple/protocols/yahoo/yahoo.c 9ab3586a680a0d70ab0d0bfdf49c244fab4f664a
@@ -145,7 +145,6 @@ static void yahoo_process_status(PurpleC
static void yahoo_process_status(PurpleConnection *gc, struct yahoo_packet *pkt)
{
PurpleAccount *account = purple_connection_get_account(gc);
- struct yahoo_data *yd = gc->proto_data;
GSList *l = pkt->hash;
YahooFriend *f = NULL;
char *name = NULL;
@@ -165,30 +164,8 @@ static void yahoo_process_status(PurpleC
switch (pair->key) {
case 0: /* we won't actually do anything with this */
+ case 1: /* we won't actually do anything with this */
break;
- case 1: /* we don't get the full buddy list here. */
- if (!yd->logged_in) {
- purple_connection_set_display_name(gc, pair->value);
- purple_connection_set_state(gc, PURPLE_CONNECTED);
- yd->logged_in = TRUE;
- if (yd->picture_upload_todo) {
- yahoo_buddy_icon_upload(gc, yd->picture_upload_todo);
- yd->picture_upload_todo = NULL;
- }
- yahoo_set_status(account, purple_account_get_active_status(account));
-
- /* this requests the list. i have a feeling that this is very evil
- *
- * scs.yahoo.com sends you the list before this packet without it being
- * requested
- *
- * do_import(gc, NULL);
- * newpkt = yahoo_packet_new(YAHOO_SERVICE_LIST, YAHOO_STATUS_OFFLINE, 0);
- * yahoo_packet_send_and_free(newpkt, yd);
- */
-
- }
- break;
case 8: /* how many online buddies we have */
break;
case 7: /* the current buddy */
@@ -547,6 +524,18 @@ static void yahoo_process_list_15(Purple
}
g_hash_table_foreach(ht, yahoo_do_group_cleanup, NULL);
+
+ /* Now that we have processed the buddy list, we can say yahoo has connected */
+ purple_connection_set_display_name(gc, purple_normalize(account, purple_account_get_username(account)));
+ purple_connection_set_state(gc, PURPLE_CONNECTED);
+ yd->logged_in = TRUE;
+ if (yd->picture_upload_todo) {
+ yahoo_buddy_icon_upload(gc, yd->picture_upload_todo);
+ yd->picture_upload_todo = NULL;
+ }
+ yahoo_set_status(account, purple_account_get_active_status(account));
+ purple_debug_info("yahoo","Authentication: Connection established\n");
+
g_hash_table_destroy(ht);
g_free(norm_bud);
}
@@ -1565,7 +1554,7 @@ static void yahoo_auth16_stage1_cb(Purpl
break;
case 1213:
/* security lock from too many failed login attempts */
- error_reason = g_strdup(_("Account locked: Too many failed login attempts"));
+ error_reason = g_strdup(_("Account locked: Too many failed login attempts.\nLogging into the Yahoo! website may fix this."));
error = PURPLE_CONNECTION_ERROR_OTHER_ERROR;
break;
case 1235:
@@ -1573,9 +1562,10 @@ static void yahoo_auth16_stage1_cb(Purpl
error_reason = g_strdup(_("Username does not exist"));
error = PURPLE_CONNECTION_ERROR_INVALID_USERNAME;
break;
+ case 1214:
case 1236:
/* indicates a lock of some description */
- error_reason = g_strdup(_("Account locked: See the debug log"));
+ error_reason = g_strdup(_("Account locked: Unknown reason.\nLogging into the Yahoo! website may fix this."));
error = PURPLE_CONNECTION_ERROR_OTHER_ERROR;
break;
case 100:
More information about the Commits
mailing list