One of my clients had a nodequeue with > 25 nodes in it. For some reason, a pager_query is being used with the value set @ 25. Since there seems to be no way to navigate to other pages, I decided to change it to a normal db_query. This may not be ideal, but IMHO is a better default so as not to confuse users.

CommentFileSizeAuthor
#1 nodequeue_admin_pager.5x-1.patch.txt865 bytesdww
nodequeue_pager.patch.txt686 bytesJacobSingh

Comments

dww’s picture

dww
we/he/they
commented
Assigned: JacobSingh » dww
Status: Active » Needs review
StatusFileSize
new nodequeue_admin_pager.5x-1.patch.txt865 bytes

As you noted, that's the wrong approach. The better way is to just add the pager links. Attached patch for DRUPAL-5. The code in DRUPAL-5--2 no longer has this problem, it seems, since the pages that display pager queries all seem to include the pager links.

brenda003’s picture

brenda003
she/her
commented
Status: Needs review » Needs work

Adding paging doesn't work, as it doesn't allow nodes from other pages to be moved to other pages. So if you are trying to move node 26, which is on the 2nd page, up to page 1, it doesn't actually do anything.

dww’s picture

dww
we/he/they
commented
Assigned: dww » Unassigned
merlinofchaos’s picture

merlinofchaos commented
Status: Needs work » Closed (won't fix)

The nodequeue 1.x line is no longer available.