Project

General

Profile

Bug #10769

Lorsque workflow_data contient des PickableUpload (formulaire backoffice) l'export JSON plante

Added by Benjamin Dauvergne almost 5 years ago. Updated over 4 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
29 Apr 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

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

Related to w.c.s. - Bug #11000: la classe PicklableUpload peut exister sous deux namespaces différentsFermé20 May 2016

Actions

Associated revisions

Revision aa617eaf (diff)
Added by Frédéric Péters almost 5 years ago

api: encode files that may exist in workflow_data (#10769)

History

#2

Updated by Benjamin Dauvergne almost 5 years ago

  • Description updated (diff)
#3

Updated by Frédéric Péters almost 5 years ago

#4

Updated by Benjamin Dauvergne almost 5 years ago

Ack.

#5

Updated by Thomas Noël almost 5 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.

#6

Updated by Frédéric Péters almost 5 years ago

S'il y a pas full=on, on pourrait zapper tout workflow_data, non ?

#7

Updated by Benjamin Dauvergne almost 5 years ago

Oui, ou filtrer les PickableUpload qui sont dans workflow_data.

#8

Updated by Frédéric Péters almost 5 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.

#9

Updated by Thomas Noël almost 5 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.

#10

Updated by Frédéric Péters almost 5 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)
#11

Updated by Thomas Noël almost 5 years ago

  • Target version set to v1.42
#12

Updated by Frédéric Péters almost 5 years ago

  • Related to Bug #11000: la classe PicklableUpload peut exister sous deux namespaces différents added
#13

Updated by Frédéric Péters over 4 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF