Projet

Général

Profil

Development #7254

export json intégrant les fichiers joints

Ajouté par Thomas Noël il y a presque 9 ans. Mis à jour il y a plus de 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
Début:
15 mai 2015
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Il serait pratique que l'export JSON des formdata intègre le contenu des fichiers joints.

« get_as_json(…, include_files=True) »


Fichiers


Demandes liées

Bloqué par w.c.s. - Bug #7258: Documents attachés absent lors du lancement du workflowFermé16 mai 2015

Actions

Révisions associées

Révision aaba8117 (diff)
Ajouté par Thomas Noël il y a presque 9 ans

api: export content of uploads (#7254)

Historique

#1

Mis à jour par Thomas Noël il y a presque 9 ans

J'ai pas de façon «propre» de faire un flag include_files=True, et en fait je n'en vois pas forcément la pertinence (à discuter).

Voici un patch qui ajoute le contenu (et le content-type) à l'existant, c'est juste une extension de l'API (ajout de deux champs).

avant:

"photo" : {
  "field_id": "8",
  "filename": "parc-de-hann-51.jpg" 
}

après:

"photo" : {
  "field_id": "8",
  "filename": "parc-de-hann-51.jpg" 
  "content_type": "image/jpeg",
  "content": "....(base64)..." 

}

#2

Mis à jour par Thomas Noël il y a presque 9 ans

  • Statut changé de Nouveau à En cours
  • Patch proposed changé de Non à Oui
#3

Mis à jour par Frédéric Péters il y a presque 9 ans

Noté par jabber, pour les API envoyant plusieurs formdata, j'ai plutôt peur de la situation où il y aura à composer un json de plusieurs centaine de mégas, avec toute une série de scans de cartes grises; pour cette API-là, plutôt que varier le contenu de l'export json du champ, je zapperais simplement les champs de type "fichier".

(et doc et test)

#4

Mis à jour par Thomas Noël il y a presque 9 ans

sans doc et sans test, voilà ce que ça donnerait... j'ai pas plus joli, et vous ?

#5

Mis à jour par Thomas Noël il y a presque 9 ans

  • Bloqué par Bug #7258: Documents attachés absent lors du lancement du workflow ajouté
#6

Mis à jour par Thomas Noël il y a presque 9 ans

Voici un patch complet, avec doc et test, et qui inclue #7258.

#7

Mis à jour par Frédéric Péters il y a presque 9 ans

Dans la doc, tu noterais la forme que ça prend quand un fichier est présent ?

#9

Mis à jour par Frédéric Péters il y a presque 9 ans

Nickel; je ne vérifie même pas si les espaces sont insécables comme il faut.

#10

Mis à jour par Frédéric Péters il y a presque 9 ans

  • Statut changé de En cours à Résolu (à déployer)

J'ai poussé le commit.

commit aaba8117c45d58888b85de41c19c7c05feef7f76
Author: Thomas NOEL <tnoel@entrouvert.com>
Date:   Fri Jun 26 16:50:37 2015 +0200

    api: export content of uploads (#7254)
#11

Mis à jour par Thomas Noël il y a presque 9 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#12

Mis à jour par Thomas Noël il y a presque 9 ans

  • Version cible mis à v1.13.0
#13

Mis à jour par Thomas Noël il y a presque 9 ans

  • Tracker changé de Bug à Development
#14

Mis à jour par Thomas Noël il y a presque 9 ans

  • Statut changé de Solution déployée à Résolu (à déployer)
#15

Mis à jour par Thomas Noël il y a plus de 8 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF