Make WordPress Core

Opened 7 years ago

Last modified 7 years ago

#41826 new defect (bug)

WP_Comment_Query can generate negative offset in LIMIT clause

Reported by: birgire's profile birgire Owned by:
Milestone: Awaiting Review Priority: normal
Severity: normal Version: 4.9
Component: Comments Keywords: has-patch has-unit-tests
Focuses: Cc:


The paged argument in WP_Comment_Query was introduced in #38268

There's a problem with the paged as 0 case.


When the paged is 0 and number is 2, WP_Comment_Query generates the LIMIT clause:

LIMIT -2, 2

instead of

LIMIT 0, 2

According to the MySQL documentation, the offset in the LIMIT clause shouldn't be negative:

I think we should handle it in the same way as in WP_Query, where the 0 case is treated as the default, that is 1.

Attachments (2)

41826.patch (1.5 KB) - added by birgire 7 years ago.
41826.2.patch (1.8 KB) - added by birgire 7 years ago.

Download all attachments as: .zip

Change History (5)

7 years ago

#1 @birgire
7 years ago

  • Keywords has-patch has-unit-tests added

The 0 case is now same as 1.

Updated: Here we focus on the calculated offset in the LIMIT clause.

Included is a test for this case.

Last edited 7 years ago by birgire (previous) (diff)

This ticket was mentioned in Slack in #core by sergey. View the logs.

7 years ago

#3 @SergeyBiryukov
7 years ago

  • Component changed from Query to Comments

7 years ago

Note: See TracTickets for help on using tickets.