Closed (outdated)
Project:
Drupal core
Version:
7.x-dev
Component:
base system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
23 Aug 2008 at 20:11 UTC
Updated:
12 Jul 2017 at 22:52 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
lilou commentedSee also this pager bug : #293664: Pager error on search module
Comment #2
jhedstromThe attached patch adds tests for the pager queries, and also for cycling through node?page=x pages (e.g., the default front page pager).
Comment #3
Anonymous (not verified) commentedThe last submitted patch failed testing.
Comment #4
jhedstromOdd, the patch still passes for me...setting back to needs review.
Comment #5
Anonymous (not verified) commentedThe last submitted patch failed testing.
Comment #6
jhedstromTests still pass...writing off to a bad day for the testing bot.
Comment #7
jhedstromoops...wrong status on the last post.
Comment #9
jhedstromComment #11
mcjim commentedPatch passes testing locally, marking as patch (code needs review).
Comment #13
mr.baileysTests do not pass on my local installation.
Looks like this assertion fails every time on my end:
Comment #14
mr.baileysFirst line is what the assertion looks for, the second line is what my pages actually contain:
Comment #15
catchMoving this out of the critical bugs queue - see #607038: Meta issue: fix gaps in code coverage.
Comment #16
jhedstrom