Make WordPress Core

Opened 13 years ago

Closed 12 years ago

#17141 closed defect (bug) (fixed)

In WP Admin -> Posts, sorting the list by Author actually sorts by Author ID

Reported by: aneesme's profile aneesme Owned by: ryan's profile ryan
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.1
Component: Administration Keywords: has-patch commit
Focuses: Cc:

Description

When I click on the author column title to sort the post listing by the author, it sorts by author id and I see the list messed up.

Attachments (1)

17141.diff (522 bytes) - added by scribu 12 years ago.
make 'author' column non-sortable

Download all attachments as: .zip

Change History (10)

#1 @scribu
13 years ago

This was always the case; now it's just more visible.

The argument is that the wp_posts table shouldn't be JOINed to the global wp_users table, which can get quite large on Multisite setups.

Last edited 13 years ago by scribu (previous) (diff)

#2 @scribu
13 years ago

  • Summary changed from In Dashboard->Posts, sorting the list by Author actually sorts by Author ID to In WP Admin -> Posts, sorting the list by Author actually sorts by Author ID

Also, we call dashboard only the first screen: wp-admin/index.php

What you're refering to is the administration area.

#3 @russellwwest
12 years ago

  • Owner set to russellwwest
  • Status changed from new to accepted

#4 @nacin
12 years ago

One could argue that the author column simply shouldn't be sorted. You can already click a single author's name to filter them down. If you can filter, and the sort is buggy, why have it? It could be implemented instead as a plugin.

#5 @scribu
12 years ago

The expectation would be that it sorts by wp_users.displayname. If that's not acceptable, I agree that we should disable sorting. We did the same on the Users screen when it came to post counts.

Last edited 12 years ago by scribu (previous) (diff)

#6 @russellwwest
12 years ago

  • Owner russellwwest deleted
  • Status changed from accepted to assigned

#7 @nofearinc
12 years ago

@scribu, I don't think that there is a way to order authors by author name without using a JOIN clause. A join would indeed affect the performance seriously having a large posts or users table but currently order_by takes place only for fields from the posts table.

There could probably be an extra screen (implemented by a plugin) for listing this, but a join-driven one.

@scribu
12 years ago

make 'author' column non-sortable

#8 @scribu
12 years ago

  • Keywords has-patch commit added
  • Milestone changed from Awaiting Review to 3.5

#9 @ryan
12 years ago

  • Owner set to ryan
  • Resolution set to fixed
  • Status changed from assigned to closed

In [21743]:

Remove not so useful author column sorting. Props scribu. fixes #17141

Note: See TracTickets for help on using tickets.