Skip to content

Block Hooks: Persist information about ignoredHookedBlocks in anchor block metadata #5712

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 Hooks: Persist information about ignoredHookedBlocks in anchor block metadata #5712

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
c74cdb2
Extract get_hooked_block_markup function
ockham Nov 28, 2023
552c66e
Set ignoredHookedBlocks metadata
ockham Nov 28, 2023
498dcf2
Rewrite a bit
ockham Nov 28, 2023
450c05f
Rewrite a bit more
ockham Nov 28, 2023
bef2301
Add ticket PHPDoc
ockham Nov 28, 2023
061cbd6
Inject hooked blocks into modified templates and parts
ockham Nov 28, 2023
acbd7f2
Relax test assertion a bit
ockham Nov 28, 2023
9e22a87
Get tests to pass
ockham Nov 28, 2023
ca6b474
Fix test
ockham Nov 28, 2023
3ad8dce
Relax tests
ockham Nov 28, 2023
df1fc75
Add some assertions for ignoredHookedBlocks
ockham Nov 28, 2023
9339ea2
Rename test
ockham Nov 28, 2023
5289f74
Add PHPDoc
ockham Nov 28, 2023
7dc3dfa
Tabs vs spaces
ockham Nov 28, 2023
dc35659
Missing .php file extension :facepalm:
ockham Nov 29, 2023
ef73917
Add test case
ockham Nov 29, 2023
41c2f1c
Coding Standards
ockham Nov 30, 2023
f3dbb7a
Revert "Inject hooked blocks into modified templates and parts"
ockham Dec 4, 2023
f4379e1
Mark get_hooked_block_markup() as private
ockham Dec 4, 2023
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
Prev Previous commit
Next Next commit
Get tests to pass
  • Loading branch information
@ockham
ockham committed Nov 28, 2023
commit 9e22a872614dba1bd417a437ee0cde02a30f62cc
4 changes: 2 additions & 2 deletions tests/phpunit/tests/blocks/getHookedBlocks.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@ public function test_loading_template_with_hooked_blocks() {
$template->content
);
$this->assertStringContainsString(
'<!-- wp:post-content {"layout":{"type":"constrained"}} /-->'
'<!-- wp:post-content {"layout":{"type":"constrained"},"metadata":{"ignoredHookedBlocks":["tests/hooked-after"]}} /-->'
. '<!-- wp:tests/hooked-after /-->',
$template->content
);
Expand Down Expand Up @@ -215,7 +215,7 @@ public function test_loading_pattern_with_hooked_blocks() {
$pattern['content']
);
$this->assertStringContainsString(
'<!-- wp:comments -->'
'<!-- wp:comments {"metadata":{"ignoredHookedBlocks":["tests/hooked-first-child"]}} -->'
. '<div class="wp-block-comments">'
. '<!-- wp:tests/hooked-first-child /-->',
str_replace( array( "\n", "\t" ), '', $pattern['content'] )
Expand Down