Ordering displays, e.g. "taxonomy/term/2/order", work with pagers and a hardcoded query limit set to 50. When having multiple pages, it is impossible to order the 51st node to being the 50th in weight by using the dragging functionality. You could achieve this by using the manual weights, but it would be better if the pager functionality is left out on this screen.

CommentFileSizeAuthor
#3 2314945-remove-query-limit-and-pager.patch1.08 KBtomvv
#1 2314945-remove-query-limit-and-pager.patch1.08 KBtomvv
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tomvv’s picture

tomvv CreditAttribution: tomvv at Finalist commented
FileSize
2314945-remove-query-limit-and-pager.patch1.08 KB

Here is a patch that does just that, removing the query limit and pager.

tomvv’s picture

tomvv CreditAttribution: tomvv commented
Status: Patch (to be ported) » Active
tomvv’s picture

tomvv CreditAttribution: tomvv at Finalist commented
FileSize
2314945-remove-query-limit-and-pager.patch1.08 KB

Patch correction.

tomvv’s picture

tomvv CreditAttribution: tomvv commented

  • dieuwe committed f675a15 on 7.x-1.x authored by tomvv 
    Issue #2314945, by tomvv: Pager on node ordering display not needed.
dieuwe’s picture

dieuwe
Location Auckland, NZ
CreditAttribution: dieuwe commented
Assigned: tomvv » Unassigned
Status: Active » Fixed
dieuwe’s picture

dieuwe
Location Auckland, NZ
CreditAttribution: dieuwe commented
Version: 7.x-1.2 » 7.x-1.x-dev

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

arifkhn46’s picture

arifkhn46 CreditAttribution: arifkhn46 commented

Hi,

what if I have 892 nodes which taking too much of memory to render, and ended up with an ugly Fatal Error

davps’s picture

davps CreditAttribution: davps as a volunteer and at DrupalJedi commented

@ak.incaendo

There is already a similar issue #402630. It is better to create a new task and describe the problem, I'll think on a solution.