forum.coppermine-gallery.net

Dev Board => cpg1.4 Testing/Bugs => cpg1.4 Testing/Bugs: FIXED/CLOSED => Topic started by: kegobeer on September 12, 2004, 03:48:44 am

Title: showdoc.php variables question
Post by: kegobeer on September 12, 2004, 03:48:44 am
There are a bunch of unset variables requested:

$remove_head = $_REQUEST['remove_head'];
$remove_stylesheet = $_REQUEST['remove_stylesheet'];
$remove_to_top = $_REQUEST['top'];
$header = $_REQUEST['h'];
$text = $_REQUEST['t'];
$style = $_REQUEST['style'];
$close = $_REQUEST['close'];
$base = $_REQUEST['base'];

And then a few if-then statements for some of them.  While it doesn't cause any problems (unless you have error reporting turned on in php) I was wondering if this for future use, or maybe no longer used at all.
Title: Re: showdoc.php variables question
Post by: Joachim Müller on September 12, 2004, 11:03:27 am
I only introduced showdoc.php a short while ago, will have to look into this. I was planning to add more functionality by implementing code related to those vars. I will do a clean-up of the code (assigning this task to me).

Moving this thread to bugs section, so I won't forget about it.

Joachim
Title: Re: showdoc.php variables question
Post by: Joachim Müller on October 04, 2004, 07:44:19 am
you were right, so I removed the unneeded vars from showdoc.php.

Joachim
Title: Re: showdoc.php variables question
Post by: kegobeer on October 06, 2004, 02:09:35 am
Has this been committed to devel yet?  I still get the same undefined index errors.
Title: Re: showdoc.php variables question
Post by: Joachim Müller on October 06, 2004, 08:39:37 am
OK, finally even I got the point  ;)
I added a check for existance for every var that might get used. Committed to devel.

Joachim