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

[WordPress Trac]

#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 filosofo):

 I got to thinking, and make_clickable_end_chars.10990.2.diff might be
 better, depending on what you want to do.  For

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

 my second patch makes the url to be

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

 but for

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

 it makes the url to be

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

 The issue is not so much what are legal characters in URLs as much as what
 is a likely use.

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