Development #20211
permettre une expression dans le dispatch automatique
0%
Description
Il y a un champ "variable" et c'est sur celle qui sera renseignée que se fera le dispatch; mais comme à mille autres endroits on autorise des expressions on peut se trouver à penser que là aussi. Alors qu'en fait, non.
Pour compatibilité avec l'existant, il faudrait malgré tout commencer par une première vérification voir si c'est une variable (en regardant dans les variables dispo si le nom matche) avant de considérer ça comme une expression.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 5 ans
- Fichier 0001-workflows-add-support-for-templates-in-auto-dispatch.patch 0001-workflows-add-support-for-templates-in-auto-dispatch.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 5 ans
Je n'arrive plus à percevoir l'intérêt de la partie ci-dessous qui «convert the given value to a few different types, to allow more diversity», et que l'on gère pour le mode "variable", mais est-ce qu'on ne voudrait pas l'avoir aussi après le compute quand on est en mode "expression" ?
if not variable_values[0]:
variable_values.append(None)
try:
variable_values.append(int(variable_values[0]))
except (ValueError, TypeError):
pass
Aussi, lors de l'appel à compute, ajouter du contexte pour de meilleurs LoggedError :
variable_values = [self.compute(self.variable, formdata=formdata, status_item=self)]Mis à jour par Frédéric Péters il y a presque 5 ans
voudrait pas l'avoir aussi après le compute quand on est en mode "expression"
Je dirais que non que c'est du truc legacy pour personne ne sait plus où (je me suis aussi posé la question mais je n'ai pas cherché).
Mis à jour par Thomas Noël il y a presque 5 ans
- Statut changé de Solution proposée à Solution validée
Frédéric Péters a écrit :
voudrait pas l'avoir aussi après le compute quand on est en mode "expression"
Je dirais que non que c'est du truc legacy pour personne ne sait plus où (je me suis aussi posé la question mais je n'ai pas cherché).
Plus personne ne sait effectivement, laissons tomber. Concernant formdata et status_item lors du compute ? (tu ajoutes ou pas, je valide de toute façon)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Yep j'ai ajouté,
commit a2fc38261cce7619089cbb79c24c5b2082a42fbe
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Fri Jul 5 13:58:01 2019 +0200
admin: format file types on a single line (#34561)
Mis à jour par Frédéric Péters il y a presque 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
workflows: add support for templates in auto dispatch (#20211)