Opened 8 weeks ago
Last modified 3 days ago
#64057 new defect (bug)
wp_die() status code Themes component updates (parent ticket: #64009)
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | Awaiting Review | Priority: | normal |
| Severity: | normal | Version: | |
| Component: | Themes | 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/themes.php | 86 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/themes.php | 106 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/theme-editor.php | 18 | User Permissions: does not have access | 500 | 403 | Forbidden |
| src/wp-admin/theme-editor.php | 73 | Missing: theme | 500 | 404 | Not Found |
| src/wp-admin/theme-install.php | 16 | 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/theme-install.php | 261 | 500 |
| src/wp-admin/themes.php | 13 | 403 |
| src/wp-admin/themes.php | 26 | 403 |
| src/wp-admin/themes.php | 41 | 403 |
| src/wp-admin/themes.php | 51 | 500 |
| src/wp-admin/themes.php | 61 | 403 |
| src/wp-admin/themes.php | 69 | 403 |
| src/wp-content/themes/twentyfifteen/inc/back-compat.php | 53 | 500 |
| src/wp-content/themes/twentyfifteen/inc/back-compat.php | 74 | 500 |
| src/wp-content/themes/twentyfourteen/inc/back-compat.php | 53 | 500 |
| src/wp-content/themes/twentyfourteen/inc/back-compat.php | 74 | 500 |
| src/wp-content/themes/twentynineteen/inc/back-compat.php | 57 | 500 |
| src/wp-content/themes/twentynineteen/inc/back-compat.php | 80 | 500 |
| src/wp-content/themes/twentyseventeen/inc/back-compat.php | 57 | 500 |
| src/wp-content/themes/twentyseventeen/inc/back-compat.php | 80 | 500 |
| src/wp-content/themes/twentysixteen/inc/back-compat.php | 59 | 500 |
| src/wp-content/themes/twentysixteen/inc/back-compat.php | 82 | 500 |
| src/wp-content/themes/twentythirteen/inc/back-compat.php | 53 | 500 |
| src/wp-content/themes/twentythirteen/inc/back-compat.php | 74 | 500 |
| src/wp-content/themes/twentytwentyone/inc/back-compat.php | 58 | 500 |
| src/wp-content/themes/twentytwentyone/inc/back-compat.php | 83 | 500 |
| src/wp-includes/theme.php | 765 | 500 |
Please have a look through the suggested changes documented here, and any/all feedback is appreciated. 😃
Change History (5)
This ticket was mentioned in PR #10096 on WordPress/wordpress-develop by @callumbw95.
8 weeks ago
#1
- Keywords has-patch added
@mindctrl commented on PR #10096:
8 weeks ago
#2
@CallumBW95 looks good. The failed checks appear to be related to a GitHub outage on macOS builds.
@callumbw95 commented on PR #10096:
8 weeks ago
#3
@mindctrl Ah ok, is there anyway I can rerun the tests as it looks like there are no issues with MacOS runners now?
@mindctrl commented on PR #10096:
8 weeks ago
#4
@CallumBW95 AFAIK the only way, without extra permissions to this repo, is to push another commit.
…s related to the Themes component
Trac ticket: #64057