Skip to content

Add theme_json_ prefix to hook name#3261

Closed
oandregal wants to merge 1 commit intoWordPress:trunkfrom
oandregal:update/get-style-nodes-hook-name
Closed

Add theme_json_ prefix to hook name#3261
oandregal wants to merge 1 commit intoWordPress:trunkfrom
oandregal:update/get-style-nodes-hook-name

Conversation

@oandregal
Copy link
Copy Markdown
Member

Trac ticket: https://core.trac.wordpress.org/ticket/56467
Backports WordPress/gutenberg#44189

This PR consolidates the name of a filter introduced in this cycle, renaming it from get_style_nodes to theme_json_get_style_nodes as to be more coherent with the other hooks that we introduce in 6.1 as well, see #3247

Copy link
Copy Markdown
Contributor

@ockham ockham left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you, @oandregal!

@SergeyBiryukov
Copy link
Copy Markdown
Member

Thanks for the PR! Merged in r54183.

@oandregal oandregal deleted the update/get-style-nodes-hook-name branch September 16, 2022 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects

Development

Successfully merging this pull request may close these issues.

3 participants