Discussion :

[romaintaz]

Bonjour,

Je dois avouer que je suis un peu bloqué sur un problème de validation, et je ne comprends pas pourquoi...

Dans un formulaire (h:form), j'ai une combobox qui me permet de choisir un profile. Voici son code :

Code :

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

1
2
3
4
<h:selectOneMenu value="#{pmb.profileId}">
    <f:selectItems value="#{pmb.profileList}" />
    <a4j:support event="onchange" reRender="permissions" actionListener="#{pmb.handleProfileSelection}" />
</h:selectOneMenu>

La méthode handleProfileSelection du bean pmb est la suivante :

Code :

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

1
2
3
4
5
6
7
8
9
public void handleProfileSelection(ActionEvent actionEvent) {
    List<Permission> permissionList = habilitationService.findPermissionsByProfileId(getProfileId());
    assignedPermissions = new ArrayList<MySelectItem>();
    if (!permissionList.isEmpty()) {
        for (Permission p : permissionList) {
            assignedPermissions.add(new MySelectItem(new Integer(p.getPermissionId()), p.getPermissionLabel()));
        }
    }
}

Rien de méchant, elle met à jour une liste contenant des éléments MySelectItem, une classe qui étend SelectItem en redéfinissant equals et hashCode (à cause de ça).

Mon listShuttle se définit comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
 
<a4j:outputPanel id="permissions">
    <rich:listShuttle fastOrderControlsVisible="false" id="permissionId"
        converter="selectItemListShuttleConverter"
        copyAllControlLabel="Add all" copyControlLabel="Add"
        removeAllControlLabel="Remove all" removeControlLabel="Remove"
        orderControlsVisible="false" var="item"
        sourceCaptionLabel="Available permissions"
        targetCaptionLabel="Assigned permissions"
        sourceValue="#{pmb.selectedPermissions}"
        targetValue="#{pmb.assignedPermissions}">
        <rich:column>
            <h:outputText value="#{item.label}"/>
        </rich:column>
    </rich:listShuttle>
</a4j:outputPanel>

Le problème c'est que quand je choisis une valeur dans mon combobox, tout se passe bien, et le listShuttle s'affiche bien. Mais si je change à nouveau une valeur dans le combobox, j'ai une erreur de validation, à savoir :

profileForm:permissionId: Validation Error: Value my.package.MySelectItem@xxxx is not valid
Pour information le code du converter :

Code :

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

1
2
3
4
5
6
7
8
9
public Object getAsObject(FacesContext fc, UIComponent compo, String val) {
    String[] str = val.split(":");
    return new MySelectItem(str[0], str[1]);
}
 
public String getAsString(FacesContext fc, UIComponent compo, Object val) {
    MySelectItem data = (MySelectItem) val;
    return data.getValue() + ":" + data.getLabel();
}

Franchement, je ne comprends pas d'où vient l'erreur...
Une idée ?

ps: La version de RichFaces utilisée est la 3.1.4.GA

[maVariable]

Salut,

étant donné ton niveau, je pense pas trop pouvoir t'aider, mais on sait jamais :
Il me semble que l'ajout d'élément marche, mais peut etre que ton converter n'arrive pas à se mettre à jour en même temps, non ? enfin, je veux dire qu'il ne traite peut etre pas les bonnes valeurs...
Tu pourrais peut etre afficher (dans la console, ou en mode debug) les valeurs renvoyées par ton converter, histoire de vérifier ce qu'il fait ?

[romaintaz]

Merci pour la flaterie, mais j'avoue que j'ai l'impression que je passe juste à côté d'un truc super bête...

J'ai mis plein de logs, et le converter fait bien son boulot... D'où mon problème. Je ne vois vraiment pas où est-ce qu'il y a une erreur !

[OButterlin]

Pourrais-tu montrer le code de MySelectItem ?

[romaintaz]

Le voici :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
public class MySelectItem extends SelectItem {
 
    public MySelectItem(Object value, String label) {
        super(value, label, null, false, true);
    }
 
    public String toString() {
        return "MySI(" + super.getValue() + " ; " + super.getLabel() + ")";
    }
 
