developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 12 years ago

Closed 12 years ago

#475 closed bug (fixed)

pagelinks : boutons page suivante et dernière page accessibles

Reported by: benjamin Owned by:
Priority: normal Milestone: Jelix 1.0.3
Component: jelix:plugins Version: 1.0
Severity: normal Keywords:
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description (last modified by laurentj)

Bonjour, je ne suis pas complètement sûr de moi mais j'ai un comportement qui me semble bizarre. Dans un template j'utilise le plugin pagelinks. J'ai 20 données à afficher et j'en affiche 5 par pages.

Je devrais donc avoir 4 pages. Ca tombe bien j'obtiens bien 4 pages.

  • Page 1 : envoie sur l'offset 0
  • Page 2 : envoie sur l'offset 5
  • Page 3 : envoie sur l'offset 10
  • Page 4 : envoie sur l'offset 15

En revanche quand je vais sur la 4ème page, les boutons "page suivante" et "dernière page" sont accessibles!?

  • Page suivante : envoie sur l'offset 20
  • Dernière page : offset 15

Ca m'étonnes que l'erreur ne soit apparue à personne avant mais qui sait, c'est peut être vraiment une erreur. Si c'est le cas, la correction semble assez simple :

 // Next link
 if (!empty($displayProperties['next-label'])) {
   echo '<li class="pagelinks-next';
   if ($nextBound '''<''' $itemsTotal) {
   ...
 }

au lieu de :

 // Next link
 if (!empty($displayProperties['next-label'])) {
   echo '<li class="pagelinks-next';
   if ($nextBound '''<=''' $itemsTotal) {
   ...
 }

et même chose pour la dernière page

Change History (4)

comment:1 Changed 12 years ago by laurentj

  • Description modified (diff)
  • Priority changed from high to normal
  • Severity changed from major to normal

comment:2 Changed 12 years ago by laurentj

  • Description modified (diff)

comment:3 Changed 12 years ago by laurentj

  • Component changed from jelix to jelix plugins

comment:4 Changed 12 years ago by laurentj

  • Milestone set to Jelix 1.0.3
  • Resolution set to fixed
  • Status changed from new to closed

Le plugin n'a pas été suffisement testé. Merci pour cette correction :-) J'ai ajouté aussi des tests unitaires. Corrigé dans le trunk et la branche 1.0.x

Note: See TracTickets for help on using tickets.