Make WordPress Core

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#8630 closed defect (bug) (fixed)

Wrong Paged Comments Logic

Reported by: filosofo's profile filosofo Owned by:
Milestone: 2.7.1 Priority: normal
Severity: normal Version: 2.7
Component: General Keywords: previous_comments_link next_comments_link cpage comments paged-comments has-patch
Focuses: Cc:


previous_comments_link and next_comments_link both erroneously assume that an empty cpage query variable value means that the comments page is page 1. This is wrong, because when "last" is the page displayed by default under Settings > Discussion, page 1 is the oldest page of comments, which is not the same as the permalink page (the page where cpage would not be specified, ordinarily).

Fortunately, this doesn't seem to affect anything as there's never any point at which *_comments_link are called but WordPress has not set the cpage query variable.

Attachments (1)

wrong_cpage_numbering.diff (545 bytes) - added by filosofo 15 years ago.

Download all attachments as: .zip

Change History (3)

#1 @ryan
15 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [10277]) Don't assume page 1 when cpage is empty. Props filosofo. fixes #8630 for trunk

#2 @ryan
15 years ago

(In [10278]) Don't assume page 1 when cpage is empty. Props filosofo. fixes #8630 for 2.7

Note: See TracTickets for help on using tickets.