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

Timestamp:

07/29/2024 05:37:48 PM (12 months 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();
         }