https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342021-02-22T13:02:21ZRedmine Entr’ouvertw.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2891322021-02-22T13:02:21ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Description</strong> mis à jour (<a title="Voir les différences" href="/journals/289132/diff?detail_id=253696">diff</a>)</li></ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2903752021-03-01T13:09:09ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/52348">0001-forms-recompute-prefills-on-live-changes-51330.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52348/0001-forms-recompute-prefills-on-live-changes-51330.patch">0001-forms-recompute-prefills-on-live-changes-51330.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><p>En fait un patch fait de quantité de petites choses :</p>
<ul>
<li>on inclut désormais la classe widget-prefilled de manière systématique pour les champs configurés avec du préremplissage; avant on ne le faisait pas quand le préremplissage donnait une chaine vide; on fait ça parce que c'est sur base de la classe widget-prefilled qu'on va mettre à jour le champ ou pas.</li>
</ul>
<ul>
<li>ça fait que côté js quand un champ est modifié on retire la classe widget-prefilled.</li>
</ul>
<ul>
<li>et dans l'appel à l'url /live pour obtenir les résultats, on passe la liste des champs préremplis (ex: ?prefilled_1=on&prefilled_4=on&...)</li>
</ul>
<ul>
<li>il y a une valeur particulière quand l'usager associé à une demande est modifié (ce qui arrive quand on est en saisie backoffice et qu'on choisit un usager en barre latérale); on passe dans ce cas ?modified_field_id=user</li>
</ul>
<ul>
<li>il y a la situation particulière de blocs de champs où on clique sur le bouton pour ajouter une ligne, dans la suite il y a court-circuitage du préremplissage et donc les champs qui étaient déjà là n'ont plus la classe widget-prefilled; ça m'ennuyait vraiment d'avoir à tout recalculer après un clic sur "ajouter" (après avoir déjà fait <a class="issue tracker-1 status-5 priority-4 priority-default closed" title="Bug: après un clic sur "ajouter" d'un bloc de champ, les champs préremplis/verrouillés n'apparaissent ... (Fermé)" href="https://dev.entrouvert.org/issues/51314">#51314</a> pour recalculer l'aspect "champ verrouillé"); c'est pour contourner cette partie du problème que j'ai fait <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: faire l'ajout d'une ligne de bloc en ajax (Fermé)" href="https://dev.entrouvert.org/issues/51369">#51369</a> pour que la partie "ajouter" soit gérée ajax et sans incidence sur les autres champs.</li>
</ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2908282021-03-03T09:41:35ZThomas Noël
<ul></ul><p>Sur ce moment :</p>
<pre> elif get_request().form.get('modified_field_id') == 'user':
# user selection in sidebar
formdata.user_id = get_request().form.get('user_id')
</pre>
<p>J'ai l'impression que ça pourrait permettre, en bidouillant le user_id envoyé au live, de voir les infos pré-remplies de n'importe quel user dans le résultat du live...? Il faudrait peut-être vérifier qu'on est bien en backoffice ?</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2908532021-03-03T10:41:44ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Yes, je vais faire ça.</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2910342021-03-04T07:31:03ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/52450">0001-forms-recompute-prefills-on-live-changes-51330.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/52450/0001-forms-recompute-prefills-on-live-changes-51330.patch">0001-forms-recompute-prefills-on-live-changes-51330.patch</a> ajouté</li></ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2910892021-03-04T08:48:56ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2910942021-03-04T08:51:00ZFré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 b091599f369fc57367df9776cd48ad4dee2867c1
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Feb 22 15:50:20 2021 +0100
forms: recompute prefills on live changes (#51330)
</pre> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913542021-03-04T21:16:21ZFré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 #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913622021-03-05T03:01:25ZPierre Crospcros@entrouvert.com
<ul></ul><p>Je m'attendais à c e que ça fonctionne ici et c'est pas le cas :<br /><a class="external" href="https://demarches-montoulouse.cutm-publik-preprod.nfrance.com/backoffice/submission/allo-toulouse-signalements/">https://demarches-montoulouse.cutm-publik-preprod.nfrance.com/backoffice/submission/allo-toulouse-signalements/</a> (EDIT : URL modifiée)</p>
<p>Peut-être parce que c'est un bloc de champ qui est employé :<br /><a class="external" href="https://demarches-montoulouse.cutm-publik-preprod.nfrance.com/backoffice/forms/blocks/3/">https://demarches-montoulouse.cutm-publik-preprod.nfrance.com/backoffice/forms/blocks/3/</a></p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913692021-03-05T07:02:53ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Tu pointes deux fois l'URL vers la définition du bloc, je pense que tu voulais pointer l'endroit où constater l'affaire en premier.</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913862021-03-05T08:19:52ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>ça fonctionne ici</p>
</blockquote>
<p>Étant donc une situation de saisie backoffice où l'usager est choisi en barre latérale et du préremplissage à partir de champs du profil configuré sur des champs d'un bloc de la démache.</p>
<p>(mais là en local, même sans l'affaire de bloc j'ai l'impression qu'un bug tardif s'est introduit).</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913882021-03-05T08:25:54ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Lié à</strong> <i><a class="issue tracker-1 status-5 priority-4 priority-default closed" href="/issues/51686">Bug #51686</a>: préremplissage dynamique en saisie backoffice avec juste du préremplissage usager</i> ajouté</li></ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2913902021-03-05T08:28:48ZFré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/51688">Development #51688</a>: préremplissage dynamique vers des champs de blocs</i> ajouté</li></ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2914242021-03-05T09:12:29ZPierre Crospcros@entrouvert.com
<ul></ul><p>J'ai rétabli la bonne URL dans mon commentaire initial.</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2917512021-03-08T09:16:52ZBrice Mallet
<ul></ul><p>J'avais fait un formulaire de test : <a class="external" href="https://demarches-mkuntz.test.entrouvert.org/brice/test-pre-remplissage-puis-reinitialisation/">https://demarches-mkuntz.test.entrouvert.org/brice/test-pre-remplissage-puis-reinitialisation/</a> (BO : <a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/</a>), je m'attendais à ce que le pré-remplissage fonctionne dans tous les cas sur la même page (sur la page 1) or ce n'est pas le cas, est-ce un problème de configuration, de ce que devait faire le développement ou un bug ?</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2917522021-03-08T09:19:53ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><p>Tu peux dire ce que tu attendais plus précisément que "dans tous les cas" ?</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2924822021-03-11T07:34:07ZBrice Mallet
<ul></ul><blockquote>
<p>Tu peux dire ce que tu attendais plus précisément que "dans tous les cas" ?</p>
</blockquote>
<p>Désolé, j'avais loupé ta réponse.<br />"dans tous les cas" : que ce qui fonctionne en pré-remplissage sur une page suivante, fonctionne maintenant en pré-remplissage sur la même page.</p>
Dans la cas de ce formulaire de test :
<ul>
<li>le déclencheur de la mise à jour est <a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/4/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/4/</a></li>
<li>cela déclenche bien la mise à jour
<ul>
<li>du champ commentaire (<a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/3/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/3/</a>)</li>
<li>et du champ texte sur la même page (<a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/5/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/5/</a>)</li>
</ul>
</li>
<li>mais pas
<ul>
<li>de la liste (<a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/12/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/12/</a>)</li>
<li>et du sous-titre (<a class="external" href="https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/15/">https://demarches-mkuntz.test.entrouvert.org/backoffice/forms/163/fields/15/</a>)<br />qui, eux, ne sont mis à jour / pré-remplis que sur la page suivante ainsi que sur la première page, mais seulement au retour sur celle-ci</li>
</ul></li>
</ul> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2924962021-03-11T07:57:55ZFrédéric Pétersfpeters@entrouvert.com
<ul></ul><blockquote>
<p>de la liste</p>
</blockquote>
<p>C'était hors scope précisé dans tracim, "Pré-remplissage dynamique des champs Texte (court et long)", mais je vais faire un ticket.</p>
<blockquote>
<p>et du sous-titre</p>
</blockquote>
<p>Il n'y a pas de notion de préremplissage sur les sous-titres.</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2925302021-03-11T09:24:52ZRenaud Boitouzet
<ul></ul><p>Bonjour, je m'immisce dans vos échanges, pardon si ce n'est pas approprié !</p>
<p>Je ne sais pas si c'est pertinent/possible de changer ça, mais en reprenant l'exemple du formulaire de test <a class="external" href="https://demarches-mkuntz.test.entrouvert.org/brice/test-pre-remplissage-puis-reinitialisation/">https://demarches-mkuntz.test.entrouvert.org/brice/test-pre-remplissage-puis-reinitialisation/</a> si on passe en page 2 sans avoir modifié manuellement le champ texte prérempli dynamiquement, puis qu'on revient en page 1, le pré-remplissage dynamique ne fonctionne plus.</p>
<p>Le comportement n'est donc pas le même selon que l'usager arrive sur la page pour la première fois ou qu'il y revienne.</p>
<p>J'ai trouvé plusieurs tickets connexes mais qui datent d'avant la mise en place du préremplissage dynamique : <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: Mise à jour d'un champ pré-rempli (Fermé)" href="https://dev.entrouvert.org/issues/43369">#43369</a>, # 45348.</p> w.c.s. - Development #51330: préremplissage dynamique d'un champ sur la même pagehttps://dev.entrouvert.org/issues/51330?journal_id=2929092021-03-12T17:22:12ZBrice Mallet
<ul></ul><blockquote>
<p>C'était hors scope précisé dans tracim, "Pré-remplissage dynamique des champs Texte (court et long)", mais je vais faire un ticket.</p>
</blockquote>
<p>La demande initiale émanait de Toulouse qui demandait explicitement uniquement pour les champs textes ; mais comme ce développement est la suite de "Pré-remplissage de champs Texte et Liste : réinitialisation après changement de condition", j'ai sur-interprété que cela s'appliquerait aussi sur les listes.<br />Nota : le ticket de Fred = <a class="issue tracker-2 status-5 priority-4 priority-default closed" title="Development: mise à jour dynamique du préremplissage d'un champ liste (Fermé)" href="https://dev.entrouvert.org/issues/51897">#51897</a></p>
<blockquote>
<p>Il n'y a pas de notion de préremplissage sur les sous-titres.</p>
</blockquote>
<p>En effet pas "pré-remplissage", "mise à jour automatique" serait plus juste ; il n'empêche que comme de plus en plus de champs de saisie et de champs de mise en forme (commentaire, titre, sous-titre) sont mis à jour depuis un déclencheur, je trouve qu'il serait bien que le comportement soit identiques pour tous les champs (i.e. mise à jour en direct même si le déclencheur est sur la page même page), je vais faire un courriel à ce sujet</p>