# HG changeset patch # User Paul Aurich # Date 1248316312 0 # Node ID fd4415c0595f96fe34999426ede37a75e3b0bfdb # Parent 1736bc22cc062c28a76d6dbcbab58af261e40621 Set xfer->status before emitting signals in purple_xfer_set_status. This was noticed by kenshin in #pidgin and doesn't interfere with the autoaccept plugin (that signal is emitted in purple_xfer_request). diff -r 1736bc22cc06 -r fd4415c0595f ChangeLog.API --- a/ChangeLog.API Wed Jul 22 21:17:50 2009 +0000 +++ b/ChangeLog.API Thu Jul 23 02:31:52 2009 +0000 @@ -87,6 +87,7 @@ * purple_find_group is now more efficient for large numbers of groups. * All DNS routines support internationalized domain names (IDNs) when libpurple is compiled with GNU libidn. + * status is set before emitting signals in purple_xfer_set_status. Deprecated: * buddy-added and buddy-removed blist signals diff -r 1736bc22cc06 -r fd4415c0595f libpurple/ft.c --- a/libpurple/ft.c Wed Jul 22 21:17:50 2009 +0000 +++ b/libpurple/ft.c Thu Jul 23 02:31:52 2009 +0000 @@ -132,6 +132,8 @@ { g_return_if_fail(xfer != NULL); + xfer->status = status; + if(xfer->type == PURPLE_XFER_SEND) { switch(status) { case PURPLE_XFER_STATUS_ACCEPTED: @@ -169,8 +171,6 @@ break; } } - - xfer->status = status; } void purple_xfer_conversation_write(PurpleXfer *xfer, char *message, gboolean is_error)