Opened 18 months ago
Closed 16 months ago
#59108 closed defect (bug) (fixed)
Preserve block style variations when securing theme json
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 6.3.2 | Priority: | normal |
Severity: | normal | Version: | 6.3 |
Component: | Editor | Keywords: | has-patch has-unit-tests dev-feedback fixed-major |
Focuses: | Cc: |
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.
18 months ago
#1
- Keywords has-patch has-unit-tests added
@ramonopoly commented on PR #5013:
18 months 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.
#4
@
18 months ago
- Owner set to isabel_brison
- Resolution set to fixed
- Status changed from new to closed
In 56502:
#5
@
18 months ago
- Keywords dev-feedback added
- Milestone changed from Awaiting Review to 6.3.2
- Resolution fixed deleted
- Status changed from closed to reopened
- Version set to 6.3
Reopening for second committer review/committing to 6.3 release branch.
@isabel_brison commented on PR #5013:
18 months 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.
17 months ago
#8
@
17 months 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
@
17 months ago
@joemcgill how do I go about wrangling that? Is there a particular method I should use?
Valid and safe block style variations were being removed by
WP_Theme_JSON::remove_insecure_properties
when 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_properties
so that they won't get removed.Trac ticket: [](https://core.trac.wordpress.org/ticket/59108)