Opened 3 years ago
Closed 3 years ago
#55178 closed defect (bug) (fixed)
Allow fully extending WP_Theme_JSON and WP_Theme_JSON_Resolver classes
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 5.9.1 | Priority: | normal |
Severity: | normal | Version: | |
Component: | Themes | Keywords: | has-patch dev-feedback fixed-major |
Focuses: | Cc: |
Description
Allow fully extending WP_Theme_JSON
and WP_Theme_JSON_Resolver
classes.
This requires changing method visibility from private
to protected
and using late static binding.
Original PRs from Gutenberg repo:
Change History (8)
#1
@
3 years ago
- Milestone changed from Awaiting Review to 5.9.1
- Owner set to audrasjb
- Status changed from new to assigned
This ticket was mentioned in PR #2325 on WordPress/wordpress-develop by kapilpaul.
3 years ago
#2
- Keywords has-patch added
#3
@
3 years ago
- Keywords dev-feedback added
@Mamaduka https://github.com/WordPress/wordpress-develop/pull/2322 is the correct PR for this ticket, right?
@kapilpaul proposed another PR in the ticket but it looks like e2e tests are failing.
It also includes more that what is in your PR
#4
@
3 years ago
@audrasjb, that's correct. My PR is mostly code backported from the Gutenberg plugin.
#5
@
3 years ago
@audrasjb my PR is failing e2e test. As there is another PR which is all good, I am closing my PR then.
trac-55178 fix: Allow fully extending WP_Theme_JSON and WP_Theme_JSON Resolver classes
Trac ticket: https://core.trac.wordpress.org/ticket/55178