WordPress.org
Get WordPress

Make WordPress Core

Ticket #29557: miqro-29557.7.patch

File miqro-29557.7.patch, 5.4 KB (added by miqrogroove, 10 years ago)

Don't match when whitespace appears before shortcode name.

  • src/wp-includes/formatting.php

     
    2828 * @return string The string replaced with html entities
    2929 */
    3030function wptexturize($text, $reset = false) {
    31         global $wp_cockneyreplace, $shortcode_tags;
     31        global $wp_cockneyreplace;
    3232        static $static_characters, $static_replacements, $dynamic_characters, $dynamic_replacements,
    3333                $default_no_texturize_tags, $default_no_texturize_shortcodes, $run_texturize = true;
    3434
    … …  
    205205
    206206        // Look for shortcodes and HTML elements.
    207207
    208         $tagnames = array_keys( $shortcode_tags );
    209         $tagregexp = join( '|', array_map( 'preg_quote', $tagnames ) );
    210         $tagregexp = "(?:$tagregexp)(?![\\w-])"; // Excerpt of get_shortcode_regex().
    211        
    212208        $regex =  '/('                  // Capture the entire match.
    213209                .       '<'             // Find start of element.
    214210                .       '(?(?=!--)'     // Is this a comment?
    … …  
    218214                .       ')'
    219215                . '|'
    220216                .       '\['            // Find start of shortcode.
    221                 .       '\[?'           // Shortcodes may begin with [[
    222                 .       '\/?'           // Closing slash may precede name.
    223                 .       $tagregexp      // Only match registered shortcodes, because performance.
    224                 .       '[^\[\]]*'      // Shortcodes do not contain other shortcodes.
     217                .       '\[?\/?'        // Shortcodes may begin with [/ or [[ or [[/
     218                .       '[^\s\[\]]'     // No whitespace before name.
     219                .       '[^\[\]]*+'     // Shortcodes do not contain other shortcodes. Possessive critical.
    225220                .       '\]'            // Find end of shortcode.
    226221                .       '\]?'           // Shortcodes may end with ]]
    227222                . ')/s';
    … …  
    243238
    244239                        continue;
    245240
    246                 } elseif ( '[' === $first && 1 === preg_match( '/^\[\[?\/?' . $tagregexp . '[^\[\]]*\]\]?$/', $curl ) ) {
     241                } elseif ( '[' === $first && 1 === preg_match( '/^\[\[?\/?[^\s\[\]][^\[\]]*+\]\]?$/', $curl ) ) {
    247242                        // This is a shortcode delimiter.
    248243
    249244                        if ( '[[' !== substr( $curl, 0, 2 ) && ']]' !== substr( $curl, -2 ) ) {

  • src/wp-includes/shortcodes.php

     
    231231        $tagregexp = join( '|', array_map('preg_quote', $tagnames) );
    232232
    233233        // WARNING! Do not change this regex without changing do_shortcode_tag() and strip_shortcode_tag()
    234         // Also, see shortcode_unautop() and shortcode.js and wptexturize().
     234        // Also, see shortcode_unautop() and shortcode.js.
    235235        return
    236236                  '\\['                              // Opening bracket
    237237                . '(\\[?)'                           // 1: Optional second opening bracket for escaping shortcodes: [[tag]]

  • tests/phpunit/tests/formatting/WPTexturize.php

     
    11881188        function data_tag_avoidance() {
    11891189                return array(
    11901190                        array(
     1191                                '[ ... ]',
     1192                                '[ &#8230; ]',
     1193                        ),
     1194                        array(
    11911195                                '[ is it wise to <a title="allow user content ] here? hmm"> maybe </a> ]',
    11921196                                '[ is it wise to <a title="allow user content ] here? hmm"> maybe </a> ]',
    11931197                        ),
    11941198                        array(
     1199                                '[is it wise to <a title="allow user content ] here? hmm"> maybe </a> ]', // HTML corruption is a known bug.  See tickets #12690 and #29557.
     1200                                '[is it wise to <a title="allow user content ] here? hmm&#8221;> maybe </a> ]',
     1201                        ),
     1202                        array(
     1203                                '[caption - is it wise to <a title="allow user content ] here? hmm"> maybe </a> ]',
     1204                                '[caption - is it wise to <a title="allow user content ] here? hmm&#8221;> maybe </a> ]',
     1205                        ),
     1206                        array(
    11951207                                '[ photos by <a href="http://example.com/?a[]=1&a[]=2"> this guy </a> ]',
    11961208                                '[ photos by <a href="http://example.com/?a[]=1&#038;a[]=2"> this guy </a> ]',
    11971209                        ),
    11981210                        array(
     1211                                '[photos by <a href="http://example.com/?a[]=1&a[]=2"> this guy </a>]',
     1212                                '[photos by <a href="http://example.com/?a[]=1&#038;a[]=2"> this guy </a>]',
     1213                        ),
     1214                        array(
    11991215                                '[gallery ...]',
    12001216                                '[gallery ...]',
    12011217                        ),
    … …  
    12121228                                '[/gallery ...]',
    12131229                        ),
    12141230                        array(
    1215                                 '[...]...[/...]', // These are potentially usable shortcodes.
    1216                                 '[&#8230;]&#8230;[/&#8230;]',
    1217                         ),
    1218                         array(
    12191231                                '[[gallery]]...[[/gallery]]', // Shortcode parsing will ignore the inner ]...[ part and treat this as a single escaped shortcode.
    12201232                                '[[gallery]]&#8230;[[/gallery]]',
    12211233                        ),
    … …  
    12241236                                '[[[gallery]]]&#8230;[[[/gallery]]]',
    12251237                        ),
    12261238                        array(
    1227                                 '[gal>ery ...]',
    1228                                 '[gal>ery &#8230;]',
    1229                         ),
    1230                         array(
    12311239                                '[gallery ...',
    12321240                                '[gallery &#8230;',
    12331241                        ),
    … …  
    17271735                        ),
    17281736                        array(
    17291737                                '[code ...]...[/code]', // code is not a registered shortcode.
    1730                                 '[code &#8230;]&#8230;[/code]',
     1738                                '[code ...]...[/code]',
    17311739                        ),
    17321740                        array(
    17331741                                '[hello ...]...[/hello]', // hello is not a registered shortcode.
    1734                                 '[hello &#8230;]&#8230;[/hello]',
     1742                                '[hello ...]&#8230;[/hello]',
    17351743                        ),
    17361744                        array(
     1745                                '[...]...[/...]', // These are potentially usable shortcodes.
     1746                                '[...]&#8230;[/...]',
     1747                        ),
     1748                        array(
     1749                                '[gal>ery ...]',
     1750                                '[gal>ery ...]',
     1751                        ),
     1752                        array(
     1753                                '[randomthing param="test"]',
     1754                                '[randomthing param="test"]',
     1755                        ),
     1756                        array(
    17371757                                '[[audio]...[/audio]...', // These are potentially usable shortcodes.  Unfortunately, the meaning of [[audio] is ambiguous unless we run the entire shortcode regexp.
    17381758                                '[[audio]&#8230;[/audio]&#8230;',
    17391759                        ),