Making WordPress.org

Opened 8 years ago

Closed 7 years ago

Last modified 7 years ago

#2694 closed defect (bug) (fixed)

Plugin Directory: Use _n() for "translated into these locales" string

Reported by: sergeybiryukov's profile SergeyBiryukov Owned by: ocean90's profile ocean90
Milestone: Plugin Directory v3.0 Priority: normal
Component: Plugin Directory Keywords: has-patch commit
Cc:

Description

'%1$s has been translated into these %2$d locales: %3$s.' needs _n() to be translated properly.

Additionally, the 1 === count( $locales ) case should be checked separately, see the note in Codex.

Attachments (1)

2694.patch (1.7 KB) - added by SergeyBiryukov 8 years ago.

Download all attachments as: .zip

Change History (4)

@SergeyBiryukov
8 years ago

#1 @ocean90
7 years ago

  • Owner set to ocean90
  • Resolution set to fixed
  • Status changed from new to closed

In 5372:

Plugin Directory: Use _n() for "translated into these locales" string.

Props SergeyBiryukov.
Fixes #2694.

#2 follow-up: @Presskopp
7 years ago

Am I right that the first line (singular) is just a 'placeholder' and never used? Translating it, it confuses me, because these %2$d locales is not singular. Can this be clearified more?

'%1$s has been translated into these %2$d locales: %3$s.',
'%1$s has been translated into these %2$d locales: %3$s.',
Version 1, edited 7 years ago by Presskopp (previous) (next) (diff)

#3 in reply to: ↑ 2 @SergeyBiryukov
7 years ago

Replying to Presskopp:

Am I right that the first line (singular) is just a 'placeholder' and never used?

It would be used in languages where the singular form is used not just for 1 item, but also for 21, 31 items, etc. (Russian and some others).

In English or German it won't be used, just translate it to whatever makes the most sense :)

Note: See TracTickets for help on using tickets.