Changeset 42860

Timestamp:

03/20/2018 09:34:15 PM (7 years ago)

Author:

SergeyBiryukov

Message:

Formatting: Avoid a PHP 7.2 warning in wp_kses_attr() when one of $allowedtags elements is an uncountable value.

Props andrei0x309, soulseekah, SergeyBiryukov.
Fixes #43312.

Location:

trunk

Files:

• src/wp-includes/kses.php (modified) (1 diff)
• tests/phpunit/tests/kses.php (modified) (1 diff)

trunk/src/wp-includes/kses.php

@@ -1048 +1048 @@
 
     // Are any attributes allowed at all for this element?
-    if ( ! isset( $allowed_html[ strtolower( $element ) ] ) || true === $allowed_html[ strtolower( $element ) ] || count( $allowed_html[ strtolower( $element ) ] ) == 0 ) {
+    $element_low = strtolower( $element );
+    if ( empty( $allowed_html[ $element_low ] ) || true === $allowed_html[ $element_low ] ) {
         return "<$element$xhtml_slash>";
     }

trunk/tests/phpunit/tests/kses.php

@@ -723 +723 @@
         $this->assertEquals( "<{$element} title=\"foo\">", wp_kses_attr( $element, $attribute, array( 'foo' => array( 'title' => true ) ), array() ) );
     }
+
+    /**
+     * @ticket 43312
+     */
+    function test_wp_kses_attr_no_attributes_allowed_with_false() {
+        $element   = 'foo';
+        $attribute = 'title="foo" class="bar"';
+
+        $this->assertEquals( "<{$element}>", wp_kses_attr( $element, $attribute, array( 'foo' => false ), array() ) );
+    }
 }