Development #54930
permettre d'écrire une expiration sous forme de gabarit "temps humain"
0%
Description
ex: "{{form_var_whatever}} jours", là où aujourd'hui ça attend comme résultat un nombre de secondes.
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 2 ans
Lors du « humanduration2seconds(timeout_str) » si le timeout_str est vide, une exception ValueError va être levée ; je pense qu'il faut donc un try/except autour (au cas où le gabarit répond du vide).
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch ajouté
En effet je n'imaginais pas ce cas, on pourrait se dire que si une chaine vide est produite c'est qu'on ne veut pas de timeout, comme si l'expiration n'avait pas été définie, mais je ne prends pas cette hypothèse, si ça produit une chaine vide c'est que ça a merdé et on le note.
Mis à jour par Thomas Noël il y a plus de 2 ans
Ah zut, raté en première lecture :
Error in timeout value (%r)') % timeout_str
on pourrait enregistrer le timeout original (il sera dans la traceback, mais ça sera plus agréable de le voir directement dans le titre de l'erreur):
_('Error in timeout value %r (computed from %r)') % (timeout_str, self.timeout),
Mis à jour par Frédéric Péters il y a plus de 2 ans
Je l'avais initialement mis puis j'ai trouvé que c'était redondant mais ok je vais le remettre.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch 0001-workflows-allow-generated-timeout-to-be-in-human-tim.patch ajouté
Et pour le coup j'ajoute aussi le test sur la situation où un gabarit donne une chaine vide.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Impec.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 5b0a9433661e5710f3d60894f2356aeebd1c1879
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Nov 1 20:56:03 2021 +0100
workflows: allow generated timeout to be in "human time" (#54930)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Frédéric Péters il y a 10 mois
- Lié à Bug #36882: gabarit dans le champ expiration d'un saut automatique ajouté
workflows: allow generated timeout to be in "human time" (#54930)