[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