Opened 7 years ago

Closed 7 years ago

Last modified 3 years ago

#8731 closed (fixed)

"Days" translation in french could be better

Reported by: Sephi Owned by: nobody
Component: Translations Version: master
Severity: Keywords: fr, french, days, journées, jours, i18n, locale
Cc: Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

The "days" translation in french is "journées", so when using the timesince tag it gives something like "1 mois, 2 journées", instead of "1 mois, 2 jours".

The attached patch simply replaces the translation "day" => "journée" into "day" => "jour" (same thing for the plural form) for the fr locale.

Attachments (3)

django.po.diff (457 bytes) - added by Sephi 7 years ago.
8759.typo.diff (1.1 KB) - added by julien 7 years ago.
Same patch as Sephi's, but applies to r8783
8731.typo.diff (488 bytes) - added by julien 7 years ago.
Same patch as Sephi's, but applies to r8783 >>> Consider this one, not "8759.typo.diff" <<<

Download all attachments as: .zip

Change History (10)

Changed 7 years ago by Sephi

comment:1 Changed 7 years ago by julien

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to invalid
  • Status changed from new to closed

The current translation is perfectly valid and "1 mois, 2 jours" is what should be displayed.

"journée" only refers to daylight time (say roughly from 6am to 6pm), while "jour", in this context, refers to the full 24 hours.
At least, I'm speaking for "mainstream" French. If you think this is incorrect in another dialect (e.g. Quebecois) please post your patch to the respective locale.

comment:2 Changed 7 years ago by Sephi

  • Resolution invalid deleted
  • Status changed from closed to reopened

Yes, as you say "1 mois, 2 jours" is what should be displayed, but currently it is "1 mois, 2 journées" which is displayed when using timesince (look at the django.po file, the "days" key, it is translated to "journées").
So my patch was correcting this translation by translating "days" into "jours" instead of "journées", which, I think, is "mainstream" french.

comment:3 Changed 7 years ago by julien

  • Triage Stage changed from Unreviewed to Ready for checkin

Whoops, my bad, I had misread your original description. I confirm that your patch is right. Thanks!

Changed 7 years ago by julien

Same patch as Sephi's, but applies to r8783

comment:4 Changed 7 years ago by julien

  • milestone set to 1.0

Argh... I first posted the wrong patch ("8759.typo.diff"), please disregard that one and consider "8731.typo.diff" instead. My brain is definitely failing these days.

Sephi, I've reposted your patch because it didn't apply properly. So I posted the same as yours, but which applies nicely to r8783.

Also bumping to 1.0, if there's a bit of time left before release to check it in. It's a trivial change but that fixes a quite silly and annoying bug in the French locale. Thanks again Sephi for the report and fix.

Changed 7 years ago by julien

Same patch as Sephi's, but applies to r8783 >>> Consider this one, not "8759.typo.diff" <<<

comment:5 Changed 7 years ago by telenieko

  • Resolution set to fixed
  • Status changed from reopened to closed

(In [8798]) Closes #8731, Typo in "fr" locale. Thanks Sephi & Julien.

comment:6 Changed 7 years ago by telenieko

Commited. Thanks for the patch!

comment:7 Changed 3 years ago by jacob

  • milestone 1.0 deleted

Milestone 1.0 deleted

Note: See TracTickets for help on using tickets.
Back to Top