pidgin: 93a6a9d2: Use purple_buddy_get_group() instead of ...

[markdoliner at pidgin.im]

-----------------------------------------------------------------
Revision: 93a6a9d25f3287ff5fa1ef6ca81f8582055293e1
Ancestor: b61e5758165be55608716938251c664cac11a633
Author: markdoliner at pidgin.im
Date: 2009-01-09T02:43:38
Branch: im.pidgin.pidgin
URL: http://d.pidgin.im/viewmtn/revision/info/93a6a9d25f3287ff5fa1ef6ca81f8582055293e1

Modified files:
        libpurple/protocols/oscar/oscar.c

ChangeLog: 

Use purple_buddy_get_group() instead of aim_ssi_itemlist_exists() to check
if the buddy is in our list or not.  Should be much more efficient.

-------------- next part --------------
============================================================
--- libpurple/protocols/oscar/oscar.c	df9c1dd9a0c07de4061d536e80bc683b3dcbe699
+++ libpurple/protocols/oscar/oscar.c	5586407473f4a24cecc37b6ee63f4443cb87768c
@@ -6200,8 +6200,9 @@ oscar_buddy_menu(PurpleBuddy *buddy) {
 		menu = g_list_prepend(menu, act);
 	}
 
-	if (aim_ssi_itemlist_exists(od->ssi.local, buddy->name) != NULL)
+	if (purple_buddy_get_group(buddy) != NULL)
 	{
+		/* We only do this if the user is in our buddy list */
 		act = purple_menu_action_new(_("Edit Buddy Comment"),
 		                           PURPLE_CALLBACK(oscar_buddycb_edit_comment),
 		                           NULL, NULL);
@@ -6240,7 +6241,7 @@ oscar_buddy_menu(PurpleBuddy *buddy) {
 #endif
 	}
 
-	if (od->ssi.received_data)
+	if (od->ssi.received_data && purple_buddy_get_group(buddy) != NULL)
 	{
 		char *gname;
 		gname = aim_ssi_itemlist_findparentname(od->ssi.local, buddy->name);