Development #36505
sélection des champs repris sur les pages de validation et demande
0%
Description
À la fois pour pouvoir ne pas en reprendre certains (typiquement un champ dans les données de traitement, technique) et pour pouvoir en reprendre d'autres (genre un commentaire sur la page de validation).
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Fichier 0001-misc-add-options-to-display-hide-fields-on-validatio.patch 0001-misc-add-options-to-display-hide-fields-on-validatio.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus de 4 ans
Reflexion du soir : quelque chose m'ennuie dans l'affichage des commentaires. L'utilisation, ça va être pour des champs commentaires qui contiennent des informations, du genre "noms des membres du bureau, résultats de l'api association". Mais il n'y aura aucun stockage : l'api sera appelée à chaque affichage du commentaire, même dans 3 semaines, pendant le traitement de la demande, quand les données de l'association auront peut-être changé. Et alors bonjour les incompréhensions pour l'agent qui traite la demande...
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Thomas Noël a écrit :
Reflexion du soir : quelque chose m'ennuie dans l'affichage des commentaires. L'utilisation, ça va être pour des champs commentaires qui contiennent des informations, du genre "noms des membres du bureau, résultats de l'api association". Mais il n'y aura aucun stockage : l'api sera appelée à chaque affichage du commentaire, même dans 3 semaines, pendant le traitement de la demande, quand les données de l'association auront peut-être changé. Et alors bonjour les incompréhensions pour l'agent qui traite la demande...
Il me semble qu'on vit déjà avec ça coté commentaire de workflow, et c'est un usage suffisamment avancé dans l'ensemble qu'on a aucun mal à l'expliquer. Ce que je veux dire c'est que les clients qui jouent avec des appels de WS dans des commentaires, c'est déjà le haut du panier.
Mis à jour par Thomas Noël il y a plus de 4 ans
Benjamin Dauvergne a écrit :
Il me semble qu'on vit déjà avec ça coté commentaire de workflow, et c'est un usage suffisamment avancé dans l'ensemble qu'on a aucun mal à l'expliquer. Ce que je veux dire c'est que les clients qui jouent avec des appels de WS dans des commentaires, c'est déjà le haut du panier.
Oui ça s'explique, mais ce que je veux dire, c'est que quasiment le seul intérêt d'afficher des commentaires dans le récap, c'est celui-là (des commentaires qui contiennent des informations "dynamiques", ie des trucs du webservice ou des session_var). Donc bon, il faut juste comprendre que ce patch n'a rien à voir avec une solution à ce soucis, même si, dans les premières secondes de la vie du formulaire, ça donne l'impression de faire le job.
C'est juste une reflexion en passant, pas du tout un argument contre ce patch (que j'ai pas encore lu)
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Thomas Noël a écrit :
Benjamin Dauvergne a écrit :
Il me semble qu'on vit déjà avec ça coté commentaire de workflow, et c'est un usage suffisamment avancé dans l'ensemble qu'on a aucun mal à l'expliquer. Ce que je veux dire c'est que les clients qui jouent avec des appels de WS dans des commentaires, c'est déjà le haut du panier.
Oui ça s'explique, mais ce que je veux dire, c'est que quasiment le seul intérêt d'afficher des commentaires dans le récap, c'est celui-là (des commentaires qui contiennent des informations "dynamiques", ie des trucs du webservice ou des session_var).
Pas seulement il y a aussi des cas ou c'est simplement un affichage par rapport à des données du formulaire, j'ai l'exemple d'un formulaire encombrant qui contient un rendu graphique des éléments sélectionnés (un frigo, deux sommiers, etc..). Ça peut servir à afficher des calculs aussi (genre calcul d'un prix) mais c'est quand même très limité par les possibilités des formules dans les templates Django (qui fait qu'on finit souvent par en faire un appel a une datasource pour déplacé le calcul dans une expression python). C'est une fonction récapitulative un peu générale, d'ailleurs je pense qu'il faudrait prévoir le cas d'un commentaire uniquement en page de validation/résumé et pas pendant la phase de remplissage du tout.
Mis à jour par Nicolas Roche il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Si ça peut débloquer, je valide malgré mon manque d'expérience sur wcs.
J'ai joué avec, et tout me semble bon.
Ensuite pour le code (mais là je pêche) j'ai juste relevé ça histoire de dire (mais en fait je n'ai toujours pas compris le rôle de cette fonction).
wcs/forms/common.py::FormStatusPage::display_fields
- if f.type in ('title', 'subtitle'):
+ if f.type in ('title', 'subtitle', 'comment') and f.include_in_summary_page:
current_page_fields.append({'field': f})
continue
if not hasattr(f, 'get_view_value'):
continue
+ if not f.include_in_summary_page:
+ continue
J'aurais mis le 3ème bloc en premier pour retirer le 'and' du premier bloc.
Mais j'aurais aussi pu me taire.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 3a496d67c6e5153f0605225a35b16bd99dfe850a
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Sat Sep 28 16:45:24 2019 +0200
misc: add options to display/hide fields on validation/summary pages (#36505)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
Mis à jour par Brice Mallet il y a plus de 4 ans
- Remplacer "Page de récapitulatif" par "Page de confirmation" car c'est le terme utilisé dans l'écran de création du formulaire (https://formulaires-meuse.test.entrouvert.org/backoffice/forms/1/)
- Ordonner les 3 termes plutôt ainsi : Page de confirmation / Tableaux de traitement / Résumé de la demande, car sera alors dans l'ordre chronologique de visualisation dans la vie d'une demande
Mis à jour par Frédéric Péters il y a plus de 4 ans
Si il est encore temps pour des ajustements, je propose :
Il faut des nouveaux tickets.
Mis à jour par Frédéric Péters il y a plus de 2 ans
- Lié à Development #11442: champs de présentation (titre/...) dans les champs backoffice ajouté
misc: add options to display/hide fields on validation/summary pages (#36505)