Changeset 28852 for trunk/src/wp-includes/formatting.php

Timestamp:

06/26/2014 05:14:47 PM (11 years ago)

Author:

wonderboymusic

Message:

Use less greedy regex in wptexturize(). Adds unit tests.

Props miqrogroove.
See #28564.

File:

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

trunk/src/wp-includes/formatting.php

@@ -199 +199 @@
         .       '.+?--\s*>' // Find end of comment
         .   '|'
-        .       '.+?>'      // Find end of element
+        .       '[^>]+>'    // Find end of element
         .   ')'
         . '|'
@@ -207 +207 @@
         .       '[^\[\]<>]' // Shortcodes do not contain other shortcodes.
         .   '|'
-        .       '<.+?>'     // HTML elements permitted. Prevents matching ] before >.
+        .       '<[^>]+>'   // HTML elements permitted. Prevents matching ] before >.
         .   ')+'
         .   '\]'        // Find end of shortcode.
@@ -225 +225 @@
             }
 
-        } elseif ( '[' === $first && 1 === preg_match( '/^\[(?:[^\[\]<>]|<.+?>)+\]$/', $curl ) ) {
+        } elseif ( '[' === $first && 1 === preg_match( '/^\[(?:[^\[\]<>]|<[^>]+>)+\]$/', $curl ) ) {
             // This is a shortcode delimeter.
 
             _wptexturize_pushpop_element( $curl, $no_texturize_shortcodes_stack, $no_texturize_shortcodes );
 
-        } elseif ( '[' === $first && 1 === preg_match( '/^\[\[?(?:[^\[\]<>]|<.+?>)+\]\]?$/', $curl ) ) {
+        } elseif ( '[' === $first && 1 === preg_match( '/^\[\[?(?:[^\[\]<>]|<[^>]+>)+\]\]?$/', $curl ) ) {
             // This is an escaped shortcode delimeter.