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

[Kirk Steffensen]

Thanks!  I'll try that tonight and if it works, I'll feed it back to the
Gallery2 folks.  

Kirk 

-----Original Message-----
From: wp-testers-bounces at lists.automattic.com
[mailto:wp-testers-bounces at lists.automattic.com] On Behalf Of Andy Skelton
Sent: Thursday, February 02, 2006 6:56 AM
To: wp-testers at lists.automattic.com
Subject: Re: [wp-testers] Can we delete this code from classes.php?

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");
_______________________________________________
wp-testers mailing list
wp-testers at lists.automattic.com
http://lists.automattic.com/mailman/listinfo/wp-testers