[Bug] Post history paginator

[alex3305]

De post history paginator wil verder dan dat je überhaupt posts hebt, waardoor je de volgende melding krijgt:

quote: GoT

Niet gevonden

Er konden geen actieve topics gevonden worden.
(interne identificatie: board::get_activetopics::no_activetopics_found)

Nu zitten hier IMHO twee bugs in, namelijk:

1. Paginator die te ver gaat (dûh)
2. Relatief vage foutmelding, aangezien het niet om actieve topics gaat. Zelfs vaak om hele dode topics

[crisp]

De paginator is gebaseerd op je totale postcount; het kan echter zo zijn dat je op basis van je rechten niet al je topics meer kan bekijken (bijvoorbeeld als een topic gemoved is naar de trashcan of een ander niet-publiek forum). Al je topics vantevoren checken op je rechtenset is echter iets dat nogal een performance impact heeft, dus dat gaan we niet doen. Dat de paginator daardoor soms verder doorloopt dan dat jij zelf kan bekijken is dan jammer, maar feitelijk wel 'by design'.

Die foutmelding kan inderdaad wel duidelijker. Onderhuids wordt gewoon get_activetopics gebruikt met een filter op user-id, maar een meer specifieke melding voor dit geval zou inderdaad niet misstaan

[alex3305]

Ik snap best wat je bedoeld crisp. Is het echter niet in php mogelijk om te controleren of een topic wel weergegeven wordt in de listing? Waardoor je automatisch bij het bekijken van een pagina met geen topics terug wordt geredirect naar de laatste pagina met wel zichtbare posts? Dan is het misschien nog niet helemaal netjes afgehandeld, maar IMHO wel beter. Dat kan natuurlijk dan nog gepaard gaan met een bericht dat je op een pagina zat zonder zichtbare topics.

[crisp]

alex3305 schreef op woensdag 12 oktober 2011 @ 15:45:
Ik snap best wat je bedoeld crisp. Is het echter niet in php mogelijk om te controleren of een topic wel weergegeven wordt in de listing? Waardoor je automatisch bij het bekijken van een pagina met geen topics terug wordt geredirect naar de laatste pagina met wel zichtbare posts? Dan is het misschien nog niet helemaal netjes afgehandeld, maar IMHO wel beter. Dat kan natuurlijk dan nog gepaard gaan met een bericht dat je op een pagina zat zonder zichtbare topics.

Dan moet je dus nog steeds weten hoeveel topics er wel zichtbaar zijn anders weet je ook niet wat de laatste pagina moet zijn (en continue met /(page-1) redirecten lijkt me ook niet echt wenselijk).