Skip to:
Content

Opened 7 years ago

Last modified 4 years ago

#1642 new defect

Problems with pagination and sticky topics

Reported by: pavelevap Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 2.0
Component: Component - Topics Keywords: needs-patch
Cc: pavelevap@…, stephen@…

Description

I have 3 topics and I set up 2 topics per page. One of three topics is also Sticky.

On first page there are shown all 3 topics (one is Sticky) and also link to second page (not needed - I can see all available topics on first page). Above topics is following problematic text:

Viewing 3 topics - 1 through 2 (of 4 total)

But I should see only 2 topics (not 3) and there are not 4 total topics (only 3).

This behaviour (combination of pagination and sticky topics) is very buggy, I guess...

Change History (6)

#1 @johnjamesjacoby
7 years ago

  • Component changed from Forums to Topics
  • Keywords needs-patch added
  • Milestone changed from 2.0 to 2.1

This is currently by design. The verbiage is a little weird, but it's done this way to avoid querying the sticky offset on each paged query. It's a little late in the 2.0 cycle to tweak this logic, so I'm bumping this to 2.1.

#2 @pavelevap
7 years ago

I investigate this problem a little more further and it can be reproduced only when you select older topic (and mark it as Sticky) from another (for example second) page. So in my example of 3 topics with settings 2 for topics per page, this problem appears only when the last topic is marked as Sticky, because this third topic is on second page of topics. When you mark any of first two topics (on first page) as Sticky everything works well.

Main problem is in showing duplicate topic (as sticky and as normal topic) and message "Viewing..." with wrong numbers.

#3 @johnjamesjacoby
7 years ago

  • Milestone changed from 2.1 to Future Release

I'm bumping this to 2.2. Agree this is annoying, but it's edge case and not a trivial fix.

#4 @netweb
7 years ago

  • Cc stephen@… added

#6 @netweb
4 years ago

#2762 was marked as a duplicate.

Note: See TracTickets for help on using tickets.