[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