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).

CommentFileSizeAuthor
#3 poll-head_0.patch1.3 KBbjaspan
#1 poll-4.7.patch1.23 KBbjaspan
poll-head.patch1.18 KBbjaspan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bjaspan’s picture

FileSize
1.23 KB

Patch attached for DRUPAL-4.7.

bjaspan’s picture

Status: Active » Needs review

Ooops, forgot to set the status.

bjaspan’s picture

Status: Needs review » Reviewed & tested by the community
FileSize
1.3 KB

Re-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.

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

I think this is a simple and straightforward patch, so committed!

Anonymous’s picture

Status: Fixed » Closed (fixed)
bjaspan’s picture

Version: 6.x-dev » 5.x-dev
Status: Closed (fixed) » Reviewed & tested by the community

This bug still exists in 5.x. The patch in #3 applies to DRUPAL-5 HEAD.

drumm’s picture

Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

Status: Fixed » Closed (fixed)