[wp-trac] [WordPress Trac] #13971: "Wordpress" being turned into CamelCase "WordPress" breaks URLs
WordPress Trac
wp-trac at lists.automattic.com
Thu Jul 8 21:04:55 UTC 2010
#13971: "Wordpress" being turned into CamelCase "WordPress" breaks URLs
--------------------------+-------------------------------------------------
Reporter: tfotherby | Owner:
Type: defect (bug) | Status: closed
Priority: lowest | Milestone: 3.0.1
Component: Formatting | Version: 3.0
Severity: normal | Resolution: fixed
Keywords: has-patch |
--------------------------+-------------------------------------------------
Comment(by dougal):
Replying to [comment:67 filosofo]:
> dougal, I'm concerned about the performance implications of using a
couple of lookbehinds and a lambda function to filter something so trivial
to begin with, especially on something ubiquitous like `the_content` and
`comment_text`. Are the possible edge cases it addresses really worth it?
Oh, almost certainly not. Honestly I think we should just ditch the damned
P function. Even with just a nice fast str_replace, I don't think it's
worth the hassle. You can keep on adding new test cases to the array of
replacements, and somebody will eventually come up with another case we
hadn't thought of yet. And as somebody else brought up, there are cross-
cultural implications to mucking around with content.
Just because it *can* be worked around, doesn't mean that it should go in.
I'm sure it seemed like a trivial, cute little change when it went in. And
if I was on the core team, I'm sure that I wouldn't have predicted the
crapstorm that it's caused, either. But I think this is just one
'enhancement' that really should be reconsidered.
If my opinion had any weight, I'd suggest leaving the function in,
removing it from default-filters, and just documenting its availability
for plugin and theme authors. Another option might be to disable it via a
define in wp-config, enabled by default, but visible and documented in the
wp-config.php file (e.g., "Uncomment the next line to disable 'WordPress'
auto case correction").
--
Ticket URL: <http://core.trac.wordpress.org/ticket/13971#comment:71>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software
More information about the wp-trac
mailing list