    public boolean equals(Object obj) {
        if (!(obj instanceof MySelectItem)) {
            return false;
        }
        MySelectItem si = (MySelectItem) obj;
        return (this.getValue().equals(si.getValue()));
    }
 
    public int hashCode() {
        return this.getValue().hashCode();
    }
 
}

[OButterlin]

Si la valeur est nulle, ça peut poser problème non ?

[romaintaz]

Ca pourrait, mais ça n'est jamais le cas...
Au pire j'aurais un NPE ce qui me serait plus facile à gérer

[romaintaz]

Je pensais avoir une piste, parce que dans la méthode handleProfileSelection, la clé était un Integer, tandis que le converter le transformait en String...
Mais même avec ce code, ça ne change rien :

Code :

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

1
2
3
4
5
6
7
8
9
public void handleProfileSelection(ActionEvent actionEvent) {
    List<Permission> permissionList = habilitationService.findPermissionsByProfileId(getProfileId());
    assignedPermissions = new ArrayList<MySelectItem>();
    if (!permissionList.isEmpty()) {
        for (Permission p : permissionList) {
            assignedPermissions.add(new MySelectItem(String.valueOf(p.getPermissionId()), p.getPermissionLabel()));
        }
    }
}

D'ailleurs, ce que je trouve bizarre c'est que le MySelectItem voit sa méthode hashCode appelée, mais jamais son equals...

[OButterlin]

J'aurais plutôt surchargé getValue() pour être sûr d'éviter le NullPointerException dans hashCode...
Je ne suis pas certains que l'exception remonte d'ailleurs, donc, le fait de ne pas l'avoir détectée ne veut pas dire qu'elle n'est pas intervenue.

C'est une idée, c'est tout...

[romaintaz]

Je prends toutes les idées, mais celle là ne semble pas m'aider, hélas
Mais qu'y a t'il de mauvais dans ce code ??

[OButterlin]

Je ne vois rien de mauvais dans ce que tu as codé, je pensais juste à une valeur "parasite" éventuellement injectée dans la liste...
(c'est tiré par les cheveux... mais comme l'erreur est liée à la valeur...)

Question en passant :

tu utilises quelle bibliothèque de tags ?

[OButterlin]

J'ai trouvé un problème similaire au tien sur un autre site...

Le gars propose de modifier le type de colonne de Integer -> String et d'enregistrer la liste en managed-property

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
The class I was sending to the new SelectItem(myClass,myLabel) had an Integer as the projectID. I changed that to a String. Tossed out my custom converter, tossed out my validator. Made sure myClass extends SelectItem
 
public class ProjectsListItem extends javax.faces.model.SelectItem implements java.io.Serializable {
 
then I added <managed-property tag to the managed bean 
 
    <managed-bean>
        <managed-bean-name>loginModel</managed-bean-name>
        <managed-bean-class>xxxxxxxx.LoginModel</managed-bean-class>
        <managed-bean-scope>session</managed-bean-scope>
        <managed-property>projectsList</managed-property>
    </managed-bean>
 
and it works. No validation error. I think the response was sending a String and decode() for the component was trying to find/convert its value and I was getting a validation error because my list of SelectItems (projectsList) contained a class that had an Integer projectID rather than a String.

Je te laisse essayer...
A+

[romaintaz]

Merci OButterlin pour cette idée, mais disposes tu encore du lien où tu as trouvé ces informations ?

[OButterlin]

No problemo (comme disait l'autre )

http://saloon.javaranch.com/cgi-bin/...&f=82&t=001126

A+

[romaintaz]

Merci beaucoup OButterlin pour ton aide, mais le problème était tout bête en fait (comme je m'en doutais)...

Le bean pmb était défini en scope request, et je l'ai passé en scope session. Cela a permis de résoudre le problème !!!

[OButterlin]

cool, l'essentiel c'est que tu ais règlé ton problème...

A+

[chahir.ammari]

Je ne vois rien de mauvais dans ce que tu as codé, je pensais juste à une valeur "parasite" éventuellement injectée dans la liste...
(c'est tiré par les cheveux... mais comme l'erreur est liée à la valeur...)

Question en passant :

tu utilises quelle bibliothèque de tags ?