| Package home | Report new bug | New search | Development Roadmap | Status: Open | Feedback | All | Closed Since Version 1.2.2 |
| Bug #16849 | WebResquest test3 is using POST and not GET to supply data. | ||
|---|---|---|---|
| Submitted: | 2009-11-30 18:21 UTC | ||
| From: | rquadling | Assigned: | izi |
| Status: | Closed | Package: | Console_CommandLine (version 1.1.1) |
| PHP Version: | 5.3.1 | OS: | Windows XP SP3 |
| Roadmaps: | (Not assigned) | ||
[2009-11-30 18:21 UTC] rquadling
(Richard Quadling)
Description:
------------
While running the tests for Console_CommandLine with my
own changes, console_commandline_webrequest_3.phpt
failed.
In looking at the test, the entry --POST-- is used. As
Console_CommandLine doesn't support $_POST'd data,
changing this to --GET-- allows the test to pass successfully.
Either the test is a for a future option to allow $_POST'd data
to be processed or is a typo and should be testing $_GET'd
data.
Patch supplied.
Comments
[2009-11-30 19:02 UTC] izi
(David Jean Louis)
This should work, Console_CommandLine should support POST (and the test passes on linux/bsd), but I've just realized that's there's a bug in the Console_CommandLine::getArgcArgv() $_GET occurences should be replaced by $_REQUEST.
Does it work if you apply theses changes ?
Thanks !
[2009-11-30 22:04 UTC] izi
(David Jean Louis)
-Status: Open
+Status: Closed
-Assigned To:
+Assigned To: izi
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.