[Struts-Layout] probleme avec pager et sort

Bonjour, j'ai un petit souci avec le pager de struts-layout, en effet, quand je change de page, cela fonctionne correctement sur la premiere page, le tri en tout cas, mais lorsque je change de page, seule la premiere colonne est mise a jour, les données des autres colonnes ne bougent pas...
voici le code de ma jsp :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

<layout:form action="processTabList.do" styleClass="FORM" key="Main form" > <layout:pager maxPageItems="10" > <layout:collection property="depList" styleClass="FORM" align="center" width="100%" indexId="index" > <layout:collectionItem title="Code" property="code" sortable="true"/> <layout:collectionInput title="Libellé" property="libelle" formProperty="libelle" sortable="true" /> <layout:collectionItem title="Etat"> <layout:radios layout="false" property="depList[${index}].etat" cols="3"> <layout:option value="etat1" key="Etat1" /> <layout:option value="etat2" key="Etat2" /> <layout:option value="etat3" key="Etat3" /> </layout:radios> </layout:collectionItem> </layout:collection> <layout:row> <layout:pagerStatus key="pager.status.pager" /> <layout:pagerStatus key="pager.status.items"/> </layout:row> </layout:pager> </layout:form>

---

Un petit coup de mains ne serait pas de refus ! ^^

As-tu fait le test du tri côté client en ajoutant l'attribut sortAction="client" dans le tag layout:collection, pour voir si ça donne le même résultat ?

A moins que tu veuilles absolument faire le tri côté serveur ?

En fait quand j'utilise le pager conjointement avec la fonction de tri, cela force le tri à se faire coté server. du coup le probleme ne change pas. de plus le tri coté client fonctionne mal sur firefox, le tri se faisant toujours sur la même colonne (meme si mon appli doit avant tout etre compatible ie6 et 7).

Effectivement, tu as raison, le tri se fait côté serveur lorsqu'on utilise le tag layout:pager.
J'en avais même déjà débattu dans cette discussion. J'avais oublié.:oops:

Utilises-tu l'Action de tri SortAction de Struts-Layout ou une Action de tri perso ?

J'utilise l'action de tri SortAction, qui est bien déclarée dans mon struts-config.xml, ainsi que PagerAction

Code:

---

1 2 3 4 5 6 7 8 9 10

<action path="/sort" type="fr.improve.struts.taglib.layout.sort.SortAction" scope="request" validate="false" /> <action path="/page" type="fr.improve.struts.taglib.layout.sort.PageAction" scope="request" validate="false" />

---

Je ne comprends pas...

Apparemment, tu utilises une ancienne version de Struts-Layout parce que l'Action PageAction n'existe plus dans la version 1.2 et on ne doit donc plus la déclarer dans le struts-config.xml.

Il serait peut-être bon de télécharger la 1.2.
Sinon, quelle version de Struts utilises-tu ?

j'utilise la version 1.2 de struts-layout (d'ailleurs j'ai retiré la déclaration de PageAction, et ca n'a rien changé) avec la version 1.3.8 de struts (que je suis obligé d'utiliser)

Citation:

---

Envoyé par Kendael

Bonjour, j'ai un petit souci avec le pager de struts-layout, en effet, quand je change de page, cela fonctionne correctement sur la premiere page, le tri en tout cas, mais lorsque je change de page, seule la premiere colonne est mise a jour, les données des autres colonnes ne bougent pas...

---

Aprsè relecture de ton message, je ne suis plus certaine d'avoir bien compris ce qui ne fonctionne pas.:oops:
Pourrais-tu être plus précis ?

Parce que normalement, même si tu utilises le layout:pager, le tri s'effectue sur toutes les occurences du champ.
Donc, si tu es sur la page 2 et que tu cliques sur le tri, le tri s'effectue et tu te retrouves automatiquement sur la page 1.
En tout cas, c'est comme cela que ça fonctionne chez moi.;)

En fait j'ai un tableau a 3 colonnes.
Sur la première page j'ai par exemple :

Code:

---

1 2 3 4

col1 || col2 || col3 1 1a 1b 2 2a 2b 3 3a 3b

---

si je passe à la page 2 j'obtiens :

Code:

---

1 2 3 4

col1 || col2 || col3 4 1a 1b 5 2a 2b 6 3a 3b

---

Seules les valeurs de la première colonnes sont "mises à jour" lors du changement de page. Les autres correspondent toujours aux valeurs de la première page... Le problème subsiste si j'enlève la fonction de tri...

Merci, c'est plus clair pour moi comme ça.;)

Effectivement, ce n'est pas vraiment normal ce qui t'arrive.
Dis-moi, ta collection contient-elle beaucoup de lignes ?

Ma collection contient 50 lignes, c'est pour tester la fonctionnalité. Je pense que le problème vient du fait que l'index

Code:

---

<layout:collection property="depList" styleClass="FORM" align="center" width="100%" indexId="index" >

---

qui est utilisé pour afficher les valeurs des 2 dernières colonnes :

Code:

---

1 2 3 4 5 6 7 8

<layout:collectionItem title="Libellé" property="libelle" sortable="true" /> <layout:collectionItem title="Etat"> <layout:radios layout="false" property="depList[${index}].etat" cols="3"> <layout:option value="etat1" key="Etat1" /> <layout:option value="etat2" key="Etat2" /> <layout:option value="etat3" key="Etat3" /> </layout:radios> </layout:collectionItem>

---

ne doit plus être correcte à cause de la pagination, en gros il reste entre 0 et maxPageItems...

Pour les boutons radio, tu devrais faire un test en utilisant les attributs selectName, selectProperty et selectType du tag layout:collection.;)

