pidgin: libgaim/protocols/msn/notification.c comparison

comparison libgaim/protocols/msn/notification.c @ 20391:0b0ecee55091

The second msn.tgz from SF Patch #1621854 from Ka-Hing Cheung. "new version, fixes some crashes because group_id is char * but in some places it's set to an int (!). skadotnet, I don't understand why you need `if (!strcmp(type, "MFN")) {` in so many places, so I didn't include most of them. I also have different versions of some of your fixes because I already fixed them. Group manipulations may still not work, at least it's not crashing :-)" committer: Richard Laager <rlaager@wiktel.com>

author	Ka-Hing Cheung <khc@hxbc.us>
date	Sun, 15 Apr 2007 02:43:47 +0000
parents	e354528c4163
children	9ba7dee775e1

comparison

equal deleted inserted replaced

-:d634f88e25d8
+:0b0ecee55091
 static MsnTable *cbs_table;
 /****************************************************************************
 * 	Local Function Prototype
 ****************************************************************************/
-void msn_notification_post_adl(MsnCmdProc *cmdproc,char *payload ,int payload_len);
+static void msn_notification_fqy_yahoo(MsnSession *session, const char *passport);
-void msn_add_contact_xml(xmlnode *mlNode,const char *passport,int list_op,int type);
+static void msn_notification_post_adl(MsnCmdProc *cmdproc, char *payload, int payload_len);
+static void msn_add_contact_xml(xmlnode *mlNode, const char *passport, int list_op, int type);
 /**************************************************************************
 * Main
 **************************************************************************/
 	account = session->account;
 	gc = gaim_account_get_connection(account);
 	if (!g_ascii_strcasecmp(cmd->params[1], "OK")){
 		/* authenticate OK */
+		/* friendly name part no longer true in msnp11 */
+#if 0
 		const char *friendly = gaim_url_decode(cmd->params[3]);
 		gaim_connection_set_display_name(gc, friendly);
+#endif
 		msn_session_set_login_step(session, MSN_LOGIN_STEP_SYN);
 //		msn_cmdproc_send(cmdproc, "SYN", "%s", "0");
 		//TODO we should use SOAP contact to fetch contact list
-	}else if (!g_ascii_strcasecmp(cmd->params[1], "TWN")){
+	} else if (!g_ascii_strcasecmp(cmd->params[1], "TWN")){
 		/* Passport authentication */
 		char **elems, **cur, **tokens;
 		session->nexus = msn_nexus_new(session);
 /**************************************************************************
 * Buddy Lists
 **************************************************************************/
 /* add contact to xmlnode */
-void
+static void
 msn_add_contact_xml(xmlnode *mlNode,const char *passport,int list_op,int type)
 {
 	xmlnode *d_node,*c_node;
 	char **tokens;
 	char *email,*domain;
 	xmlnode_insert_child(d_node, c_node);
 	g_free(tokens);
 }
-void
+static void
 msn_notification_post_adl(MsnCmdProc *cmdproc,char *payload, int payload_len)
 {
 	MsnTransaction *trans;
+	const char *display_name;
+	const char *friendly;
+	display_name = gaim_connection_get_display_name(cmdproc->session->account->gc);
+	if (display_name) {
+		friendly = gaim_url_encode(display_name);
+		msn_cmdproc_send(cmdproc, "PRP", "MFN %s", friendly);
+	}
 	gaim_debug_info("MaYuan","Send ADL{%s}\n",payload);
 	trans = msn_transaction_new(cmdproc, "ADL","%d",strlen(payload));
 	msn_transaction_set_payload(trans, payload, strlen(payload));
 	msn_cmdproc_send_trans(cmdproc, trans);
 	msn_notification_post_adl(session->notification->cmdproc,payload,payload_len);
 }
 /*Post FQY to NS,Inform add a Yahoo User*/
-void
+static void
-msn_notification_fqy_yahoo(MsnSession *session,char *passport)
+msn_notification_fqy_yahoo(MsnSession *session, const char *passport)
 {
 	MsnTransaction *trans;
 	MsnCmdProc *cmdproc;
 	char* email,*domain,*payload;
 	char **tokens;
 }
 static void
 rml_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 {
+#if 0
 	MsnTransaction *trans;
 	char * payload;
+#endif
 	gaim_debug_info("MaYuan","Process ADL\n");
 #if 0
 	trans = msn_transaction_new(cmdproc, "RML","");
 	group_id = atoi(cmd->params[3]);
 	group_name = gaim_url_decode(cmd->params[2]);
-	msn_group_new(session->userlist, group_id, group_name);
+	msn_group_new(session->userlist, cmd->params[3], group_name);
 	/* There is a user that must me moved to this group */
 	if (cmd->trans->data)
 	{
 		/* msn_userlist_move_buddy(); */
 			msn_user_set_home_phone(session->user, NULL);
 		else if (!strcmp(type, "PHW"))
 			msn_user_set_work_phone(session->user, NULL);
 		else if (!strcmp(type, "PHM"))
 			msn_user_set_mobile_phone(session->user, NULL);
+		else if (!strcmp(type, "MFM")) {
+			type = cmd->params[1];
+			gaim_connection_set_display_name(
+				gaim_account_get_connection(session->account),
+				gaim_url_decode(cmd->params[2]));
+		}
 	}
 }
 static void
 reg_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 {
 	MsnSession *session;
-	int group_id;
+	const char *group_id, *group_name;
-	const char *group_name;
 	session = cmdproc->session;
-	group_id = atoi(cmd->params[2]);
+	group_id = cmd->params[2];
 	group_name = gaim_url_decode(cmd->params[3]);
 	msn_userlist_rename_group_id(session->userlist, group_id, group_name);
 }
 static void
 rem_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 {
 	MsnSession *session;
 	MsnUser *user;
-	const char *list;
+	const char *group_id, *list, *passport;
-	const char *passport;
 	MsnListId list_id;
-	int group_id;
 	session = cmdproc->session;
 	list = cmd->params[1];
 	passport = cmd->params[3];
 	user = msn_userlist_find_user(session->userlist, passport);
 	g_return_if_fail(user != NULL);
 	list_id = msn_get_list_id(list);
 	if (cmd->param_count == 5)
-		group_id = atoi(cmd->params[4]);
+		group_id = cmd->params[4];
 	else
-		group_id = -1;
+		group_id = NULL;
 	msn_got_rem_user(session, user, list_id, group_id);
 	msn_user_update(user);
 }
 static void
 rmg_cmd(MsnCmdProc *cmdproc, MsnCommand *cmd)
 {
 	MsnSession *session;
-	int group_id;
+	const char *group_id;
 	session = cmdproc->session;
-	group_id = atoi(cmd->params[2]);
+	group_id = cmd->params[2];
 	msn_userlist_remove_group_id(session->userlist, group_id);
 }
 static void
 rmg_error(MsnCmdProc *cmdproc, MsnTransaction *trans, int error)
 {
-	int group_id;
+	const char *group_id;
 	char **params;
 	params = g_strsplit(trans->params, " ", 0);
 	group_id = atoi(params[0]);
 {
 	MsnSession *session;
 	GaimAccount *account;
 	GaimConnection *gc;
 	MsnUser *user;
-	const char *passport, *psm_str;
+	const char *passport;
+	char *psm_str;
 	/*get the payload content*/
 //	gaim_debug_info("MaYuan","UBX {%s} payload{%s}\n",cmd->params[0], cmd->payload);
 	session = cmdproc->session;
 	msn_userlist_load(session);
 	msn_session_set_bnode(session);
 	session->contact = msn_contact_new(session);
 	clLastChange = gaim_blist_node_get_string(msn_session_get_bnode(session),"CLLastChange");
-	msn_get_contact_list(session->contact,clLastChange);
+	msn_get_contact_list(session->contact, clLastChange);
 //	msn_contact_connect(session->contact);
 }
 static void
 initial_email_msg(MsnCmdProc *cmdproc, MsnMessage *msg)
 msn_notification_add_buddy(MsnNotification *notification, const char *list,
 						   const char *who, const char *store_name,
 						   const char *group_id)
 {
 	MsnCmdProc *cmdproc;
-	MsnTransaction *trans;
 	xmlnode *adl_node;
 	char *payload;
 	int payload_len;
 	cmdproc = notification->servconn->cmdproc;

Mercurial > pidgin

comparison libgaim/protocols/msn/notification.c @ 20391:0b0ecee55091