[wp-trac] [WordPress Trac] #53938: replace core uses of wp_parse_url() with PHP's native parse_url() (was: replace core uses of wp_parse_url())

WordPress Trac noreply at wordpress.org
Mon Aug 16 21:06:22 UTC 2021


#53938: replace core uses of wp_parse_url() with PHP's native parse_url()
-------------------------------------+------------------------------
 Reporter:  pbiron                   |       Owner:  (none)
     Type:  enhancement              |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  HTTP API                 |     Version:
 Severity:  normal                   |  Resolution:
 Keywords:  needs-patch 2nd-opinion  |     Focuses:
-------------------------------------+------------------------------
Changes (by pbiron):

 * milestone:  5.9 => Awaiting Review


Old description:

> [https://developer.wordpress.org/reference/functions/wp_parse_url/
> wp_parse_url()] was introduced in #34408 to get around URL parsing
> failures of PHP's [https://www.php.net/manual/en/function.parse-url
> parse_url()] in PHP < 5.4.7.
>
> With the minimum supported PHP for core now 5.6.20, `wp_parse_url()` no
> longer seems necessary.  For background on this ticket, see the
> [https://wordpress.slack.com/archives/C02RQBWTW/p1629146151384100 slack
> thread].

New description:

 [https://developer.wordpress.org/reference/functions/wp_parse_url/
 wp_parse_url()] was introduced in #34408 to get around URL parsing
 failures of PHP's [https://www.php.net/manual/en/function.parse-url
 parse_url()] in PHP < 5.4.7.

 With the minimum supported PHP for core now 5.6.20, `wp_parse_url()` no
 longer seems necessary and `parse_url()` can be used directly.  For
 background on this ticket, see the
 [https://wordpress.slack.com/archives/C02RQBWTW/p1629146151384100 slack
 thread].

--

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/53938#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list