[wp-trac] [WordPress Trac] #10990: Include trailing ) in make_clickable if it is part of the URL

WordPress Trac wp-trac at lists.automattic.com
Wed Oct 21 16:13:01 UTC 2009


#10990: Include trailing ) in make_clickable if it is part of the URL
--------------------------+-------------------------------------------------
 Reporter:  josephscott   |       Owner:  azaozz   
     Type:  defect (bug)  |      Status:  new      
 Priority:  normal        |   Milestone:  2.9      
Component:  Formatting    |     Version:  2.8.4    
 Severity:  normal        |    Keywords:  has-patch
--------------------------+-------------------------------------------------

Comment(by josephscott):

 @filosofo

 Thanks for improving this again.  Using your patch and the test cases
 mentioned above (including the two from westi) I think there's only one
 case left where things don't work as expected:

 {{{
 (http://en.wikipedia.org/wiki/PC_Tools_(Central_Point_Software))
 }}}

 on a line by itself.  The opening paren before the URL is correctly left
 out, but the trailing paren is included in the URL, when it shouldn't be.
 This only triggers if that is on a line by itself, since this works fine:

 {{{
 blah (http://en.wikipedia.org/wiki/PC_Tools_(Central_Point_Software))
 }}}

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


More information about the wp-trac mailing list