Produits Entr'ouvert » Chrono

Valentin Deniaud a écrit :

Pas compliqué (la trad est assurée dans str).

Mais Jenkins dit que c'est pas si simple.

Ouais, comme d'hab, ici le pb c'est qu'on veut bien juste le label et pas ce qui est renvoyé par str, qui ajoute date et heure entre parenthèses.

• Patcher juste ces deux lignes à coup de {% if slot.from_settings %}{% trans slot.label %}{% else %}{{ slot.label }}{% endif %} (c'est moche, et ça sera oublié à chaque fois qu'il faudra réutiliser ce label dans les templates).
• Renommer label en raw_label et avoir une propriété label qui s'occupe renvoyer raw_label traduit (tous les objets chrono ont un attribut label, pas cool d'avoir un truc spécifique pour les exceptions).
• Changer la sémantique des méthodes pour avoir un truc plus clair (introduire label_with_date à la place de str, faire que str ne retourne que le label).

Mais bon en vrai je pense que c'était une mauvaise idée de vouloir traduire les trucs à la volée, la bonne solution c'est de mettre les chaînes traduites en base et ne plus s'en occuper par la suite, non ?

Mais bon en vrai je pense que c'était une mauvaise idée de vouloir traduire les trucs à la volée, la bonne solution c'est de mettre les chaînes traduites en base et ne plus s'en occuper par la suite, non ?

Tout à fait.

C'est rouge.