https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342019-09-28T15:36:59ZRedmine Entr’ouvertw.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1937072019-09-28T15:36:59ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/37639">0001-misc-add-options-to-display-hide-fields-on-validatio.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/37639/0001-misc-add-options-to-display-hide-fields-on-validatio.patch">0001-misc-add-options-to-display-hide-fields-on-validatio.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>Nouveau</i> à <i>Solution proposée</i></li><li><strong>Patch proposed</strong> changé de <i>Non</i> à <i>Oui</i></li></ul> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1937212019-09-28T20:05:23ZThomas Noël
<ul></ul><p>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...</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1937232019-09-28T21:00:55ZBenjamin Dauvergne
<ul></ul><p>Thomas Noël a écrit :</p>
<blockquote>
<p>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...</p>
</blockquote>
<p>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.</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1937392019-09-29T13:19:43ZThomas Noël
<ul></ul><p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>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.</p>
<p>C'est juste une reflexion en passant, pas du tout un argument contre ce patch (que j'ai pas encore lu)</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1937422019-09-29T13:48:56ZBenjamin Dauvergne
<ul></ul><p>Thomas Noël a écrit :</p>
<blockquote>
<p>Benjamin Dauvergne a écrit :</p>
<blockquote>
<p>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.</p>
</blockquote>
<p>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).</p>
</blockquote>
<p>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.</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1949722019-10-04T12:50:21ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Si ça peut débloquer, je valide malgré mon manque d'expérience sur wcs.<br />J'ai joué avec, et tout me semble bon.<br />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).</p>
<p><em>wcs/forms/common.py::FormStatusPage::display_fields</em><br /><pre>
- 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
</pre></p>
<p>J'aurais mis le 3ème bloc en premier pour retirer le 'and' du premier bloc.<br />Mais j'aurais aussi pu me taire.</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1949752019-10-04T12:54:36ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>Résolu (à déployer)</i></li></ul><pre>
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)
</pre> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1950032019-10-04T14:15:13ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1950732019-10-04T17:11:21ZBrice Mallet
<ul></ul>Si il est encore temps pour des ajustements, je propose :
<ul>
<li>Remplacer "Page de récapitulatif" par "Page de confirmation" car c'est le terme utilisé dans l'écran de création du formulaire (<a class="external" href="https://formulaires-meuse.test.entrouvert.org/backoffice/forms/1/">https://formulaires-meuse.test.entrouvert.org/backoffice/forms/1/</a>)</li>
<li>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</li>
</ul> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=1950932019-10-05T17:17:11ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>Si il est encore temps pour des ajustements, je propose :</p>
</blockquote>
<p>Il faut des nouveaux tickets.</p> w.c.s. - Development #36505: sélection des champs repris sur les pages de validation et demandehttps://dev.entrouvert.org/issues/36505?journal_id=3352762021-11-01T09:06:28ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-2 status-5 priority-4 priority-default closed" href="/issues/11442">Development #11442</a>: champs de présentation (titre/...) dans les champs backoffice</i> ajouté</li></ul>