Show user roles when deleting users

[Presskopp]

When deleting an admin user, who has published something before, you will be asked to transfer the ownership of posts:

Attribute all content to:

Then there's a list with ALL users of the site, having all sorts of roles and therefore it can be huge.

My proposal is to somehow differenciate between user roles here so one can easily pick another admin.

[Presskopp]

related #15855

[lopo]

It would make sense to me, but it looks like we'd need to change the underlying wp_dropdown_users() function.
The easiest thing would probably be adding a value for the show parameter to e.g. display Display Name (login_name) - role.
A smarter and more versatile choice could be adding a bool group_by_role parameter to display users grouped in one <outgroup> for each role.

Trac ticket:

Trac ticket:

[lphoumpakka]

Hello @madejtax,

The patch is working as expected.
I left comments in the PR: ​https://github.com/WordPress/wordpress-develop/pull/4331

[pooja1210]

Hi,

Tested the patch and the patch is working fine. ​https://github.com/WordPress/wordpress-develop/pull/4331

The user roles are visible in Attribute all content to dropdown.

Just noticed that: role is visible as rol. I think it should be role.

Reference: ​https://prnt.sc/DjdnLZwy274b

Thanks!

Looks like there are some coding standards issues in the file changes. Can u fix them?

[hztyfoon]

Looks like the PR ​https://github.com/WordPress/wordpress-develop/pull/4331 is failing some coding standard tests.

[madejtax]

Sorry, I made the changes and they would already be in the corresponding branch.

​https://github.com/madejtax/wordpress-develop/tree/Changes-when-deleting-a-user-ticket-58103

I need to create the pull request, currently github is giving me a bug, when I solve it I will make the pull request.

Trac ticket: #58103

[madejtax]

I have a problem with PHPCS, the case is that it is giving me an error in a line that has no problem...

​https://github.com/WordPress/wordpress-develop/commit/2b6662ec7f8135de50716c78fa53b92322c3c7ac

As you can see, I've already had several pulls.... But it still gives the same problem

[pooja1210]

Hi,

Test Report for patch - ​https://github.com/WordPress/wordpress-develop/pull/4441

Environment:
WordPress- 6.2
Device- Mac
Browser: Chrome

Expected Result: The user roles should be visible in Attribute all content to: dropdown while deleting the user ☑️

Screenshots:
​https://prnt.sc/pQOJk5TG9vX2

However, @madejtax can we update the 'role' in dropdown to 'Role'. Make first letter capitalize.
Other than this rest looks fine to me.

Thanks!

[madejtax]

Modified, ready to implement

Replying to pooja1210:

Hi,

Test Report for patch - ​https://github.com/WordPress/wordpress-develop/pull/4441

Environment:
WordPress- 6.2
Device- Mac
Browser: Chrome

Expected Result: The user roles should be visible in Attribute all content to: dropdown while deleting the user ☑️

Screenshots:
​https://prnt.sc/pQOJk5TG9vX2

However, @madejtax can we update the 'role' in dropdown to 'Role'. Make first letter capitalize.
Other than this rest looks fine to me.

Thanks!

Closed in favor of #4441

[audrasjb]

Before patch

[audrasjb]

After patch: we loose the user slug

[audrasjb]

The patch looks almost good to me, but I wonder why the username (slug) was removed?
Also I don't think we need to add the strings "User:" and "Role:". And if we want to keep these string, they should at least be translatable.

Please note that tomorrow is 6.3 beta 1. Without a patch ready to go we will probably move this to the next milestone, 6.4 :)

[madejtax]

What do you think of this second option?

Where do I show the user's name, slug and role?

Edit: Edit: I thought it was a bit silly to show the name instead of the slug, in the end when you have the user you already know what name he/she has.

Replying to audrasjb:

The patch looks almost good to me, but I wonder why the username (slug) was removed?
Also I don't think we need to add the strings "User:" and "Role:". And if we want to keep these string, they should at least be translatable.

Please note that tomorrow is 6.3 beta 1. Without a patch ready to go we will probably move this to the next milestone, 6.4 :)

Trac ticket:

[audrasjb]

Hey thanks for updating the patch @madejtax!

As there is not much time before 6.3 beta 1 I'd recommend to simplify the patch and to keep it as close as possible to the current behavior.

Here is what I suggest:

"Display Name (username) - administrator"

echo '<option value="' . esc_attr( $user->ID ) . '">' . esc_html( $user->display_name ) . ' (' . esc_html( $user->user_login ) . ') - ' . esc_html( $user->roles[0] ) . '</option>';

[ocean90]

Replying to audrasjb:

Make sure to use translate_user_role() for displaying role names.

[madejtax]

Here is the change as they say commented

​https://github.com/WordPress/wordpress-develop/pull/4703/commits/8f4b16a82833fd8ce83057ac3b175a41fd2a33e2

Replying to audrasjb:

Hey thanks for updating the patch @madejtax!

As there is not much time before 6.3 beta 1 I'd recommend to simplify the patch and to keep it as close as possible to the current behavior.

Here is what I suggest:

"Display Name (username) - administrator"

echo '<option value="' . esc_attr( $user->ID ) . '">' . esc_html( $user->display_name ) . ' (' . esc_html( $user->user_login ) . ') - ' . esc_html( $user->roles[0] ) . '</option>';

Trac ticket

[audrasjb]

Thanks!
However, I have a small concern about $user->roles[0] because on many websites, users can have several roles.

As per today's bug scrub and since WP 6.3 beta 1 is scheduled today, let's move this ticket to milestone 6.4 to give it more time for discussion/feedback.

[Presskopp]

I was even thinking of something like:

[knutsp]

Replying to Presskopp:

I was even thinking of something like:

Nice, but then every WP_Role object, including custom ones, need a plural label for translation.

[knutsp]

Replying to audrasjb:

Thanks!
However, I have a small concern about $user->roles[0] because on many websites, users can have several roles.

Ordering them alpabetically will not work well for all languages, so I suggest keep it simple. But if not simple, indicate it. That means show first one, then suffix it with '(+n)' when necessary where n = count(roles).

And don't forget no role situation.

[oglekler]

@madejtax can you please look through the suggestions and concerns above? This is an enhancement and to land it into 6.4, it needs to be in Trunk before Beta 1. Thank you 🙏

[oglekler]

I wonder if a better approach will be to create a new function wp_dropdown_users_with_roles() instead of the current approach, this way it can be used elsewhere if needed. And patch cannot be applied, there are changes in wp_dropdown_users() arguments.

[oglekler]

This ticket was discussed during bug scrub.

Due to lack of activity lately and some changes that needs to be done, I am moving it into the next milestone.

Add props to @mukesh27

[audrasjb]

As per today's bug scrub: we have a PR but it still needs some testing/refinement to make sure it works well with multiple roles.

[huzaifaalmesbah]

i try to test but PR not working. i think need refresh.

[audrasjb]

As per today's bug scrub, moving this ticket to Future Release as we still need some work and testing.