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

Timestamp:

02/22/2023 08:53:41 PM (3 years ago)

Author:

hellofromTonya

Message:

HTML API: Fix finding bookmarks set on closing tag WP_HTML_Tag_Processor.

Setting a bookmark on a tag should set its "start" position before the opening "<", e.g.:

<div> Testing a <b>Bookmark</b>
----------------^

The previous calculation assumed this is always one byte to the left from $tag_name_starts_at.

However, in a closing tag that index points to a solidus symbol "/":

<div> Testing a <b>Bookmark</b>
----------------------------^

The bookmark should therefore start two bytes before the tag name:

<div> Testing a <b>Bookmark</b>
---------------------------^

This changeset achieves this by:

• Using the correct starting index for closing tag bookmarks.
• Adding array( 'tag_closers' => 'visit' ) in WP_HTML_Tag_Processor::seek().

Follow-up to [55203].

Props zieladam, dmsnell, flixos90.
Fixes #57787.
See #57575.

File:

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

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

@@ -723 +723 @@
 
         $this->bookmarks[ $name ] = new WP_HTML_Span(
-            $this->tag_name_starts_at - 1,
+            $this->tag_name_starts_at - ( $this->is_closing_tag ? 2 : 1 ),
             $this->tag_ends_at
         );
@@ -1505 +1505 @@
         $this->bytes_already_copied = $this->bytes_already_parsed;
         $this->output_buffer        = substr( $this->html, 0, $this->bytes_already_copied );
-        return $this->next_tag();
+        return $this->next_tag( array( 'tag_closers' => 'visit' ) );
     }