Development #42371
api de listing des demandes/fiches, inclure des infos sur les fichiers quand ?full=on
0%
Description
Aujourd'hui les champs fichiers sont juste totalement zappé ("pour ne pas alourdir l'export en mode full=on, les champs de type « Fichier » ne sont pas exportés"), on pourrait inclure nom, type mime, url, sans alourdir.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-api-include-file-info-in-list-API-42371.patch 0001-api-include-file-info-in-list-API-42371.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 4 ans
Je suis un peu partagé entre le renvoie de "none" ou l'absence totale de la clé qui est proposée ici (et qui "casse" un peu le schéma). Mais bon, comme on n'a pas vraiment de schéma, supprimons.
Je n'ai pas compris l'idée de la modif de FileField.get_json_value dans fields.py et l'ajout d'une clé "url". De plus selon moi il faut éviter de divulguer les URL de redirection, qui ne doivent être visibles que des personnes qui ont accès au fichier (ie à la demande liée).
Mis à jour par Frédéric Péters il y a presque 4 ans
- Fichier 0001-api-include-file-info-in-list-API-42371.patch 0001-api-include-file-info-in-list-API-42371.patch ajouté
Je n'ai pas compris l'idée de la modif de FileField.get_json_value dans fields.py et l'ajout d'une clé "url".
Ajouter une clé "url", qui est en fait le souhait dans #42364. (mais c'est tellement fondamental que je passe peut-être à coté de la question).
De plus selon moi il faut éviter de divulguer les URL de redirection, qui ne doivent être visibles que des personnes qui ont accès au fichier (ie à la demande liée).
C'était pour m'éviter de vérifier le comportement adéquat sur download?... pour ces fichiers mais c'est ok, zappé donc ici.
Mis à jour par Thomas Noël il y a presque 4 ans
- Statut changé de Solution proposée à Solution validée
Ok pour l'ajout de 'url', j'avais trop vite lu le ticket.
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 07617b5f1ba189579b4b67ab60a2dee7240e3e9e Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri May 1 17:39:53 2020 +0200 api: include file info in list API (#42371)
Mis à jour par Frédéric Péters il y a presque 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
api: include file info in list API (#42371)