FAQ
Edit report at http://pear.php.net/bugs/bug.php?id=17036&edit=1

ID: 17036
Updated by: till@php.net
Reported By: lukas dot emile at gmail dot com
Summary: Problem with parsed query string
-Status: Open
+Status: Feedback
Type: Bug
Package: Net_URL2
Operating System: linux
Package Version: 0.3.1
PHP Version: 5.1.3
-Assigned To:
+Assigned To: till
Roadmap Versions:
New Comment:

-Status: Open
+Status: Feedback
-Assigned To:
+Assigned To: till
Lukas,

you mind providing some example code which I could wrap into a unit
test?

Till


Previous Comments:
------------------------------------------------------------------------

[2010-02-08 17:12:40] schmidt

The patch does not respect the various OPTION_xxx options.

------------------------------------------------------------------------

[2010-02-03 15:58:51] lukas

I've maid a patch.
The script use parse_str and http_build_query instead of foreach and
explode functions.

------------------------------------------------------------------------

[2010-02-03 15:57:10] lukas

Added #patch bug:17036;patch:query_string_parsing;revision:1265212630;.

------------------------------------------------------------------------

[2010-01-28 17:19:18] schmidt

True, this is a bug.

------------------------------------------------------------------------

[2010-01-26 16:09:28] lukas

modification :
public function setQueryVariable($name, $value)
{
$keys = preg_split('/[\[\]]+/', trim($name, '[]'));
$array = $this->getQueryVariables();
$var =& $array;
foreach ($keys as $key) {
if (!array_key_exists($key, $var)) {
$var[$key] = array();
}
$var =& $var[$key];
}
$var = $value;
$this->setQueryVariables($array);
}

------------------------------------------------------------------------

The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
http://pear.php.net/bugs/bug.php?id=17036

