Skip to content

Block Bindings: Allow more generic setting of block attributes (use WP_HTML_Text_Replacement) #6

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.

Sign up for GitHub

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

Closed
ockham wants to merge 19 commits into from
Closed

Block Bindings: Allow more generic setting of block attributes (use WP_HTML_Text_Replacement) #6

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
2d7b4f3
Introduce data provider to allow extending test coverage
ockham Aug 13, 2025
a137176
Add test coverage for Button block's text attribute
ockham Aug 13, 2025
83b3ae1
Block Bindings: Simplify replace_html() method
ockham Aug 13, 2025
15d619b
Add WP_Block_Bindings_Processor class
ockham Aug 18, 2025
9dc0aba
Use WP_Block_Bindings_Processor for block bindings
ockham Aug 18, 2025
d9132f8
Add kses back :/
ockham Aug 18, 2025
c011d25
WPCS
ockham Aug 18, 2025
9295227
Remove obsolete var
ockham Aug 18, 2025
3c3f908
Indentation
ockham Aug 18, 2025
c46300d
Return true upon success
ockham Aug 18, 2025
f528702
Add basic PHPDoc
ockham Aug 18, 2025
73efaa7
Add more PHPDoc
ockham Aug 19, 2025
d5798c1
Add basic test coverage
ockham Aug 19, 2025
26a0a29
Allow setting attributes
ockham Aug 19, 2025
0dbe235
Increase test coverage
ockham Aug 19, 2025
3b9eac6
Increase test coverage
ockham Aug 20, 2025
e98185e
Add more commentary and warnings
ockham Aug 20, 2025
c6acfad
Block Bindings Processor: Tweak test to break with current implementa…
ockham Aug 20, 2025
1a61046
Block Bindings Processor: Base implementation on WP_HTML_Text_Replace…
ockham Aug 20, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Introduce data provider to allow extending test coverage
  • Loading branch information
@ockham
ockham committed Aug 13, 2025
commit 2d7b4f3f06beab2859f24afef76774661bff3f6a
42 changes: 31 additions & 11 deletions tests/phpunit/tests/block-bindings/render.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,14 +61,30 @@ public static function wpTearDownAfterClass() {
unregister_block_type( 'test/block' );
}

public function data_update_block_with_value_from_source() {
return array(
'paragraph block' => array(
'content',
<<<HTML
<!-- wp:paragraph -->
<p>This should not appear</p>
<!-- /wp:paragraph -->
HTML,
'<p>test source value</p>'
)
);
}

/**
* Test if the block content is updated with the value returned by the source.
*
* @ticket 60282
*
* @covers ::register_block_bindings_source
*
* @dataProvider data_update_block_with_value_from_source
*/
public function test_update_block_with_value_from_source() {
public function test_update_block_with_value_from_source( $bound_attribute, $block_content, $expected_result ) {
$get_value_callback = function () {
return 'test source value';
};
Expand All @@ -81,22 +97,26 @@ public function test_update_block_with_value_from_source() {
)
);

$block_content = <<<HTML
<!-- wp:paragraph {"metadata":{"bindings":{"content":{"source":"test/source"}}}} -->
<p>This should not appear</p>
<!-- /wp:paragraph -->
HTML;
$parsed_blocks = parse_blocks( $block_content );
$block = new WP_Block( $parsed_blocks[0] );
$result = $block->render();

$parsed_blocks[0]['attrs']['metadata'] = array(
'bindings' => array(
$bound_attribute => array(
'source' => self::SOURCE_NAME,
),
)
);

$block = new WP_Block( $parsed_blocks[0] );
$result = $block->render();

$this->assertSame(
'test source value',
$block->attributes['content'],
"The 'content' attribute should be updated with the value returned by the source."
$block->attributes[ $bound_attribute ],
"The '{$bound_attribute}' attribute should be updated with the value returned by the source."
);
$this->assertSame(
'<p>test source value</p>',
$expected_result,
trim( $result ),
'The block content should be updated with the value returned by the source.'
);
Expand Down