After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 574700 - Messages from Python plugins don't show up localized
Messages from Python plugins don't show up localized
Status: RESOLVED FIXED
Product: totem
Classification: Core
Component: Plugins
2.25.x
Other Linux
: Normal blocker
: ---
Assigned To: General Totem maintainer(s)
General Totem maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2009-03-10 02:08 UTC by Gabor Kelemen
Modified: 2009-03-10 14:35 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Simple patch (1.12 KB, patch)
2009-03-10 02:19 UTC, Gabor Kelemen
none Details | Review

Description Gabor Kelemen 2009-03-10 02:08:57 UTC
Strings from the jamendo.py, opensubtitles.py and youtube.py files are translated in the po file, but they do not show up translated.
Comment 1 Gabor Kelemen 2009-03-10 02:19:36 UTC
Created attachment 130361 [details] [review]
Simple patch 

I don't say I completely understand why this is necessary or why does it work, but this solved the problem for me :)
Comment 2 Philip Withnall 2009-03-10 07:14:52 UTC
We should probably request a code freeze break to get this in for 2.26.
Comment 4 Bastien Nocera 2009-03-10 14:35:40 UTC
I've fixed this differently (by setting the gettext domain in the totem python module instead).

2009-03-10  Bastien Nocera  <hadess@hadess.net>

        * src/plugins/totem/__init__.py: Import gettext and set the 
        translation domain when importing the module

        * src/plugins/coherence_upnp/coherence_upnp.py:
        * src/plugins/jamendo/jamendo.py:
        * src/plugins/opensubtitles/opensubtitles.py:
        * src/plugins/youtube/youtube.py: Use the gettext set in the 
        totem Python module

        Spotted by Gabor Kelemen <kelemeng@gnome.hu> (Closes: #574700)

Note that the Coherence plugin is still only half translated because the widgets aren't translated, see:
http://coherence.beebits.net/ticket/195#preview