Development #17162
json data store : expression pour le calcul d'une clé "text"
0%
Description
Pour que les données stockées puissent être exploitées par w.c.s., il manque une clé "text". Idée d'un paramètre de configuration "template for text key" où on pourrait taper une ligne de template django.
Ça serait calculé lors du create/update, pour ne pas tuer les perfs de la récup des infos.
Fichiers
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-jsondatastore-add-possibility-of-a-template-to-creat.patch 0001-jsondatastore-add-possibility-of-a-template-to-creat.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a presque 7 ans
s/text_key_template/text_value_template/ ?
Mais bon, pas très important, ack.
Mis à jour par Frédéric Péters il y a presque 7 ans
- Fichier 0001-jsondatastore-add-possibility-of-a-template-to-creat.patch 0001-jsondatastore-add-possibility-of-a-template-to-creat.patch ajouté
Patch refait avec cette modification dans le nom de l'attribut mais aussi :
- création de la valeur text dans la méthode save() de JsonData (c'est en voulant faire ça que j'ai remarqué qu'on n'avait pas de lien vers le store)
- modification pour passer uniquement le dictionnaire du contenu au template, pour ne pas avoir à expliquer/répéter
data.content.
Mis à jour par Thomas Noël il y a presque 7 ans
Pas fan du «.strip()» (même si je vois bien contre quoi il lutte)
Tout à fait ok pour Context(self.content) : et là, je me dis qu'on ne teste pas que ce content (donc request.body du save ou update) doit être un dict.
Ajouter des 'err':0 dans les réponses (déjà fait dans le delete). De fait, pour uniformiser (et encore simplifier la doc), tout passer en {err:0, data:...}. Et donc utiliser partout le mode 'json-api' pour gagner le err:1 en cas d'exception.
Mis à jour par Thomas Noël il y a presque 7 ans
Thomas Noël a écrit :
Tout à fait ok pour Context(self.content) : et là, je me dis qu'on ne teste pas que ce content (donc request.body du save ou update) doit être un dict.
Ajouter des 'err':0 dans les réponses (déjà fait dans le delete). De fait, pour uniformiser (et encore simplifier la doc), tout passer en {err:0, data:...}. Et donc utiliser partout le mode 'json-api' pour gagner le err:1 en cas d'exception.
Ces deux trucs peuvent venir dans 2 autres patches ; je tickette.
Mis à jour par Frédéric Péters il y a presque 7 ans
- Statut changé de En cours à Résolu (à déployer)
Ok; la suite dans les autres tickets.
commit 816700efd7f176388652fce2cabb05fa7d950810 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Fri Jun 23 18:44:54 2017 +0200 jsondatastore: add possibility of a template to create a "text" key (#17162)
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Fermé
jsondatastore: add possibility of a template to create a "text" key (#17162)