Skip to content

HTML API: Fix CDATA lookalike matching invalid CDATA #5992

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
sirreal wants to merge 8 commits into from
Closed

HTML API: Fix CDATA lookalike matching invalid CDATA #5992

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
0b1c43e
Add failing test
sirreal Jan 31, 2024
7b370bb
fixup! Add failing test
sirreal Jan 31, 2024
fc01129
Fix bad CDATA comment
sirreal Jan 31, 2024
058e29a
Fix up test, add ticket number
sirreal Jan 31, 2024
b24f42d
Add invalid html comment type assertion to another test
sirreal Jan 31, 2024
4caaebf
Describe indicies example comment
sirreal Jan 31, 2024
d4150b0
Remove diagram in comment.
dmsnell Jan 31, 2024
31ade96
Merge remote-tracking branch 'upstream/trunk' into html-api/fix-cdata…
dmsnell Feb 1, 2024
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
fixup! Add failing test
  • Loading branch information
@sirreal
sirreal committed Jan 31, 2024
commit 7b370bb3fb4d7ddb54b6a1a0bb8a1a0bdd0ba7cb
4 changes: 2 additions & 2 deletions tests/phpunit/tests/html-api/wpHtmlTagProcessor-token-scanning.php
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -357,7 +357,7 @@ public function test_basic_assertion_cdata_section() {
* @covers WP_HTML_Tag_Processor::next_token
*/
public function test_cdata_comment_with_incorrect_closer() {
$processor = new WP_HTML_Tag_Processor( '<![CDATA[this "cdata" is missing a closing square bracket. It should end at the first ">" ]>' );
$processor = new WP_HTML_Tag_Processor( '<![CDATA[this is missing a closing square bracket]>' );
$processor->next_token();

$this->assertSame(
Expand All @@ -373,7 +373,7 @@ public function test_cdata_comment_with_incorrect_closer() {
);

$this->assertSame(
'[CDATA[this "cdata" is missing a closing square bracket. It should end at the first "',
'[CDATA[this is missing a closing square bracket]',
$processor->get_modifiable_text(),
'Found incorrect modifiable text.'
);
Expand Down