Skip to content

Templates: Deprecate _inject_theme_attribute_in_block_template_content(). #5307

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.

Sign up for GitHub

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

Closed
ockham wants to merge 4 commits into from
Closed

Templates: Deprecate _inject_theme_attribute_in_block_template_content(). #5307

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6c83a13
Templates: Deprecate _inject_theme_attribute_in_template_part_block().
ockham Sep 26, 2023
8cd9350
Add expectedDeprecated
ockham Sep 26, 2023
5572a1d
Add 3rd arg to deprecation notice
ockham Sep 26, 2023
eef7056
More PHPDoc
ockham Sep 26, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Templates: Deprecate _inject_theme_attribute_in_template_part_block().
  • Loading branch information
@ockham
ockham committed Sep 26, 2023
commit 6c83a133cbee9a7bbb22d9fb5080e2d73a4c7ba1
3 changes: 3 additions & 0 deletions src/wp-includes/block-template-utils.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -475,12 +475,15 @@ function _flatten_blocks( &$blocks ) {
* stylesheet as a theme attribute into each wp_template_part
*
* @since 5.9.0
* @deprecated 6.4.0 Use _inject_theme_attribute_in_template_part_block() with traverse_and_serialize_blocks() instead.
* @access private
*
* @param string $template_content serialized wp_template content.
* @return string Updated 'wp_template' content.
*/
function _inject_theme_attribute_in_block_template_content( $template_content ) {
_deprecated_function( __FUNCTION__, '6.4.0' );
Copy link
Member

@gziolo gziolo Sep 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you can also put the message with the new recommendation as 3rd argument in the call.

Copy link
Contributor Author

@ockham ockham Sep 26, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5572a1d


$has_updated_content = false;
$new_content = '';
$template_blocks = parse_blocks( $template_content );
Expand Down