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. Merges [36097] to the 4.4 branch, minus a string change. Props aaroncampbell. Fixes #34939. git-svn-id: https://develop.svn.wordpress.org/branches/4.4@36098 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
parent
a148640ec4
commit
0ad7833b73
@ -216,7 +216,7 @@ function wptexturize( $text, $reset = false ) {
|
||||
|
||||
// Look for shortcodes and HTML elements.
|
||||
|
||||
preg_match_all( '@\[/?([^<>&/\[\]\x00-\x20]++)@', $text, $matches );
|
||||
preg_match_all( '@\[/?([^<>&/\[\]\x00-\x20=]++)@', $text, $matches );
|
||||
$tagnames = array_intersect( array_keys( $shortcode_tags ), $matches[1] );
|
||||
$found_shortcodes = ! empty( $tagnames );
|
||||
$shortcode_regex = $found_shortcodes ? _get_wptexturize_shortcode_regex( $tagnames ) : '';
|
||||
|
@ -95,7 +95,7 @@ function add_shortcode($tag, $func) {
|
||||
return;
|
||||
}
|
||||
|
||||
if ( 0 !== preg_match( '@[<>&/\[\]\x00-\x20]@', $tag ) ) {
|
||||
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 );
|
||||
_doing_it_wrong( __FUNCTION__, $message, '4.4.0' );
|
||||
@ -210,7 +210,7 @@ function do_shortcode( $content, $ignore_html = false ) {
|
||||
return $content;
|
||||
|
||||
// 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] );
|
||||
|
||||
if ( empty( $tagnames ) ) {
|
||||
@ -578,7 +578,7 @@ function strip_shortcodes( $content ) {
|
||||
return $content;
|
||||
|
||||
// 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] );
|
||||
|
||||
if ( empty( $tagnames ) ) {
|
||||
|
@ -647,4 +647,14 @@ EOF;
|
||||
|
||||
}
|
||||
|
||||
/**
|
||||
* @ticket 34939
|
||||
*
|
||||
* Test the (not recommended) [shortcode=XXX] format
|
||||
*/
|
||||
function test_unnamed_attribute() {
|
||||
$out = do_shortcode('[dumptag=https://wordpress.org/]');
|
||||
$expected = "0 = =https://wordpress.org\n";
|
||||
$this->assertEquals($expected, $out);
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user