Changeset 55407 for trunk/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.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/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.php (modified) (1 diff)

trunk/tests/phpunit/tests/html-api/wpHtmlTagProcessor-bookmark.php

@@ -61 +61 @@
             $p->get_updated_html(),
             'Did not seek to the intended bookmark locations'
+        );
+    }
+
+    /**
+     * @ticket 57787
+     *
+     * @covers WP_HTML_Tag_Processor::seek
+     */
+    public function test_seeks_to_tag_closer_bookmark() {
+        $p = new WP_HTML_Tag_Processor( '<div>First</div><span>Second</span>' );
+        $p->next_tag( array( 'tag_closers' => 'visit' ) );
+        $p->set_bookmark( 'first' );
+        $p->next_tag( array( 'tag_closers' => 'visit' ) );
+        $p->set_bookmark( 'second' );
+
+        $p->seek( 'first' );
+        $p->seek( 'second' );
+
+        $this->assertSame(
+            'DIV',
+            $p->get_tag(),
+            'Did not seek to the intended bookmark location'
         );
     }