Rafraîchissement combobox suite à un onchange="submit()"

Version imprimable

Bonjour à tous,

A chaque changement de valeur d'une liste déroulante, je souhaite recharger la page. Pour cela j'utilise l'attribut onchange avec la fonction submit() :
Code:

1 2 3 <h:selectOneMenu value="#{controller.object.id}" onchange="submit();" valueChangeListener="#{controller.updateValue}"> <f:selectItems value="#{controller.itemList}"/> </h:selectOneMenu>
Le problème est qu'aucune combobox ne prend la valeur du modèle (rafraîchissement?). J'ai fait un test en introduisant un bouton :

Code:

<h:commandButton action="controller.updateValue2"/>

Et là, toutes les combos prennent les bonnes valeurs. Le rafraîchissement de la page suite à une action sur un bouton a l'air différent d'un rafraîchissement provoqué par un submit javascript. Comment palier à ce problème?
Comment forcer le rafraîchissement d'une liste déroulante?

J'utilise JSF 1.2 sans framework complémentaire (ex: pas de richfaces).

Merci

03/08/2010, 10h27
Sniper37

Essaye:

Code:

onchange="this.form.submit();"
03/08/2010, 13h53
scorplayer

J'ai essayé et ça ne change rien :calim2:
03/08/2010, 14h20
Sniper37

tu peux montrer le code de la page et de la fonction listener

Le composant :

Code:

1
2
3
<h:selectOneMenu value="#{controller.schedule.id}" onchange="submit();" valueChangeListener="#{controller.updateValue}">
      <f:selectItems value="#{controller.scheduleList}"/>
</h:selectOneMenu>

La méthode updateValue :

Code:

1
2
3
4
5
6
public void updateValue(ValueChangeEvent valueChangeEvent) {
        System.out.println(" --- updateValue --- ");
        if (valueChangeEvent.getNewValue() != null) {
            deduceInformation(new Integer(valueChangeEvent.getNewValue().toString()));
        }
    }

La méthode deduceInformation :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public void deduceInformation (int scheduleId) {
        Office office = null;
        schedule = scheduleService.get(scheduleId);
        if (schedule.getItineraryList() != null && schedule.getItineraryList().size() > 0) {
            for (Itinerary eachItinerary : schedule.getItineraryList()) {
                    office = eachItinerary.getOffice();
                    if (eachItinerary.getFirstOffice() == 1) {
                        firstOffice = new Office(office.getId());
                    } else {
                        getOfficeList().add(office);
                    }
 
            }
        }
        // Refresh table
        dataModel = null;
    }

La combo box qui ne se rafraîchit pas est bindé à l'attribut firstOffice.

Merci Sniper37

05/08/2010, 15h55
scorplayer

Sniper37, tu vois d'où peut venir le problème? Ou pas d'idée?

Merci
06/08/2010, 11h09
DevServlet

Essaie de faire un peu un truc tout con , mets un

Code:

onchange="alert(\'bonjour\')"

pour voir si le js est bien interpreté à cet endroit. ensuite si ca marche essaie ca

Code:

document.forms[0].submit()
06/08/2010, 15h28
scorplayer

Merci pour ta réponse DevServlet.

Le "alert()" marche.
document.forms[0].submit(), this.form.submit() et submit() ont le même résultat : les combobox ne se rafraîchissent pas.

Est-il possible de provoquer la mise à jour d'une liste déroulante en javascript ou autre?
09/08/2010, 10h07
Sniper37
Citation:

Envoyé par scorplayer

Merci pour ta réponse DevServlet.

Le "alert()" marche.
document.forms[0].submit(), this.form.submit() et submit() ont le même résultat : les combobox ne se rafraîchissent pas.

Est-il possible de provoquer la mise à jour d'une liste déroulante en javascript ou autre?
- le submit va surement rafraichir toute la page pas que la liste déroulante, tu peux simuler le submit avec la touche F5.
- vérifie si tu n'a pas d'erreurs javascript, d'erreur de conversion/validation avec h:messages.
- tu peux aussi faire un outputText avec la valeur de la liste, voir à quoi elle correspond.
- Ton bean doit être en scope session sinon ses valeurs seront perdues à chaque requete.
09/08/2010, 11h09
scorplayer

