-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Update the packages for 5.8 beta1 batch 2 #1351
Update the packages for 5.8 beta1 batch 2 #1351
Conversation
*/ | ||
function gutenberg_register_legacy_query_loop_block() { | ||
$registry = WP_Block_Type_Registry::get_instance(); | ||
if ( $registry->is_registered( 'core/query-loop' ) ) { |
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.
Since we are removing now query-loop
from core we can loose this check.
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 file is automatically brought from Gutenberg, so we can't do this here.
Another thing is that the function still has the gutenberg_
prefix (I think this should be removed in a follow-up).
These changes can be done after beta1 though.
@@ -318,7 +318,8 @@ function _unhook_block_registration() { | |||
remove_action( 'init', 'register_block_core_post_terms' ); | |||
remove_action( 'init', 'register_block_core_post_title' ); | |||
remove_action( 'init', 'register_block_core_query' ); | |||
remove_action( 'init', 'register_block_core_query_loop' ); | |||
remove_action( 'init', 'register_block_core_post_template' ); | |||
remove_action( 'init', 'gutenberg_register_legacy_query_loop_block' ); |
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.
Should we remove the gutenberg
prefix here?
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.
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.
Tested and looks good without the GB plugin with existing and new Query blocks
committed. |
This updates WP to the latest WP packages. It includes the backports from WordPress/gutenberg#32508
Trac ticket: https://core.trac.wordpress.org/ticket/52991
See the diff in https://github.com/WordPress/gutenberg/pull/32508/files