Re: cvs: phpweb /include layout.inc

From: Date: Sun, 16 Jan 2005 14:12:45 +0000
Subject: Re: cvs: phpweb /include layout.inc
References: 1 2 3 4  Groups: php.mirrors 
Request: Send a blank email to php-mirrors+get-27242@lists.php.net to get a copy of this message
Gabor Hojtsy wrote:
I did it in layout.inc because it was the only file mentionning control-structures.php :) I just had a look to error.php, and cannot find where I need to fix the code. Can you fix it for me ? (I'll learn with your commit)
If you look at the code after the stuff you added, you will see that it is supposed to check for an existing exact filename. Now if you use # in the filename, it is not going to exist :)
There was a return in my patch, the code after it wasn't executed. But I agree, I've been dump on this one :)
Now I moved the if redirection to the error.php handler, where other such stuff is located.
Thanks, I'll be able to do it properly next time. didou

« previous php.mirrors (#27242) next »