wp_extract_urls is not returning full url

[pushplaybang]

wp_extract_urls doesn't seem to return the query part of an extracted url. for example

​http://www.youtube.com/watch?v=exvUH2qKLTU

is returned as

​http://www.youtube.com/watch

this is an incredibly useful function and surely should be able to handle this sort of URL?

[SergeyBiryukov]

Note that the function is primarily used for pings/trackbacks (it was added in [25313] for #9064).

[hauvong]

Patch to fix link containing "?" and issue extracting url from shortcode tags: [shortcode attr="​http://.."]http:...[/shortcode]

[hauvong]

I had similar issue where we have shortcode in the form [shortcode image="​http://.."]http://..[/shortcode]
the function would extract the url value as: ​http://..."]http://..

Attached patch file is the suggest modification to fix both issues.

[dlh]

Update test_wp_extract_urls()

[dlh]

This patch adds a URL of the form reported here to test_wp_extract_urls(). Are additional URLs or tests needed?

[wonderboymusic]

In 29756:

wp_extract_urls() needs to also grab URLs with query strings and hashes.

Updates unit test.

Props hauvong, dlh.
Fixes #29314.