WordPress.org

Make WordPress Core

Opened 6 months ago

Last modified 5 weeks ago

#22959 new enhancement

Show all roles in user list table

Reported by: scribu Owned by:
Priority: normal Milestone: Future Release
Component: Role/Capability Version:
Severity: normal Keywords: has-patch
Cc: jeff@…, phill@…, justin@…, pippin@…

Description (last modified by scribu)

From [22686]:

we should show a comma-separated list of all roles, editable or otherwise, and this list should be filterable, either by user, or by the roles which can appear. Probably both.

Previously: #22361

Attachments (2)

22959.diff (3.0 KB) - added by scribu 6 months ago.
22959.1.diff (2.9 KB) - added by JustinSainton 5 weeks ago.

Download all attachments as: .zip

Change History (12)

scribu6 months ago

comment:1 scribu6 months ago

  • Description modified (diff)

comment:2 scribu6 months ago

  • Description modified (diff)

comment:3 deltafactory6 months ago

  • Cc jeff@… added

Where should filtering be added, and to what end? If we're expected to find existing elements in the array, perhaps it should be filtered pre-translation. If not, this at least gives an opportunity to add elements.

Before: return implode( ', ', $role_str );

After: return implode( ', ', apply_filters( 'wp_user_list_get_role_str', $role_str, $user_object ) );

If the intention was to modify the column's output after the implode(), another filter would be necessary.

comment:4 phill_brown6 months ago

  • Cc phill@… added

comment:5 greenshady5 months ago

  • Cc justin@… added

comment:6 MikeHansenMe5 weeks ago

  • Keywords needs-refresh added

JustinSainton5 weeks ago

comment:7 JustinSainton5 weeks ago

  • Keywords needs-refresh removed

comment:8 mordauk5 weeks ago

  • Cc pippin@… added

comment:9 ryan5 weeks ago

  • Milestone changed from 3.6 to Future Release

comment:10 deltafactory5 weeks ago

I know this ticket is now for a future release. I'm still interested in the inclusion of filtering of the roles, pre-translation. Am I the only one, or should I submit a patch to that effect?

Note: See TracTickets for help on using tickets.