[Struts-Layout] probleme avec pager et sort

Version imprimable

Voir 40 message(s) de cette discussion en une page

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 ! ^^

03/08/2007, 14h14
c_nvy

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 ?
03/08/2007, 14h19
Kendael

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).
03/08/2007, 14h35
c_nvy

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...

03/08/2007, 15h03
c_nvy

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 ?
03/08/2007, 15h09
Kendael

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)
03/08/2007, 15h43
c_nvy

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.;)
03/08/2007, 15h53
Kendael
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...
03/08/2007, 15h59
c_nvy

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...

03/08/2007, 16h34
c_nvy

Pour les boutons radio, tu devrais faire un test en utilisant les attributs selectName, selectProperty et selectType du tag layout:collection.;)
03/08/2007, 16h39
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 ;).
La je vais tester la dernière nightly-build, apparemment c'est un bug qui a été fixé depuis.
03/08/2007, 16h43
c_nvy

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.;)
03/08/2007, 17h01
Kendael

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...
04/08/2007, 15h35
c_nvy

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.;)
04/08/2007, 18h45
Kendael

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>

06/08/2007, 13h24
Kendael
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.
06/08/2007, 16h14
c_nvy

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"/>

Voir 40 message(s) de cette discussion en une page