Reuse block metadata in WP_Theme_JSON::get_valid_block_style_variations() for better performance

[mukesh27]

In WP_Theme_JSON::get_valid_block_style_variations(), the method was calling self::get_blocks_metadata() even though the metadata was already retrieved in the parent function. This update reuses the existing block metadata instead of calling it again.

A new optional parameter, $blocks_metadata, has been added to the function, allowing it to use pre-fetched metadata when available, improving efficiency.

Fewer self::get_blocks_metadata() calls mean faster processing, especially in themes with many blocks.

Let me know if you need help backporting to the plugin.

Yes please if you have moment. Thanks

Gutenberg backport PR:

• ​https://github.com/WordPress/gutenberg/pull/66539

Is this ready to be committed? The ​Gutenberg backport PR is merged.

Is this ready to be committed? The ​Gutenberg backport PR is merged.

Yes. If you have moment then commit it.

[ramonopoly]

In 59359:

Performance: reuse block metadata in WP_Theme_JSON::get_valid_block_style_variations()

In WP_Theme_JSON::get_valid_block_style_variations(), the method was calling self::get_blocks_metadata() even though the metadata was already retrieved in the parent function. This update reuses the existing block metadata instead of calling it again.

A new optional parameter, $blocks_metadata, has been added to the function, allowing it to use pre-fetched metadata when available, improving efficiency.

Fewer self::get_blocks_metadata() calls mean faster processing, especially in themes with many blocks.

Props mukesh27, ramonopoly, aaronrobertshaw, flixos90.
Fixes #62291.

Committed in r59359 / 5ff971744e4b2d246c66f58d436cac5eff2a0662