Changes in protocols/msn/sb.c [fe79de66:dcf155d]

File:

• protocols/msn/sb.c (modified) (10 diffs)

protocols/msn/sb.c

@@ -2 +2 @@
   * BitlBee -- An IRC to other IM-networks gateway                     *
   *                                                                    *
-  * Copyright 2002-2012 Wilmer van der Gaast and others                *
+  * Copyright 2002-2010 Wilmer van der Gaast and others                *
   \********************************************************************/
 
@@ -122 +122 @@
 }
 
-struct msn_switchboard *msn_sb_by_handle( struct im_connection *ic, const char *handle )
+struct msn_switchboard *msn_sb_by_handle( struct im_connection *ic, char *handle )
 {
         struct msn_data *md = ic->proto_data;
@@ -308 +308 @@
         struct msn_switchboard *sb = data;
         struct im_connection *ic;
-        struct msn_data *md;
         char buf[1024];
         
@@ -316 +315 @@
         
         ic = sb->ic;
-        md = ic->proto_data;
         
         if( source != sb->fd )
@@ -334 +332 @@
         
         if( sb->session == MSN_SB_NEW )
-                g_snprintf( buf, sizeof( buf ), "USR %d %s;{%s} %s\r\n", ++sb->trId, ic->acc->user, md->uuid, sb->key );
+                g_snprintf( buf, sizeof( buf ), "USR %d %s %s\r\n", ++sb->trId, ic->acc->user, sb->key );
         else
-                g_snprintf( buf, sizeof( buf ), "ANS %d %s;{%s} %s %d\r\n", ++sb->trId, ic->acc->user, md->uuid, sb->key, sb->session );
+                g_snprintf( buf, sizeof( buf ), "ANS %d %s %s %d\r\n", ++sb->trId, ic->acc->user, sb->key, sb->session );
         
         if( msn_sb_write( sb, "%s", buf ) )
@@ -455 +453 @@
                         char buf[1024];
                         
-                        /* For as much as I understand this MPOP stuff now, a
-                           switchboard has two (or more) roster entries per
-                           participant. One "bare JID" and one JID;UUID. Ignore
-                           the latter. */
-                        if( !strchr( cmd[4], ';' ) )
+                        if( num == 1 )
                         {
-                                /* HACK: Since even 1:1 chats now have >2 participants
-                                   (ourselves included) it gets hard to tell them apart
-                                   from rooms. Let's hope this is enough: */
-                                if( sb->chat == NULL && num != tot )
-                                {
-                                        g_snprintf( buf, sizeof( buf ), "MSN groupchat session %d", sb->session );
-                                        sb->chat = imcb_chat_new( ic, buf );
-                                        
-                                        g_free( sb->who );
-                                        sb->who = NULL;
-                                }
+                                g_snprintf( buf, sizeof( buf ), "MSN groupchat session %d", sb->session );
+                                sb->chat = imcb_chat_new( ic, buf );
                                 
-                                if( sb->chat )
-                                        imcb_chat_add_buddy( sb->chat, cmd[4] );
-                        }
-                        
-                        /* We have the full roster, start showing the channel to
-                           the user. */
-                        if( num == tot && sb->chat )
+                                g_free( sb->who );
+                                sb->who = NULL;
+                        }
+                        
+                        imcb_chat_add_buddy( sb->chat, cmd[4] );
+                        
+                        if( num == tot )
                         {
                                 imcb_chat_add_buddy( sb->chat, ic->acc->user );
@@ -521 +506 @@
                         return( 0 );
                 }
-                
-                /* See IRO above. Handle "bare JIDs" only. */
-                if( strchr( cmd[1], ';' ) )
-                        return 1;
                 
                 if( sb->who && g_strcasecmp( cmd[1], sb->who ) == 0 )
@@ -560 +541 @@
                         return( st );
                 }
-                else if( strcmp( cmd[1], ic->acc->user ) == 0 )
-                {
-                        /* Well, gee thanks. Thanks for letting me know I've arrived.. */
-                }
                 else if( sb->who )
                 {
@@ -636 +613 @@
                            The server will clean it up when it's idle for too long. */
                 }
-                else if( sb->chat && !strchr( cmd[1], ';' ) )
+                else if( sb->chat )
                 {
                         imcb_chat_remove_buddy( sb->chat, cmd[1], "" );
@@ -652 +629 @@
                 /* If the person is offline, send an offline message instead,
                    and don't report an error. */
+                /* TODO: Support for OIMs that works. (#874) */
+                /*
                 if( num == 217 )
-                        msn_ns_oim_send_queue( ic, &sb->msgq );
+                        msn_soap_oim_send_queue( ic, &sb->msgq );
                 else
+                */
                         imcb_error( ic, "Error reported by switchboard server: %s", err->text );