Changeset 59535

Timestamp:

12/18/2024 06:38:58 PM (3 months ago)

Author:

desrosj

Message:

HTML API: Use case insensitive tag_name comparison in ::next_tag.

The HTML API ::next_tag method now performs case-insensitive matching when searching for tags by name. For example, searching for 'DIV' will match both '<div>' and '<DIV>' tags.

Reviewed by desrosj.
Merges [59422] to the 6.7 branch.

Props jonsurrell, dmsnell, czapla.
Fixes #62427.

Location:

branches/6.7

Files:

• . (modified) (1 prop)
• src/wp-includes/html-api/class-wp-html-processor.php (modified) (1 diff)
• tests/phpunit/tests/html-api/wpHtmlProcessor.php (modified) (1 diff)

branches/6.7/src/wp-includes/html-api/class-wp-html-processor.php

@@ -558 +558 @@
         }
 
+        if ( isset( $query['tag_name'] ) ) {
+            $query['tag_name'] = strtoupper( $query['tag_name'] );
+        }
+
         $needs_class = ( isset( $query['class_name'] ) && is_string( $query['class_name'] ) )
             ? $query['class_name']

branches/6.7/tests/phpunit/tests/html-api/wpHtmlProcessor.php

@@ -883 +883 @@
         $this->assertTrue( $processor->is_tag_closer() );
     }
+
+    /**
+     * Ensure that lowercased tag_name query matches tags case-insensitively.
+     *
+     * @group 62427
+     */
+    public function test_next_tag_lowercase_tag_name() {
+        // The upper case <DIV> is irrelevant but illustrates the case-insentivity.
+        $processor = WP_HTML_Processor::create_fragment( '<section><DIV>' );
+        $this->assertTrue( $processor->next_tag( array( 'tag_name' => 'div' ) ) );
+
+        // The upper case <RECT> is irrelevant but illustrates the case-insentivity.
+        $processor = WP_HTML_Processor::create_fragment( '<svg><RECT>' );
+        $this->assertTrue( $processor->next_tag( array( 'tag_name' => 'rect' ) ) );
+    }
 }