unexpected T_VARIABLE in

geohix

I am trying to add wysiwyg editor in my form and get this error on line 20 when validating my directory. Can anyone assist with this. Line 20 in my code is right under // validate the directory

// validate the directory
$_GET['dir'] = $_POST['dir'] ? $_POST['dir'] : $_GET['dir'];
if($_GET['dir']) {
	if(substr($_GET['dir'], -1, 1)!='/') {
		$_GET['dir'] = $_GET['dir'] . '/';
	}
	$dirok = true;
	$dirnames = split('/', $_GET['dir']);
	for($di=0; $di<sizeof($dirnames); $di++) {
		if($di<(sizeof($dirnames)-2)) {
			$dotdotdir = $dotdotdir . $dirnames[$di] . '/';
		}
	}
	if(substr($_GET['dir'], 0, 1)=='/') {
		$dirok = false;
	}

if($_GET['dir'] == $leadon) {
	$dirok = false;
}

if($dirok) {
	$leadon = $_GET['dir'];
}
}

thanks

laserlight

The error may lie on the previous line, e.g., a missing terminating semi-colon.

david8787

$_GET['dir'] = $_POST['dir'] ? $_POST['dir'] : $_GET['dir'];

I think you need ==, so it should be:

$_GET['dir'] == $_POST['dir'] ? $_POST['dir'] : $_GET['dir'];

Assuming that's what you meant.

laserlight

david8787 wrote:
I think you need ==, so it should be:

No, that looks correct, i.e., it assigns to $GET['dir'] either $POST['dir'] or $GET['dir'], depending on the evaluation of $POST['dir']. That said, one might just use $REQUEST['dir'] instead, and perhaps the intention is to use isset or empty instead of evaluating $POST['dir'].

geohix

That was it, the line directly above it did not have a ; added it and it worked perfectly. Thanks I spent hours trying to figure this out. I guess that is why i am a newbie

laserlight

No worries, we learn from our mistakes 🙂

You can mark this thread as resolved using the thread tools, unless there is something related that you wish to ask 🙂

bradgrafelman

Don't forget to mark this thread resolved using the link on the Thread Tools menu.

Also, in the future, please post all parse errors in the thread stickied in the Coding forum entitled Parse Error: syntax error.

EDIT: Woops - simultaneous post again. :p