zestedesavoir / zds-site

Cœur du projet technique de Zeste de Savoir
https://zestedesavoir.com
Other
268 stars 161 forks source link

Pagination : Page avec uniquement des messages masqués #6280

Open FougereBle opened 2 years ago

FougereBle commented 2 years ago

Description du bug

Si une page sur les forums ne contient que des messages masqués, on se retrouve avec une page "vide" et inutile.

Exemple juste ici :

https://beta.zestedesavoir.com/forums/sujet/6900/comite-de-redaction-de-contenu-sur-c/?page=2

Comment reproduire ?

La liste des étapes qui permet de reproduire le bug :

  1. Allez sur le site
  2. Ajouter assez de messages pour avoir une autre page
  3. Masquer tous les messages

Comportement attendu

Il ne devrait pas y avoir une page entière consacrée à des messages masqués.

Capture d'écran

Capture d’écran 2022-03-31 à 21 04 28

Informations techniques

Arnaud-D commented 2 years ago

J'ai réfléchi aux solutions possibles et je pense qu'une piste serait de spécialiser notre vue qui fait la pagination sur le site (messages du forum, topics, commits dans les publications, etc.). En particulier, cette vue spécialisée utiliserait un Paginator custom, où on réécrirait cette fonction : Paginator.page.

Par contre, je me questionne sur de possibles impacts sur la performance si ce n'est pas fait proprement, parce qu'il va falloir compter intelligemment, et on ne connaît pas les bornes a priori comme dans l'implémentation actuelle.