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

Timestamp:

10/23/2024 10:03:23 PM (7 months ago)

Author:

westonruter

Message:

HTML API: Fix extensibility of WP_HTML_Processor::next_token().

Break out logic from the next_token() method into a private method which may call itself recursively. This allows for subclasses to override the next_token() method and be assured that each call to next_token() corresponds with the consumption of one single token. This also parallels how WP_HTML_Tag_Processor::next_token() wraps a private base_class_next_token() method.

Props westonruter, jonsurrell.
Fixes #62269.

File:

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

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

@@ -605 +605 @@
 
     /**
-     * Ensures internal accounting is maintained for HTML semantic rules while
-     * the underlying Tag Processor class is seeking to a bookmark.
+     * Finds the next token in the HTML document.
      *
      * This doesn't currently have a way to represent non-tags and doesn't process
@@ -613 +612 @@
      *
      * @since 6.5.0 Added for internal support; do not use.
+     * @since 6.7.1 Refactored so subclasses may extend.
+     *
+     * @return bool Whether a token was parsed.
+     */
+    public function next_token(): bool {
+        return $this->_next_token();
+    }
+
+    /**
+     * Ensures internal accounting is maintained for HTML semantic rules while
+     * the underlying Tag Processor class is seeking to a bookmark.
+     *
+     * This doesn't currently have a way to represent non-tags and doesn't process
+     * semantic rules for text nodes. For access to the raw tokens consider using
+     * WP_HTML_Tag_Processor instead.
+     *
+     * @since 6.7.1 Added for internal support; do not use.
      *
      * @access private
@@ -618 +634 @@
      * @return bool
      */
-    public function next_token(): bool {
+    private function _next_token(): bool {
         $this->current_element = null;
 
@@ -636 +652 @@
          */
         if ( empty( $this->element_queue ) && $this->step() ) {
-            return $this->next_token();
+            return $this->_next_token();
         }
 
@@ -647 +663 @@
             }
 
-            return empty( $this->element_queue ) ? false : $this->next_token();
+            return empty( $this->element_queue ) ? false : $this->_next_token();
         }
 
@@ -658 +674 @@
          */
         if ( 'root-node' === $this->current_element->token->bookmark_name ) {
-            return $this->next_token();
+            return $this->_next_token();
         }
 
@@ -670 +686 @@
         // Avoid sending close events for elements which don't expect a closing.
         if ( $is_pop && ! $this->expects_closer( $this->current_element->token ) ) {
-            return $this->next_token();
+            return $this->_next_token();
         }