Development #59674
retirer les parties de workflow_data tapées à la "racine" dans l'/inspect
0%
Description
workflow_data est un vrac pas rangé et en plus de form_workflow_data_… on expose tout son contenu sans préfixe.
Je serais pour zapper ça, au moins encourager à viser form_workflow_data_…
Ça devrait pouvoir se faire ainsi :
for k in sorted(substvars.get_flat_keys()): if k in ('attachments',): # blacklist, legacy continue + if k in (self.filled.workflow_data or {}).keys(): + continue
Fichiers
Révisions associées
Historique
Mis à jour par Thomas Noël il y a plus de 2 ans
Frédéric Péters a écrit :
Ça devrait pouvoir se faire ainsi :
C'est sans doute un petit peu plus compliqué, workflow_data contient des clés qui peuvent être des dictionnaires/listes/etc qui vont être aplatis par get_flat_keys(). Et donc le "in" ne va pas fonctionner.
Peut-être un with_workflow_data=True à ajouter dans FormData::get_substitution_variables qui est appelé au début de l'inspect ( substvars.update(self.filled.get_substitution_variables(with_workflow_data=False)) ) ?
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Fichier 0001-misc-limit-inspect-view-to-form_-variables-59674.patch 0001-misc-limit-inspect-view-to-form_-variables-59674.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Assigné à mis à Frédéric Péters
- Patch proposed changé de Non à Oui
Alternative simple et directe, ignorer tout ce qui ne commence pas par form_.
Mis à jour par Thomas Noël il y a plus de 2 ans
- Statut changé de Solution proposée à Solution validée
Ca me semble bête comme j'aime.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c20803ba48d5b9f91888f3d2e550d2a1b42412a4 Author: Frédéric Péters <fpeters@entrouvert.com> Date: Sat Dec 18 11:03:00 2021 +0100 misc: limit inspect view to form_* variables (#59674)
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Pierre Cros il y a plus de 2 ans
J'imagine que c'est lié un retour chrono et maintenant stocké dans {{ form_workflow_data_reservation_response_* }}
Mis à jour par Frédéric Péters il y a plus de 2 ans
C'était déjà le cas, l'information existait dans form_workflow_data_reservation_response_ et reservation_response_. (via #49196).
misc: limit inspect view to form_* variables (#59674)