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

Timestamp:

07/29/2024 05:37:48 PM (2 years ago)

Author:

dmsnell

Message:

HTML API: Close all elements at the end of a document.

When the model of breadcrumb generation in the HTML Processor and node
traversal was simplified, the change introduced a bug whereby unclosed
nodes at the end of a document would remain unvisited and unclosed.

In this patch, a fix is applied to ensure that all open elements close
while traversing a document. A couple of minor documentation typos are
fixed in the patch as well.

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

Follow-up to [58713].

Props: dmsnell, gziolo, jonsurrell.
See #61576.

File:

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

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

@@ -610 +610 @@
                  *       tokens works in the meantime and isn't obviously wrong.
                  */
-                while ( empty( $this->element_queue ) && $this->step() ) {
-                        continue;
+                if ( empty( $this->element_queue ) && $this->step() ) {
+                        return $this->next_token();
                 }
 
@@ -617 +617 @@
                 $this->current_element = array_shift( $this->element_queue );
                 if ( ! isset( $this->current_element ) ) {
-                        return false;
+                        // There are no tokens left, so close all remaining open elements.
+                        while ( $this->state->stack_of_open_elements->pop() ) {
+                                continue;
+                        }
+
+                        return empty( $this->element_queue ) ? false : $this->next_token();
                 }
 
@@ -624 +629 @@
                 /*
                  * The root node only exists in the fragment parser, and closing it
-                 * indicates that the parse is complete. Stop before popping if from
+                 * indicates that the parse is complete. Stop before popping it from
                  * the breadcrumbs.
                  */
                 if ( 'root-node' === $this->current_element->token->bookmark_name ) {
-                        return ! $is_pop && $this->next_token();
+                        return $this->next_token();
                 }
 
@@ -639 +644 @@
 
                 // Avoid sending close events for elements which don't expect a closing.
-                if ( $is_pop && ! static::expects_closer( $this->current_element->token ) ) {
+                if ( $is_pop && ! $this->expects_closer( $this->current_element->token ) ) {
                         return $this->next_token();
                 }