Ticket #6444: 6444-no-p-wrap-shortcodes-r7811.patch

File 6444-no-p-wrap-shortcodes-r7811.patch, 1.8 kB (added by tellyworth, 3 months ago)

compromise: don't p-wrap stand-alone shortcodes

  • wordpress-2.5/wp-includes/shortcodes.php

    old new  
    7272        if (empty($shortcode_tags) || !is_array($shortcode_tags)) 
    7373                return $content; 
    7474 
     75        $pattern = get_shortcode_regex(); 
     76        return preg_replace_callback('/'.$pattern.'/s', 'do_shortcode_tag', $content); 
     77} 
     78 
     79function get_shortcode_regex() { 
     80        global $shortcode_tags; 
    7581        $tagnames = array_keys($shortcode_tags); 
    7682        $tagregexp = join( '|', array_map('preg_quote', $tagnames) ); 
    7783 
    78         $pattern = '/\[('.$tagregexp.')\b(.*?)(?:(\/))?\](?:(.+?)\[\/\1\])?/s'; 
    79  
    80         return preg_replace_callback($pattern, 'do_shortcode_tag', $content); 
     84        return '\[('.$tagregexp.')\b(.*?)(?:(\/))?\](?:(.+?)\[\/\1\])?'; 
    8185} 
    8286 
    8387function do_shortcode_tag($m) { 
  • wordpress-2.5/wp-includes/formatting.php

    old new  
    7474        $pee = preg_replace("/\n\n+/", "\n\n", $pee); // take care of duplicates 
    7575        $pee = preg_replace('/\n?(.+?)(?:\n\s*\n|\z)/s', "<p>$1</p>\n", $pee); // make paragraphs, including one at the end 
    7676        $pee = preg_replace('|<p>\s*?</p>|', '', $pee); // under certain strange conditions it could create a P of entirely whitespace 
     77        $pee = preg_replace('/<p>(\s*?' . get_shortcode_regex() . '\s*)<\/p>/s', '$1', $pee); // don't auto-p wrap post-formatting shortcodes    
    7778        $pee = preg_replace('!<p>([^<]+)\s*?(</(?:div|address|form)[^>]*>)!', "<p>$1</p>$2", $pee); 
    7879        $pee = preg_replace( '|<p>|', "$1<p>", $pee ); 
    7980        $pee = preg_replace('!<p>\s*(</?' . $allblocks . '[^>]*>)\s*</p>!', "$1", $pee); // don't pee all over a tag