[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:22:52 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 westi):

 Replying to [comment:8 josephscott]:
 > @westi -
 >
 > What WP version are you running those tests against?  I applied the last
 patch provided by filosofo on a -trunk install then submitted a comment
 with these lines:
 >
 > {{{
 > http://wordpress.org/hello.html
 > There was a spoon named http://wordpress.org. Alice!
 > There was a spoon named http://wordpress.org, said Alice.
 > There was a spoon named http://wordpress.org; said Alice.
 > There was a spoon named http://wordpress.org: said Alice.
 > There was a spoon named (http://wordpress.org) said Alice.
 > }}}
 >
 > They all linked correctly.

 The test was run against trunk + patch.

 All those examples work (they are from
 {{{test_strip_trailing_with_protocol}}}) the ones that fail are in
 {{{test_strip_trailing_with_protocol_nothing_afterwards}}}.

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


More information about the wp-trac mailing list