Search Discussions

  • Till at Mar 13, 2011 at 12:36 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=17036&edit=1

    ID: 17036
    Updated by: till@php.net
    Reported By: lukas dot emile at gmail dot com
    Summary: Problem with parsed query string
    Status: Feedback
    Type: Bug
    Package: Net_URL2
    Operating System: linux
    Package Version: 0.3.1
    PHP Version: 5.1.3
    Assigned To: till
    Roadmap Versions:
    New Comment:

    I think this might have been fixed by another bugfix I did. Anyway, I
    tried to make sense of the example code and
    committed a unit test with it. The unit test runs.

    http://svn.php.net/viewvc?view=revision&revision=309168

    Wouldn't mind if ANYONE could take a look so this is finished and we can
    move on a new release.


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-13 02:49:36] till

    -Status: Open
    +Status: Feedback
    -Assigned To:
    +Assigned To: till
    Lukas,

    you mind providing some example code which I could wrap into a unit
    test?

    Till

    ------------------------------------------------------------------------

    [2010-02-08 17:12:40] schmidt

    The patch does not respect the various OPTION_xxx options.

    ------------------------------------------------------------------------

    [2010-02-03 15:58:51] lukas

    I've maid a patch.
    The script use parse_str and http_build_query instead of foreach and
    explode functions.

    ------------------------------------------------------------------------

    [2010-02-03 15:57:10] lukas

    Added #patch bug:17036;patch:query_string_parsing;revision:1265212630;.

    ------------------------------------------------------------------------

    [2010-01-28 17:19:18] schmidt

    True, this is a bug.

    ------------------------------------------------------------------------

    The remainder of the comments for this report are too long. To view
    the rest of the comments, please view the bug report online at
    http://pear.php.net/bugs/bug.php?id=17036
  • Cweiske at Mar 14, 2011 at 7:47 am
    Edit report at http://pear.php.net/bugs/bug.php?id=17036&edit=1

    ID: 17036
    Updated by: cweiske@php.net
    Reported By: lukas dot emile at gmail dot com
    Summary: Problem with parsed query string
    Status: Feedback
    Type: Bug
    Package: Net_URL2
    Operating System: linux
    Package Version: 0.3.1
    PHP Version: 5.1.3
    Assigned To: till
    Roadmap Versions:
    New Comment:

    Yes, your patch fixes the problem.


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-13 13:39:44] till

    I think this might have been fixed by another bugfix I did. Anyway, I
    tried to make sense of the example code and
    committed a unit test with it. The unit test runs.

    http://svn.php.net/viewvc?view=revision&revision=309168

    Wouldn't mind if ANYONE could take a look so this is finished and we can
    move on a new release.

    ------------------------------------------------------------------------

    [2011-03-13 02:49:36] till

    -Status: Open
    +Status: Feedback
    -Assigned To:
    +Assigned To: till
    Lukas,

    you mind providing some example code which I could wrap into a unit
    test?

    Till

    ------------------------------------------------------------------------

    [2010-02-08 17:12:40] schmidt

    The patch does not respect the various OPTION_xxx options.

    ------------------------------------------------------------------------

    [2010-02-03 15:58:51] lukas

    I've maid a patch.
    The script use parse_str and http_build_query instead of foreach and
    explode functions.

    ------------------------------------------------------------------------

    [2010-02-03 15:57:10] lukas

    Added #patch bug:17036;patch:query_string_parsing;revision:1265212630;.

    ------------------------------------------------------------------------

    The remainder of the comments for this report are too long. To view
    the rest of the comments, please view the bug report online at
    http://pear.php.net/bugs/bug.php?id=17036
  • Till at Mar 14, 2011 at 1:13 pm
    Edit report at http://pear.php.net/bugs/bug.php?id=17036&edit=1

    ID: 17036
    Updated by: till@php.net
    Reported By: lukas dot emile at gmail dot com
    Summary: Problem with parsed query string
    -Status: Feedback
    +Status: Closed
    Type: Bug
    Package: Net_URL2
    Operating System: linux
    Package Version: 0.3.1
    PHP Version: 5.1.3
    Assigned To: till
    Roadmap Versions:
    New Comment:

    -Status: Feedback
    +Status: Closed
    This bug has been fixed in SVN.

    If this was a documentation problem, the fix will appear on pear.php.net
    by the end of next Sunday (CET).

    If this was a problem with the pear.php.net website, the change should
    be live shortly.

    Otherwise, the fix will appear in the package's next release.

    Thank you for the report and for helping us make PEAR better.

    Epic win!!!11


    Previous Comments:
    ------------------------------------------------------------------------

    [2011-03-14 08:50:34] cweiske

    Yes, your patch fixes the problem.

    ------------------------------------------------------------------------

    [2011-03-13 13:39:44] till

    I think this might have been fixed by another bugfix I did. Anyway, I
    tried to make sense of the example code and
    committed a unit test with it. The unit test runs.

    http://svn.php.net/viewvc?view=revision&revision=309168

    Wouldn't mind if ANYONE could take a look so this is finished and we can
    move on a new release.

    ------------------------------------------------------------------------

    [2011-03-13 02:49:36] till

    -Status: Open
    +Status: Feedback
    -Assigned To:
    +Assigned To: till
    Lukas,

    you mind providing some example code which I could wrap into a unit
    test?

    Till

    ------------------------------------------------------------------------

    [2010-02-08 17:12:40] schmidt

    The patch does not respect the various OPTION_xxx options.

    ------------------------------------------------------------------------

    [2010-02-03 15:58:51] lukas

    I've maid a patch.
    The script use parse_str and http_build_query instead of foreach and
    explode functions.

    ------------------------------------------------------------------------

    The remainder of the comments for this report are too long. To view
    the rest of the comments, please view the bug report online at
    http://pear.php.net/bugs/bug.php?id=17036

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
grouppear-bugs @
categoriesphp
postedMar 13, '11 at 1:46a
activeMar 14, '11 at 1:13p
posts4
users2
websitepear.php.net

2 users in discussion

Till: 3 posts Cweiske: 1 post

People

Translate

site design / logo © 2021 Grokbase