Project

General

Profile

Actions

Bug #1366

closed

View Generator - Pagination : Erreur de l'affichage du numéro de la page visualisée

Added by Philippe Bazart about 11 years ago. Updated almost 11 years ago.

Status:
Closed
Priority:
Normal
Category:
ViewGenerator
Start date:
12/06/2010
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:
Navigateur:
Tous
Votre version de Silverpeas:
5.4
Système d'exploitation:
Votre base de données:
Toutes
Livraison en TEST:
Livraison en PROD:

Description

Quand on arrive sur un affichage multipage, utilisant la pagination et que la page à afficher est la dernière. Le numéro de la page affichée dans la pagination peut être erroné.

En effet, le numéro de la page courante est calculé à l'aide du numéro du premier élément qui la compose. Si la page courante est la dernière page et que le nombre total d'éléments à afficher n'est pas un multiple du nombre d'éléments à afficher par page, le premier élément affiché dans la dernière page appartient en fait à l'avant dernière page, c'est donc l'avant dernière page qui va être prise en compte dans l'affichage de la pagination.

Par exemple, s'il existe 14 lignes à afficher, avec un affichage de 10 éléments par page.
Quand la dernière page est affichée, elle contiend les 10 derniers éléments : les éléments 5 à 14.
Comme le calcul de la page courante est réalisé en tenant compte du 1er élément qui la compose, dans cet exemple le 5ème élément, la pagination affiche que la page courante est la page 1, page d'appartenance du 5ème élément, alors que c'est la page 2 qui est affichée.

Pour reproduire le bug : Avec 14 lignes à afficher et un affichage de 10 éléments par page, aller sur la dernière page, changer d'écran puis revenir sur la page.

Evaluation de la correction : calculer le numéro de la page courante avec le dernier élément qui la compose (méthode getLastItemIndex())
Package : com.stratelia.webactiv.util.viewGenerator.html.pagination
Classe : AbstractPagination.java
Methode : getCurrentPage()

Actions #1

Updated by Emmanuel Hugonnet about 11 years ago

  • Category set to ViewGenerator
  • Status changed from New to In progress...
  • Assignee set to Emmanuel Hugonnet
  • Target version set to Version 5.4
Actions #2

Updated by Nicolas Eysseric about 11 years ago

  • Target version changed from Version 5.4 to Version 5.5
Actions #3

Updated by Emmanuel Hugonnet almost 11 years ago

  • % Done changed from 0 to 100

Le problème ne concernait pas la pagination en tant que telle mais la gestion de la pagination dans les ArrayPanes.
Mise à jour du code pur une gestion plus 'propre' et plus simple des arraypanes

Actions #4

Updated by Emmanuel Hugonnet almost 11 years ago

  • Status changed from In progress... to Closed
Actions

Also available in: Atom PDF