Viewing pagers beyond the second page.
Viewing pagers with different number of results per page.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | pager.test.patch | 5.32 KB | jhedstrom |
Viewing pagers beyond the second page.
Viewing pagers with different number of results per page.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | pager.test.patch | 5.32 KB | jhedstrom |
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