Opened 3 years ago
Closed 3 years ago
#59108 closed defect (bug) (fixed)
Preserve block style variations when securing theme json
| Reported by: | dean | Owned by: | isabel_brison |
|---|---|---|---|
| Priority: | normal | Milestone: | 6.3.2 |
| Component: | Editor | Version: | 6.3 |
| Severity: | normal | Keywords: | has-patch has-unit-tests dev-feedback fixed-major |
| Cc: | Focuses: |
Description
I'd like to sync a GB PR that fixes an issue where block style variations are not preserved when securing theme json
Change History (12)
This ticket was mentioned in PR #5013 on WordPress/wordpress-develop by @dsas.
3 years ago
#1
- Keywords has-patch has-unit-tests added
@ramonopoly commented on PR #5013:
3 years ago
#3
Thanks for the ping! We're now looking for a core committer to give final review.
Though I think it might now be punted to 6.3.2 (?)
At any rate, I added it to the 6.3.x board a while back so it's on the radar.
#5
@
3 years ago
- Keywords dev-feedback added
- Milestone Awaiting Review → 6.3.2
- Resolution fixed
- Status closed → reopened
- Version → 6.3
Reopening for second committer review/committing to 6.3 release branch.
@isabel_brison commented on PR #5013:
3 years ago
#6
Committed to trunk in r56502; leaving the PR open pending commit to the release branch.
This ticket was mentioned in Slack in #core by joemcgill. View the logs.
3 years ago
#8
@
3 years ago
- Keywords fixed-major added
It would be good to get a performance analysis of the changes in r56502, but it looks good at first glance.
#9
@
3 years ago
@joemcgill how do I go about wrangling that? Is there a particular method I should use?
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Valid and safe block style variations were being removed by
WP_Theme_JSON::remove_insecure_propertieswhen securing the theme.json. When this was a problem varied depending upon site configuration, but out-of-the-box it was a problem for administrators on multi-site installs.This change adds explicit processing of variations in
remove_insecure_propertiesso that they won't get removed.Trac ticket: [](https://core.trac.wordpress.org/ticket/59108)