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 658730 - ngettext (Plural forms) needed for "Imported %u buddies from '%s'."
ngettext (Plural forms) needed for "Imported %u buddies from '%s'."
Status: RESOLVED FIXED
Product: folks
Classification: Platform
Component: general
git master
Other All
: Normal normal
: folks-0.6.3
Assigned To: folks-maint
folks-maint
Depends on:
Blocks:
 
 
Reported: 2011-09-11 07:02 UTC by Marek Černocký
Modified: 2011-09-13 16:56 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
Use ngettext() instead of gettext() for a plural string (1.48 KB, patch)
2011-09-12 21:01 UTC, Philip Withnall 		committed Details | Review

Description Marek Černocký 2011-09-11 07:02:31 UTC 
Message "Imported %u buddies from '%s'." (tools/import-pidgin.vala) requires ngettext due plural forms.
http://live.gnome.org/TranslationProject/DevGuidelines/Plurals
Comment 1 Philip Withnall 2011-09-12 21:01:12 UTC 
Created attachment 196305 [details] [review]
Use ngettext() instead of gettext() for a plural string

https://www.gitorious.org/folks/folks/commits/658730-gettext-plurals

Requesting a string freeze break momentarily.
Comment 2 Philip Withnall 2011-09-12 22:10:05 UTC 
I've done that thing again where I forget to update NEWS when posting a patch.
Comment 3 Philip Withnall 2011-09-13 16:56:02 UTC 
Comment on attachment 196305 [details] [review]
Use ngettext() instead of gettext() for a plural string

Committed with the change to NEWS. I forgot we don't need to request string freeze breaks, since we're an external module.

commit f5358a7a0920103f2943ea02298842d123c0f743
Author: Philip Withnall <philip@tecnocode.co.uk>
Date:   Mon Sep 12 21:55:04 2011 +0100

    Bug 658730 — ngettext (Plural forms) needed…
    
    Use ngettext() instead of gettext() for a plural string.
    
    Closes: bgo#658730

 NEWS                     |    2 ++
 tools/import-pidgin.vala |   10 ++++++----
 2 files changed, 8 insertions(+), 4 deletions(-)