Pb selectOneMenu dans dataTable

Version imprimable

Bonjour,

Je développe depuis peu une nouvelle application avec JSF 1.2 et richFaces 3.3.0GA.

En suivant la 3ème méthode de la FAQ, j'arrive bien à MAJ chacune de mes lignes en utilisant un setPropertyActionListener.
Par contre, je me trouve devant une impasse lorsque j'utilise un selectOneMenu dans une de mes colonnes. :evilred: :evilred:
En effet, seule la dernière ligne de ma table prends en compte la nouvelle valeur alors que les autres gardent l'ancienne lors de la MAJ. 8O

Existe-t-il une bonne pratique que je n'ai pas mise en œuvre ?
Es-ce un problème connu ? Quels solutions adopter ?

Ma page xhtml:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
<rich:dataTable id="tableMessage" cellpadding="0" cellspacing="0" 
                    width="700" border="0" var="mess" value="#{messageBean.messages}"
                    rowClasses="fondBlanc,fondCouleur">
 
	    			<f:facet name="header">
                        <rich:columnGroup styleClass="fondCouleur">
                            <rich:column breakBefore="true"><h:outputText value="Message" /></rich:column>
                            <rich:column width="1%"><h:outputText value="Cible" /></rich:column>
                            <rich:column width="1%"><rich:spacer width="17" height="10"/></rich:column>
                        </rich:columnGroup>
                    </f:facet>
 
	    			<rich:column colspan="1">
	    				<h:inputText styleClass="sansBord" value="#{mess.corpsMessage}"/>
	    			</rich:column>
 
	    			<rich:column colspan="1">
	    				<h:selectOneMenu id="selectTypeId" styleClass="sansBord" value="#{mess.typeUtilisateur.idTypeUtilisateur}" >
	    					<f:selectItems value="#{messageBean.tousLesTypesUtilisateurs}"/>
	    				</h:selectOneMenu>
	    			</rich:column>
 
	    			<rich:column colspan="1">
	    				<h:commandLink action="#{messageBean.processSauvMessage}" >
	    					<h:graphicImage value="../images/valider.jpg" width="17" height="17" title="Valider les modifications" />
	    						<f:setPropertyActionListener value="#{mess}" target="#{messageBean.messToSave}" />
	    				</h:commandLink>
	    			</rich:column>
 
	    		</rich:dataTable>

Mon BackingBean:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
public class MessageBBean {
 
	private List<Message> messages;
	private List<SelectItem> tousLesTypesUtilisateurs;
	private Message messToSave;
 
	public MessageBBean() {
	}
 
	public List<Message> getMessages() {
		return messages;
	}
 
	public void setMessages(List<Message> messages) {
		this.messages = messages;
	}
 
	public List<SelectItem> getTousLesTypesUtilisateurs() {
		return tousLesTypesUtilisateurs;
	}
 
	public void setTousLesTypesUtilisateurs(
			List<SelectItem> tousLesTypesUtilisateurs) {
		this.tousLesTypesUtilisateurs = tousLesTypesUtilisateurs;
	}
 
	public Message getMessToSave() {
		return messToSave;
	}
 
	public void setMessToSave(Message messToSave) {
		this.messToSave = messToSave;
	}
 
	public String initMessages(){
		try {
			List<TypeUtilisateur> l = MessageDelegate.getAllTypeUtilisateurs();
			this.tousLesTypesUtilisateurs = new ArrayList<SelectItem>();
			for(TypeUtilisateur t : l){
				this.tousLesTypesUtilisateurs.add(new SelectItem(t.getIdTypeUtilisateur(),t.getLibelleTypeUtilisateur()));
			}
			this.messages = MessageDelegate.getAllMessages();
		} catch (Exception e) {
			e.printStackTrace();
			return "retourAccueil";
		}
		return "h1MessageGestion";
	}
 
	public void processSauvMessage(){
 
		this.messToSave = MessageDelegate.updateMessage(this.messToSave);		
		String rien = initMessages();
		FacesContext.getCurrentInstance().renderResponse();
	}

21/04/2009, 19h12
Sniper37

tu fais où la MAJ?

Bonjour et merci de ton intérêt,

Je fais la MAJ via un commandLink sur chaque ligne. J'utilise un "setPropertyActionListener" pour passer l'objet sélectionne comme dans la méthode 3 de la FAQ. Le tout pointe vers la fonction:
Code:

1 2 3 4 5 6 public void processSauvMessage(){ this.messToSave = MessageDelegate.updateMessage(this.messToSave); String rien = initMessages(); FacesContext.getCurrentInstance().renderResponse(); }
J'attaque mes EJB3 via une classe MessageDelegate à qui je délègue tous les appels JNDI (je suis sur jboss 4.x qui ne supporte pas l'injection @EJB).

