[wp-trac] [WordPress Trac] #11678: wpautop() fails on uppercase closing tags

WordPress Trac wp-trac at lists.automattic.com
Fri Oct 7 00:45:23 UTC 2011


#11678: wpautop() fails on uppercase closing tags
-----------------------------------------+-----------------------------
 Reporter:  joehoyle                     |       Owner:
     Type:  defect (bug)                 |      Status:  new
 Priority:  normal                       |   Milestone:  Future Release
Component:  Template                     |     Version:  2.9
 Severity:  minor                        |  Resolution:
 Keywords:  gsoc has-patch dev-feedback  |
-----------------------------------------+-----------------------------
Changes (by kurtpayne):

 * cc: kpayne@… (added)
 * keywords:  gsoc => gsoc has-patch dev-feedback


Comment:

 Looks like there's two ways to handle it.

 1.) Fix the text to have lowercase tag names before performing the
 transformations.  This makes the most sense to me since the text is being
 changed anyway and the patch is minimally invasive.

 2.) Change the regular expressions to be case insensitive.  The
 performance overhead should be negligible.  My latest profiles show that
 `wpautop()` is being called ~11 times.  This patch added ~0.6ms compared
 to a previous profile.  This patch is more invasive, but the output may
 more closely resemble the original text.

 I've submitted a patch for each approach.  Let me know what you think.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/11678#comment:7>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list