[wp-trac] [WordPress Trac] #15549: WordPress fails to ping whole url if there's an ampersand

WordPress Trac wp-trac at lists.automattic.com
Mon Sep 10 06:36:44 UTC 2012


#15549: WordPress fails to ping whole url if there's an ampersand
------------------------------+-----------------------------
 Reporter:  unpix             |       Owner:  markjaquith
     Type:  defect (bug)      |      Status:  accepted
 Priority:  normal            |   Milestone:  Future Release
Component:  Pings/Trackbacks  |     Version:  3.1
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |
------------------------------+-----------------------------
Changes (by wonderboymusic):

 * keywords:  needs-patch => has-patch


Comment:

 The regex used to extract URLs for pinging is old and odd - I updated it
 to use this one, which I tend to prefer:
 http://daringfireball.net/2010/07/improved_regex_for_matching_urls

 I also moved it into a function so the regex isn't duped across functions.
 Patch attached.

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


More information about the wp-trac mailing list