Je précise que mon entity Message comprend un objet TypeUtilisateur et que c'est justement celui là qui n'est pas mis à jour. Le problème étant que "messToSave" ne contient pas la valeur modifiée par le "selectOneMenu" lorsque je le reçoit dans ma fonction de MAJ.

Merci d'avance...

22/04/2009, 14h31
wushual1

Personne n'aurait une idée ?
Quelque chose que j'aurais mal fait ou que je n'aurais pas fait ?
22/04/2009, 15h37
Sniper37

Citation:

Envoyé par wushual1

Personne n'aurait une idée ?
Quelque chose que j'aurais mal fait ou que je n'aurais pas fait ?

Je ne comprend pas ton problème..

la valeur du selectOneMenu correspond à chaque ligne du bean, pour faire l'update tu récupère une ligne..
cette valeur ne correspond pas à ce que tu as saisi?
22/04/2009, 16h55
wushual1

Exactement !!

En fait, lorsque je récupère ma ligne et que je regarde la valeur correspondant à ce que doit modifier mon selectOneMenu (en l'occurrence un objet TypeUtilisateur), j'ai l'ancienne valeur.

De plus, je me suis aperçu que seule la dernière ligne de mon tableau récupère la valeur modifiée (celle que je veux effectivement sauvegarder)...

Ajout:

En fait dans chaque ligne de mon tableau, j'ai un objet message:
Long id
String corpsMessage
TypeUtilisateur type
Date dateFinVal

Dans la colonne correspondant au type, j'utilise un selectOneMenu qui est une List<SelectItem> remplie avec (idType.toString(), libType).
23/04/2009, 09h25
wushual1

Je dois mal m'y prendre.

Lorsque vous avez une liste de Produits lProduits qui contiennent tous leur catégorie correspondante:
P1[idProduit,nomProduit,cat1] avec cat1[idCateg,nomCateg]
P2...
P3...

Vous désirez les mettre dans un dataTable (ça c'est bon) mais pour gérer les changements de catégories, vous désirez utiliser un selectOneMenu (on a à dispo une liste des catégories lCategorie(selectItem[idCateg,nomCateg])).

Pouvez-vous me mettre un exemple de code permettant d'afficher la bonne catégorie correspondante à la ligne du produit et côté BackingBean, la fonction d'update Correspondante.

J'ai l'impression d'être passé à côté de quelque chose d'important mais je ne vois pas quoi...
23/04/2009, 09h44
Sniper37

le selectOneMenu ne se positionne pas sur la bonne valeur quand la table est affichée.?
23/04/2009, 09h58
wushual1

Si, lorsque j'arrive sur ma page, la table est initialisée avec les bonnes valeurs.

Là, je change la valeur du typeUtilisateur d'un message du tableau (aucune action effectuée côté BackingBean) puis je clique sur mon commandLink de validation de la ligne.

Côté BackingBean, le message que je reçois a toujours l'ancienne valeur du typeUtilisateur et non, celle que j'ai positionné.

Chose étrange: seule la dernière ligne de ma liste de Message reçoit la valeur que j'ai modifiée (en bref, ça fonctionne uniquement sur le dernière ligne). De plus, ça fonctionne aussi lorsqu'après avoir modifié la dernière ligne, j'assigne à la ligne précédente, la même valeur que la dernière ligne...
23/04/2009, 10h05
Sniper37

dans la méthode dans le bean

récupère la ligne courante comme ça :

Code:

TonBean bean = (TonBean ) FacesContext.getCurrentInstance().getExternalContext().getRequestMap().get("mess");

et regarde la valeur du selectOneMenu si c'est la bonne valeur ou pas..
23/04/2009, 10h14
wushual1

Malheureusement, j'obtiens la même valeur que dans mon messToSave passé via:

Code:

<f:setPropertyActionListener value="#{mess}" target="#{messageBean.messToSave}"

à savoir l'ancienne valeur...

Doit-on normalement associer une action particulière lorsqu'on manipule le selectOneMenu ?
23/04/2009, 10h26
Sniper37

Citation:

Envoyé par wushual1

Malheureusement, j'obtiens la même valeur que dans mon messToSave passé via:

Code:

<f:setPropertyActionListener value="#{mess}" target="#{messageBean.messToSave}"

à savoir l'ancienne valeur...

Doit-on normalement associer une action particulière lorsqu'on manipule le selectOneMenu ?

bizard :roll:

essaye de faire dans le selectOneMenu:

Code:

<a4j:support event="onchange" reRender="table" />
23/04/2009, 10h33
wushual1

Dans ce cas là, juste après la modification de selectOneMenu, il me raffiche la table avec l'ancienne valeur...
23/04/2009, 10h39
Sniper37

Citation:

Envoyé par wushual1

Dans ce cas là, juste après la modification de selectOneMenu, il me raffiche la table avec l'ancienne valeur...

tu as bien la table dans un formulaire..?
normalement ouii d'autres input fonctionne bien...
je teste chez moi..je te tiens au courant ;)
23/04/2009, 10h45
wushual1

Oui, la table est bien dans un formulaire ;)

