Re: cvs: php4 /main rfc1867.c
| From: | Sterling Hughes | Date: | Thu, 13 Dec 2001 20:23:26 +0000 |
| Subject: | Re: cvs: php4 /main rfc1867.c | ||
| References: | 1 2 3 | Groups: | php.dev |
| Request: | Send a blank email to php-dev+get-73609@lists.php.net to get a copy of this message | ||
> > Sterling: Thursday, December 13, 2001 7:19 PM
>
> > Please provide a more descriptive note about what you changed and
> > what was wrong. Also, it would be great if you could add some
> > test cases, which show what the problem was, and how your fixes made
> > the problem go away.
>
> Next time i will follow your advise. I did not want to write redundant
> comments.
> Array uploads did not work because php_register_variable truncates [*] from
> the variable name and therefore had to be moved a few lines down. I marked
> that as comment in the patch. And one cannot really write test cases for the
> bugs
> i fixed without writing something that send malformed/untypical POST
> headers.
>
Hrrm, yeah, I'm thinking that the test suite should contain a simple
webserver which we can run PHP with (as a CGI even), it would make
the test suite more robust as there are a lot of features that rely
on the presence of a webserver... (we might also provide an FTP
server, that doesn't need to run PHP, just to test certain request
based services..)
-Sterling