Include IDs for users and sites on their respective views in the network admin

[danielbachhuber]

It would be nice if the IDs for Users and Sites were displayed in columns on their respective views in the network admin (with the option to hide in screen options of course)

[danielbachhuber]

Add ID columns to ms-users and ms-sites tables

[danielbachhuber]

id.18661.1.patch adds ID columns to ms-users and ms-sites tables. For the latter, there's already a switch value for the column (legacy code?) so we just need to register the column.

[Latz]

There are plugins (e.g. "Reveal IDs" ​http://wordpress.org/extend/plugins/reveal-ids-for-wp-admin-25/) which can do this task if you really need to know the IDs. No need to confuse the users with this information since it's not needed for normal operation.

[nacin]

This is only for the network admin, where it's not users, as much as network administrators. I find needing these IDs all the time.

[danielbachhuber]

+1 nacin. The hack for me right now is to load the edit URL for the user and grab the ID from the query string. Which is an ugly habit. I'd argue that easy access to the user ID is core for the network admin (where you're most likely managing a large number of users).

[Latz]

OK, sorry, I seem to have missed the "network" part in the ticket.

[westi]

Replying to nacin:

This is only for the network admin, where it's not users, as much as network administrators. I find needing these IDs all the time.

I'm interested to know what for.

Because I don't think we have anywhere in the Network Admin where you need to know this info to looks something up.

(I don't think adding something to the UI soley to help people look things up directly in the DB is worthwhile - I would rather we address the reasons that mean you go straight to the db)

[nacin]

For me, it's typically when I'm doing migrations and other DB level tasks. In particular, getting the ID of a blog, that way I can go inspect tables. User IDs are less helpful. My comment wasn't to support this for core, only that I would find it useful.

Can we add the renderer for the ID field for ms-users? We have some precedence there (like mature actions) which at least makes it super easy to register the column.

[danielbachhuber]

Replying to nacin:

Can we add the renderer for the ID field for ms-users? We have some precedence there (like mature actions) which at least makes it super easy to register the column.

Who is this directed towards?

[wonderboymusic]

18661.diff​ love at first sight

[SergeyBiryukov]

In 34289:

Network Admin: Include IDs for users and sites on their respective views.

Props danielbachhuber, wonderboymusic.
Fixes #18661.

[ocean90]

This wasn't tested on mobile, see 18661-mobile.png​.

[jeremyfelt]

Can we make the ID column optional and off by default? I'm not sure it's of high value to the majority. I know it's useful for me when I go to use WP-CLI, but I also use WP-CLI to get the ID at that point.

We should get before/after screenshots of the change too.

[helen]

Replying to jeremyfelt:

Can we make the ID column optional and off by default? I'm not sure it's of high value to the majority.

Please. I also don't think it's a good idea to have it as the first column, not just because of small screen treatment, but also because it's not the primary piece of human-identifiable data.

[jeremyfelt]

18661.2.diff moves ID to the second position for both sites and users. It also hides the column by default if a user has not adjusted their settings.

I tried putting it as the last column, but that didn't really look right in mobile. It seems to fit best as the second, but I'm open for suggestions.

[ocean90]

Replying to jeremyfelt:

Can we make the ID column optional and off by default? I'm not sure it's of high value to the majority.

That sounds like we shouldn't add the column at all.

[dd32]

I don't mind the ID field being available however as most users shouldn't ever need it, having it disabled by default would be best IMHO. I'd previously have suggested this is plugin material though.

Nacins suggestion earlier was somewhat similar:

Can we add the renderer for the ID field for ms-users? We have some precedence there (like mature actions) which at least makes it super easy to register the column.

The idea being that the handler would be in the code, but a plugin would still have to come along and register the column for display.

[jeremyfelt]

Yeah, let's take the registration of the field back out and leave the renderer. A super simple plugin can then register the fields and desired placement if needed.

[jeremyfelt]

In 34750:

MS: Remove registration of ID column in Sites and Users list tables.

Reverts a portion of [34289] and leaves the render for the column behind. A plugin can register these fields for display via filter.

Fixes #18661.