comparison src/gtkblist.c @ 8986:8cf32769ba1b

[gaim-migrate @ 9761] " This patch adds a Plugin Actions menu item after the Account Actions menu. The Plugin Actions menu is populated from the added 'actions' slot in GaimPluginInfo. As a demonstration, the Idle Maker plugin has been converted to no longer require GTK code and the Preferences interface just to perform its actions. Instead, it uses a Plugin Action to spawn a Fields Request. There's also a minor fix for consistency in the menu building for buddy actions. The pre-existing method for instructing a menu list to display a separator was to insert a NULL rather than a proto_buddy_menu into the GList of actions. The code for the buddy menus was instead checking for a proto_buddy_menu with a '-' label. This has been fixed, and it now correctly uses NULL to indicate a separator." "Date: 2004-05-16 02:25 Sender: taliesein Logged In: YES user_id=77326 I need to add a callback to this patch to watch for loading/unloading of plugins (to determine when to rebuild the menu). Since the appropriate way to handle Plugin Actions is still mildly up for debate, I'm holding of on correcting the patch until I know for sure whether I should fix this patch, or scrap it and write a new one using a different method." "Date: 2004-05-18 12:26 Sender: taliesein Logged In: YES user_id=77326 I've completed changes to this patch to also add plugin load and unload signals (it looks like plugin.c actually had pre-signal callbacks in place, but they were never used or converted to signals) This patch now will correctly update the Plugin Action menu as plugins load and unload." I'm not entirely sure i like the ui of a plugins actions menu, but i think that having some way for plugins to add actions on an account is a good thing, and i'm not sure that every viable action fits under the accounts actions menu. we may want to merge the two (the existing accounts actions and this plugins actions), but both times it came up in #gaim no one seemed to want to comment, and on one commented to the gaim-devel post either. committer: Tailor Script <[e-mail]>

8985:8abc99ed5d93

#include "debug.h"
#include "multi.h"
#include "notify.h"
#include "prpl.h"
#include "prefs.h"
#include "request.h"
#include "signals.h"
#include "sound.h"
#include "stock.h"
#include "util.h"

	GList *entries;
} GaimGtkJoinChatData;


static GtkWidget *protomenu = NULL;

