emptyitem et menulist obligatoire

[foxmask]

​la doc dit que l'on peut utiliser <emptyitem> pour indiquer un libellé pour le choix "vide" (ce qui de surcroit rend la page valid W3C ; c'est qui n'est pas le cas quand il n'y a pas de libelle pour le choix vide d'ailleurs)

Cela ne fonctionne plus dans un cas particulier :

   <menulist ref="ville" required="true">
      <label>votre ville :</label>
      <emptyitem>faites votre choix</emptyitem>
      <datasource class="monmodule~listData" />
  </menulist>

dans ce cas là, si la datasource ne retourne aucune ville. Le formulaire contiendra une liste déroulante vide, sans même le <emptyitem>.

[foxmask]

le contenu de emptyitem n'est pas affiché des lors qu'on met required=true

[laurentj]

Je n'avais pas fait attention à l'attribut required. Mais cela parait logique. Si le control est required, cela veut dire que l'utilisateur doit absolument indiquer un choix. Et faire un choix implique bien entendu une valeur. Donc ce ne peut etre le choix du emptyitem. Donc pas de emptyitem. Car sinon, si on autorisait la valeur vide pour un champs required, comment coté serveur savoir si l'utilisateur a bien fait un choix ?

Bref, si le control est required, à toi de faire en sorte que le dao fournisse toujours quelque chose. Si il est susceptible de ne rien renvoyer, alors cela veut dire que la saisie dans ton control n'est pas obligatoire.

INVALID donc pour moi.

[foxmask]

understood

[laurentj]

Reouverture du bug. Cet emptyitem avec un menulist required est de plus en plus demandé, et finalement mon argumentation est assez faible. Rien n’empêche effectivement de checker que la valeur selectionnée correspond bien à une valeur de la liste...

[Julien]

Replying to laurentj:

Reouverture du bug. Cet emptyitem avec un menulist required est de plus en plus demandé, et finalement mon argumentation est assez faible. Rien n’empêche effectivement de checker que la valeur selectionnée correspond bien à une valeur de la liste...

​https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jForms-menulist-should-have-empty-item-even-when-required.patch

;)

[Julien]

and also :

​https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jForms-emptyitem-in-listbox.patch

because some people don't know how to deselect things in listboxes (ctrk-key)

ps: fot this patch and the one above, I have to complete unit-tests.

[laurentj]

​https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jForms-menulist-should-have-empty-item-even-when-required.patch

ok, but some tests should be adapted, no?

​https://bitbucket.org/julieni/jelix-trunk-patches/src/128b1c3f30f5/jForms-emptyitem-in-listbox.patch

It seems it doesn't apply on the trunk? And modifications on jFormsCompiler_jf_1_0 should be made in fact on jFormsCompiler_jf_1_1.

And probably some tests should be adapted.

[Julien]

yup, my ps said tests should be completed/upgraded

for the listbox patch, should work, but I may need to re-order my patch queue because it's quite at the end right now...

[laurentj]

jForms-menulist-should-have-empty-item-even-when-required.patch has been pushed with tests updated ​https://bitbucket.org/jelix/jelix-trunk/changeset/736c4434d8d4

[laurentj]

pushed jForms-emptyitem-in-listbox.patch with tests

​https://bitbucket.org/jelix/jelix-trunk/changeset/c3d87a14248a