https://dev.entrouvert.org/https://dev.entrouvert.org/favicon.ico?15861920342022-08-01T10:07:56ZRedmine Entr’ouvertw.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3981532022-08-01T10:07:56ZThomas Noël
<ul><li><strong>Assigné à</strong> mis à <i>Thomas Noël</i></li></ul> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3981622022-08-01T10:28:57ZThomas Noël
<ul><li><strong>Fichier</strong> <a href="/attachments/68240">0001-fields-allow-none-prefill-in-items-fields-67843.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68240/0001-fields-allow-none-prefill-in-items-fields-67843.patch">0001-fields-allow-none-prefill-in-items-fields-67843.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. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3981702022-08-01T12:27:41ZNicolas 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>Je suis embêté pour valider ici, parce que oui ça paraît être la bonne correction,<br />cependant le test ne couvre pas les 2 types de traces relevées dans les tickets liés, parce qu'ici on sort avant d'entrer dans get_prefill_value.<br />Sur le test :<br /><pre>
# check with a "none" explicit prefill, or a None value
for none_prefill_value in [
{'type': 'none'},
</pre></p>
<p>dans le code, on sort avant d'atteindre la correction, sur :<br /><pre>
> /home/nroche/src/wcs/wcs/forms/root.py(421)apply_field_prefills()
417 if field.prefill:
</pre></p>
<p>J'imagine que je chipote pour rien, auquel cas je valide.</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3981792022-08-01T13:03:33ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution validée</i> à <i>En cours</i></li></ul><p>Effectivement, ça ne va pas... Le test ne plante pas avec <code>prefill={'type': 'none'}</code>.</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3981922022-08-01T13:25:25ZNicolas Roche (absent jusqu'au 3 avril)
<ul><li><strong>Fichier</strong> <a href="/attachments/68247">form-reservation-halle-toiles.wcs</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68247/form-reservation-halle-toiles.wcs">form-reservation-halle-toiles.wcs</a> ajouté</li></ul><p>J'arrive à reproduire avec ce petit formulaire (ici d'un export du formulaire d'un des 2 tickets lié : #67651)</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3982172022-08-01T14:19:42ZThomas Noël
<ul><li><strong>Fichier</strong> <a href="/attachments/68249">0001-fields-allow-none-prefill-in-items-fields-67843.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68249/0001-fields-allow-none-prefill-in-items-fields-67843.patch">0001-fields-allow-none-prefill-in-items-fields-67843.patch</a> ajouté</li><li><strong>Statut</strong> changé de <i>En cours</i> à <i>Nouveau</i></li><li><strong>Assigné à</strong> <del><i>Thomas Noël</i></del> supprimé</li></ul><p>Voici une idée de patch que je pense bonne... mais un test manque qui passerait par <code>reset_locked_data</code> dans <code>wcs/forms/root.py</code> avec un <code>ItemsField</code> configuré avec un <code>prefill={'type': 'none'}</code></p>
<p>Mystère trop profond pour moi, je ne sais pas écrire de test sur le sujet... :/</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3994782022-08-08T16:51:34ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Assigné à</strong> mis à <i>Frédéric Péters</i></li><li><strong>Patch proposed</strong> changé de <i>Oui</i> à <i>Non</i></li></ul> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3994852022-08-09T04:54:18ZFrédéric Pétersfpeters@entrouvert.com
<ul><li><strong>Fichier</strong> <a href="/attachments/68448">0002-fields-allow-none-prefill-in-items-fields-67843.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68448/0002-fields-allow-none-prefill-in-items-fields-67843.patch">0002-fields-allow-none-prefill-in-items-fields-67843.patch</a> ajouté</li><li><strong>Fichier</strong> <a href="/attachments/68447">0001-fields-use-accessor-to-get-prefill-configuration-678.patch</a> <a class="icon-only icon-download" title="Télécharger" href="/attachments/download/68447/0001-fields-use-accessor-to-get-prefill-configuration-678.patch">0001-fields-use-accessor-to-get-prefill-configuration-678.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>J'ai repris le commit mais j'y ajouté avant un autre, qui ajoute une méthode get_prefill_configuration, qui assure que tout le temps le dictionnaire "prefill" sera vide pour signifier qu'il n'y a pas de préremplissage, jamais {"type": "none"}.</p>
<p>Rebasé le commit là-dessus, c'était lui enlever la partie touchant à wcs/forms/root.py. J'ai aussi modifié le test pour passer sur le cas prefill dictionnaire vide (même si ça n'apporte pas grand chose) + y corriger le commentaire "it will use foo,bar as selected in the first part of this test" qui ne correspondait pas à ce qui était attendu/vérifié.</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3995312022-08-09T08:00:49ZThomas Noël
<ul><li><strong>Statut</strong> changé de <i>Solution proposée</i> à <i>Solution validée</i></li></ul><p>Clairement plus solide.</p> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3996902022-08-09T14:25:56ZFré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 4050ddd838366ff23baa4fbb1acdc76efd7207e3
Author: Thomas Noël <tnoel@entrouvert.com>
Date: Mon Aug 1 12:28:23 2022 +0200
fields: allow none prefill in items fields (#67843)
commit dd79662c1b0e23191a8e0a17a55c5e045ec316ec
Author: Frédéric Péters <fpeters@entrouvert.com>
Date: Mon Aug 8 18:57:30 2022 +0200
fields: use accessor to get prefill configuration (#67843)
This allows an uniform handling of the unexpected {'type': 'none'}
situation.
</pre> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=3996992022-08-09T15:14:29ZTransition automatique
<ul><li><strong>Statut</strong> changé de <i>Résolu (à déployer)</i> à <i>Solution déployée</i></li></ul> w.c.s. - Bug #67843: Le prefill sur les listes à choix multiple doit accepter la valeur Nonehttps://dev.entrouvert.org/issues/67843?journal_id=4130272022-10-09T02:42:01ZTransition automatique
<ul></ul><p>Automatic expiration</p>