Project

General

Profile

Development #36505

sélection des champs repris sur les pages de validation et demande

Added by Frédéric Péters 3 months ago. Updated 2 months ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
28 Sep 2019
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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).

0001-misc-add-options-to-display-hide-fields-on-validatio.patch View (24.4 KB) Frédéric Péters, 28 Sep 2019 05:36 PM

Associated revisions

Revision 3a496d67 (diff)
Added by Frédéric Péters 2 months ago

misc: add options to display/hide fields on validation/summary pages (#36505)

History

#2 Updated by Frédéric Péters 3 months ago

#3 Updated by Thomas Noël 3 months ago

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...

#4 Updated by Benjamin Dauvergne 3 months ago

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.

#5 Updated by Thomas Noël 2 months ago

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)

#6 Updated by Benjamin Dauvergne 2 months ago

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.

#7 Updated by Nicolas Roche 2 months ago

  • Status changed from Solution proposée to 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.

#8 Updated by Frédéric Péters 2 months ago

  • Status changed from Solution validée to 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)

#9 Updated by Frédéric Péters 2 months ago

  • Status changed from Résolu (à déployer) to Solution déployée

#10 Updated by Brice Mallet 2 months ago

Si il est encore temps pour des ajustements, je propose :
  • 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

#11 Updated by Frédéric Péters 2 months ago

Si il est encore temps pour des ajustements, je propose :

Il faut des nouveaux tickets.

Also available in: Atom PDF