Opened 14 years ago
Closed 14 years ago
#21121 closed enhancement (maybelater)
Remove _http_build_query compatibility function
| Reported by: | kurtpayne | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | HTTP API | Version: | |
| Severity: | normal | Keywords: | close |
| Cc: | Focuses: |
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)
#1
follow-up:
↓ 2
@
14 years ago
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.
#2
in reply to: ↑ 1
@
14 years ago
- 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.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Deprecate _http_build_query