If a contact did not have a status-message or mood set, the "status_text" callback would return an empty ("") string. mxit

Sat, 13 Aug 2011 15:08:30 +0000

author
Andrew Victor <andrew.victor@mxit.com>
date
Sat, 13 Aug 2011 15:08:30 +0000
branch
mxit
changeset 32169
b500edda2f55
parent 32168
ce76a1093b21
child 32170
1d5f4a7a0790

If a contact did not have a status-message or mood set, the "status_text" callback would return an empty ("") string.
This causes the buddy list (with buddy details enabled) to display the contact's alias not in line with the status icon - which looked odd.

Now we return NULL if there is no status-message or mood set.

libpurple/protocols/mxit/mxit.c file | annotate | diff | comparison | revisions
--- a/libpurple/protocols/mxit/mxit.c	Sat Aug 13 13:13:19 2011 +0000
+++ b/libpurple/protocols/mxit/mxit.c	Sat Aug 13 15:08:30 2011 +0000
@@ -309,19 +309,18 @@
  */
 char* mxit_status_text( PurpleBuddy* buddy )
 {
+	char* text = NULL;
 	struct contact*	contact = purple_buddy_get_protocol_data(buddy);
 
 	if ( !contact )
 		return NULL;
 
-	if ( contact->statusMsg ) {
-		/* status message */
-		return g_strdup( contact-> statusMsg );
-	}
-	else {
-		/* mood */
-		return g_strdup( mxit_convert_mood_to_name( contact->mood ) );
-	}
+	if ( contact->statusMsg )							/* status message */
+		text = g_strdup( contact-> statusMsg );
+	else if ( contact->mood != MXIT_MOOD_NONE )			/* mood */
+		text = g_strdup( mxit_convert_mood_to_name( contact->mood ) );
+
+	return text;
 }
 
 

mercurial