| CARVIEW |
Select Language
HTTP/2 200
date: Sat, 11 Oct 2025 09:46:07 GMT
content-type: text/html; charset=utf-8
cache-control: no-cache
content-security-policy: default-src 'none'; base-uri 'self'; child-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/; connect-src 'self' uploads.github.com www.githubstatus.com collector.github.com raw.githubusercontent.com api.github.com github-cloud.s3.amazonaws.com github-production-repository-file-5c1aeb.s3.amazonaws.com github-production-upload-manifest-file-7fdce7.s3.amazonaws.com github-production-user-asset-6210df.s3.amazonaws.com *.rel.tunnels.api.visualstudio.com wss://*.rel.tunnels.api.visualstudio.com github.githubassets.com objects-origin.githubusercontent.com copilot-proxy.githubusercontent.com proxy.individual.githubcopilot.com proxy.business.githubcopilot.com proxy.enterprise.githubcopilot.com *.actions.githubusercontent.com wss://*.actions.githubusercontent.com productionresultssa0.blob.core.windows.net/ productionresultssa1.blob.core.windows.net/ productionresultssa2.blob.core.windows.net/ productionresultssa3.blob.core.windows.net/ productionresultssa4.blob.core.windows.net/ productionresultssa5.blob.core.windows.net/ productionresultssa6.blob.core.windows.net/ productionresultssa7.blob.core.windows.net/ productionresultssa8.blob.core.windows.net/ productionresultssa9.blob.core.windows.net/ productionresultssa10.blob.core.windows.net/ productionresultssa11.blob.core.windows.net/ productionresultssa12.blob.core.windows.net/ productionresultssa13.blob.core.windows.net/ productionresultssa14.blob.core.windows.net/ productionresultssa15.blob.core.windows.net/ productionresultssa16.blob.core.windows.net/ productionresultssa17.blob.core.windows.net/ productionresultssa18.blob.core.windows.net/ productionresultssa19.blob.core.windows.net/ github-production-repository-image-32fea6.s3.amazonaws.com github-production-release-asset-2e65be.s3.amazonaws.com insights.github.com wss://alive.github.com wss://alive-staging.github.com api.githubcopilot.com api.individual.githubcopilot.com api.business.githubcopilot.com api.enterprise.githubcopilot.com; font-src github.githubassets.com; form-action 'self' github.com gist.github.com copilot-workspace.githubnext.com objects-origin.githubusercontent.com; frame-ancestors 'none'; frame-src viewscreen.githubusercontent.com notebooks.githubusercontent.com; img-src 'self' data: blob: github.githubassets.com media.githubusercontent.com camo.githubusercontent.com identicons.github.com avatars.githubusercontent.com private-avatars.githubusercontent.com github-cloud.s3.amazonaws.com objects.githubusercontent.com release-assets.githubusercontent.com secured-user-images.githubusercontent.com/ user-images.githubusercontent.com/ private-user-images.githubusercontent.com opengraph.githubassets.com marketplace-screenshots.githubusercontent.com/ copilotprodattachments.blob.core.windows.net/github-production-copilot-attachments/ github-production-user-asset-6210df.s3.amazonaws.com customer-stories-feed.github.com spotlights-feed.github.com objects-origin.githubusercontent.com *.githubusercontent.com; manifest-src 'self'; media-src github.com user-images.githubusercontent.com/ secured-user-images.githubusercontent.com/ private-user-images.githubusercontent.com github-production-user-asset-6210df.s3.amazonaws.com gist.github.com; script-src github.githubassets.com; style-src 'unsafe-inline' github.githubassets.com; upgrade-insecure-requests; worker-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/
referrer-policy: no-referrer-when-downgrade
server-timing: pull_request_layout-fragment;desc="pull_request_layout fragment";dur=424.927042,conversation_content-fragment;desc="conversation_content fragment";dur=3714.388336,conversation_sidebar-fragment;desc="conversation_sidebar fragment";dur=399.827371,nginx;desc="NGINX";dur=0.85497,glb;desc="GLB";dur=139.672922
strict-transport-security: max-age=31536000; includeSubdomains; preload
vary: X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, X-Requested-With,Accept-Encoding, Accept, X-Requested-With
x-content-type-options: nosniff
x-frame-options: deny
x-voltron-version: aab62e3
x-xss-protection: 0
server: github.com
content-encoding: gzip
accept-ranges: bytes
set-cookie: _gh_sess=pSdC5HMMa8dGqZgT0D%2Fe13BrMFeGrtRxRqUDqeThQEYrbdhLe8Br%2FjTzH5MDNDLUKQTWUx2cAq9bDUhHnYPXCQHoFtgdqhcMwbBnKnlk2CUw1xQ61%2BvguGZ%2BKdoA1CQakqbPxtfmMvnVzee6Stybpa%2FVyDHhCOruceecip8te1U3Tp2nKXprUYFVz8hrsyl9P5ToNcRigFnKhsJ8tf2B0KHUw1ZnxUa7Uhae%2Fnu1nde0kmleLDEq%2F6r28HhvuCnYMhTBIAcEs6bgAgpk%2BnvSjA%3D%3D--uFZNrIYZ%2BqVtlhQy--EDypkERRHCc0ux0DfAGSFA%3D%3D; Path=/; HttpOnly; Secure; SameSite=Lax
set-cookie: _octo=GH1.1.1637571256.1760175963; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 09:46:03 GMT; Secure; SameSite=Lax
set-cookie: logged_in=no; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 09:46:03 GMT; HttpOnly; Secure; SameSite=Lax
x-github-request-id: EDC2:9DC1D:53E369:716EC9:68EA275B
Template activation: fix /lookup endpoint to use custom resolve fn by ellatrix · Pull Request #72049 · WordPress/gutenberg · GitHub
ellatrix
force-pushed
the
fix/template-activation-lookup
branch
from
October 3, 2025 14:55
ellatrix
added
the
Backport to Gutenberg RC
Pull request that needs to be backported to a Gutenberg release candidate (RC)
label
Oct 7, 2025
cbravobernal
removed
the
Backport to Gutenberg RC
Pull request that needs to be backported to a Gutenberg release candidate (RC)
label
Oct 8, 2025
Skip to content
Navigation Menu
{{ message }}
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
Template activation: fix /lookup endpoint to use custom resolve fn #72049
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ellatrix
force-pushed
the
fix/template-activation-lookup
branch
from
0ca0d47 to
b45cf60
Compare
|
The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message. To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook. |
|
Flaky tests detected in 469ba69. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/18226813568
|
ellatrix
commented
Oct 3, 2025
| .click(); | ||
| await page.getByRole( 'menuitem', { name: 'Change template' } ).click(); | ||
| await page.getByRole( 'option', { name: 'Custom' } ).click(); | ||
| await expect( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test was previously giving a false positive. It only checks for Single Entries at the very end, but it failed to set the template Custom in the first place.
|
Size Change: -6 B (0%) Total Size: 1.96 MB
ℹ️ View Unchanged
|
|
Merging this because template activation basically doesn't work in the editor. Adjusted the e2e test for better coverage. |
ramonjd
reviewed
Oct 7, 2025
| @@ -0,0 +1,17 @@ | |||
| <?php | |||
|
|
|||
| class Gutenberg_REST_Old_Templates_Controller extends WP_REST_Templates_Controller { | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ellatrix I need to extend WP_REST_Templates_Controller for another feature.
Can I piggy back off Gutenberg_REST_Old_Templates_Controller?
Could you explain how it relates to Gutenberg_REST_Templates_Controller? Some comments in the code might help folks working on 6.9.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we discussed, it would make sense to add the method to override for all 3 controllers if you need to filter templates.
ellatrix
added a commit
that referenced
this pull request
Oct 7, 2025
ellatrix
added
the
Backport to Gutenberg RC
cbravobernal
pushed a commit
that referenced
this pull request
Oct 8, 2025
|
I just cherry-picked this PR to the release/21.8 branch to get it included in the next release: 682b5be |
cbravobernal
removed
the
Backport to Gutenberg RC
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
You can’t perform that action at this time.
What?
The rewrite (#71811) adjusting template resolution use an earlier filter instead of
get_block_templatesshould have also adjusted the/lookupendpoint, which is still using the coreresolve_block_templatefunction. It should use modified version instead.Why?
Currently when you create a new page template and edit a page, that inactive page template will be shown instead of the active theme one. That's because the old resolve template function is still called, and WP thinks that edit should be active.
How?
We override the endpoint callback.
Testing Instructions
Duplicate and edit a page template. Do not activate it. Then create a new page. Use the "show template" preview button. You should see the theme template, not the newly created inactive template.
Testing Instructions for Keyboard
Screenshots or screencast