php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Bug #54418 http_build_query not accepting enc_type
Submitted: 2011-03-29 18:26 UTC Modified: 2012-01-31 15:43 UTC
Votes:3
Avg. Score:4.3 ± 0.5
Reproduced:3 of 3 (100.0%)
Same Version:3 (100.0%)
Same OS:0 (0.0%)
From: chris at sedlmayr dot co dot uk Assigned: jpauli (profile)
Status: Closed Package: Documentation problem
PHP Version: 5.3.6 OS: CentOS 5
Private report: No CVE-ID: None
View Developer Edit
Welcome! If you don't have a Git account, you can't do anything here.
If you reported this bug, you can edit this bug over here.
(description)
Block user comment
Status: Assign to:
Package:
Bug Type:
Summary:
From: chris at sedlmayr dot co dot uk
New email:
PHP Version: OS:

 

 [2011-03-29 18:26 UTC] chris at sedlmayr dot co dot uk 
Description:
------------
According to the latest documentation, http_build_query should accept a fourth 
argument so you can specify the alternate RFC for encoding of spaces to %20 rather 
than +

Checking the source however this does not appear to be the case, and this has not 
yet been implemented.

Test script:
---------------
Source:

ZEND_BEGIN_ARG_INFO_EX(arginfo_http_build_query, 0, 0, 1)
	ZEND_ARG_INFO(0, formdata)
	ZEND_ARG_INFO(0, prefix)
	ZEND_ARG_INFO(0, arg_separator)
ZEND_END_ARG_INFO()

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2011-03-29 19:39 UTC] dtajchreber@php.net
-Package: *General Issues +Package: Documentation problem
 [2011-03-29 19:39 UTC] dtajchreber@php.net 
Exists in trunk but not 5_3: http://svn.php.net/viewvc/php/php-
src/trunk/ext/standard/http.c?view=markup#l209

if (zend_parse_parameters(ZEND_NUM_ARGS() TSRMLS_CC, "z|ssl", &formdata, &prefix, 
&prefix_len, &arg_sep, &arg_sep_len, &enc_type) != SUCCESS)
 [2011-03-29 19:44 UTC] chris at sedlmayr dot co dot uk 
OK That's great.
The docs state it's part of the 5.3.6 release.
Can we get the docs amended or have this included in the 5.3.7 release?

Thanks either way.

Chris
 [2012-01-31 15:43 UTC] jpauli@php.net 
Please try using this snapshot:

  http://snaps.php.net/php5.4-latest.tar.gz
 
For Windows:

  http://windows.php.net/snapshots/

Implemented in PHP5.4
 [2012-01-31 15:43 UTC] jpauli@php.net
-Status: Open +Status: Closed -Assigned To: +Assigned To: jpauli
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Thu Jul 17 14:04:04 2025 UTC