Skip to content

HTML API: Detect CDATA by adjusted node namespace#38

Open
sirreal wants to merge 1 commit into
trunkfrom
fix/html-api-cdata-namespace
Open

HTML API: Detect CDATA by adjusted node namespace#38
sirreal wants to merge 1 commit into
trunkfrom
fix/html-api-cdata-namespace

Conversation

@sirreal

@sirreal sirreal commented Jun 9, 2026

Copy link
Copy Markdown
Owner

Summary

  • tracks CDATA eligibility separately from integration-point HTML tokenization
  • preserves SVG/MathML namespaces for CDATA sections inside HTML integration points
  • adds regression coverage for SVG foreignObject and MathML annotation-xml

Testing

  • WP_TESTS_SKIP_INSTALL=1 ./vendor/bin/phpunit --filter 'test_cdata_sections_in_(svg|mathml)_html_integration_points' tests/phpunit/tests/html-api/wpHtmlProcessor.php
  • WP_TESTS_SKIP_INSTALL=1 ./vendor/bin/phpunit tests/phpunit/tests/html-api/wpHtmlProcessor.php
  • ./vendor/bin/phpcs --standard=phpcs.xml.dist src/wp-includes/html-api/class-wp-html-processor.php src/wp-includes/html-api/class-wp-html-tag-processor.php tests/phpunit/tests/html-api/wpHtmlProcessor.php
  • codex review --base trunk

gemini-code-assist[bot]

This comment was marked as off-topic.

@sirreal sirreal marked this pull request as ready for review June 9, 2026 20:05
@github-actions

This comment was marked as off-topic.

@sirreal

sirreal commented Jun 10, 2026

Copy link
Copy Markdown
Owner Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

1 similar comment
@sirreal

sirreal commented Jun 10, 2026

Copy link
Copy Markdown
Owner Author

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

🤖 Generated with Claude Code

- If this code review was useful, please react with 👍. Otherwise, react with 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant