Changeset 36097 for trunk/src/wp-includes/shortcodes.php

Timestamp:

12/26/2015 04:45:26 AM (9 years ago)

Author:

dd32

Message:

Shortcodes: = is a reserved character in shortcode names, mark it as such.

This allows for shortcodes such as [shortcode=attribute] to work, which while never intentionally supported were widely used in the pre-shortcode days.

Props aaroncampbell.
Fixes #34939 for trunk.

File:

• trunk/src/wp-includes/shortcodes.php (modified) (3 diffs)

trunk/src/wp-includes/shortcodes.php

@@ -96 +96 @@
     }
 
-    if ( 0 !== preg_match( '@[<>&/\[\]\x00-\x20]@', $tag ) ) {
-        /* translators: %s: shortcode name */
-        $message = sprintf( __( 'Invalid shortcode name: %s. Do not use spaces or reserved characters: & / < > [ ]' ), $tag );
+    if ( 0 !== preg_match( '@[<>&/\[\]\x00-\x20=]@', $tag ) ) {
+        /* translators: 1: shortcode name, 2: space separated list of reserved characters */
+        $message = sprintf( __( 'Invalid shortcode name: %1$s. Do not use spaces or reserved characters: %2$s' ), $tag, '& / < > [ ] =' );
         _doing_it_wrong( __FUNCTION__, $message, '4.4.0' );
         return;
@@ -211 +211 @@
 
     // Find all registered tag names in $content.
-    preg_match_all( '@\[([^<>&/\[\]\x00-\x20]++)@', $content, $matches );
+    preg_match_all( '@\[([^<>&/\[\]\x00-\x20=]++)@', $content, $matches );
     $tagnames = array_intersect( array_keys( $shortcode_tags ), $matches[1] );
 
@@ -579 +579 @@
 
     // Find all registered tag names in $content.
-    preg_match_all( '@\[([^<>&/\[\]\x00-\x20]++)@', $content, $matches );
+    preg_match_all( '@\[([^<>&/\[\]\x00-\x20=]++)@', $content, $matches );
     $tagnames = array_intersect( array_keys( $shortcode_tags ), $matches[1] );