[wp-trac] [WordPress Trac] #13971: "Wordpress" being turned into CamelCase "WordPress" breaks URLs
WordPress Trac
wp-trac at lists.automattic.com
Mon Jun 21 18:05:27 UTC 2010
#13971: "Wordpress" being turned into CamelCase "WordPress" breaks URLs
--------------------------+-------------------------------------------------
Reporter: tfotherby | Owner:
Type: defect (bug) | Status: new
Priority: normal | Milestone: 3.0.1
Component: General | Version: 3.0
Severity: normal | Keywords:
--------------------------+-------------------------------------------------
Comment(by markjaquith):
> There's a lot of talk around what should sit in plugin land and what
shouldn't, and I very much feel that were not Matt behind this one, there
could be no justification for it being part of core.
This was run past the other leads. I brought up the argument that this
might break a great many links. Matt said that they'd been running it on
!WordPress.com for several years with, I believe, only one recorded
instance of it breaking anything. That satisfied me that this reported bug
would occur infrequently.
However, what if, instead of a regular expression, with its speed and
complexity concerns, we do three search/replace pairs (explicitly running
the filter on priority 11):
{{{" Wordpress" => "WordPress"}}} — that is, {SPACE}Wordpress to
{SPACE}!WordPress. That should eliminate all URL/src false
positives.[[BR]]
{{{">Wordpress" => ">WordPress"}}} — which should get the HTML-wrapped or
beginning-of-paragraph uses.
{{{"“Wordpress" => "“WordPress"}}} — which should get all
instances of WordPress at the beginning of a double quote.
Using those three replacements should greatly decrease our false
positives.
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13971#comment:15>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list