Repository navigation
Conversation
…ut `user_login`. When `$_POST['user_login']` is not set while adding a user, `edit_user()` never set the `user_login` property, so the later checks read an undefined property and the "Please enter a username" error was not added. Set the property to an empty string in that case so the existing validation reports the missing username. See #36303. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. Core Committers: Use this line as a base for the props when committing in SVN: To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. Props Bot commandsYou can trigger Props Bot actions by commenting on this PR:
|
Test using WordPress PlaygroundThe changes in this pull request can previewed and tested using a WordPress Playground instance. WordPress Playground is an experimental project that creates a full WordPress instance entirely within the browser. Some things to be aware of
For more details about these limitations and more, check out the Limitations page in the WordPress Playground documentation. |
When
$_POST['user_login']is not set while adding a user,edit_user()never set theuser_loginproperty, so the later checks read an undefined property (PHP warnings) and the "Please enter a username" error was not added. This sets it to an empty string in that case, so the existing validation reports the missing username: the same approach as 36303.patch on the ticket, with a unit test.Trac ticket: https://core.trac.wordpress.org/ticket/36303
Testing
vendor/bin/phpunit --filter test_edit_user_missing_user_loginfails without the patch withUndefined property: stdClass::$user_loginand passes with it.vendor/bin/phpunit --group userpasses (1464 tests).Use of AI Tools
AI assistance: Yes
Tool(s): Claude Code
Model(s): Claude Opus 5.5
Used for: reproducing the warnings in trunk, drafting the fix and test, and running them. I reviewed every line.
This Pull Request is for code review only. Please keep all other discussion in the Trac ticket. Do not merge this Pull Request. See GitHub Pull Requests for Code Review in the Core Handbook for more details.
🤖 Generated with Claude Code