WordPress.org
Get WordPress

Make WordPress Core

Opened 18 years ago

Closed 18 years ago

#7807 closed defect (bug) (fixed)

String need plural form

Reported by: xibe Owned by:
Priority: normal Milestone: 2.7
Component: I18N Version: 2.7
Severity: trivial Keywords: has-patch commit
Cc: Focuses:

Description

Strings from the plugin installer (wp-admin/includes/plugin-install.php) need ability to pluzalize:

Attachments (2)

7807.diff​ (1.4 KB ) - added by DD32 18 years ago.
num-ratings-plural.diff​ (2.3 KB ) - added by nbachiyski 18 years ago.

Download all attachments as: .zip

Change History (11)

#1 @xibe
18 years ago

And by "pluzalize", I mean "pluralize", which in the end is rather a way of saying it needs to handle single-rating note gracefully :)

#2 @DD32
18 years ago

  • Keywords needs-patch added
  • Owner changed from anonymous to DD32
  • Status new → assigned
  • Version → 2.7

Just taking owner of the ticket so i remember about it when time comes to submit the next plugin-install related patch

@DD32
18 years ago

#3 @DD32
18 years ago

  • Keywords has-patch commit added; needs-patch removed

#4 @DD32
18 years ago

(Can someone check i did that right? :))

#5 @ryan
18 years ago

You don't need __() inside of ngettext. Also, the number substituted into the strings should be run through i18n_number_format() and subbed with %s.

#6 @DD32
18 years ago

  • Owner changed from DD32 to anonymous
  • Status assigned → new

@nbachiyski
18 years ago

#7 @DD32
18 years ago

nbachiyski: Theres also one on line ~335 which it appears i missed in the patch i added.

#8 @ryan
18 years ago

(In [9222]) i18n fixes for plugin intaller from DD32 and nbachiyski. see #7807

#9 @nbachiyski
18 years ago

  • Resolution → fixed
  • Status new → closed

I got it in the second patch.

Note: See TracTickets for help on using tickets.