Skip to content
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

FSE: Add template_type guards #36318

Merged
merged 1 commit into from
Nov 10, 2021
Merged

FSE: Add template_type guards #36318

merged 1 commit into from
Nov 10, 2021

Conversation

ockham
Copy link
Contributor

@ockham ockham commented Nov 8, 2021

Carried over from WordPress/wordpress-develop#1796; see https://github.com/WordPress/wordpress-develop/pull/1796/files#r744135286 for background.

props @hellofromtonya

Description

In two block template theme helper functions that attempt to access theme subdirectories based on $template_type, add guards to make sure that it's either wp_template or wp_template_part.

How has this been tested?

Verify that block-based themes continue to work.

Types of changes

Minor improvement.

@ockham ockham self-assigned this Nov 8, 2021
@ockham ockham requested review from a team, fullofcaffeine and karmatosed and removed request for a team November 8, 2021 17:46
@Mamaduka Mamaduka merged commit b1b4547 into trunk Nov 10, 2021
@Mamaduka Mamaduka deleted the add/template-type-guards branch November 10, 2021 13:22
@github-actions github-actions bot added this to the Gutenberg 12.0 milestone Nov 10, 2021
@ockham ockham added the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 10, 2021
@noisysocks noisysocks removed the Backport to WP 6.7 Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 11, 2021
noisysocks pushed a commit that referenced this pull request Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants