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

Timestamp:

02/02/2024 10:55:27 PM (2 years ago)

Author:

dmsnell

Message:

HTML API: Reset parser state after seeking to bookmark.

When parser states were introduced, nothing in the seek() method reset the
parser state. This is problematic because it could leave the parser in the
wrong state.

In this patch the parser state is reset so that it's properly adjusted on
the successive call to next_token().

Developed in ​https://github.com/WordPress/wordpress-develop/pull/6021
Discussed in https://core.trac.wordpress.org/ticket/60428

Follow-up to [57211]

Props dmsnell, kevin940726
Fixes #60428

File:

• trunk/src/wp-includes/html-api/class-wp-html-tag-processor.php (modified) (1 diff)

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

@@ -2336 +2336 @@
         // Point this tag processor before the sought tag opener and consume it.
         $this->bytes_already_parsed = $this->bookmarks[ $bookmark_name ]->start;
+        $this->parser_state         = self::STATE_READY;
         return $this->next_token();
     }