Table sort is currently broken on admin/content.

Comments

heather’s picture

I tested this out.

On a fresh install from HEAD, yes, the content listing page was not sorting on the sortable columns.

After a fresh install and applying the patch, the columns sort.

wd!

jody lynn’s picture

Status: Needs review » Reviewed & tested by the community

Thanks for confirming!

webchick’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD. Thanks!

Status: Fixed » Closed (fixed)

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

Status: Closed (fixed) » Needs review

nurmuhammad requested that failed test be re-tested.

Status: Needs review » Needs work

The last submitted patch failed testing.

jody lynn’s picture

Status: Needs work » Closed (fixed)
tszming’s picture

Status: Closed (fixed) » Active

Have installed the 7.x-dev (2010-Aug-29), seems there is a regression:

1. Install 7.x-dev (2010-Aug-29), with standard profile
2. Create two articles in order, Test1 & Test2
3. In the URL, http://www.example.com/admin/content?sort=desc&order=Updated, expected order should be:

Test2
Test1

But the order showed:

Test1
Test2

4. Have further checked the node.admin.inc, the $nids returned are in correct order, so there must be some codes which altered the order after that query


  $nids = $query
    ->fields('n',array('nid'))
    ->limit(50)
    ->orderByHeader($header)
    ->execute()
    ->fetchCol();

  var_dump($nids); // Showing correct order, i.e. (2, 1)
sivaji_ganesh_jojodae’s picture

Title: Sorting on admin/content table is broken » Sorting on admin/content table is incorrect
StatusFileSize
new7.08 KB

#8 is true, see the attachment for proof. Changing the title to reflect the bug.

rayasa’s picture

The bug lies in the form system.
Check the patch at #847852: Table Sort Doesn't Quite Sort

heine’s picture

Status: Active » Closed (fixed)

Restoring issue status. The problem experienced in #8 is caused by another problem. This is handled in #700380: Regression: Tableselect sorting is altered if '#weight' is not set.