wp_tag_cloud(): title='1 topics', should be 1 topic

[atppp]

I am picky...

[webrocker]

been there done that :)

[Nazgul]

This patch kills right-to-left language support.

[foolswisdom]

Please don't set the milestone, leave that for a core contributor.

[westi]

Replying to Nazgul:

This patch kills right-to-left language support.

Indeed, patch should be reworked to use ​__ngettext rather than __

[webrocker]

Replying to Nazgul:

This patch kills right-to-left language support.

Ooops. Is this because of the way how the digit is 'handwired' in front of the 'topics' label (like '1 topic'), whereas a right-to-left needs something like 'topic 1'? In the original it is sprintf( __('%d topics'), $count ), so the digit is in front of the label anyway? Sorry, I don't get the difference between a hardcoded 'topics' after the %d and the variable used in my patch sprintf( __('%d '.$topictxt), $count ).

Replying to foolswisdom:

Please don't set the milestone, leave that for a core contributor.

Ok, sorry for that.

Replying to westi:

Indeed, patch should be reworked to use ​__ngettext rather than __

well the original used __ , so I didn't messed with that.

[webrocker]

Replying to westi:

Indeed, patch should be reworked to use ​__ngettext rather than __

I see... would this work
original:
attribute_escape( sprintf( __('%d topics'), $count ) )
patch:
attribute_escape( __ngettext( 'topic', 'topics', $count )
?

[foolswisdom]

Replying to webrocker:

Replying to foolswisdom:

Please don't set the milestone, leave that for a core contributor.

Ok, sorry for that.

No serious problem of course! ;-) Keep up the great work!

[westi]

Replying to webrocker:

Replying to westi:

Indeed, patch should be reworked to use ​__ngettext rather than __

I see... would this work
original:
attribute_escape( sprintf( __('%d topics'), $count ) )
patch:
attribute_escape( __ngettext( 'topic', 'topics', $count )
?

Nearly __ngettext allows you to use plural forms in your translations e.g.:

__ngettext( '%d topic', '%d topics', $count)

This will output '1 topic' or '3 topics' giving you the correct plural form while still allowing translators to handle the "right to left" issues by translating the '%d topic' as something like 'topic %d'.

[JeremyVisser]

Not a good idea to go in 2.3.1 – string changes are bad for translators.

[webrocker]

Replying to westi:

__ngettext( '%d topic', '%d topics', $count)

This will output '1 topic' or '3 topics' giving you the correct plural form while still allowing translators to handle the "right to left" issues by translating the '%d topic' as something like 'topic %d'.

Ok, I think I'm getting it... so it still needs to be used inside the sprintf() function, so that the %d thingie gets replaced with $count, while the __ngettext() needs another instance of $count to determine if $single or $plural is to be used?
So this should work:
attribute_escape( sprintf( __ngettext( '%d topic', '%d topics', $count ), $count ) )

[webrocker]

I have tested the above in my 2.3 install, and it works, so I attached another patch-file.

[westi]

Relates to #5172 and #5171 all tag cloud changes.

I will get these all in soon.

[westi]

(In [6227]) Translation fix for wp_tag_cloud(). Fixes #5131 props webrocker