Produits Entr'ouvert » Authentic 2

Effectivement, en reformulant la phrase on arrive à nos fins. Il faudra faire gaffe à garder une tournure qui fonctionne au moment des traductions (« one year ago » devient « il y a une année »).

Je cale le filtre dans le code et pas dans le template, un peu parce que comme ça on aura pas à s'en soucier dans publik-base-theme, mais en vrai c'est surtout qu'utiliser humanize dans le template ne fonctionne pas, sans que je trouve d'explication (oui j'ai pensé à l'ajouter à INSTALLED_APPS).

Rebasé sur #40760 et avec des tests qui devraient passer.

Rerebasé, sur #26909.

Rererebasé.

Pourquoi avoir laissé « In {{ days_to_deletion }} days […]» dans le template ? Est-ce qu'on ne pourrait pas faire quelque chose du genre naturaltime(now()+timedelta(days=user.ou.clean_unused_accounts_deletion - alert_delay)) pour avoir un affichage du même genre pour cette date future ?

Paul Marillonnet a écrit :

Pourquoi avoir laissé « In {{ days_to_deletion }} days […]» dans le template ? Est-ce qu'on ne pourrait pas faire quelque chose du genre naturaltime(now()+timedelta(days=user.ou.clean_unused_accounts_deletion - alert_delay)) pour avoir un affichage du même genre pour cette date future ?

Parce que le problème du ticket c'est d'afficher un nombre de jours inintelligible. Or le délai entre alerte et suppression est de maximum un mois, et afficher « 30 jours », c'est intelligible (si le délai est plus élevé, le problème est ailleurs).
La deuxième raison est technique, naturaltime, pour un nombre de jours élevé du genre autour de 365 ou 720, va renvoyer « 1 année » ou 2, c'est à dire arrondir correctement et ne pas faire quelque chose de précis et moche. En revanche si days_to_deletion vaut 14, naturaltime dit « une semaine et 6 jours », c'est moche et moins bien que « 14 jours ».

Autre remarque, sur la forme, naturaltime c'est un filtre de gabarit, ça me paraît plus logique de faire, dans le gabarit

Comme noté plus haut, j'avais essayé sans réussir, mais je m'y suis repenché et j'ai trouvé le problème, on ne peut pas utiliser de filtres à l'intérieur de {% blocktrans %}, il faut passer par with. C'est donc corrigé.

Valentin Deniaud a écrit :

Parce que le problème du ticket c'est d'afficher un nombre de jours inintelligible. Or le délai entre alerte et suppression est de maximum un mois, et afficher « 30 jours », c'est intelligible (si le délai est plus élevé, le problème est ailleurs).
La deuxième raison est technique, naturaltime, pour un nombre de jours élevé du genre autour de 365 ou 720, va renvoyer « 1 année » ou 2, c'est à dire arrondir correctement et ne pas faire quelque chose de précis et moche. En revanche si days_to_deletion vaut 14, naturaltime dit « une semaine et 6 jours », c'est moche et moins bien que « 14 jours ».

Oui t'as complètement raison.

Comme noté plus haut, j'avais essayé sans réussir, mais je m'y suis repenché et j'ai trouvé le problème, on ne peut pas utiliser de filtres à l'intérieur de {% blocktrans %}, il faut passer par with. C'est donc corrigé.

Yep nickel, ack.

commit edab053f92495b0eda214697a4a6b3d9541e96b8
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Mar 12 12:22:57 2020 +0100

    commands: human duration in unused account email (#40521)