Ces attributs ne me servent pas, puisqu'ils servent à la sélection de lignes dans la collection, mes radios me permettent juste de switcher un attribut. Par exemple pour imaginons une liste de portes, je renseigne si elle est "ouverte", "fermée" ou "entrebaillée". Rien avoir donc avec les attributs de layout:collection, mais merci bcp de creuser avec moi ;).
La je vais tester la dernière nightly-build, apparemment c'est un bug qui a été fixé depuis.

Citation:

---

Envoyé par Kendael

Ces attributs ne me servent pas, puisqu'ils servent à la sélection de lignes dans la collection, mes radios me permettent juste de switcher un attribut. Par exemple pour imaginons une liste de portes, je renseigne si elle est "ouverte", "fermée" ou "entrebaillée". Rien avoir donc avec les attributs de layout:collection, mais merci bcp de creuser avec moi .

---

Ok, autant pour moi.:oops:

Citation:

---

Envoyé par Kendael

La je vais tester la dernière nightly-build, apparemment c'est un bug qui a été fixé depuis.

---

C'est une bonne idée.
Tiens-nous au courant.;)

Bon eh bien la solution de la nightly-build n'a pas fonctionné, je reste toujours dans l'incapacité de faire fonctionner correctement le pager avec des collection:input ou tout autre élément qui nécessite un index de bean courant...

En fait, j'ai fait un test et j'ai exactement le même problème que toi.;)

Après recherche, dans la page TODO de Struts-Layout, j'ai trouvé ceci :

Citation:

---

pager: allow editable collections

---

Ceci est donc prévu dans une prochaine release.

Il va donc falloir attendre pour que cela fonctionne.;)

aïe ! bon eh bien en tout cas merci bcp pour ton aide ! je vais chercher une autre solution...

En fait, il y a une solution en calculant la valeur de l'index en fonction de la page sélectionnée et du nombre maximum d'éléments affichés par page (soit 10 dans ton cas).

Il faut donc pour que cela fonctionne récupérer la valeur du paramètre pagerPage de l'Action de path /sort, si il existe.

Voici donc une solution avec la JSTL :

:arrow: Récupérer la valeur du paramètre pagerPage et mettre sa valeur dans une variable pagerPage. S'il n'existe pas, lui affecter la valeur 0 :

Code:

---

1 2 3 4 5 6 7 8

<c:choose> <c:when test="${param['pagerPage']==null}"> <c:set var="pagerPage" value="0"/> </c:when> <c:otherwise> <c:set var="pagerPage" value="${param['pagerPage']}"/> </c:otherwise> </c:choose>

---

:arrow: Calculer la valeur de l'index et stocker le résultat dans une variable newIndex :

Code:

---

1 2 3 4 5 6 7 8

<c:choose> <c:when test="${pagerPage==0}"> <c:set var="newIndex" value="${index}"/> </c:when> <c:otherwise> <c:set var="newIndex" value="${index+(10*pagerPage)}"/> </c:otherwise> </c:choose>

---

:arrow: Utiliser la nouvelle variable d'index en remplaçant le collectionInput sur le libelle par ceci :

Code:

---

1 2 3

<layout:collectionItem title="Libelle"> <layout:text property="depList[${newIndex}].libelle" layout="false" /> </layout:collectionItem>

---

:arrow: Et enfin, utiliser la nouvelle variable d'index pour les boutons radio :

Code:

---

1 2 3 4 5 6 7

<layout:collectionItem title="Etat"> <layout:radios layout="false" property="depList[${newIndex}].etat" cols="3"> <layout:option value="etat1" key="Etat1" /> <layout:option value="etat2" key="Etat2" /> <layout:option value="etat3" key="Etat3" /> </layout:radios> </layout:collectionItem>

---

magnifique ! merci beaucoup ! cela fonctionne très bien. en revanche le tri ne peut plus se faire sur l'objet

Code:

---

1 2 3

<layout:collectionItem title="Libelle" > <layout:text property="depList[${newIndex}].libelle" layout="false" /> </layout:collectionItem>

---

puisque celui-ci ne renseigne plus d'attribut "property"... mais merci beaucoup en tout cas, cela résoud une grande partie du problème.

Effectivement, j'avais oublié que tu souhaitais pouvoir faire un tri sur le libelle.:oops:

Mais je viens de m'apercevoir qu'il existait un attribut formIndex dans le tag collectionInput qui permet de spécifier le nom de la variable d'index.

Tu peux donc réutiliser le tag collectionInput en spécifiant la nouvelle variable d'index newIndex dans l'attribut formIndex :

Code:

---

<layout:collectionInput title="Libellé" property="libelle" formProperty="libelle" formIndex="newIndex" sortable="true"/>

---

Comme on dit : "pas de nouvelles, bonnes nouvelles".;)

Le problème est-il complétement résolu ?
Si oui, pense à mettre le tag :resolu:.

en effet excuse-moi, en fait j'ai testé ta derniere ligne de code juste avant de partir hier, j'avais donc oublié de répondre.
donc en effet cela fonctionne tres bien, merci beaucoup pour le service !

De rien.;)
Ravie d'avoir trouvé une solution à ton problème.:)