Je te remercie d'avance de regarder car, je commence à vraiment sécher sur le problème là ;p
23/04/2009, 10h49
Sniper37

Citation:

Envoyé par wushual1

Oui, la table est bien dans un formulaire ;)

Je te remercie d'avance de regarder car, je commence à vraiment sécher sur le problème là ;p

ça fonctionne bien chez moi..

regarde la déclaration de ton bean dans le faces-config

est ce que typeUtilisateur est bien déclaré..

Hum, je n'ai pas de bean typeUtilisateur dans mon faceconfig !!

J'ai juste:
Code:

1 2 3 4 5 <managed-bean> <managed-bean-name>messageBean</managed-bean-name> <managed-bean-class>bean.MessageBBean</managed-bean-class> <managed-bean-scope>session</managed-bean-scope> </managed-bean>
Qu'entends-tu exactement ?

23/04/2009, 11h45
Sniper37

faut l'ajouter et ajoute la property typeUtilisateur avec la valeur #{typeUtilisateur }

Je ne comprends pas bien.
Je dois ajouter un property dans mon messageBean avec la valeur #{typeUtilisateur} tel que:
Code:

1 2 3 4 5 6 7 8 9 10 <managed-bean> <managed-bean-name>messageBean</managed-bean-name> <managed-bean-class>bean.MessageBBean</managed-bean-class> <managed-bean-scope>session</managed-bean-scope> <managed-property> <property-name>typeUtilisateur</property-name> <property-class>com.sigat.entity.TypeUtilisateur</property-class> <value>#{typeUtilisateur }</value> </managed-property> </managed-bean>
Et ajouter dans mon messageBean:
Code:

1 2 3 4 5 6 7 8 9 private TypeUtilisateur typeUtilisateur; public TypeUtilisateur getTypeUtilisateur() { return typeUtilisateur; } public void setTypeUtilisateur(TypeUtilisateur typeUtilisateur) { this.typeUtilisateur = typeUtilisateur; }
Si c'est bien ça, rien ne change dans le comportement de mon tableau...

Edit: Si tu pouvais me montrer ce que tu as fait stp...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<managed-bean>
  <managed-bean-name>messageBean</managed-bean-name>
  <managed-bean-class>bean.MessageBBean</managed-bean-class>
  <managed-bean-scope>session</managed-bean-scope>
  <managed-property>
   <property-name>typeUtilisateur</property-name>
   <value>#{typeUtilisateur }</value>
  </managed-property>
 </managed-bean>
<managed-bean>
  <managed-bean-name>typeUtilisateur</managed-bean-name>
  <managed-bean-class>com....TypeUtilisateur</managed-bean-class>
  <managed-bean-scope>session</managed-bean-scope>
 </managed-bean>

23/04/2009, 14h07
wushual1

Alors si je te suis, je déclare un bean qui pointe vers mon POJO TypeUtilisateur.
J'ajoute une propriété dans mon messageBean du même nom que le bean précédemment créé.
Je suppose que je dois retoucher mon selectOneMenu pour qu'il utilise la propriété typeUtilisateur ?

Je récupère toujours l'ancienne valeur !!

Je ne comprends pas du tout quel est le lien entre ce nouveau bean et le selectOneMenu sur le typeUtilisateur de mon objet métier Message représentant une ligne de mon tableau en fait...

non si je comprend bien la valeur du selectOneMenu peut être déclaré comme ça:
Code:

1 2 3 4 5 6 7 8 MessageBean mess = new MessageBean(); TypeUtilisateur typeUtilisateur = new TypeUtilisateur(); typeUtilisateur.setTypeUtilisateurId(value); mess.setTypeUtilisateur(typeUtilisateur );
JSF doit chercher à donner mettre la valeur dans le bean TypeUtilisateur, donc il faut qu'il soit 'managé' par JSF.

Je te donne cette piste parceque en testant chez moi avec avec une valeur appartenant au bean déclaré, seulement MessageBean ça marchait bien.

23/04/2009, 14h53
wushual1

On pourrait dire que c'est le code sous-jasent qu'il y aurait derrière le selectOneMenu.

