Make WordPress Core

Opened 14 years ago

Closed 10 years ago

#18995 closed defect (bug) (wontfix)

Superfluous check in WP_Users_List_Table::display_rows()

Reported by: scribu's profile scribu Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.2.1
Component: Users Keywords: has-patch needs-refresh
Focuses: Cc:

Description

Specifically, this one:

if ( is_multisite() && empty( $role ) )
    continue;

It seems to have been a misguided attempt to fix something that should have been (and currently is) handled at the SQL level: #12711

Attachments (2)

18995.diff (675 bytes) - added by scribu 14 years ago.
18995.2.diff (1.8 KB) - added by scribu 14 years ago.

Download all attachments as: .zip

Change History (7)

@scribu
14 years ago

#1 @scribu
14 years ago

Also note that single_row() is perfectly capable of handling a user without a role on the current blog (it just shows 'None' in the Role column).

@scribu
14 years ago

#2 @scribu
14 years ago

While we're at it, we can just let single_row() get the role for itself. See 18995.2.diff.

#3 @nacin
14 years ago

Works for me.

#4 @coffee2code
13 years ago

  • Keywords needs-refresh added
  • Version set to 3.2.1

Patch no longer applies cleanly against trunk as of r21435.

#5 @wonderboymusic
10 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

The role logic is a lot more robust now - we also shouldn't change the method signature 4 years later

Note: See TracTickets for help on using tickets.