Dashboard "Right Now" counts don't correctly use plurals

[zerocrates]

[2802] changed the way the counts were created to call number_format on each count first, then pass the formatted number to __ngettext. This causes __ngettext to incorrectly pick plurals when the format includes punctuation like a comma.

Example: the dashboard says I have "1,416 topic", but "6,069 posts." Since __ngettext's third argument is an integer, the comma inserted by number_format is screwing up the plural selection.

Instead of, for example:

$rn_topics = number_format( get_total_topics() );
$rn_topics = sprintf(__ngettext('<span>%s</span> topic', '<span>%s</span> topics', $rn_topics), $rn_topics ); 

it needs to be:

$rn_topics = get_total_topics();
$rn_topics = sprintf(__ngettext('<span>%s</span> topic', '<span>%s</span> topics', $rn_topics), number_format($rn_topics) ); 

[jclermont]

Are items with 1.x milestones still open for work? Would it be on the 2.0 branch or on the older 1.x branch?

I'd be willing to jump in and take this one once you clear that up for me. Thanks!

[johnjamesjacoby]

Tickets in the 1.x milestones are specific to that older code. You're welcome to fix them if they are outstanding, and you'd want to work from the /branches/1.1/ folder if you do. 1.1 is the final release of the stand-alone version, and all efforts are going into 2.0 and beyond.

[johnjamesjacoby]

Which is also to say, /trunk is about to get emptied out, and replaced by /branches/plugin soon.