Changeset 58841

Timestamp:

08/01/2024 10:51:11 PM (2 years ago)

Author:

dmsnell

Message:

HTML API: Add support for IN SELECT IN TABLE parsing.

As part of work to add more spec support to the HTML API, this patch adds
support for the IN SELECT IN TABLE insertion mode. This small section of the
spec handles rules for the <select> element and its children when found
inside of a <table>.

Developed in ​https://github.com/wordpress/wordpress-develop/pull/7044
Discussed in https://core.trac.wordpress.org/ticket/61576

Props: dmsnell, jonsurrell.
See #61576.

File:

• trunk/src/wp-includes/html-api/class-wp-html-processor.php (modified) (2 diffs)

trunk/src/wp-includes/html-api/class-wp-html-processor.php

@@ -3772 +3772 @@
          * logic for the generalized WP_HTML_Processor::step() function.
          *
-         * @since 6.7.0 Stub implementation.
+         * @since 6.7.0
          *
          * @throws WP_HTML_Unsupported_Exception When encountering unsupported HTML input.
@@ -3782 +3782 @@
          */
         private function step_in_select_in_table(): bool {
-                $this->bail( 'No support for parsing in the ' . WP_HTML_Processor_State::INSERTION_MODE_IN_SELECT_IN_TABLE . ' state.' );
+                $token_name = $this->get_token_name();
+                $token_type = $this->get_token_type();
+                $op_sigil   = '#tag' === $token_type ? ( parent::is_tag_closer() ? '-' : '+' ) : '';
+                $op         = "{$op_sigil}{$token_name}";
+
+                switch ( $op ) {
+                        /*
+                         * > A start tag whose tag name is one of: "caption", "table", "tbody", "tfoot", "thead", "tr", "td", "th"
+                         */
+                        case '+CAPTION':
+                        case '+TABLE':
+                        case '+TBODY':
+                        case '+TFOOT':
+                        case '+THEAD':
+                        case '+TR':
+                        case '+TD':
+                        case '+TH':
+                                // @todo Indicate a parse error once it's possible.
+                                $this->state->stack_of_open_elements->pop_until( 'SELECT' );
+                                $this->reset_insertion_mode();
+                                return $this->step( self::REPROCESS_CURRENT_NODE );
+
+                        /*
+                         * > An end tag whose tag name is one of: "caption", "table", "tbody", "tfoot", "thead", "tr", "td", "th"
+                         */
+                        case '-CAPTION':
+                        case '-TABLE':
+                        case '-TBODY':
+                        case '-TFOOT':
+                        case '-THEAD':
+                        case '-TR':
+                        case '-TD':
+                        case '-TH':
+                                // @todo Indicate a parse error once it's possible.
+                                if ( ! $this->state->stack_of_open_elements->has_element_in_table_scope( $token_name ) ) {
+                                        return $this->step();
+                                }
+                                $this->state->stack_of_open_elements->pop_until( 'SELECT' );
+                                $this->reset_insertion_mode();
+                                return $this->step( self::REPROCESS_CURRENT_NODE );
+                }
+
+                /*
+                 * > Anything else
+                 */
+                return $this->step_in_select();
         }