GSList *gaim_gtk_blist_sort_methods = NULL;
static struct gaim_gtk_blist_sort_method *current_sort_method = NULL;
static GtkTreeIter sort_method_none(GaimBlistNode *node, GaimBuddyList *blist, GtkTreeIter groupiter, GtkTreeIter *cur);

		list = prpl_info->buddy_menu(b->account->gc, b->name);

		for(l = list; l; l = l->next) {
			struct proto_buddy_menu *pbm = l->data;

			/* draw "-" titled menu items as a separator. Since the
			   pbm is not being used in a callback, it needs to be
			   freed. Also, do some simple checking to prevent
			   doubled-up separators */
			if('-' == *pbm->label) {
				if(! dup_separator) {
					gaim_separator(menu);
					dup_separator = TRUE;
				}
				g_free(pbm);
				continue;
			} else {
				dup_separator = FALSE;
			}

	   plugins. */
	list = gaim_buddy_get_extended_menu(b);
	for(l = list; l; l = l->next) {
		struct proto_buddy_menu *pbm = l->data;

		/* draw "-" titled menu items as a separator.  see previous,
		   identical-looking code. */
		if('-' == *pbm->label) {
			if(! dup_separator) {
				gaim_separator(menu);
				dup_separator = TRUE;
			}
			g_free(pbm);

	list = gaim_group_get_extended_menu(g);
	for(l = list; l; l = l->next) {
		struct proto_group_menu *pgm = l->data;

		/* draw "-" titled menu items as a separator.  see previous,
		   identical-looking code. (in make_buddy_menu)*/
		if(pgm == NULL) {
			if(! dup_separator) {
				gaim_separator(menu);
				dup_separator = TRUE;

	/* Tools */
	{ N_("/_Tools"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/_Away"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/Buddy _Pounce"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/Account Ac_tions"), NULL, NULL, 0, "<Branch>" },
	{ "/Tools/sep1", NULL, NULL, 0, "<Separator>" },
	{ N_("/Tools/A_ccounts"), "<CTL>A", gaim_gtk_accounts_window_show, 0, "<StockItem>", GAIM_STOCK_ACCOUNTS },
	{ N_("/Tools/_File Transfers"), NULL, gaim_show_xfer_dialog, 0, "<StockItem>", GAIM_STOCK_FILE_TRANSFER },
	{ N_("/Tools/R_oom List"), NULL, gaim_gtk_roomlist_dialog_show, 0, NULL },
	{ N_("/Tools/Pr_eferences"), "<CTL>P", gaim_gtk_prefs_show, 0, "<StockItem>", GTK_STOCK_PREFERENCES },

	widget = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Privacy"));
	gtk_widget_set_sensitive(widget, gaim_gtk_privacy_is_showable());
}

/* this is called on all sorts of signals, and we have no reason to pass
 * it anything, so it remains without arguments. If you need anything
 * more specific, do as below, and create another callback that calls
 * this */
static void

	gtkblist->bpmenu = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Buddy Pounce"));
	gaim_gtkpounce_menu_build(gtkblist->bpmenu);

	protomenu = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Account Actions"));
	gaim_gtk_blist_update_protocol_actions();
	/****************************** GtkTreeView **********************************/
	sw = gtk_scrolled_window_new(NULL,NULL);
	gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW(sw), GTK_SHADOW_IN);
	gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(sw), GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC);

	gaim_signal_connect(gaim_connections_get_handle(), "signed-on",
						gtkblist, GAIM_CALLBACK(sign_on_off_cb), list);
	gaim_signal_connect(gaim_connections_get_handle(), "signed-off",
						gtkblist, GAIM_CALLBACK(sign_on_off_cb), list);

	/* emit our created signal */
	gaim_signal_emit(gaim_gtk_blist_get_handle(), "gtkblist-created", list);
}

/* XXX: does this need fixing? */

	gtkblist->idle_column = NULL;
	gtkblist->warning_column = gtkblist->buddy_icon_column = NULL;
	gtkblist->bbox = NULL;
	g_object_unref(G_OBJECT(gtkblist->ift));
	protomenu = NULL;
	awaymenu = NULL;
	gtkblist = NULL;

	while(blist_prefs_callbacks) {
		gaim_prefs_disconnect_callback(GPOINTER_TO_INT(blist_prefs_callbacks->data));

{
	if (pam->callback && pam->gc)
		pam->callback(pam->gc);
}

void
gaim_gtk_blist_update_protocol_actions(void)
{
	GtkWidget *menuitem;
	GtkWidget *submenu;

					gaim_separator(submenu);
			}
		}
	}
}

8986:8cf32769ba1b

#include "debug.h"
#include "multi.h"
#include "notify.h"
#include "prpl.h"
#include "prefs.h"
#include "plugin.h"
#include "request.h"
#include "signals.h"
#include "sound.h"
#include "stock.h"
#include "util.h"

	GList *entries;
} GaimGtkJoinChatData;


static GtkWidget *protomenu = NULL, *pluginmenu = NULL;

GSList *gaim_gtk_blist_sort_methods = NULL;
static struct gaim_gtk_blist_sort_method *current_sort_method = NULL;
static GtkTreeIter sort_method_none(GaimBlistNode *node, GaimBuddyList *blist, GtkTreeIter groupiter, GtkTreeIter *cur);

		list = prpl_info->buddy_menu(b->account->gc, b->name);

		for(l = list; l; l = l->next) {
			struct proto_buddy_menu *pbm = l->data;

			/* draw NULL menu items as a separator. Since the
			   pbm is not being used in a callback, it needs to be
			   freed. Also, do some simple checking to prevent
			   doubled-up separators */
			if(pbm == NULL) {
				if(! dup_separator) {
					gaim_separator(menu);
					dup_separator = TRUE;
				}
				continue;
			} else {
				dup_separator = FALSE;
			}

	   plugins. */
	list = gaim_buddy_get_extended_menu(b);
	for(l = list; l; l = l->next) {
		struct proto_buddy_menu *pbm = l->data;

		/* draw NULL menu items as a separator.  see previous,
		   identical-looking code. */
		if(pbm == NULL) {
			if(! dup_separator) {
				gaim_separator(menu);
				dup_separator = TRUE;
			}
			g_free(pbm);

	list = gaim_group_get_extended_menu(g);
	for(l = list; l; l = l->next) {
		struct proto_group_menu *pgm = l->data;

		/* draw NULL menu items as a separator.  see previous,
		   identical-looking code. (in make_buddy_menu)*/
		if(pgm == NULL) {
			if(! dup_separator) {
				gaim_separator(menu);
				dup_separator = TRUE;

	/* Tools */
	{ N_("/_Tools"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/_Away"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/Buddy _Pounce"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/Account Ac_tions"), NULL, NULL, 0, "<Branch>" },
	{ N_("/Tools/Pl_ugin Actions"), NULL, NULL, 0, "<Branch>" },
	{ "/Tools/sep1", NULL, NULL, 0, "<Separator>" },
	{ N_("/Tools/A_ccounts"), "<CTL>A", gaim_gtk_accounts_window_show, 0, "<StockItem>", GAIM_STOCK_ACCOUNTS },
	{ N_("/Tools/_File Transfers"), NULL, gaim_show_xfer_dialog, 0, "<StockItem>", GAIM_STOCK_FILE_TRANSFER },
	{ N_("/Tools/R_oom List"), NULL, gaim_gtk_roomlist_dialog_show, 0, NULL },
	{ N_("/Tools/Pr_eferences"), "<CTL>P", gaim_gtk_prefs_show, 0, "<StockItem>", GTK_STOCK_PREFERENCES },

	widget = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Privacy"));
	gtk_widget_set_sensitive(widget, gaim_gtk_privacy_is_showable());
}


static void
plugin_changed_cb(GaimPlugin *p, gpointer *data)
{
	gaim_gtk_blist_update_plugin_actions();
}


/* this is called on all sorts of signals, and we have no reason to pass
 * it anything, so it remains without arguments. If you need anything
 * more specific, do as below, and create another callback that calls
 * this */
static void

	gtkblist->bpmenu = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Buddy Pounce"));
	gaim_gtkpounce_menu_build(gtkblist->bpmenu);

	protomenu = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Account Actions"));
	gaim_gtk_blist_update_protocol_actions();

	pluginmenu = gtk_item_factory_get_widget(gtkblist->ift, N_("/Tools/Plugin Actions"));
	gaim_gtk_blist_update_plugin_actions();
	/****************************** GtkTreeView **********************************/
	sw = gtk_scrolled_window_new(NULL,NULL);
	gtk_scrolled_window_set_shadow_type (GTK_SCROLLED_WINDOW(sw), GTK_SHADOW_IN);
	gtk_scrolled_window_set_policy(GTK_SCROLLED_WINDOW(sw), GTK_POLICY_AUTOMATIC, GTK_POLICY_AUTOMATIC);

	gaim_signal_connect(gaim_connections_get_handle(), "signed-on",
						gtkblist, GAIM_CALLBACK(sign_on_off_cb), list);
	gaim_signal_connect(gaim_connections_get_handle(), "signed-off",
						gtkblist, GAIM_CALLBACK(sign_on_off_cb), list);

	gaim_signal_connect(gaim_plugins_get_handle(), "plugin-load",
			gtkblist, GAIM_CALLBACK(plugin_changed_cb), NULL);
	gaim_signal_connect(gaim_plugins_get_handle(), "plugin-unload",
			gtkblist, GAIM_CALLBACK(plugin_changed_cb), NULL);

	/* emit our created signal */
	gaim_signal_emit(gaim_gtk_blist_get_handle(), "gtkblist-created", list);
}

/* XXX: does this need fixing? */

	gtkblist->idle_column = NULL;
	gtkblist->warning_column = gtkblist->buddy_icon_column = NULL;
	gtkblist->bbox = NULL;
	g_object_unref(G_OBJECT(gtkblist->ift));
	protomenu = NULL;
	pluginmenu = NULL;
	awaymenu = NULL;
	gtkblist = NULL;

	while(blist_prefs_callbacks) {
		gaim_prefs_disconnect_callback(GPOINTER_TO_INT(blist_prefs_callbacks->data));

{
	if (pam->callback && pam->gc)
		pam->callback(pam->gc);
}


static void
plugin_act(GtkObject *obk, struct plugin_actions_menu *pam)
{
	if (pam->callback && pam->plugin)
		pam->callback(pam->plugin);
}


void
gaim_gtk_blist_update_protocol_actions(void)
{
	GtkWidget *menuitem;
	GtkWidget *submenu;

					gaim_separator(submenu);
			}
		}
	}
}



void
gaim_gtk_blist_update_plugin_actions(void)
{
	GtkWidget *menuitem;
	GtkWidget *submenu;
	GaimPluginInfo *plugin_info = NULL;
	GList *l;
	GList *c;
	struct plugin_actions_menu *pam;
	int count = 0;

	if (pluginmenu == NULL)
		return;

	for (l = gtk_container_get_children(GTK_CONTAINER(pluginmenu));
		 l != NULL;
		 l = l->next) {

		menuitem = l->data;
		pam = g_object_get_data(G_OBJECT(menuitem), "plugin_actions_menu");
		g_free(pam);

		gtk_container_remove(GTK_CONTAINER(pluginmenu), GTK_WIDGET(menuitem));
	}

	for (c = gaim_plugins_get_loaded(); c != NULL; c = c->next) {
		plugin_info = ((GaimPlugin *)c->data)->info;
		if (plugin_info->actions)
			count++;
	}

	if (count == 0) {
		menuitem = gtk_menu_item_new_with_label(_("No actions available"));
		gtk_menu_shell_append(GTK_MENU_SHELL(pluginmenu), menuitem);
		gtk_widget_show(menuitem);

	}
	else if (count == 1) {
		GList *act;
		GaimPlugin *plugin = NULL;

		for (c = gaim_plugins_get_loaded(); c != NULL; c = c->next) {
			plugin = (GaimPlugin *) c->data;
			plugin_info = plugin->info;
			if (plugin_info->actions != NULL)
				break;
		}

		for (act = plugin_info->actions(plugin); act != NULL; act = act->next) {
			if (act->data) {
				struct plugin_actions_menu *pam = act->data;
				menuitem = gtk_menu_item_new_with_label(pam->label);
				gtk_menu_shell_append(GTK_MENU_SHELL(pluginmenu), menuitem);
				g_signal_connect(G_OBJECT(menuitem), "activate",
						G_CALLBACK(plugin_act), pam);
				g_object_set_data(G_OBJECT(menuitem), "plugin_actions_menu", pam);
				gtk_widget_show(menuitem);
			}
			else
				gaim_separator(pluginmenu);
		}
	}
	else {
		for (c = gaim_plugins_get_loaded(); c != NULL; c = c->next) {
			GList *act;
			GaimPlugin *plugin;

			plugin = (GaimPlugin *) c->data;
			plugin_info = plugin->info;
			if (plugin_info->actions == NULL)
				continue;

			menuitem = gtk_image_menu_item_new_with_label(plugin_info->name);
			gtk_menu_shell_append(GTK_MENU_SHELL(pluginmenu), menuitem);
			gtk_widget_show(menuitem);

			submenu = gtk_menu_new();
			gtk_menu_item_set_submenu(GTK_MENU_ITEM(menuitem), submenu);
			gtk_widget_show(submenu);

			for (act = plugin_info->actions(plugin); act != NULL; act = act->next) {
				if (act->data) {
					struct plugin_actions_menu *pam = act->data;
					menuitem = gtk_menu_item_new_with_label(pam->label);
					gtk_menu_shell_append(GTK_MENU_SHELL(submenu), menuitem);
					g_signal_connect(G_OBJECT(menuitem), "activate",
							G_CALLBACK(plugin_act), pam);
					g_object_set_data(G_OBJECT(menuitem), "plugin_actions_menu", pam);
					gtk_widget_show(menuitem);
				}
				else {
					gaim_separator(submenu);
				}
			}
		}
	}
}