Changeset 12302

Timestamp:

12/01/2009 07:46:36 AM (15 years ago)

Author:

azaozz

Message:

Separate the removal of <p> wrapping from shortcodes into another function and apply it with different filter, props miqrogroove, props mdawaffe, see #11257, see #11249

Location:

trunk/wp-includes

Files:

• default-filters.php (modified) (2 diffs)
• formatting.php (modified) (2 diffs)

trunk/wp-includes/default-filters.php

@@ -89 +89 @@
 // Format text area for display.
 foreach ( array( 'term_description' ) as $filter ) {
-    add_filter( $filter, 'wptexturize'   );
-    add_filter( $filter, 'convert_chars' );
-    add_filter( $filter, 'wpautop'       );
+    add_filter( $filter, 'wptexturize'      );
+    add_filter( $filter, 'convert_chars'    );
+    add_filter( $filter, 'wpautop'          );
+    add_filter( $filter, 'shortcode_unautop');
 }
 
@@ -108 +109 @@
 add_filter( 'the_content', 'convert_chars'      );
 add_filter( 'the_content', 'wpautop'            );
+add_filter( 'the_content', 'shortcode_unautop'  );
 add_filter( 'the_content', 'prepend_attachment' );
 
-add_filter( 'the_excerpt',     'wptexturize'     );
-add_filter( 'the_excerpt',     'convert_smilies' );
-add_filter( 'the_excerpt',     'convert_chars'   );
-add_filter( 'the_excerpt',     'wpautop'         );
-add_filter( 'get_the_excerpt', 'wp_trim_excerpt' );
+add_filter( 'the_excerpt',     'wptexturize'      );
+add_filter( 'the_excerpt',     'convert_smilies'  );
+add_filter( 'the_excerpt',     'convert_chars'    );
+add_filter( 'the_excerpt',     'wpautop'          );
+add_filter( 'the_excerpt',     'shortcode_unautop');
+add_filter( 'get_the_excerpt', 'wp_trim_excerpt'  );
 
 add_filter( 'comment_text', 'wptexturize'            );

trunk/wp-includes/formatting.php

@@ -180 +180 @@
  */
 function wpautop($pee, $br = 1) {
+
     if ( trim($pee) === '' )
         return '';
@@ -217 +218 @@
         $pee = preg_replace_callback('!(<pre[^>]*>)(.*?)</pre>!is', 'clean_pre', $pee );
     $pee = preg_replace( "|\n</p>$|", '</p>', $pee );
-    $pee = preg_replace('/<p>\s*?(' . get_shortcode_regex() . ')\s*<\/p>/s', '$1', $pee); // don't auto-p wrap shortcodes that stand alone
+
+    return $pee;
+}
+
+/**
+ * Don't auto-p wrap shortcodes that stand alone
+ *
+ * Ensures that shortcodes are not wrapped in <<p>>...<</p>>.
+ *
+ * @since 2.9.0
+ *
+ * @param string $pee The content.
+ * @return string The filtered content.
+ */
+function shortcode_unautop($pee) {
+    global $shortcode_tags;
+
+    if ( !empty($shortcode_tags) && is_array($shortcode_tags) ) {
+        $tagnames = array_keys($shortcode_tags);
+        $tagregexp = join( '|', array_map('preg_quote', $tagnames) );
+        $pee = preg_replace('/<p>\\s*?(\\[(' . $tagregexp . ')\\b.*?\\/?\\](?:.+?\\[\\/\\2\\])?)\\s*<\\/p>/s', '$1', $pee);
+    }
 
     return $pee;