Add support for `ancestor` block.json property

[dd32]

​As reported in Slack, the block directory validator is rejecting plugins which use the new WordPress 6.0+ ancestor property instead of the more limiting parent property.

This results in plugins that may be eligible for the block directory being rejected as containing multiple blocks.

This also appears to cause the plugins listing to list all of the blocks, rather than just the parent blocks.

Here's an example block.json for a block that was detected as being a non-child block from the Slack thread: ​https://plugins.trac.wordpress.org/browser/gutena-forms/trunk/build/form-field/block.json?rev=2777775

[tellyworth]

Add is_top_level_block function with parent/ancestor logic

[tellyworth]

The patch attachment:6465.diff​ seems to do the trick. It essentially treats parent and ancestor as the same. It's a bit stricter than the old check though, requiring that the parent/ancestor block exists within the same plugin. That's so that a block with a parent like core/column is correctly treated as top-level for validation purposes.

It works ok for me at a quick test; do I have the logic right?

[dd32]

It works ok for me at a quick test; do I have the logic right?

I think the logic is right. For the list below, parent means ancestor or parent.

• If the block has no parent it's top-level
• If the block has a parent that's not provided by the plugin, it's top-level.
• If the block has a parent that's provided by the plugin, it's child.

A plugin with a block that requires a parent element that's provided by another plugin will be treated as a top-level block here (ie. A plugin provides "File Upload Block" for "Form Builder Plugin") that'll be treated as top-level here, which seems somewhat appropriate.

[randhirexpresstech]

Thanks! @dd32 and @tellyworth for fixing this issue. May I test this fix on live block-plugin-validator ​https://wordpress.org/plugins/developers/block-plugin-validator/.

[dd32]

@randhirexpresstech This patch still needs to be deployed to production, you'll see a followup here when that happens and you can then test it in the Block validator.

[tellyworth]

In 12356:

Plugin Directory: better child block handling in validator.

Fixes #6465.

[randhirexpresstech]

Hi! @dd32 and @tellyworth . There are some issue with on-page block plugin search, and insertion. Will you please look into this ticket: https://meta.trac.wordpress.org/ticket/6735.
Thanks