Bug #17233
variable "foo_var_file_url" vide depuis un formulaire de workflow "foo" avec un champ fichier "file"
Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
-
Début:
28 juin 2017
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Description
workflow joint :
- statut 1 : affiche un formulaire "foo" avec un champ "file" ; puis saut à la soumission vers status 2
- statut 2 : enregistre dans le journal un message utilisant [foo_var_file_url] : ça plante, parce que cette variable existe, mais est vide
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 7 ans
- Fichier 0001-fix-workflow_data-upload_url-substitution-variables-.patch 0001-fix-workflow_data-upload_url-substitution-variables-.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Le bogue :
workflow_data est un dico avec pour un champ upload de fichier : {"fichier": "toto.pdf", "fichier_raw": Upload(...), "fichier_url": None}
Lors de la génération des variables de substitution, on parcourt ce dico. Quand on trouve un "Upload" alors on calcule son «_url».
Mais c'est un parcours de dico, et parfois on va lire le _url avant (dans ce cas, tout va bien, il va être écrasé par le calcul précédent), parfois non et paffff, la valeur est remise à None dans le dico résultat.
Il faut donc écraser les _url seulement après le parcours.
Je prends un meilleur message de commit.
Mis à jour par Thomas Noël il y a presque 7 ans
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-tests-add-test-for-file-_url-computation-in-substitu.patch 0001-tests-add-test-for-file-_url-computation-in-substitu.patch ajouté
Ack, avec un patch test pour accompagner.
Mis à jour par Thomas Noël il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
Thanks a lot, test mergé dans le patch poussé :
commit 21e0603aa791b7a72a1fab6cf2aa71e1eaee7be4
Author: Thomas NOEL <tnoel@entrouvert.com>
Date: Wed Jun 28 16:23:31 2017 +0200
fix compute _url variable of workflow data uploads (#17233)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
fix compute _url variable of workflow data uploads (#17233)