Discussion :

[wazup]

j'ai un formulaire dans lequel je veux mettre une liste multiselectable

couplée à une collection dans mon formbean.

je veux donc qu'au load, elle charge la liste avec ma collection et qu'au submit, elle prenne les éléments sélectionnés pour remettre à jour ma collection.

mon jsp contient donc :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
<label for="maCollection"><bean:message key="label.Formulaire.maCollection"/></label><br>
<html:select property="maCollection" size="6" multiple="true">
	<html:options property="maCollection"/>
</html:select>

Tout ceci marche bien au chargement, maliste est initialisée avac la collection,

mais lorsque je fait un submit, je merécupère :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
01:52:19,826 DEBUG [BeanUtils]   setProperty(net.forms.MyBodyForm@161f39e, maCollection, [marc.dupont@wanadoo.fr,mdupont@aol.com])
01:52:19,826 DEBUG [ConvertUtils] Convert string 'marc.dupont@wanadoo.fr' to class 'java.util.Collection'
01:52:19,826 DEBUG [ConvertUtils]   Using converter org.apache.commons.beanutils.converters.StringConverter@1961581
01:52:19.831 WARN!! Exception for /appli/do/MonFormWrite
java.lang.IllegalArgumentException: argument type mismatch

Donc apparemment, chaque entrée de la liste est interprétée comme une collection et non comme un string de la collection.

Quelqu'un a-t-il une idée ?

PS
getter/setter côté form :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
public Collection getMaCollection() {
	return MaCollection(;
}
 
public void setMaCollection((Collection newCollection() {
	this.MaCollection = new ArrayList();
	for (Iterator ite = newCollection.iterator(); ite.hasNext();) {
		this.MaCollection.add(ite.next());
	}
}

[wazup]

bon ben je vais me répondre tout seul !

Il faut donc intercepter la requete dans le Form.Reset() et remplir soi-même la collection avec les parameters values de la requete apparemment..

Bof, pas terrible, mais ça marche au moins, a titre d'info.

[cowa]

public void setMaCollection((Collection newCollection() {
this.MaCollection = new ArrayList();
for (Iterator ite = newCollection.iterator(); ite.hasNext() {
this.MaCollection.add(ite.next());
}
}

Juste par curiosité, est-ce que ca marche ton bout de code ?
J'aurais fait un

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
 
public void setMaCollection(Collection newCollection) {
   this.maCollection = newCollection;
}

A ma connaissance, ca marche.

Pour remplir une liste avant d'afficher le formulaire, la convention veut qu'on utilise l'Action qui appelle le formulaire (avant le forward vers le formulaire, tu peux spécifier les valeurs initiales du formulaire) par des

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
MonFormulaire monForm = (MonFormulaire) form;
form.setXXX(...);

[wazup]

Oui, pour le setter, tu as raison, mais là n'est pas le problème.

(les deux marchent, la différence tient à ce que ton setter ne renvoie qu'une référence sur la collection déjà exxistante, alors que pour d'autres raisons, j'ai besoin d'une recopie en profondeur, maus passons)

Pour le reste, effectivement, tu as raison aussi, quoique je n'ai pas essayé.

C'est effectivement plus conventionnel que de redéfinir le Reset(), mais je trouve que les actions doivent être les plus simples possible, surtout pour faire du populate de formulaire.

En fait, au lieu de décortiquer les paramètres du POST dans l'action, je le fait dans le validatorForm.reset()

Merci!

[RanDomX]

il y a deux choses:

1/ tu utilises mal le tag html:options -> tu devrais regarde html:optionsCollection

2/ ds la méthode reset tu dois systématiquement vider la collection pour qu'elle ait une taille nulle.


@+

[wazup]

Oui, finalement en regardant la doc du <html:options> j'ai vu que le problème venait tout smplement du fait que

j'ai utilisé un ArrayList().

En utilisant un String[], il n'y a plus de problème et effectivement je peux à présent être "propre" en vidant tout dans Reset(), et sans rajouter de boucle dans Action.execute(..).

Par contre, je maintiens que le tag <html:optionsCollection> n'est pas nécessaire ici.