Bug #17904
cellule json: permettre que les varnames prennent le dessus
0%
Description
Actuellement pour prendre en compte des variables depuis la querystring, on a ce code :
for varname in self.varnames: if varname in context['request'].GET and varname not in context: context[varname] = context['request'].GET[varname]
le varname not in context
empêche toute redéfinition d'une variable déjà présente dans le contexte, par exemple dans les parameters.
En fait ce n'est pas utile (si une variable ne doit rien écraser alors il suffit de la nommer correctement), mais surtout ça empêche un usage : on défini une valeur de base dans les parameters, et on permet qu'elle soit redéfinie dans la query-string.
Exemple : un paramètre "limit" configuré par 10 par défaut dans la cellule, mais on autorise ?limit=100 dans la query-string.
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a presque 7 ans
- Fichier 0001-json-cell-priorize-varnames-in-context-17904.patch 0001-json-cell-priorize-varnames-in-context-17904.patch ajouté
- Statut changé de Nouveau à En cours
- Patch proposed changé de Non à Oui
Au passage il faut modifier la gestion de get_cell_extra_context pour que les paramètres n'écrasent pas les varnames.
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Poussé en corrigeant s/priorize/prioritise/.
commit 04f1fd1d827c2d75fd81b0d1f0c46eb26ca6a909 Author: Thomas NOEL <tnoel@entrouvert.com> Date: Mon Aug 7 15:02:01 2017 +0200 json cell: prioritise varnames in context (#17904)
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
json cell: prioritise varnames in context (#17904)