[wp-testers] Can we delete this code from classes.php?

Andy Skelton skeltoac at gmail.com
Thu Feb 2 11:55:55 GMT 2006


On 2/2/06, Kirk Steffensen <blogger at steffensenfamily.com> wrote:
> Andy,
>
> Thanks!  That worked!
>
> I'm still not sure why, at first glance through the code, though.  (Gotta
> shower for work, so I haven't been able to dig deep.)  Can you explain what
> you think is happening?

I think line 368 of gallery2/main.php
	$phpVm->header("Location: $redirectUrl");
is not setting the HTTP status code to 302.

I read a bug about this on php.net. Not that I understood it all, but
it had to do with PHP not implementing the CGI spec on setting HTTP
headers. Something like that.

Maybe we can help the Gallery project. Try undoing all prior fixes and
adding this line directly after the one referenced above:
	$phpVm->header("HTTP/1.1 302");


More information about the wp-testers mailing list