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).
Files
Related issues
Associated revisions
History
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
- File 0001-api-encode-files-that-may-exist-in-workflow_data-107.patch 0001-api-encode-files-that-may-exist-in-workflow_data-107.patch added
- Status changed from Nouveau to En cours
- Patch proposed changed from No to Yes
Updated by Thomas Noël over 6 years ago
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.
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
S'il y a pas full=on, on pourrait zapper tout workflow_data, non ?
Updated by Benjamin Dauvergne over 6 years ago
Oui, ou filtrer les PickableUpload qui sont dans workflow_data.
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
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.
Updated by Thomas Noël over 6 years ago
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.
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
- Status changed from En cours to 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)
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
- Related to Bug #11000: la classe PicklableUpload peut exister sous deux namespaces différents added
Updated by Frédéric Péters (de retour le 10/10) over 6 years ago
- Status changed from Résolu (à déployer) to Fermé
api: encode files that may exist in workflow_data (#10769)