Opened 11 months ago
Closed 11 months ago
#21121 closed enhancement (maybelater)
Remove _http_build_query compatibility function
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | HTTP | Version: | |
| Severity: | normal | Keywords: | close |
| Cc: |
Description
WordPress requires PHP 5.2.4+ which includes http_build_query(). http_build_query() is already used in update.php and class-http.php. We should remove the compatibility function and be more consistent.
Attachments (1)
Change History (4)
Although this function started out life as a compatibility function.. It's got some non-php-core functionality in it, which is why it's been used directly rather than through a compat wrapper.
Specifically, the PHP version doesn't support the final parameter, $urlencode, which core uses to produce url's usable for Location headers (IIRC) amongst other things.
- Keywords close added; dev-feedback removed
Replying to dd32:
Specifically, the PHP version doesn't support the final parameter, $urlencode, which core uses to produce url's usable for Location headers (IIRC) amongst other things.
In PHP 5.4, there's an $enc_type parameter. Probably best to leave this function in until WordPress requires php 5.4.

Deprecate _http_build_query