Changeset 44293

Timestamp:

12/18/2018 04:45:54 PM (5 years ago)

Author:

desrosj

Message:

KSES: Conditionally remove the <form> element from $allowedposttags.

To avoid backwards compatibility issues, <form> is re-added if a custom filter has added the <input> or <select> elements to $allowedposttags.

Merges [43994] to trunk.

Location:

trunk

Files:

• . (modified) (1 prop)
• src/wp-includes/kses.php (modified) (3 diffs)

trunk/src/wp-includes/kses.php

@@ -187 +187 @@
             'lang'     => true,
             'xml:lang' => true,
-        ),
-        'form'       => array(
-            'action'         => true,
-            'accept'         => true,
-            'accept-charset' => true,
-            'enctype'        => true,
-            'method'         => true,
-            'name'           => true,
-            'target'         => true,
         ),
         'h1'         => array(
@@ -825 +816 @@
  *
  * @since 3.5.0
+ * @since 5.0.1 `form` removed as allowable HTML tag.
  *
  * @global array $allowedposttags
@@ -853 +845 @@
         case 'post':
             /** This filter is documented in wp-includes/kses.php */
-            return apply_filters( 'wp_kses_allowed_html', $allowedposttags, $context );
+            $tags = apply_filters( 'wp_kses_allowed_html', $allowedposttags, $context );
+
+            // 5.0.1 removed the `<form>` tag, allow it if a filter is allowing it's sub-elements `<input>` or `<select>`.
+            if ( ! CUSTOM_TAGS && ! isset( $tags['form'] ) && ( isset( $tags['input'] ) || isset( $tags['select'] ) ) ) {
+                $tags = $allowedposttags;
+
+                $tags['form'] = array(
+                    'action' => true,
+                    'accept' => true,
+                    'accept-charset' => true,
+                    'enctype' => true,
+                    'method' => true,
+                    'name' => true,
+                    'target' => true,
+                );
+
+                /** This filter is documented in wp-includes/kses.php */
+                $tags = apply_filters( 'wp_kses_allowed_html', $tags, $context );
+            }
+
+            return $tags;
 
         case 'user_description':