search

BlogoText / blogotext

A little more than a lightweight SQLite Blog-Engine.
Other
137 stars 30 forks source link

Problème avec la pagination #79

Closed remrem closed 7 years ago

remrem commented 8 years ago

J'ai remarqué un problème sur la pagination des pages du blog, la dernière page propose d’accéder aux "éléments plus anciens", ce qui renvoie vers une page ne contenant aucun article ...

Pour exemple : http://couleur-science.eu?&p=27 propose l’accès à la page http://couleur-science.eu?&p=28 qui ne propose aucun article.

Désolé, pas motivé pour poster en anglais :/

remrem commented 8 years ago

Pull corrigé, cf. #80

remrem commented 8 years ago

Vu les commentaires de #80 , il faudrait faire une requêtes en prenant en compte bt_statut et bt_date (comme pour ?liste), le problème est que je ne vois pas trop comment gérer ça (cache ou compteur dans un fichier...)...

timovn commented 8 years ago

Je ne sais pas encore non plus. Perso je préfère recycler les requêtes SQL au maximum, question de performances.

Le bug est bien présent, mais seulement dans 10% du temps. Il n’est pas non plus critique selon moi.

BoboTiG commented 7 years ago

Est-ce que le dernier commit corrige le soucis ? J'ai testé avec 120 articles (dont 28 non publiés) et la pagination semble correcte.

remrem commented 7 years ago

De mémoire, il ne me semble pas, @timovn avait annulé le commit. Le problème étant que pour la génération de la pagination, il fallait faire une requête SQL pour avoir le nombre réel d'articles(...) (et il fallait que je revois les requêtes). Je vois ça ASAP (d'ici lundi).

BoboTiG commented 7 years ago

Je parlais de mon dernier commit. Quand tu feras des tests, pense à mettre à jour ta branche de travail.

remrem commented 7 years ago

Je parlais de mon dernier commit.¨

My bad, ça m'apprendra à lire en diagonale Je vois ça ASAP (d'ici lundi).

remrem commented 7 years ago

Apparemment, le problème n'est pas réglé, sur une install fraîche de la branche dev, j'ai 9 articles dont 1 non publié et l'affichage configuré à 5 articles par pages : "/" (5 art. affichés) > "article plus anciens" envoie sur "/?p=2" (0 art. affichés) "/?p=2" (0 art. affichés) > "éléments plus récents" envoie sur "/?p=1" (3 art. affichés)

Si je supprime les articles pour n'en conserver que 5 (tous publiés) et l'affichage configuré à 5 articles par pages : "/" (5 art. affichés) > "article plus anciens" envoie sur "/?p=2" (0 art. affichés) "/?p=2" (0 art. affichés) > "éléments plus récents" envoie sur "/?p=1" (0 art. affichés) "/?p=1" (0 art. affichés) > "éléments plus récents" envoie sur "/?p=0" (5 art. affichés)

remrem commented 7 years ago

Bon ... bah ... désolé, je ne m'en sors pas avec l'application github pour pousser proprement les commits ...

BoboTiG commented 7 years ago

Merci @remrem, ça fonctionne nickel.