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 556767 - Plural form is required for maximum password length
Plural form is required for maximum password length
Status: RESOLVED FIXED
Product: vino
Classification: Applications
Component: Preferences Dialog
2.24.x
Other All
: Normal minor
: ---
Assigned To: Vino Maintainer(s)
Vino Maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2008-10-17 18:37 UTC by Marcel Telka
Modified: 2009-02-13 07:27 UTC
See Also:
GNOME target: 2.26.x
GNOME version: ---


Attachments
tiny adjust! (1.07 KB, patch)
2008-10-21 18:52 UTC, Jorge Pereira
rejected Details | Review
Use ngettext (848 bytes, patch)
2009-02-12 21:12 UTC, Claude Paroz
committed Details | Review

Description Marcel Telka 2008-10-17 18:37:19 UTC
This string:

#: ../tools/vino-passwd.c:115
#, c-format
msgid ""
"ERROR: Maximum length of password is %d characters. Please, re-enter the "
"password.\n"

should use plural forms. It is not possible to correctly translate this string into languages where more than one plural form is used (for example Slovak language).

Thanks.

Other information:
Here is description about plural forms for developers:

http://live.gnome.org/TranslationProject/DevGuidelines/Plurals
Comment 1 Jorge Pereira 2008-10-21 18:52:17 UTC
Created attachment 121049 [details] [review]
tiny adjust!

tiny patch, solve this!
Comment 2 Jonh Wendell 2008-10-21 19:12:18 UTC
Jorge, your patch is an example of what do NOT do :)

Anyway, the patch is quite trivial and I will do it for 2.26. I don't want to bother i18n guys with a request for string freeze.
Comment 3 André Klapper 2008-10-21 19:27:03 UTC
Jorge: Please read http://live.gnome.org/TranslationProject/DevGuidelines/Plurals to understand why the patch won't fix the issue at all. :)
Comment 4 Claude Paroz 2009-02-12 21:12:24 UTC
Created attachment 128588 [details] [review]
Use ngettext
Comment 5 Jonh Wendell 2009-02-12 23:43:09 UTC
Thanks Claude. Please send the string change announcement email.