Number of answers in tooltip on French Discourse incorrect

dev translations
#1

Hi, there is a bug in the French Discourse translation, the number of answers to a topic is incorrectly shown as 1 in the tooltip even for topics that have no answers:

image
image1575×393 67.6 KB

This bug does not exists in the English Discourse.

4 Likes
Stuff randomly deleted from translation
#2

Translations can be corrected at Contribute a translation to Discourse

4 Likes
#3

Well once again, it’s weird :weary:
If I’m not mistaken it’s js.posts_likes_MF
which is

This topic has {count, plural, one {1 reply} other {# replies}} {ratio, select,
low {with a high like to post ratio}
med {with a very high like to post ratio}
high {with an extremely high like to post ratio}
other {}}

where zero is plural (?) and which is translated in

Ce sujet a {count, plural, one {1 réponse} other {# réponses}} {ratio, select,
low {avec un ratio élevé de J'aime par message}
med {avec un ratio trÚs élevé de J'aime par message}
high {avec un ratio extrĂȘmement Ă©levĂ© de J'aime par message}
other {}}

where for some reason zero is singular :thinking:
It’s not the accents, I tried removing them (and the ratio part)

5 Likes
#4

I am not a specialist but I found this translation too, it looks like it is the source of the problem but it’s the same as the English version, so, as you said, mystery
 for the time being.

1 Like
#5

Oh, I see the problem. This is the same problem I described in Always use %{count} variable when translating pluralized strings.

It should be one {# rĂ©ponse} instead of one {1 rĂ©ponse}. So, replace the “1” inside the one case with “#” to make it work. The admin_js.admin.user.delete_all_posts_confirm_MF string is affected too.

You can fix the French translations at translate.discourse.org.

We should probably update the English strings to always use the # symbol in order to make it easier for translators.

6 Likes
#6

Thanks so much ! It worked !

1 Like
#7

I think js.flagging.delete_confirm_MF also is (but js.topic.read_more_MF doesn’t seem to be).

3 Likes
#8

Thanks for noticing. I updated our linter and two translations.

4 Likes
#9