Donc, il faut que le typeUtilisateur soit 'managé' par JSF car contrairement aux types communs (String...), JSF ne sais pas où trouver les get/set correspondants...

Bref, si tu pouvais me mettre en vrac tes fichiers de tests que je jette un coup d'oeil (voir s'il me manque quelque chose) sur la solution que tu as faite tourner...

Merci de tes efforts, jvais continuer à chercher et posterai dès que je trouverai une solution ;p
23/04/2009, 14h57
Sniper37

non j'ai testé sur une application
j'ai testé aussi des valeurs de type String...
23/04/2009, 15h10
wushual1

OK
Par contre, je n'ai aucun problème avec des valeurs de type String en effet.
Je vais donc creuser ta piste et voir ce que ça me donne...

Bon, ça y est, j'ai trouvé ma solution.

Voilà ce que je faisais avant:

Code:

1
2
3
<h:selectOneMenu id="selectTypeId" styleClass="sansBord" value="#{mess.typeUtilisateur.idTypeUtilisateur}" >
	    					<f:selectItems value="#{messageBean.tousLesTypesUtilisateurs}"/>
	    				</h:selectOneMenu>

Code:

1
2
3
4
5
List<TypeUtilisateur> l = MessageDelegate.getAllTypeUtilisateurs();
			this.tousLesTypesUtilisateurs = new ArrayList<SelectItem>();
			for(TypeUtilisateur t : l){
				this.tousLesTypesUtilisateurs.add(new SelectItem(t.getIdTypeUtilisateur(),t.getLibelleTypeUtilisateur()));
			}

Et ce que je fais maintenant:

Code:

1
2
3
<h:selectOneMenu id="selectTypeId" styleClass="sansBord" value="#{mess.typeUtilisateur}" converter="javax.faces.TypeUtilisateur">
	    					<f:selectItems value="#{messageBean.tousLesTypesUtilisateurs}"/>
	    				</h:selectOneMenu>

Code:

1
2
3
4
5
List<TypeUtilisateur> l = MessageDelegate.getAllTypeUtilisateurs();
			this.tousLesTypesUtilisateurs = new ArrayList<SelectItem>();
			for(TypeUtilisateur t : l){
				this.tousLesTypesUtilisateurs.add(new SelectItem(t,t.getLibelleTypeUtilisateur()));
			}

J'ai donc du créer un convertisseur et le déclarer dans le fece-config.xml:

Code:

1
2
3
4
<converter>
  <converter-id>javax.faces.TypeUtilisateur</converter-id>
  <converter-class>convertisseur.TypeUtilisateurConverter</converter-class>
 </converter>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
public class TypeUtilisateurConverter implements javax.faces.convert.Converter {
 
	/* (non-Javadoc)
	 * @see javax.faces.convert.Converter#getAsObject(javax.faces.context.FacesContext, javax.faces.component.UIComponent, java.lang.String)
	 */
	public Object getAsObject(FacesContext arg0, UIComponent arg1, String arg2) {
		System.out.println("getAsObject " + arg2);
		TypeUtilisateur t = new TypeUtilisateur();
		String[] params = arg2.split("@");
		t.setIdTypeUtilisateur(Long.parseLong(params[0]));
		t.setLibelleTypeUtilisateur(params[1]);
		return t;
	}
 
	/* (non-Javadoc)
	 * @see javax.faces.convert.Converter#getAsString(javax.faces.context.FacesContext, javax.faces.component.UIComponent, java.lang.Object)
	 */
	public String getAsString(FacesContext arg0, UIComponent arg1, Object arg2) {
		System.out.println("getAsString " + arg2);
		if(arg2 instanceof String){
			return (String) arg2;
		}else{
			return ((TypeUtilisateur)arg2).toString();
		}
	}
 
}

Et ne pas oublier de redéfinir les méthodes HashCode et Equals de votre objet métier (perso seulement sur l'ID):

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
@Override
	public int hashCode() {
		final int prime = 31;
		int result = 1;
		result = prime
				* result
				+ ((idTypeUtilisateur == null) ? 0 : idTypeUtilisateur
						.hashCode());
		return result;
	}
 
	@Override
	public boolean equals(Object obj) {
		if (this == obj)
			return true;
		if (obj == null)
			return false;
		if (getClass() != obj.getClass())
			return false;
		final TypeUtilisateur other = (TypeUtilisateur) obj;
		if (idTypeUtilisateur == null) {
			if (other.idTypeUtilisateur != null)
				return false;
		} else if (!idTypeUtilisateur.equals(other.idTypeUtilisateur))
			return false;
		return true;
	}