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 665421 - Port number explanations do not appear translated in the account editor
Port number explanations do not appear translated in the account editor
Status: RESOLVED FIXED
Product: evolution-data-server
Classification: Platform
Component: general
3.2.x (obsolete)
Other Linux
: Normal normal
: ---
Assigned To: Evolution Shell Maintainers Team
Evolution QA team
Depends on:
Blocks:
 
 
Reported: 2011-12-02 18:29 UTC by Gabor Kelemen
Modified: 2011-12-02 20:32 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Screenshot of the problem (9.32 KB, image/png)
2011-12-02 18:29 UTC, Gabor Kelemen
  Details
Localize port numbers with the rest of the provider strings (1.10 KB, patch)
2011-12-02 18:33 UTC, Gabor Kelemen
committed Details | Review

Description Gabor Kelemen 2011-12-02 18:29:39 UTC
Created attachment 202642 [details]
Screenshot of the problem

On the Receiving Email and Sending Email pages of the account editor, you can set the port number of the servers. When you click the arrow next to the Port box, you can see a short explanation of the default ports, like "IMAP default port" "IMAP over SSL" etc, which do not appear translated, despite they are translated in the po file.

This is because the strings are parts of static arrays, and gettext() is never called on them when they are actually used.
Comment 1 Gabor Kelemen 2011-12-02 18:33:46 UTC
Created attachment 202643 [details] [review]
Localize port numbers with the rest of the provider strings

This patch worked for me in jhbuild.
Comment 2 Matthew Barnes 2011-12-02 18:48:32 UTC
What does P_() do differently from _() ?  Just curious.  Hard to Google these things.  :)
Comment 3 Gabor Kelemen 2011-12-02 19:22:54 UTC
P_() is a macro around dgettext(), defined a in the same function:

http://git.gnome.org/browse/evolution-data-server/tree/camel/camel-provider.c#n246
Comment 4 Matthew Barnes 2011-12-02 19:55:24 UTC
Oh hey look at that.  No wonder Google came up empty.

Patch approved for gnome-3-2 and master branches.  Can't commit it myself right at the moment, but if you have commit rights then go ahead.
Comment 5 Gabor Kelemen 2011-12-02 20:31:22 UTC
Review of attachment 202643 [details] [review]:

Pushed, thanks for the quick reply!
Comment 6 Gabor Kelemen 2011-12-02 20:32:21 UTC
Review of attachment 202643 [details] [review]:

Set status for real :)