There is a little mistake on the format_plural() page. At the example with additional replacements is a unnecessary bracket.

array('%old-type' => $info->old_type, '%new-type' => $info->new_type)));

fixed:

array('%old-type' => $info->old_type, '%new-type' => $info->new_type));

http://api.drupal.org/api/drupal/includes%21common.inc/function/format_p...

The failure is also on the drupal 6 http://api.drupal.org/api/drupal/includes%21common.inc/function/format_p... and drupal 8
http://api.drupal.org/api/drupal/core%21includes%21common.inc/function/f... site

CommentFileSizeAuthor
#3 format_plural-1783348-3.patch615 bytesjfhovinne

Comments

Anonymous’s picture

Anonymous (not verified) commented
Version: 7.15 » 8.x-dev

So, this needs fixed in the 8.x-dev version and backported once resolved.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Issue tags: +Novice, +Needs backport to D7

Thanks! Just needs a patch...

jfhovinne’s picture

jfhovinne commented
Status: Active » Needs review
StatusFileSize
new format_plural-1783348-3.patch615 bytes

Here it is.

lucascaro’s picture

lucascaro commented

Seems legit, RTBC?

javi-er’s picture

javi-er commented
Status: Needs review » Reviewed & tested by the community

The patch seems fine.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Thanks! Committed to 8.x and 7.x.

Automatically closed -- issue fixed for 2 weeks with no activity.