Changeset 59535 for branches/6.7

Timestamp:

12/18/2024 06:38:58 PM (20 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' ) ) );
+        }
 }