# HG changeset patch
# User Christian Hammond <chipx86@chipx86.com>
# Date 1052899323 0
# Node ID d8ebee2f749c2f231cf59e4bc5cbf3f2799da9c3
# Parent  20519fff34553bbcc8a32392bf1fa55edd2757d6
[gaim-migrate @ 5751]
Clean up, do it right.

committer: Tailor Script <tailor@pidgin.im>

diff -r 20519fff3455 -r d8ebee2f749c src/protocols/msn/notification.c
--- a/src/protocols/msn/notification.c	Wed May 14 07:57:33 2003 +0000
+++ b/src/protocols/msn/notification.c	Wed May 14 08:02:03 2003 +0000
@@ -434,6 +434,7 @@
 	MsnSession *session = servconn->session;
 	struct gaim_connection *gc = session->account->gc;
 	const char *passport, *type, *value;
+	struct buddy *b;
 	MsnUser *user;
 
 	passport = params[1];
@@ -443,8 +444,18 @@
 	user = msn_users_find_with_passport(session->users, passport);
 
 	if (value != NULL) {
-		if (!strcmp(type, "MOB"))
-			user->mobile = (!strcmp(value, "Y"));
+		if (!strcmp(type, "MOB")) {
+			if (!strcmp(value, "Y")) {
+				user->mobile = TRUE;
+
+				if ((b = gaim_find_buddy(gc->account, passport)) != NULL) {
+					if (GAIM_BUDDY_IS_ONLINE(b)) {
+						serv_got_update(gc, (char *)passport,
+										1, 0, 0, 0, b->uc);
+					}
+				}
+			}
+		}
 		else if (!strcmp(type, "PHH"))
 			msn_user_set_home_phone(user, msn_url_decode(value));
 		else if (!strcmp(type, "PHW"))
@@ -453,15 +464,6 @@
 			msn_user_set_mobile_phone(user, msn_url_decode(value));
 	}
 
-	if (!strcmp(type, "MOB") || !strcmp(type, "MBE")) {
-		struct buddy *b;
-
-		if ((b = gaim_find_buddy(gc->account, passport)) != NULL) {
-			if (GAIM_BUDDY_IS_ONLINE(b))
-				serv_got_update(gc, (char *)passport, 1, 0, 0, 0, b->uc);
-		}
-	}
-
 	return TRUE;
 }