Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
poll_page() uses pager_query() with a query including GROUP BY without a custom count query. The pager ends up either missing or having an incorrect number of page links depending on the vagaries of the primary query.
Patch attached for HEAD and DRUPAL-5 (same).
Comment | File | Size | Author |
---|---|---|---|
#3 | poll-head_0.patch | 1.3 KB | bjaspan |
#1 | poll-4.7.patch | 1.23 KB | bjaspan |
poll-head.patch | 1.18 KB | bjaspan | |
Comments
Comment #1
bjaspan CreditAttribution: bjaspan commentedPatch attached for DRUPAL-4.7.
Comment #2
bjaspan CreditAttribution: bjaspan commentedOoops, forgot to set the status.
Comment #3
bjaspan CreditAttribution: bjaspan commentedRe-rolled to use proper patch format (from Drupal root, not the module/poll directory). After six months, I'm marking this RTBC, and fully expect to be flamed for doing so.
Comment #4
Gábor HojtsyI think this is a simple and straightforward patch, so committed!
Comment #5
(not verified) CreditAttribution: commentedComment #6
bjaspan CreditAttribution: bjaspan commentedThis bug still exists in 5.x. The patch in #3 applies to DRUPAL-5 HEAD.
Comment #7
drummCommitted to 5.x.
Comment #8
(not verified) CreditAttribution: commented