Citation:

le submit va surement rafraichir toute la page pas que la liste déroulante

On est d'accord.

Citation:

Ton bean doit être en scope session

Mon Bean est bien en session.

Citation:

vérifie si tu n'a pas d'erreurs javascript, d'erreur de conversion/validation

Je n'ai pas d'erreurs de conversion/validation, je vais voir pour le javascript.

Citation:

tu peux aussi faire un outputText avec la valeur de la liste, voir à quoi elle correspond

Je vais tester ça tout de suite.
09/08/2010, 12h10
scorplayer

Oula, je ne comprend pas tout! D'après la trace que j'ai mis dans le getter de firstOffice (variable bindé à la combo), firstOffice est bien modifiée lors de son passage dans la méthode deduceInformation() mais quelque chose la remet à son ancienne valeur juste derrière!
09/08/2010, 12h13
DevServlet

Code:

Est-il possible de provoquer la mise à jour d'une liste déroulante en javascript ou autre?

Pour d'autres composants oui tout à fait, mais les listes j'ai jamais essayé, si le but de ton submit c juste rafraichir tes combo, alors essaies de générer ton code source de la page et aller dedans voir la structure DOM de l'objetcombo box généré, et essaie de le modifier avec ton javascript, c'est ce que je faisais dans certains projets et ca marchait bien.
09/08/2010, 14h22
Sniper37

tu peux poster le vrai code, qui ne fonctionne pas, celui que tu as donné ne concerne pas la valeur que tu change: firstOffice.

Je te redonne (mon 3ème post) le code de la méthode deduceInformation() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public void deduceInformation (int scheduleId) {
        Office office = null;
        schedule = scheduleService.get(scheduleId);
        if (schedule.getItineraryList() != null && schedule.getItineraryList().size() > 0) {
            for (Itinerary eachItinerary : schedule.getItineraryList()) {
                    office = eachItinerary.getOffice();
                    if (eachItinerary.getFirstOffice() == 1) {
                        firstOffice = new Office(office.getId());
                    } else {
                        getOfficeList().add(office);
                    }
 
            }
        }
        // Refresh table
        dataModel = null;
    }

firstOffice intervient à la ligne 8.

09/08/2010, 14h50
Sniper37

la valeur du select ne peux ps prendre un objet sans lui associer un converter, si la valeur est un id, dans ta fonction tu modifie un objet.
09/08/2010, 15h26
scorplayer
Le composant lié à firstOffice :
Code:

1 2 3 4 <h:outputText value="#{properties.FIRST_OFFICE}"/><br/> <h:selectOneMenu value="#{controller.firstOffice.id}"> <f:selectItems value="#{controller.officeList}"/> </h:selectOneMenu>
Je suis donc bindé à l'Id de l'objet (pas besoin de passer par un converter dans ce cas?). Et dans la méthode deduceInformation(), je ne modifie que l'Id aussi. L'objet est vide, seul l'Id est manipulé. C'est correct?
C'est vrai que j'aurais pu me binder directement sur un int sans passer par l'objet. Mais là, au moins, si j'ai besoin de l'objet complet, j'ai juste à le récupérer en base grâce à son Id.
09/08/2010, 15h32
Sniper37

ton problème vient surement de là, utilise un simple Integer en value de ton selectOneMenu et à priori tout rentrera dans l'ordre :P
09/08/2010, 15h46
scorplayer

Comment expliquer que la même chose (sans javascript) marchait avec un commandButton alors?

Je teste avec un Integer.
09/08/2010, 16h27
scorplayer

Ça ne marche pas :calim2:

