# HG changeset patch # User Elliott Sales de Andrade # Date 1234840030 0 # Node ID 3620b113e046debdc4345edc223cff0d709e5028 # Parent fb9118dea0ed263310f68b93acb970c9d27163d1 Buddies on both the Allow and Block list are now automatically removed from the Allow list. Users with this problem will now no longer receive an ADL 241 error. The problematic buddy should now appear on the buddy list and can be removed or unblocked as desired. Fixes #6702. diff -r fb9118dea0ed -r 3620b113e046 ChangeLog --- a/ChangeLog Tue Feb 17 00:27:34 2009 +0000 +++ b/ChangeLog Tue Feb 17 03:07:10 2009 +0000 @@ -47,6 +47,10 @@ * We no longer try to send empty messages. This could happen when a message contained only formatting and that formatting was not supported on MSN. + * Buddies on both the Allow and Block list are now automatically + removed from the Allow list. Users with this problem will now no + longer receive an ADL 241 error. The problematic buddy should now + appear on the buddy list and can be removed or unblocked as desired. XMPP: * Resources using __HOSTNAME__ substitution will now grab only the short diff -r fb9118dea0ed -r 3620b113e046 libpurple/protocols/msn/notification.c --- a/libpurple/protocols/msn/notification.c Tue Feb 17 00:27:34 2009 +0000 +++ b/libpurple/protocols/msn/notification.c Tue Feb 17 03:07:10 2009 +0000 @@ -623,6 +623,18 @@ if (user->passport && !strcmp(user->passport, "messenger@microsoft.com")) continue; + if ((user->list_op & MSN_LIST_OP_MASK) == (MSN_LIST_AL_OP | MSN_LIST_BL_OP)) { + /* The server will complain if we send it a user on both the + Allow and Block lists. So assume they're on the Block list + and remove them from the Allow list in the membership lists to + stop this from happening again. */ + purple_debug_warning("msn", + "User %s is on both Allow and Block list," + "removing from Allow list.\n", + user->passport); + msn_userlist_rem_buddy_from_list(session->userlist, user->passport, MSN_LIST_AL); + } + msn_add_contact_xml(session, adl_node, user->passport, user->list_op & MSN_LIST_OP_MASK, user->networkid);