developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#755 closed bug (fixed)

jForms : Problème de sélection d'item

Reported by: Surfoo Owned by: Julien
Priority: normal Milestone: jelix 1.1
Component: jelix:forms Version: trunk
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

En utilisant un <menulist> dont une des valeurs vaut 0, celle ci est sélectionné alors qu'il y a l'élément vide qui devrait être sélectionné avant.

Ça vient de la valeur attribué qui est 0 qui provoque ce soucis, si je la change, la sélection se fait bien sur l'élément vide.

Attachments (4)

patch.diff (1.6 KB) - added by Surfoo 12 years ago.
patch.2.diff (1.7 KB) - added by Surfoo 12 years ago.
Patch à partir de la racine !
patch.3.diff (1.7 KB) - added by Julien 12 years ago.
added type cast to rpevious patch, so strict comparison will always work
755-jForms-menulist-selected-item-problem-when-value-is-zero.diff (2.2 KB) - added by Julien 12 years ago.
final good patch

Download all attachments as: .zip

Change History (13)

Changed 12 years ago by Surfoo

comment:1 Changed 12 years ago by Surfoo

Bon mon premier patch, est ce que les modifications conviennent ? Je l'ai testé sur mon appli et c'est bon pour html.jformsbuilder.php, et pour htmllight.jformsbuilder.php c'est la même chose. J'ai pas modifié les infos des headers, pour un '=' en plus..

comment:2 Changed 12 years ago by laurentj

  • Component changed from jelix to jelix:forms
  • Milestone set to jelix 1.1
  • review set to review?
  • Version set to trunk

vérifier si le bug existe aussi dans la branche 1.0.x.

Les tests unitaires doivent être vérifiés, et peut être complété pour tester ce patch. Le patch doit être fait à partir de la racine de la branche.

Changed 12 years ago by Surfoo

Patch à partir de la racine !

Changed 12 years ago by Julien

added type cast to rpevious patch, so strict comparison will always work

comment:3 Changed 12 years ago by Julien

Hello,

j'ai juste (un peu) modifié ce patch afin d'ajouter un transtypage, comme cela avait était fait sur d'autres contrôles (checkboxes, radios, ...) afin de palier au même type de problème.

Changed 12 years ago by Julien

final good patch

comment:4 Changed 12 years ago by Julien

voilà le patch final, après vérif des tests unitaires.

oubliez patch.3.diff, il est incomplet.

comment:5 Changed 12 years ago by laurentj

  • review changed from review? to review+

Ok pour le patch. Pour le commit, ne pas oublier p=surfoo+julien

comment:6 Changed 12 years ago by Julien

vi bien entendu

comment:7 Changed 12 years ago by Julien

  • Owner set to Julien
  • Status changed from new to assigned

comment:8 Changed 12 years ago by Julien

  • Resolution set to fixed
  • Status changed from assigned to closed

commited in the trunk, r1186

I think we need to backport it to 1.0.x branch, no ?

comment:9 Changed 12 years ago by laurentj

yes please

Note: See TracTickets for help on using tickets.