Add filters to wp_list_users()

[tmatsuur]

The wp_list_users function has been added in version 5.9.

In the source code of this function, the full name is "first_name last_name".

Japanese people's full names are usually "last_name first_name" and it would be useful to have a filter for full names.

$name = apply_filters( 'wp_list_users_fullname', "$user->first_name $user->last_name", $user, $args );

[swissspidy]

#55076 was marked as a duplicate.

[swissspidy]

I think there's an opportunity here do add a couple more filters,

Some thoughts:

• Allow filtering the query args passed to get_users() (see also #55076)
• Remove the detailed PHPDoc for the $args param and just point to WP_User_Query::prepare_query(), like get_users() does.
• Add filter for the full name option, as you suggest.

[rafiahmedd]

Modify as suggested

[rafiahmedd]

Sorry I forgot to add filter hook for get_users() args, so adding another patch for this

[swissspidy]

Thanks for contributing a patch here!

Please note that filters need to be documented appropriately as per our guidelines, see ​https://developer.wordpress.org/coding-standards/inline-documentation-standards/php/#4-hooks-actions-and-filters.

Also, apply_filters( 'wp_list_users', $args ); on its own does not do anything. You would need to do $args = apply_filters( 'wp_list_users', $args ); for it to have any effect.

You might want to try submitting your change as a GitHub pull request so that you'll get notified if there's anything wrong with inline documentation and things like that. Makes everything a bit easier.

[rafiahmedd]

doc blocks added, moreover I remove the $args from wp_list_users_fullname filter because I think we don’t need this $args parameter just for changing a name

[rafiahmedd]

Sorry for another patch there was a mistake in doc block so I just fixed it and adding this patch

[tmatsuur]

Thanks @swissspidy @rafiahmedd

Thank you for changing the summary and creating the patch.

I don't think $args is a must-have feature either. However, in the WordPress source code, there are many filters whose parameters are added when upgrading.

That's a good thing, because there are many cases where the more information you have, the more helpful it is for the user of the filter, and I would like to have room to change the processing depending on the contents of $args.

[rafiahmedd]

Replying to tmatsuur:

Thanks @swissspidy @rafiahmedd

Thank you for changing the summary and creating the patch.

I don't think $args is a must-have feature either. However, in the WordPress source code, there are many filters whose parameters are added when upgrading.

That's a good thing, because there are many cases where the more information you have, the more helpful it is for the user of the filter, and I would like to have room to change the processing depending on the contents of $args.

@tmatsuur welcome, I will wait for @swissspidy confirmation if he also confirms that $args is needed here then I will update a new patch.

Thanks, both of you 😊