Development #47906
traduction jours fériés
0%
Description
Je pensais qu'on avait arrangé les choses pour soit ne pas avoir de libellés, soit les traduire nous-mêmes, mais je vois les noms en anglais.
Fichiers
Demandes liées
Révisions associées
agendas: do not translate label when creating exception (#47906)
Historique
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-manifest-add-workalendar-translation-files-47906.patch 0001-manifest-add-workalendar-translation-files-47906.patch ajouté
- Tracker changé de Bug à Development
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Ça je suppose.
Et ensuite on est sauvé par le fait que la traduction est tentée deux fois :- À la création :
TimePeriodException.objects.create( ... label=_(label), external=True, )
Raté pour ce cas là donc. - Et au rendu :
if self.label: label = _(self.label) if self.external else self.label
Pas intentionnel et sûrement lié à l'incertitude initiale concernant les traductions. Pourquoi pas en profiter pour enlever le premier ? (ou osef ?)
Mis à jour par Thomas Noël il y a plus de 3 ans
Valentin Deniaud a écrit :
Pas intentionnel et sûrement lié à l'incertitude initiale concernant les traductions. Pourquoi pas en profiter pour enlever le premier ? (ou osef ?)
En théorie ça ne devrait pas être fait... la traduction de _('Mardi') ne fonctionne que par chance (parce que y'a pas de "Mardi" en anglais).
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Thomas Noël a écrit :
Valentin Deniaud a écrit :
Pas intentionnel et sûrement lié à l'incertitude initiale concernant les traductions. Pourquoi pas en profiter pour enlever le premier ? (ou osef ?)
En théorie ça ne devrait pas être fait... la traduction de _('Mardi') ne fonctionne que par chance (parce que y'a pas de "Mardi" en anglais).
Tu veux dire si label n'est pas un truc workalendar mais un label normal, qui se retrouverait traduit alors qu'on ne veut pas ? On est protégé par le if self.external
normalement.
Mis à jour par Thomas Noël il y a plus de 3 ans
Je vois ça d'un côté :
label=_(label), external=True,
Donc quand external=True le label est traduit.
puis ensuite
label = _(self.label) if self.external else self.label
Donc quand external est vrai on traduit encore...
Pour moi c'est pas bon. Non ?
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Fichier 0001-manifest-add-workalendar-translation-files-47906.patch 0001-manifest-add-workalendar-translation-files-47906.patch ajouté
- Fichier 0002-agendas-do-not-translate-label-when-creating-excepti.patch 0002-agendas-do-not-translate-label-when-creating-excepti.patch ajouté
Ah oui tu parlais de traduire la chaîne déjà traduite et de tomber sur un truc différent, c'est méta. Mais on est d'accord et on en revient à ce que je proposais, virer le premier, de toute façon c'est pas propre de mettre des choses traduites en base.
Mis à jour par Frédéric Péters il y a plus de 3 ans
En external il n'y pas les ICS aussi ?, qu'on ne voudrait pas passer dans gettext.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
Frédéric Péters a écrit :
En external il n'y pas les ICS aussi ?, qu'on ne voudrait pas passer dans gettext.
Nop, ça a été introduit par et pour ces sources définies dans les settings, oui le nom est pas fou. (sinon on aurait pu juste utiliser l'attribut source, pas besoin d'un flag en plus)
Mis à jour par Thomas Noël il y a plus de 3 ans
- Statut changé de Solution proposée à Solution validée
Je valide ça, mais oui, pour clarifier pour plus tard, peut-être faire un ticket qui proposerait de renommer ce flag external en "from_settings", ou d'utiliser source.
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit fba6eff780f8090d80c2260080a54a46810b2201 Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Thu Oct 22 13:59:37 2020 +0200 agendas: do not translate label when creating exception (#47906) commit 75a04a468ac10855cfc73ed20db890f54fdc5b7b Author: Valentin Deniaud <vdeniaud@entrouvert.com> Date: Wed Oct 21 10:31:31 2020 +0200 manifest: add workalendar translation files (#47906)
Mis à jour par Frédéric Péters il y a plus de 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Valentin Deniaud il y a plus de 3 ans
- Lié à Development #48028: traduction jours fériés, toujours pas bon ajouté
manifest: add workalendar translation files (#47906)