feat: add restrictBlocksForPostTypes to Gutenberg.php

[YvetteNikolov]

Voor context, zie deze PR: yardinternet/brave#79

Je kunt nu via de gutenberg.php config file blocks restricten op basis van post type.

[Copilot]

Pull request overview

This PR adds a Gutenberg hook to restrict allowed blocks per WordPress post type using values from the gutenberg.php configuration, extending the existing Gutenberg-related config-driven behavior in this package.

Changes:

• Add an allowed_block_types_all filter handler to derive an allowed block list from gutenberg.postTypeBlockRestrictions.
• Support composing the final allowlist from a configured block set plus optional add/remove overrides.
• Attempt to respect earlier filters by intersecting when $allowedBlockTypes is already an array.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

When $allowedBlockTypes is true (WordPress default: allow all blocks), this method computes $finalAllowedBlocks but ultimately returns the original boolean, so the post type restriction is never applied. Consider returning $finalAllowedBlocks when $allowedBlockTypes === true, and preserving false when $allowedBlockTypes === false (and still intersecting when it is an array).

Suggested change

	// When WordPress passes `true` (allow all by default), apply our computed restriction.
	if ($allowedBlockTypes === true) {
	return $finalAllowedBlocks;
	}
	// Preserve explicit `false` (no blocks allowed) from earlier filters.

[Copilot]

$blockSet is validated (including trim()), but the config lookup uses $restriction['blockSet'] instead. This can cause unexpected behavior (e.g., whitespace in the config value passes validation but looks up the wrong key) and duplicates logic. Use the sanitized $blockSet variable consistently for the gutenberg.blockSets.* lookup.

Suggested change

	$blockSet = $restriction['blockSet'] ?? null;
	if (! is_string($blockSet) || '' === trim($blockSet)) {
	return $allowedBlockTypes;
	}
	$baseBlocks = config("gutenberg.blockSets.{$restriction['blockSet']}", []);
	$blockSet = isset($restriction['blockSet']) ? trim((string) $restriction['blockSet']) : null;
	if (! is_string($blockSet) || '' === $blockSet) {
	return $allowedBlockTypes;
	}
	$baseBlocks = config("gutenberg.blockSets.{$blockSet}", []);

[mvdhoek1]

⏫

[Copilot]

$baseBlocks is assumed to be an array, but if the gutenberg.blockSets.* config value is misconfigured (non-array), the array unpack (...$baseBlocks) will throw a TypeError. Add an is_array($baseBlocks) guard (or cast to array) before unpacking.

[Copilot]

Docblock grammar: "Restrict blocks based for post types" reads incorrectly. Consider changing to "Restrict blocks based on post types" (or "Restrict blocks for post types").

Suggested change

	* Restrict blocks based for post types through the gutenberg.php config file.
	* Restrict blocks for post types through the gutenberg.php config file.

[mvdhoek1]

AI doet het reviewen al voor ons haha, ik approve hem vast onder voorbehoud van de comments.

[mvdhoek1]

⏫