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

WordPress Trac noreply at wordpress.org
Tue Aug 17 00:55:14 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 close  |     Focuses:
-------------------------------------------+------------------------------

Comment (by jrf):

 @pbiron While I would very much welcome more tests, in this case, the
 tests are only relevant for those calls which ''may'' run into the `:`
 issue. So I suspect that would be the calls to `parse_url()` without the
 `$component` parameter or when the `$component` would be any of these:
 `PHP_URL_QUERY`, `PHP_URL_PORT`, but I haven't looked into the colon issue
 in detail today to bring back the fine details, so the above suggestion
 should be verified carefully against the PHP core ticket:
 https://bugs.php.net/bug.php?id=74780

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


More information about the wp-trac mailing list