gnucash maint: Add a missing gettext in plugin page owner tree

Geert Janssens geert.gnucash at kobaltwit.be
Fri Jan 8 04:39:50 EST 2016


On Friday 01 January 2016 19:21:49 Frank H. Ellenberger wrote:
> Updated	 via  https://github.com/Gnucash/gnucash/commit/c2598f89
> (commit) from  https://github.com/Gnucash/gnucash/commit/b4f39ebf
> (commit)
> 
> 
> 
> commit c2598f89c59c5f44640e4b924222f6f224b7e372
> Author: fell <f.ellenberger at online.de>
> Date:   Sat Jan 2 01:21:42 2016 +0100
> 
>     Add a missing gettext in plugin page owner tree
> 
>     Thanks to Mechtilde on IRC
> 
> diff --git a/src/business/business-gnome/gnc-plugin-page-owner-tree.c
> b/src/business/business-gnome/gnc-plugin-page-owner-tree.c index
> 1faa339..84d8530 100644
> --- a/src/business/business-gnome/gnc-plugin-page-owner-tree.c
> +++ b/src/business/business-gnome/gnc-plugin-page-owner-tree.c
> @@ -626,7 +626,7 @@ gnc_plugin_page_owner_tree_create_widget
> (GncPluginPage *plugin_page) "show-column-menu", TRUE, NULL);
> 
> -    g_object_set(G_OBJECT(plugin_page), "page-name", label, NULL);
> +    g_object_set(G_OBJECT(plugin_page), "page-name", _(label), NULL);
> 
>      priv->tree_view = tree_view;
>      selection = gtk_tree_view_get_selection(tree_view);
> 
> 
> 
> Summary of changes:
>  src/business/business-gnome/gnc-plugin-page-owner-tree.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Frank,

I'm not sure this is the right solution to the problem. Instead of marking the variable name as 
translatable I'd be inclined to mark the label definitions themselves as translatable. These are a 
few lines up in the code and currently set as N_(...). We could change them there to _(...) to 
avoid marking a variable instead of a constant as translatable.

What do you think ?

Geert


More information about the gnucash-devel mailing list