[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