Par contre peut-être une piste :
J'ai enlevé le setter de la valiable et le outputText du test m'indique que le modèle est à la bonne valeur. Conclusion : le modèle est à la bonne valeur avant le rafraîchissement de la page; au submit du formulaire, la combo valide son contenu (pas la valeur du modèle mais l'ancienne) et écrase la bonne valeur du modèle.
09/08/2010, 16h49
Sniper37

tu peux redonner le dernier code partie JSF et partie java..

Composant lançant le rafraîchissement de la page :

Code:

1
2
3
<h:selectOneMenu value="#{controller.schedule.id}" onchange="submit();" valueChangeListener="#{controller.updateValue}">
      <f:selectItems value="#{controller.scheduleList}"/>
</h:selectOneMenu>

Méthode updateValue() :

Code:

1
2
3
4
5
6
public void updateValue(ValueChangeEvent valueChangeEvent) {
        System.out.println(" --- updateValue --- ");
        if (valueChangeEvent.getNewValue() != null) {
            deduceInformation(new Integer(valueChangeEvent.getNewValue().toString()));
        }
    }

Méthode deduceInformation() lancé par updateValue() :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
public void deduceInformation (int scheduleId) {
        Office office = null;
        schedule = scheduleService.get(scheduleId);
        if (schedule.getItineraryList() != null && schedule.getItineraryList().size() > 0) {
            for (Itinerary eachItinerary : schedule.getItineraryList()) {
                    office = eachItinerary.getOffice();
                    if (eachItinerary.getFirstOffice() == 1) {
                        firstOfficeId = office.getId();
                    } else {
                        getOfficeList().add(office);
                    }
 
            }
        }
        // Refresh table
        dataModel = null;
    }

Composant impacté par la méthode deduceInformation() :

Code:

1
2
3
4
<h:outputText value="#{properties.FIRST_OFFICE}"/><br/>
<h:selectOneMenu value="#{controller.firstOfficeId}">
     <f:selectItems value="#{controller.officeList}"/>
</h:selectOneMenu>

Ce dernier composant est maintenant bindé à un Integer.

10/08/2010, 09h02
Sniper37

tu n'a pas un immediate qui traine quelque part??
Sinon ajoute:

Code:

FacesContext.getCurrentInstance().renderResponse();

à la fin de la méthode du listener.
10/08/2010, 09h20
scorplayer

Code:

FacesContext.getCurrentInstance().renderResponse();

Ce code est l'équivalent de l'attribut immediate="true"? J'avais déjà essayé avec.
J'ai essayé ton code sans succès.
10/08/2010, 09h55
Sniper37

Citation:

Envoyé par scorplayer

Code:

FacesContext.getCurrentInstance().renderResponse();

Ce code est l'équivalent de l'attribut immediate="true"? J'avais déjà essayé avec.
J'ai essayé ton code sans succès.

le renderResponse pour forcer le passage par la phase Update Model lorsque le composant est avec immediate true, dans ton cas elle ne sert à rien.

Sinon ajoute un bindind à ton selectOneMenu et dans le listener tu modifie la valeur directyement dans le binding:

Code:

htmlSelectOneMenu.setValue(newValue);

J'ai testé et ça marche ;)
10/08/2010, 10h35
scorplayer

Je n'ai jamais utilisé l'attribut binding. Quel est la différence entre les attributs binding et value? Peut-on les utiliser tous les 2 en même temps? Quel type de donnée doit-on avoir dans binding?
10/08/2010, 10h41
Sniper37

le binding permet de contrôler le composant en java, il sert généralement à générer le composant dans le backing bean. il est conseillé de l'utiliser dans un bean en scope request.
le type de la valeur doit être HtmlSelectOneMenu pour le selectItem.
10/08/2010, 11h27
scorplayer

Peut-on utiliser l'attribut binding avec l'attribut value? Ou c'est soit l'un, soit l'autre?
10/08/2010, 11h34
Sniper37

Citation:

Envoyé par scorplayer

Peut-on utiliser l'attribut binding avec l'attribut value? Ou c'est soit l'un, soit l'autre?

seul le binding est nécessaire.

Un grand merci pour ta patience et tes explications Sniper37 :ccool:

J'utilise les 2 attributs finalement :
Code:

1 2 3 4 <h:outputText value="#{properties.FIRST_OFFICE}"/><br/> <h:selectOneMenu value="#{controller.firstOfficeId}" binding="#{controller.firstOfficeSelectOneMenu}"> <f:selectItems value="#{controller.officeList}"/> </h:selectOneMenu>