[Struts-Layout] probleme avec pager et sort

**Kendael** · 03/08/2007, 15h07

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 :

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

**c_nvy** · 03/08/2007, 15h14

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 ?

**Kendael** · 03/08/2007, 15h19

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

**c_nvy** · 03/08/2007, 15h35

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

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

**Kendael** · 03/08/2007, 15h47

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

Code :

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

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

**c_nvy** · 03/08/2007, 16h03

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 ?

**Kendael** · 03/08/2007, 16h09

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)

**c_nvy** · 03/08/2007, 16h43

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.

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.

**Kendael** · 03/08/2007, 16h53

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

Code :

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

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 :

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

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

**c_nvy** · 03/08/2007, 16h59

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 ?

**Kendael** · 03/08/2007, 17h11

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

Code :

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

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

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

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

**c_nvy** · 03/08/2007, 17h34

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

**Kendael** · 03/08/2007, 17h39

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.

**c_nvy** · 03/08/2007, 17h43

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.

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.

**Kendael** · 03/08/2007, 18h01

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

**c_nvy** · 04/08/2007, 16h35

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 :

pager: allow editable collections

Ceci est donc prévu dans une prochaine release.

Il va donc falloir attendre pour que cela fonctionne.

**Kendael** · 04/08/2007, 19h45

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

**c_nvy** · 06/08/2007, 13h30

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 :

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 :

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

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>

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

Code :

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

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>

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

Code :

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

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

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

Code :

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

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>

**Kendael** · 06/08/2007, 14h24

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

Code :

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

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.

**c_nvy** · 06/08/2007, 17h14

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

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 :

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

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

[Struts-Layout] probleme avec pager et sort

Struts 1 Java

Discussions similaires

Partager

Partager