Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#10326 closed defect (bug) (fixed)

strip_shortcodes() greedily strips surrounding bytes

Reported by: markjaquith's profile markjaquith Owned by: markjaquith's profile markjaquith
Milestone: 2.9 Priority: high
Severity: critical Version: 2.8
Component: Shortcodes Keywords: has-patch needs-testing
Focuses: Cc:


strip_shortcodes() which is run on excerpts is too greedy.





As you can see, the characters immediately preceding and following the shortcode have been stripped out. These characters should be preserved.

Attachments (1)

10326.001.diff (751 bytes) - added by markjaquith 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @markjaquith
14 years ago

  • Keywords has-patch needs-testing added
  • Owner set to markjaquith
  • Status changed from new to accepted

Patch uploaded. Return the 1st and 6th matches, like do_shortcode_tag() does. Works for me. Can I get someone to verify?

#3 @westi
14 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

#5 @miqrogroove
13 years ago

  • Severity changed from major to critical

Increase severity because this bug is causing UTF-8 corruption and Google was rejecting my feeds until I applied this patch.

#6 @miqrogroove
13 years ago

  • Summary changed from strip_shortcodes() greedily strips surrounding characters to strip_shortcodes() greedily strips surrounding bytes
Note: See TracTickets for help on using tickets.