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 628601 - Untranslated strings in IM account setting dialog
Untranslated strings in IM account setting dialog
Status: RESOLVED FIXED
Product: empathy
Classification: Core
Component: Accounts
2.30.x
Other Linux
: Normal normal
: ---
Assigned To: empathy-maint
Depends on:
Blocks:
 
 
Reported: 2010-09-02 10:09 UTC by Gary Ching-Pang Lin
Modified: 2010-09-02 13:37 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
screenshot of the dialog with untranslated strings (29.44 KB, image/png)
2010-09-02 10:09 UTC, Gary Ching-Pang Lin
  Details
proposed patch (1.46 KB, patch)
2010-09-02 10:11 UTC, Gary Ching-Pang Lin
rejected Details | Review
After applying the patch (29.66 KB, image/png)
2010-09-02 10:11 UTC, Gary Ching-Pang Lin
  Details
http://git.Collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/translate-param-628601 (843 bytes, patch)
2010-09-02 11:08 UTC, Guillaume Desmottes
none Details | Review
http://git.Collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/translate-param-628601 (1.12 KB, patch)
2010-09-02 11:25 UTC, Guillaume Desmottes
accepted-commit_after_freeze Details | Review

Description Gary Ching-Pang Lin 2010-09-02 10:09:29 UTC
Created attachment 169338 [details]
screenshot of the dialog with untranslated strings

While I added an new account to empathy, I noticed there are some untranslated strings in the dialog.

1. Launch the account setting dialog by Edit->Accounts-> Add...
2. Choose different protocols and check the strings "Account:" and "Password:"

In the beginning, I think those two strings were not in the po file, but the strings actually exist in zh_TW.po and are translated. After tracing the source code, it seems g_strdup_printf() is not aware of the translations, so the strings were not replaced with the translated strings.
Comment 1 Gary Ching-Pang Lin 2010-09-02 10:11:11 UTC
Created attachment 169340 [details] [review]
proposed patch

I moved _() from g_strdup_printf() to gtk_lable_new(), and it works for me.
Comment 2 Gary Ching-Pang Lin 2010-09-02 10:11:48 UTC
Created attachment 169341 [details]
After applying the patch
Comment 3 Gary Ching-Pang Lin 2010-09-02 10:16:34 UTC
An additional remark: this issue happens on the protocols which uses the generic widget, such as zephyr, Yahoo! Japan, sipe, simple, silc, sametime, myspace, and gadugadu. The protocol with a specific widget ui file like Jabber doesn't have this issue.
Comment 4 Guillaume Desmottes 2010-09-02 11:08:25 UTC
Review of attachment 169340 [details] [review]:

This is actually the wrong way to fix that. I have a branch providing a better fix.
Comment 5 Guillaume Desmottes 2010-09-02 11:08:56 UTC
Created attachment 169350 [details] [review]
http://git.Collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/translate-param-628601

 libempathy-gtk/empathy-account-widget.c |   22 ++++++++++++++++++++++
 1 files changed, 22 insertions(+), 0 deletions(-)
Comment 6 Guillaume Desmottes 2010-09-02 11:25:52 UTC
Created attachment 169352 [details] [review]
http://git.Collabora.co.uk/?p=user/cassidy/empathy;a=shortlog;h=refs/heads/translate-param-628601

 libempathy-gtk/empathy-account-widget.c |   23 +++++++++++++++++++++++
 1 files changed, 23 insertions(+), 0 deletions(-)
Comment 7 Guillaume Desmottes 2010-09-02 11:37:34 UTC
Branch has been reviewed. I'll merge it once I got the approval of the translation team.
Comment 8 Guillaume Desmottes 2010-09-02 13:37:47 UTC
This problem has been fixed in the development version. The fix will be available in the next major software release. Thank you for your bug report.