Bug #10769
Lorsque workflow_data contient des PickableUpload (formulaire backoffice) l'export JSON plante
0%
Description
Solution facile, traduire les pickable upload en un dico comme pour les champs fichiers, solution compliquée parcourir tous les WorkflowStatusItem qui doivent eux même traduire le contenu de workflow_data en JSON à leur convenance, on ne reprend plus workflow_data tel quel (de fait ça ne marchera pas pour les données qui ne viennent pas d'un WorkflowStatusItem ou ne sont pas nommées/structurées comme celles venant des triggers).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 8 ans
- Fichier 0001-api-encode-files-that-may-exist-in-workflow_data-107.patch 0001-api-encode-files-that-may-exist-in-workflow_data-107.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 8 ans
Le « 'content': base64.b64encode(obj.get_content()), » m'ennuie un peu parce que dans l'api, si y'a pas "full=on", on n'envoie pas les contenus des champs fichiers de la demande. Et donc on devrait faire pareil pour les fichiers dans le workflow_data.
Mis à jour par Frédéric Péters il y a presque 8 ans
S'il y a pas full=on, on pourrait zapper tout workflow_data, non ?
Mis à jour par Benjamin Dauvergne il y a presque 8 ans
Oui, ou filtrer les PickableUpload qui sont dans workflow_data.
Mis à jour par Frédéric Péters il y a presque 8 ans
C'est un peu tordu naze cette API "?full=on", ça ne veut pas dire "et bien inclus aussi les contenus de fichier" mais bien "là je te demandais un listing mets moi aussi les données des formulaires dedans mais pas le contenu de fichiers qu'ils pourraient contenir".
J'ignorerais la question.
Mis à jour par Thomas Noël il y a presque 8 ans
Frédéric Péters a écrit :
J'ignorerais la question.
Ok pour moi, si la question revient avec des arguments, on verra (le full=on c'est uniquement pour les dump de listing complets, pas vraiment pertinent, sans doute à revoir un jour, oui).
Donc pour moi, le patch est ok ainsi.
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de En cours à Résolu (à déployer)
commit aa617eaf3d94ed30fcb8b9fa0ee6e41385212b23
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Wed May 4 14:48:49 2016 +0200
api: encode files that may exist in workflow_data (#10769)
Mis à jour par Frédéric Péters il y a presque 8 ans
- Lié à Bug #11000: la classe PicklableUpload peut exister sous deux namespaces différents ajouté
Mis à jour par Frédéric Péters il y a presque 8 ans
- Statut changé de Résolu (à déployer) à Fermé
api: encode files that may exist in workflow_data (#10769)