Opened 2 weeks ago
Last modified 2 weeks ago
#64059 new defect (bug)
wp_die() status code Upgrade/Install component updates (parent ticket: #64009)
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | trunk |
| Component: | Users | Keywords: | has-patch |
| Focuses: | Cc: |
Description
If you have come here directly, please refer back to the parent ticket here: #64009
This ticket is regarding the wp_die() function calls within the Media component and will cover the following locations. Please note links were correct at the time of testing and writing this up, but line numbers may have changed slightly:
| Filepath | Line Number | Context | Status Code | New Code | Code Meaning |
| src/wp-admin/users.php | 249 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/export-personal-data.php | 13 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/erase-personal-data.php | 13 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/user-edit.php | 28 | Error: invalid params | 500 | 400 | Bad Request |
| src/wp-admin/user-edit.php | 30 | Error: invalid params | 500 | 400 | Bad Request |
| src/wp-admin/user-edit.php | 104 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/user-edit.php | 136 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/user-edit.php | 195 | User Permissions: does not have access | 500 | 403 | Forbidden |
There are however the following function calls I have not touched as I felt they were already valid with their current status code:
| Filepath | Line Number | Status Code |
| src/wp-admin/includes/user.php | 62 | 403 |
| src/wp-admin/user-new.php | 14 | 302 |
| src/wp-admin/user-new.php | 21 | 403 |
| src/wp-admin/user-new.php | 55 | 403 |
| src/wp-admin/user-new.php | 193 | 403 |
| src/wp-admin/users.php | 14 | 403 |
| src/wp-admin/users.php | 114 | 403 |
| src/wp-admin/users.php | 131 | 403 |
| src/wp-admin/users.php | 143 | 403 |
| src/wp-admin/users.php | 157 | 403 |
| src/wp-admin/users.php | 173 | 400 |
| src/wp-admin/users.php | 193 | 403 |
| src/wp-admin/users.php | 201 | 403 |
| src/wp-admin/users.php | 276 | 400 |
| src/wp-admin/users.php | 461 | 403 |
| src/wp-admin/users.php | 470 | 403 |
| src/wp-admin/users.php | 493 | 400 |
Please have a look through the suggested changes documented here, and any/all feedback is appreciated. 😃
Change History (1)
This ticket was mentioned in PR #10098 on WordPress/wordpress-develop by @callumbw95.
2 weeks ago
#1
- Keywords has-patch added
Note: See
TracTickets for help on using
tickets.
…s related to the media component
Trac ticket: #64059