Context Navigation

← Previous Change
Next Change →

wpHtmlProcessorBreadcrumbs.php

Timestamp:

01/08/2024 02:03:40 PM (3 years ago)

Author:

Bernhard Reiter

Message:

HTML API: Add explicit handling or failure for all tags.

The HTML API HTML processor does not yet support all tags. Many tags (e.g. list elements) have some complicated rules in the "in body" insertion mode.

Implementing these special rules is blocking the implementation for a catch-all rule for "any other tag" because we need to prevent special rules from being handled by the catch-all.

Any other start tag
Reconstruct the active formatting elements, if any.

Insert an HTML element for the token.

…

This change ensures the HTML Processor fails when handling special tags. This is the same as existing behavior, but will allow us to implement the catch-all "any other tag" handling without unintentionally handling special elements.

Additionally, we add tests that assert the special elements are unhandled. As these tags are implemented, this should help to ensure they're removed from the unsupported tag list.

Props jonsurrell, dmsnell.
Fixes #60092.

File:

: 1 edited

trunk/tests/phpunit/tests/html-api/wpHtmlProcessorBreadcrumbs.php (modified) (7 diffs)

Legend:

: Unmodified
: Added
: Removed

trunk/tests/phpunit/tests/html-api/wpHtmlProcessorBreadcrumbs.php

-              r57186
+              r57248
                 $supported_elements = array(
                         'A',
+                        'ABBR',
+                        'ACRONYM', // Neutralized
                         'ADDRESS',
                         'ARTICLE',
                         'ASIDE',
+                        'AUDIO',
                         'B',
+                        'BDI',
+                        'BDO',
                         'BIG',
+                        'BLINK', // Deprecated
                         'BUTTON',
+                        'CANVAS',
                         'CENTER', // Neutralized
+                        'CITE',
                         'CODE',
+                        'DATA',
+                        'DATALIST',
+                        'DFN',
+                        'DEL',
                         'DETAILS',
                         'DIALOG',
 …
                         'I',
                         'IMG',
+                        'INS',
+                        'ISINDEX', // Deprecated
+                        'KBD',
+                        'LABEL',
+                        'LEGEND',
                         'MAIN',
+                        'MAP',
+                        'MARK',
                         'MENU',
+                        'METER',
+                        'MULTICOL', // Deprecated
                         'NAV',
+                        'NEXTID', // Deprecated
+                        'OUTPUT',
                         'P',
+                        'PICTURE',
+                        'PROGRESS',
+                        'Q',
+                        'RUBY',
+                        'SAMP',
                         'SEARCH',
                         'SECTION',
+                        'SLOT',
                         'SMALL',
+                        'SPACER', // Deprecated
                         'SPAN',
                         'STRIKE',
                         'STRONG',
+                        'SUB',
                         'SUMMARY',
+                        'SUP',
+                        'TIME',
                         'TT',
                         'U',
+                        'VAR',
+                        'VIDEO',
                 );
 …
         public function data_unsupported_elements() {
                 $unsupported_elements = array(
-                        'ABBR',
-                        'ACRONYM', // Neutralized
                         'APPLET', // Deprecated
                         'AREA',
-                        'AUDIO',
                         'BASE',
-                        'BDI',
-                        'BDO',
                         'BGSOUND', // Deprecated; self-closing if self-closing flag provided, otherwise normal.
-                        'BLINK', // Deprecated
                         'BODY',
                         'BR',
-                        'CANVAS',
                         'CAPTION',
-                        'CITE',
                         'COL',
                         'COLGROUP',
-                        'DATA',
-                        'DATALIST',
                         'DD',
-                        'DEL',
-                        'DEFN',
                         'DT',
                         'EMBED',
 …
                         'IFRAME',
                         'INPUT',
-                        'INS',
-                        'ISINDEX', // Deprecated
-                        'KBD',
                         'KEYGEN', // Deprecated; void
-                        'LABEL',
-                        'LEGEND',
                         'LI',
                         'LINK',
                         'LISTING', // Deprecated, use PRE instead.
-                        'MAP',
-                        'MARK',
                         'MARQUEE', // Deprecated
                         'MATH',
                         'META',
-                        'METER',
-                        'MULTICOL', // Deprecated
-                        'NEXTID', // Deprecated
                         'NOBR', // Neutralized
                         'NOEMBED', // Neutralized
 …
                         'OPTGROUP',
                         'OPTION',
-                        'OUTPUT',
-                        'PICTURE',
                         'PLAINTEXT', // Neutralized
                         'PRE',
-                        'PROGRESS',
-                        'Q',
                         'RB', // Neutralized
                         'RP',
                         'RT',
                         'RTC', // Neutralized
-                        'RUBY',
-                        'SAMP',
                         'SCRIPT',
                         'SELECT',
-                        'SLOT',
                         'SOURCE',
-                        'SPACER', // Deprecated
                         'STYLE',
-                        'SUB',
-                        'SUP',
                         'SVG',
                         'TABLE',
 …
                         'TH',
                         'THEAD',
-                        'TIME',
                         'TITLE',
                         'TR',
                         'TRACK',
                         'UL',
-                        'VAR',
-                        'VIDEO',
                         'WBR',
                         'XMP', // Deprecated, use PRE instead.
-                        // Made up elements, custom elements.
-                        'X-NOT-AN-HTML-ELEMENT',
-                        'HUMAN-TIME',
                 );
 …
                         'H5 after unclosed H4 inside H2'        => array( '<h2><span>Major<h4>Minor</span></h3><h5 target>', array( 'HTML', 'BODY', 'H2', 'SPAN', 'H5' ), 1 ),
                         'H5 after H4 inside H2'                 => array( '<h2><span>Major<h4>Minor</h4></span></h3><h5 target>', array( 'HTML', 'BODY', 'H5' ), 1 ),
+                        // Custom elements.
+                        'WP-EMOJI'                              => array( '<div><wp-emoji target></wp-emoji></div>', array( 'HTML', 'BODY', 'DIV', 'WP-EMOJI' ), 1 ),
+                        'WP-EMOJI then IMG'                     => array( '<div><wp-emoji></wp-emoji><img target></div>', array( 'HTML', 'BODY', 'DIV', 'IMG' ), 1 ),
                 );
+        }

Note: See TracChangeset for help on using the changeset viewer.

Make WordPress Core

Context Navigation

Changeset 57248 for trunk/tests/phpunit/tests/html-api/wpHtmlProcessorBreadcrumbs.php

Legend:

trunk/tests/phpunit/tests/html-api/wpHtmlProcessorBreadcrumbs.php

Download in other formats: