GNOME Bugzilla – Bug 332833
The i18n handling of gnome-default-applications-properties does not work.
Last modified: 2006-02-28 20:21:03 UTC
Please describe the problem: The 2.13.x version of gnome-default-applications-properties does not show i18N translated messages on my locale. The other components in the same version of gnome-control-center work fine. It looks that the domain names in glade_xml_new in "gnome-da-capplet.c" are wrong, which is passed as PACKAGE from Makefile.am, because the running gnome-default-applications-properties calls control-center.mo instead of control-center-2.0 as locale data in strace log. My work arrround is attached. Steps to reproduce: 1. Click "Preferred Applications" from "Desktop"->"Setting" on Japanese locale (ja_JP.utf8). 2. A gnome-default-applications-properties is run. 3. All of messages are not translated, except any entries in pull-down menus which are retrieved from gnome-default-applications.xml has already generated with Japanese translation by intltool. Actual results: i18N translated messages are disappered. Expected results: i18N translated messages are appered (Japanese here). Does this happen every time? Yes. Other information: I attached a patch to work arround this issue.
Created attachment 60313 [details] [review] Change domain name in glade_xml_new() to NULL (and so on). The domain name in glade_xml_new looks pointed to the control-center instead of control-cdnter-2.0.
Created attachment 60322 [details] [review] default-applications-text-domain-name.patch Takeshi, thanks for the patch. Indeed I missed the correct text domain. I did a very very small rework of the patch. OK to commit?
Yes, please. Thanks!
Why do you use the "#ifdef ENABLE_NLS" for that?
Sebastien, isn't that defined in config.h when i18n is enabled? If not needed consider the #ifdef removed from the patch. btw, also other capplets use it.
according to a grep only ones capplet use it, anyway no objection from my part, feel free to commit
Done. Since both versions seem to be used by gnome modules, I kept the #ifdef.