-
Notifications
You must be signed in to change notification settings - Fork 43
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DevHub] Wrong default mentioned for wp_list_users
(exclude_admin
)
#1849
Comments
Heads up @WordPress/docs-issues-coordinators, we have a new issue open. Time to use 'em labels. |
Heads up @docs-reviewers - the "[Status] Review" label was applied to this issue. |
🤷♂️ This does not add anything to the issue report. Also, self-assigning the issue and then asking someone to fix it is not what self-assigning means. It means YOU will fix it. So @karthick-murugan , please keep the contributing guidelines and code of conduct in mind, and either fix the mentioned issue or unassign yourself from it so it can be picked up by someone else. |
@vHeemstra – Apologies for the misunderstanding. I have now unassigned myself. Thanks for your clarification. |
I can't figure out how to send in a PR with the needed change. But for anyone how knows and can change it, it's this comment line that needs changing: https://core.trac.wordpress.org/browser/trunk/src/wp-includes/user.php#L867 |
Issue Description
In the Dev docs on
wp_list_users
, the default value ofexclude_admin
is wrong. It saysfalse
, but it should betrue
according to the source code.URL of the Page with the Issue
https://developer.wordpress.org/reference/functions/wp_list_users/
Section of Page with the issue
Parameters
Why is this a problem?
Wrong default value mentioned.
Suggested Fix
Change
false
totrue
in the docs/comment in the source code.The text was updated successfully, but these errors were encountered: