[wp-trac] Re: [WordPress Trac] #9064: URLs with commas are not pinged

WordPress Trac wp-trac at lists.automattic.com
Sun Apr 19 09:03:34 GMT 2009


#9064: URLs with commas are not pinged
--------------------------+-------------------------------------------------
 Reporter:  sirzooro      |       Owner:  anonymous              
     Type:  defect (bug)  |      Status:  new                    
 Priority:  high          |   Milestone:  2.8                    
Component:  General       |     Version:  2.7                    
 Severity:  major         |    Keywords:  has-patch needs-testing
--------------------------+-------------------------------------------------

Comment(by sirzooro):

 One more thing to consider: looks like original regexp allows to extract
 all URLs from page, both clickable (wrapped in <a> tag) and non-clickable.
 I think that only clickable ones should be pinged. Therefore we can
 replace whole rx with something like this:

 {{{
 preg_match_all( '/\bhref\s*=\s*["\']?([^"\'\s]+)/', $content,
 $post_links_temp );
 }}}

 This will also require change in {{{foreach}}} loop below - it should
 iterate over {{{$post_links_temp[1]}}}.

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


More information about the wp-trac mailing list