Changeset fb020ac for protocols


Ignore:
Timestamp:
2010-03-07T18:43:23Z (15 years ago)
Author:
Wilmer van der Gaast <wilmer@…>
Branches:
master
Children:
279607e
Parents:
e08e53c (diff), c32f492 (diff)
Note: this is a merge changeset, the changes displayed below correspond to the merge itself.
Use the (diff) links above to see all the changes relative to each parent.
Message:

Merging in mainline, including improved away/status stuff.

Location:
protocols
Files:
14 edited

Legend:

Unmodified
Added
Removed

  • protocols/jabber/jabber.c

    re08e53c rfb020ac  
    8080        s = set_add( &acc->set, "xmlconsole", "false", set_eval_bool, acc );
    8181        s->flags |= ACC_SET_OFFLINE_ONLY;
     82       
     83        acc->flags |= ACC_FLAG_AWAY_MESSAGE | ACC_FLAG_STATUS_MESSAGE;
    8284}
    8385
    … …  
    364366        while( bud )
    365367        {
    366                 imcb_log( ic, "Buddy %s (%d) information:\nAway state: %s\nAway message: %s",
    367                                    bud->full_jid, bud->priority,
    368                                    bud->away_state ? bud->away_state->full_name : "(none)",
    369                                    bud->away_message ? : "(none)" );
     368                imcb_log( ic, "Buddy %s (%d) information:", bud->full_jid, bud->priority );
     369                if( bud->away_state )
     370                        imcb_log( ic, "Away state: %s", bud->away_state->full_name );
     371                imcb_log( ic, "Status message: %s", bud->away_message ? : "(none)" );
     372               
    370373                bud = bud->next;
    371374        }
    … …  
    377380{
    378381        struct jabber_data *jd = ic->proto_data;
    379         struct jabber_away_state *state;
    380        
    381         /* Save all this info. We need it, for example, when changing the priority setting. */
    382         state = (void *) jabber_away_state_by_name( state_txt );
    383         jd->away_state = state ? state : (void *) jabber_away_state_list; /* Fall back to "Away" if necessary. */
     382       
     383        /* state_txt == NULL -> Not away.
     384           Unknown state -> fall back to the first defined away state. */
     385        jd->away_state = state_txt ? jabber_away_state_by_name( state_txt )
     386                         ? : jabber_away_state_list : NULL;
     387       
    384388        g_free( jd->away_message );
    385389        jd->away_message = ( message && *message ) ? g_strdup( message ) : NULL;

  • protocols/jabber/jabber.h

    re08e53c rfb020ac  
    8484        /* After changing one of these two (or the priority setting), call
    8585           presence_send_update() to inform the server about the changes. */
    86         struct jabber_away_state *away_state;
     86        const struct jabber_away_state *away_state;
    8787        char *away_message;
    8888       

  • protocols/jabber/jabber_util.c

    re08e53c rfb020ac  
    228228{
    229229        { "away",  "Away" },
    230         { "chat",  "Free for Chat" },
     230        { "chat",  "Free for Chat" },   /* WTF actually uses this? */
    231231        { "dnd",   "Do not Disturb" },
    232232        { "xa",    "Extended Away" },
    233         { "",      "Online" },
    234233        { "",      NULL }
    235234};
    … …  
    238237{
    239238        int i;
     239       
     240        if( code == NULL )
     241                return NULL;
    240242       
    241243        for( i = 0; jabber_away_state_list[i].full_name; i ++ )
    … …  
    249251{
    250252        int i;
     253       
     254        if( name == NULL )
     255                return NULL;
    251256       
    252257        for( i = 0; jabber_away_state_list[i].full_name; i ++ )

  • protocols/jabber/presence.c

    re08e53c rfb020ac  
    190190                int is_away = 0;
    191191
    192                 if( send_presence->away_state && !( *send_presence->away_state->code == 0 ||
    193                     strcmp( send_presence->away_state->code, "chat" ) == 0 ) )
     192                if( send_presence->away_state &&
     193                    strcmp( send_presence->away_state->code, "chat" ) != 0 )
    194194                        is_away = OPT_AWAY;
    195195
    196196                imcb_buddy_status( ic, send_presence->bare_jid, OPT_LOGGED_IN | is_away,
    197                                    ( is_away && send_presence->away_state ) ?
    198                                    send_presence->away_state->full_name : NULL,
     197                                   is_away ? send_presence->away_state->full_name : NULL,
    199198                                   send_presence->away_message );
    200199        }
    … …  
    209208        struct jabber_data *jd = ic->proto_data;
    210209        struct xt_node *node, *cap;
    211         char *show = jd->away_state->code;
    212         char *status = jd->away_message;
    213210        struct groupchat *c;
    214211        int st;
    … …  
    216213        node = jabber_make_packet( "presence", NULL, NULL, NULL );
    217214        xt_add_child( node, xt_new_node( "priority", set_getstr( &ic->acc->set, "priority" ), NULL ) );
    218         if( show && *show )
    219                 xt_add_child( node, xt_new_node( "show", show, NULL ) );
    220         if( status )
    221                 xt_add_child( node, xt_new_node( "status", status, NULL ) );
     215        if( jd->away_state )
     216                xt_add_child( node, xt_new_node( "show", jd->away_state->code, NULL ) );
     217        if( jd->away_message )
     218                xt_add_child( node, xt_new_node( "status", jd->away_message, NULL ) );
    222219       
    223220        /* This makes the packet slightly bigger, but clients interested in

  • protocols/msn/msn.c

    re08e53c rfb020ac  
    139139       
    140140        if( l == NULL )
    141                 for( i = 0; msn_away_state_list[i].number > -1; i ++ )
    142                         l = g_list_append( l, (void*) msn_away_state_list[i].name );
     141                for( i = 0; *msn_away_state_list[i].code; i ++ )
     142                        if( *msn_away_state_list[i].name )
     143                                l = g_list_append( l, (void*) msn_away_state_list[i].name );
    143144       
    144145        return l;
    … …  
    149150        char buf[1024];
    150151        struct msn_data *md = ic->proto_data;
    151         const struct msn_away_state *st;
    152        
    153         if( strcmp( state, GAIM_AWAY_CUSTOM ) == 0 )
    154                 st = msn_away_state_by_name( "Away" );
     152       
     153        if( state )
     154                md->away_state = msn_away_state_by_name( state ) ? :
     155                                 msn_away_state_list + 1;
    155156        else
    156                 st = msn_away_state_by_name( state );
    157        
    158         if( !st ) st = msn_away_state_list;
    159         md->away_state = st;
    160        
    161         g_snprintf( buf, sizeof( buf ), "CHG %d %s\r\n", ++md->trId, st->code );
     157                md->away_state = msn_away_state_list;
     158       
     159        g_snprintf( buf, sizeof( buf ), "CHG %d %s\r\n", ++md->trId, md->away_state->code );
    162160        msn_write( ic, buf, strlen( buf ) );
    163161}

  • protocols/msn/msn.h

    re08e53c rfb020ac  
    9797struct msn_away_state
    9898{
    99         int number;
    10099        char code[4];
    101100        char name[16];

  • protocols/msn/msn_util.c

    re08e53c rfb020ac  
    374374        *list = NULL;
    375375       
    376         imcb_log( ic, ret->str );
     376        imcb_log( ic, "%s", ret->str );
    377377        g_string_free( ret, TRUE );
    378378}

  • protocols/msn/ns.c

    re08e53c rfb020ac  
    420420                {
    421421                        /* FIXME: Warn/Bomb about unknown away state? */
    422                         st = msn_away_state_list;
    423                 }
    424                
    425                 imcb_buddy_status( ic, cmd[3], OPT_LOGGED_IN |
    426                                    ( st->number ? OPT_AWAY : 0 ), st->name, NULL );
     422                        st = msn_away_state_list + 1;
     423                }
     424               
     425                imcb_buddy_status( ic, cmd[3], OPT_LOGGED_IN |
     426                                   ( st != msn_away_state_list ? OPT_AWAY : 0 ),
     427                                   st->name, NULL );
    427428        }
    428429        else if( strcmp( cmd[0], "FLN" ) == 0 )
    … …  
    449450                {
    450451                        /* FIXME: Warn/Bomb about unknown away state? */
    451                         st = msn_away_state_list;
    452                 }
    453                
    454                 imcb_buddy_status( ic, cmd[2], OPT_LOGGED_IN |
    455                                    ( st->number ? OPT_AWAY : 0 ), st->name, NULL );
     452                        st = msn_away_state_list + 1;
     453                }
     454               
     455                imcb_buddy_status( ic, cmd[2], OPT_LOGGED_IN |
     456                                   ( st != msn_away_state_list ? OPT_AWAY : 0 ),
     457                                   st->name, NULL );
    456458        }
    457459        else if( strcmp( cmd[0], "RNG" ) == 0 )
    … …  
    663665                                }
    664666                               
    665                                 if( arg1 ) g_free( arg1 );
    666                                 if( mtype ) g_free( mtype );
     667                                g_free( arg1 );
     668                                g_free( mtype );
    667669                        }
    668670                        else if( g_strncasecmp( ct, "text/x-msmsgsprofile", 20 ) == 0 )
    … …  
    672674                        else if( g_strncasecmp( ct, "text/x-msmsgsinitialemailnotification", 37 ) == 0 )
    673675                        {
    674                                 char *inbox = msn_findheader( body, "Inbox-Unread:", blen );
    675                                 char *folders = msn_findheader( body, "Folders-Unread:", blen );
    676                                
    677                                 if( inbox && folders && set_getbool( &ic->acc->set, "mail_notifications" ) )
     676                                if( set_getbool( &ic->acc->set, "mail_notifications" ) )
    678677                                {
    679                                         imcb_log( ic, "INBOX contains %s new messages, plus %s messages in other folders.", inbox, folders );
     678                                        char *inbox = msn_findheader( body, "Inbox-Unread:", blen );
     679                                        char *folders = msn_findheader( body, "Folders-Unread:", blen );
     680
     681                                        if( inbox && folders )
     682                                                imcb_log( ic, "INBOX contains %s new messages, plus %s messages in other folders.", inbox, folders );
     683                                       
     684                                        g_free( inbox );
     685                                        g_free( folders );
    680686                                }
    681                                
    682                                 g_free( inbox );
    683                                 g_free( folders );
    684687                        }
    685688                        else if( g_strncasecmp( ct, "text/x-msmsgsemailnotification", 30 ) == 0 )
    686689                        {
    687                                 char *from = msn_findheader( body, "From-Addr:", blen );
    688                                 char *fromname = msn_findheader( body, "From:", blen );
    689                                
    690                                 if( from && fromname && set_getbool( &ic->acc->set, "mail_notifications" ) )
     690                                if( set_getbool( &ic->acc->set, "mail_notifications" ) )
    691691                                {
    692                                         imcb_log( ic, "Received an e-mail message from %s <%s>.", fromname, from );
     692                                        char *from = msn_findheader( body, "From-Addr:", blen );
     693                                        char *fromname = msn_findheader( body, "From:", blen );
     694                                       
     695                                        if( from && fromname )
     696                                                imcb_log( ic, "Received an e-mail message from %s <%s>.", fromname, from );
     697
     698                                        g_free( from );
     699                                        g_free( fromname );
    693700                                }
    694701                        }

  • protocols/msn/tables.c

    re08e53c rfb020ac  
    2929const struct msn_away_state msn_away_state_list[] =
    3030{
    31         {  0, "NLN", "Available" },
    32         {  1, "BSY", "Busy" },
    33         {  3, "IDL", "Idle" },
    34         {  5, "BRB", "Be Right Back" },
    35         {  7, "AWY", "Away" },
    36         {  9, "PHN", "On the Phone" },
    37         { 11, "LUN", "Out to Lunch" },
    38         { 13, "HDN", "Hidden" },
    39         { -1, "",    "" }
     31        { "NLN", "" },
     32        { "AWY", "Away" },
     33        { "BSY", "Busy" },
     34        { "IDL", "Idle" },
     35        { "BRB", "Be Right Back" },
     36        { "PHN", "On the Phone" },
     37        { "LUN", "Out to Lunch" },
     38        { "HDN", "Hidden" },
     39        { "",    "" }
    4040};
    41 
    42 const struct msn_away_state *msn_away_state_by_number( int number )
    43 {
    44         int i;
    45        
    46         for( i = 0; msn_away_state_list[i].number > -1; i ++ )
    47                 if( msn_away_state_list[i].number == number )
    48                         return( msn_away_state_list + i );
    49        
    50         return( NULL );
    51 }
    5241
    5342const struct msn_away_state *msn_away_state_by_code( char *code )
    … …  
    5544        int i;
    5645       
    57         for( i = 0; msn_away_state_list[i].number > -1; i ++ )
     46        for( i = 0; *msn_away_state_list[i].code; i ++ )
    5847                if( g_strcasecmp( msn_away_state_list[i].code, code ) == 0 )
    5948                        return( msn_away_state_list + i );
    6049       
    61         return( NULL );
     50        return NULL;
    6251}
    6352
    … …  
    6655        int i;
    6756       
    68         for( i = 0; msn_away_state_list[i].number > -1; i ++ )
     57        for( i = 0; *msn_away_state_list[i].code; i ++ )
    6958                if( g_strcasecmp( msn_away_state_list[i].name, name ) == 0 )
    7059                        return( msn_away_state_list + i );
    7160       
    72         return( NULL );
     61        return NULL;
    7362}
    7463

  • protocols/nogaim.c

    re08e53c rfb020ac  
    22  * BitlBee -- An IRC to other IM-networks gateway                     *
    33  *                                                                    *
    4   * Copyright 2002-2006 Wilmer van der Gaast and others                *
     4  * Copyright 2002-2010 Wilmer van der Gaast and others                *
    55  \********************************************************************/
    66
    … …  
    276276        ic->flags |= OPT_LOGGED_IN;
    277277       
    278         /* Also necessary when we're not away, at least for some of the
    279            protocols. */
    280         imc_set_away( ic, u->away );
     278        /* Necessary to send initial presence status, even if we're not away. */
     279        imc_away_send_update( ic );
    281280       
    282281        /* Apparently we're connected successfully, so reset the
    … …  
    10781077}
    10791078
    1080 static char *imc_away_alias_find( GList *gcm, char *away );
    1081 
    1082 int imc_set_away( struct im_connection *ic, char *away )
    1083 {
    1084         GList *m, *ms;
    1085         char *s;
    1086        
    1087         if( !away ) away = "";
    1088         ms = m = ic->acc->prpl->away_states( ic );
    1089        
    1090         while( m )
    1091         {
    1092                 if( *away )
    1093                 {
    1094                         if( g_strncasecmp( m->data, away, strlen( m->data ) ) == 0 )
    1095                                 break;
    1096                 }
    1097                 else
    1098                 {
    1099                         if( g_strcasecmp( m->data, "Available" ) == 0 )
    1100                                 break;
    1101                         if( g_strcasecmp( m->data, "Online" ) == 0 )
    1102                                 break;
    1103                 }
    1104                 m = m->next;
    1105         }
    1106        
    1107         if( m )
    1108         {
    1109                 ic->acc->prpl->set_away( ic, m->data, *away ? away : NULL );
    1110         }
    1111         else
    1112         {
    1113                 s = imc_away_alias_find( ms, away );
    1114                 if( s )
    1115                 {
    1116                         ic->acc->prpl->set_away( ic, s, away );
    1117                         if( set_getbool( &ic->irc->set, "debug" ) )
    1118                                 imcb_log( ic, "Setting away state to %s", s );
    1119                 }
    1120                 else
    1121                         ic->acc->prpl->set_away( ic, GAIM_AWAY_CUSTOM, away );
    1122         }
    1123        
    1124         return( 1 );
     1079static char *imc_away_state_find( GList *gcm, char *away, char **message );
     1080
     1081int imc_away_send_update( struct im_connection *ic )
     1082{
     1083        char *away, *msg;
     1084       
     1085        away = set_getstr( &ic->acc->set, "away" ) ?
     1086             : set_getstr( &ic->irc->set, "away" );
     1087        if( away && *away )
     1088        {
     1089                GList *m = ic->acc->prpl->away_states( ic );
     1090                msg = ic->acc->flags & ACC_FLAG_AWAY_MESSAGE ? away : NULL;
     1091                away = imc_away_state_find( m, away, &msg ) ? : m->data;
     1092        }
     1093        else if( ic->acc->flags & ACC_FLAG_STATUS_MESSAGE )
     1094        {
     1095                away = NULL;
     1096                msg = set_getstr( &ic->acc->set, "status" ) ?
     1097                    : set_getstr( &ic->irc->set, "status" );
     1098        }
     1099       
     1100        ic->acc->prpl->set_away( ic, away, msg );
     1101       
     1102        return 1;
    11251103}
    11261104
    … …  
    11371115};
    11381116
    1139 static char *imc_away_alias_find( GList *gcm, char *away )
     1117static char *imc_away_state_find( GList *gcm, char *away, char **message )
    11401118{
    11411119        GList *m;
    11421120        int i, j;
    11431121       
     1122        for( m = gcm; m; m = m->next )
     1123                if( g_strncasecmp( m->data, away, strlen( m->data ) ) == 0 )
     1124                {
     1125                        /* At least the Yahoo! module works better if message
     1126                           contains no data unless it adds something to what
     1127                           we have in state already. */
     1128                        if( strlen( m->data ) == strlen( away ) )
     1129                                *message = NULL;
     1130                       
     1131                        return m->data;
     1132                }
     1133       
    11441134        for( i = 0; *imc_away_alias_list[i]; i ++ )
    11451135        {
     1136                int keep_message;
     1137               
    11461138                for( j = 0; imc_away_alias_list[i][j]; j ++ )
    11471139                        if( g_strncasecmp( away, imc_away_alias_list[i][j], strlen( imc_away_alias_list[i][j] ) ) == 0 )
     1140                        {
     1141                                keep_message = strlen( away ) != strlen( imc_away_alias_list[i][j] );
    11481142                                break;
     1143                        }
    11491144               
    11501145                if( !imc_away_alias_list[i][j] )        /* If we reach the end, this row */
    … …  
    11541149                for( j = 0; imc_away_alias_list[i][j]; j ++ )
    11551150                {
    1156                         m = gcm;
    1157                         while( m )
    1158                         {
     1151                        for( m = gcm; m; m = m->next )
    11591152                                if( g_strcasecmp( imc_away_alias_list[i][j], m->data ) == 0 )
    1160                                         return( imc_away_alias_list[i][j] );
    1161                                 m = m->next;
    1162                         }
    1163                 }
    1164         }
    1165        
    1166         return( NULL );
     1153                                {
     1154                                        if( !keep_message )
     1155                                                *message = NULL;
     1156                                       
     1157                                        return imc_away_alias_list[i][j];
     1158                                }
     1159                }
     1160               
     1161                /* No need to look further, apparently this state doesn't
     1162                   have any good alias for this protocol. */
     1163                break;
     1164        }
     1165       
     1166        return NULL;
    11671167}
    11681168

  • protocols/nogaim.h

    re08e53c rfb020ac  
    4949
    5050#define WEBSITE "http://www.bitlbee.org/"
    51 #define GAIM_AWAY_CUSTOM "Custom"
    5251
    5352/* Sharing flags between all kinds of things. I just hope I won't hit any
    … …  
    219218       
    220219        /* You can tell what away states your protocol supports, so that
    221          * BitlBee will try to map the IRC away reasons to them, or use
    222          * GAIM_AWAY_CUSTOM when calling skype_set_away(). */
     220         * BitlBee will try to map the IRC away reasons to them. If your
     221         * protocol doesn't have any, just return one generic "Away". */
    223222        GList *(* away_states)(struct im_connection *ic);
    224223       
    … …  
    316315
    317316/* Actions, or whatever. */
    318 int imc_set_away( struct im_connection *ic, char *away );
     317int imc_away_send_update( struct im_connection *ic );
    319318int imc_buddy_msg( struct im_connection *ic, char *handle, char *msg, int flags );
    320319int imc_chat_msg( struct groupchat *c, char *msg, int flags );

  • protocols/oscar/oscar.c

    re08e53c rfb020ac  
    380380                s->flags |= ACC_SET_OFFLINE_ONLY;
    381381        }
     382       
     383        acc->flags |= ACC_FLAG_AWAY_MESSAGE;
    382384}
    383385
    … …  
    19521954static void oscar_set_away_aim(struct im_connection *ic, struct oscar_data *od, const char *state, const char *message)
    19531955{
     1956        if (state == NULL)
     1957                state = "";
    19541958
    19551959        if (!g_strcasecmp(state, _("Visible"))) {
    … …  
    19591963                aim_setextstatus(od->sess, od->conn, AIM_ICQ_STATE_INVISIBLE);
    19601964                return;
    1961         } /* else... */
     1965        } else if (message == NULL) {
     1966                message = state;
     1967        }
    19621968
    19631969        if (od->rights.maxawaymsglen == 0)
    … …  
    20022008        }
    20032009
    2004         if (!g_strcasecmp(state, "Online")) {
     2010        if (state == NULL) {
    20052011                aim_setextstatus(od->sess, od->conn, AIM_ICQ_STATE_NORMAL);
    20062012        } else if (!g_strcasecmp(state, "Away")) {
    … …  
    20272033                aim_setextstatus(od->sess, od->conn, AIM_ICQ_STATE_INVISIBLE);
    20282034                ic->away = g_strdup(msg);
    2029         } else if (!g_strcasecmp(state, GAIM_AWAY_CUSTOM)) {
     2035        } else {
    20302036                if (no_message) {
    20312037                        aim_setextstatus(od->sess, od->conn, AIM_ICQ_STATE_NORMAL);
    … …  
    22762282{
    22772283        struct oscar_data *od = ic->proto_data;
    2278         GList *m = NULL;
    2279 
    2280         if (!od->icq)
    2281                 return g_list_append(m, GAIM_AWAY_CUSTOM);
    2282 
    2283         m = g_list_append(m, "Online");
    2284         m = g_list_append(m, "Away");
    2285         m = g_list_append(m, "Do Not Disturb");
    2286         m = g_list_append(m, "Not Available");
    2287         m = g_list_append(m, "Occupied");
    2288         m = g_list_append(m, "Free For Chat");
    2289         m = g_list_append(m, "Invisible");
    2290 
    2291         return m;
     2284
     2285        if (od->icq) {
     2286                static GList *m = NULL;
     2287                m = g_list_append(m, "Away");
     2288                m = g_list_append(m, "Do Not Disturb");
     2289                m = g_list_append(m, "Not Available");
     2290                m = g_list_append(m, "Occupied");
     2291                m = g_list_append(m, "Free For Chat");
     2292                m = g_list_append(m, "Invisible");
     2293                return m;
     2294        } else {
     2295                static GList *m = NULL;
     2296                m = g_list_append(m, "Away");
     2297                return m;
     2298        }
    22922299}
    22932300

  • protocols/yahoo/libyahoo2.c

    re08e53c rfb020ac  
    15301530                        newbud = y_new0(struct yahoo_buddy, 1);
    15311531                        newbud->id = strdup(pair->value);
    1532                         if(cur_group)
     1532                        if (cur_group) {
    15331533                                newbud->group = strdup(cur_group);
    1534                         else {
    1535                                 struct yahoo_buddy *lastbud = (struct yahoo_buddy *)y_list_nth(
    1536                                                                 yd->buddies, y_list_length(yd->buddies)-1)->data;
    1537                                 newbud->group = strdup(lastbud->group);
     1534                        } else {
     1535                                YList *last;
     1536                                struct yahoo_buddy *lastbud;
     1537                               
     1538                                for (last = yd->buddies; last && last->next; last = last->next);
     1539                                if (last) {
     1540                                        lastbud = last->data;
     1541                                        newbud->group = strdup(lastbud->group);
     1542                                } else {
     1543                                        newbud->group = strdup("Buddies");
     1544                                }
    15381545                        }
    15391546
    … …  
    23932400{
    23942401        struct yahoo_https_auth_data *had = req->data;
    2395         struct yahoo_input_data *yid = had->yid;
    2396         struct yahoo_data *yd = yid->yd;
     2402        struct yahoo_input_data *yid;
     2403        struct yahoo_data *yd;
    23972404        int st;
     2405       
     2406        if (y_list_find(inputs, had->yid) == NULL)
     2407                return;
     2408       
     2409        yid = had->yid;
     2410        yd = yid->yd;
    23982411       
    23992412        if (req->status_code != 200) {
    … …  
    24362449{
    24372450        struct yahoo_https_auth_data *had = req->data;
    2438         struct yahoo_input_data *yid = had->yid;
    2439         struct yahoo_data *yd = yid->yd;
     2451        struct yahoo_input_data *yid;
     2452        struct yahoo_data *yd;
    24402453        struct yahoo_packet *pack;
    2441         char *crumb;
     2454        char *crumb = NULL;
    24422455        int st;
     2456       
     2457        if (y_list_find(inputs, had->yid) == NULL)
     2458                return;
     2459       
     2460        yid = had->yid;
     2461        yd = yid->yd;
    24432462       
    24442463        md5_byte_t result[16];
    … …  
    40804099
    40814100        yd = yid->yd;
    4082 
    40834101        old_status = yd->current_status;
    4084 
    4085         if (msg && strncmp(msg,"Invisible",9)) {
    4086                 yd->current_status = YAHOO_STATUS_CUSTOM;
    4087         } else {
    4088                 yd->current_status = state;
    4089         }
     4102        yd->current_status = state;
    40904103
    40914104        /* Thank you libpurple :) */
    … …  
    41024115        snprintf(s, sizeof(s), "%d", yd->current_status);
    41034116        yahoo_packet_hash(pkt, 10, s);
    4104          
    4105         if (yd->current_status == YAHOO_STATUS_CUSTOM) {
    4106                 yahoo_packet_hash(pkt, 19, msg);
    4107         } else {
    4108                 yahoo_packet_hash(pkt, 19, "");
    4109         }
    4110        
     4117        yahoo_packet_hash(pkt, 19, msg && state == YAHOO_STATUS_CUSTOM ? msg : "");
    41114118        yahoo_packet_hash(pkt, 47, (away == 2)? "2": (away) ?"1":"0");
    4112 
    41134119        yahoo_send_packet(yid, pkt, 0);
    41144120        yahoo_packet_free(pkt);

  • protocols/yahoo/yahoo.c

    re08e53c rfb020ac  
    130130{
    131131        set_add( &acc->set, "mail_notifications", "false", set_eval_bool, acc );
     132       
     133        acc->flags |= ACC_FLAG_AWAY_MESSAGE | ACC_FLAG_STATUS_MESSAGE;
    132134}
    133135
    … …  
    197199{
    198200        struct byahoo_data *yd = (struct byahoo_data *) ic->proto_data;
    199         char *away;
    200        
    201         away = NULL;
    202        
    203         if( state && msg && g_strcasecmp( state, msg ) != 0 )
    204         {
    205                 yd->current_status = YAHOO_STATUS_CUSTOM;
    206                 away = "";
    207         }
    208         else if( state )
    209         {
    210                 /* Set msg to NULL since (if it isn't NULL already) it's equal
    211                    to state. msg must be empty if we want to use an existing
    212                    away state. */
    213                 msg = NULL;
    214                
    215                 away = "";
    216                 if( g_strcasecmp( state, "Available" ) == 0 )
    217                 {
    218                         yd->current_status = YAHOO_STATUS_AVAILABLE;
    219                         away = NULL;
    220                 }
    221                 else if( g_strcasecmp( state, "Be Right Back" ) == 0 )
     201       
     202        if( state && msg == NULL )
     203        {
     204                /* Use these states only if msg doesn't contain additional
     205                   info since away messages are only supported with CUSTOM. */
     206                if( g_strcasecmp( state, "Be Right Back" ) == 0 )
    222207                        yd->current_status = YAHOO_STATUS_BRB;
    223208                else if( g_strcasecmp( state, "Busy" ) == 0 )
    … …  
    239224                else if( g_strcasecmp( state, "Invisible" ) == 0 )
    240225                        yd->current_status = YAHOO_STATUS_INVISIBLE;
    241                 else if( g_strcasecmp( state, GAIM_AWAY_CUSTOM ) == 0 )
    242                 {
    243                         yd->current_status = YAHOO_STATUS_AVAILABLE;
    244                        
    245                         away = NULL;
    246                 }
    247         }
     226                else
     227                        yd->current_status = YAHOO_STATUS_CUSTOM;
     228        }
     229        else if( state )
     230                yd->current_status = YAHOO_STATUS_CUSTOM;
    248231        else
    249232                yd->current_status = YAHOO_STATUS_AVAILABLE;
    250233       
    251         yahoo_set_away( yd->y2_id, yd->current_status, msg, away != NULL ? 2 : 0 );
     234        yahoo_set_away( yd->y2_id, yd->current_status, msg, state ? 2 : 0 );
    252235}
    253236
    … …  
    258241        if( m == NULL )
    259242        {
    260                 m = g_list_append( m, "Available" );
    261243                m = g_list_append( m, "Be Right Back" );
    262244                m = g_list_append( m, "Busy" );
    … …  
    269251                m = g_list_append( m, "Stepped Out" );
    270252                m = g_list_append( m, "Invisible" );
    271                 m = g_list_append( m, GAIM_AWAY_CUSTOM );
    272253        }
    273254       
Note: See TracChangeset for help on using the changeset viewer.