Bug #30885
les urls des actions dans les cellules JSON ne reçoit pas tout le contexte
0%
Description
Notamment sur la cellule json a un form qui décrit un champ "foobar", les URL d'actions ne reçoivent pas de {{foobar}}
Et dans l'idée que les URL d'actions doivent réagir comme les autres URLs, ne pas lancer l'action sur l'URL finalement calculée est vide (pour permettre les {% if foobar %}url{% endif %}
)
Fichiers
Révisions associées
misc: add back a "parameters" key in json cell context (#30885)
Historique
Mis à jour par Frédéric Péters il y a environ 5 ans
- Fichier 0001-misc-pass-configured-json-cell-parameters-to-action-.patch 0001-misc-pass-configured-json-cell-parameters-to-action-.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a environ 5 ans
- Statut changé de Solution proposée à Solution validée
Je pensais qu'on pouvait aussi oser un :
diff --git a/combo/data/models.py b/combo/data/models.py index 1a0e501..1473a71 100644 --- a/combo/data/models.py +++ b/combo/data/models.py @@ -1186,6 +1186,9 @@ class JsonCellBase(CellBase): logger.warning('error in templated URL (%s): %s', self.actions[action]['url'], e) raise PostException(error_message) + if not url: + return None + json_response = utils.requests.request( method, url,
dans l'idée que ce patch serait "que les url d'actions se comportent comme les autres".
Mais ack ainsi ; un autre ticket/patch à faire si un jour on a besoin de cela sur les actions.
(et merci)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
Mais ack ainsi ; un autre ticket/patch à faire si un jour on a besoin de cela sur les actions.
Oui c'est autre chose.
commit c47ef6fd4dd321b4c9bf1a0c1a9947bf1bf50b9c Author: Frédéric Péters <fpeters@entrouvert.com> Date: Mon Feb 25 20:52:49 2019 +0100 misc: pass configured json cell parameters to action URL templates (#30885)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à En cours
Bon un truc merde, on transmettait visiblement avant les paramètres également dans une variable dédiée, je m'en occupe.
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de En cours à Résolu (à déployer)
L'accès à ces éléments via "parameters" c'est ce qui est documenté sur https://doc-publik.entrouvert.com/dev/developpement-d-une-cellule-json/ et c'est clairement mieux, je l'ai restauré plutôt que juste retirer la ligne des tests et modifier la documentation en douce…
commit b1f13644d36008230512b1c860bd35df847d7ccb Author: Frédéric Péters <fpeters@entrouvert.com> Date: Tue Feb 26 10:47:16 2019 +0100 misc: add back a "parameters" key in json cell context (#30885)
Mis à jour par Frédéric Péters il y a environ 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: pass configured json cell parameters to action URL templates (#30885)