[Hirthas]

Hi i could really use some help here. I am making a account management site in php and I am getting this error that I'm not sure how to fix or why i am getting it. Here is the error that comes up when I try to register a new account:

Notice: Undefined index: userID in c:\Inetpub\wwwroot\356\vmicchia\Project1\createNewAccount.php on line 5

Notice: Undefined index: passwd in c:\Inetpub\wwwroot\356\vmicchia\Project1\createNewAccount.php on line 7

Notice: Undefined index: passwd2 in c:\Inetpub\wwwroot\356\vmicchia\Project1\createNewAccount.php on line 8

Notice: Undefined index: userID in c:\Inetpub\wwwroot\356\vmicchia\Project1\createNewAccount.php on line 9

Here is the form that is being submitted:

 <form method="post" id="form0" action="createNewAccount.php">
		<ul>
		  <li>
			<label for="login">User Name:</label>
			<input id="userID" name="userID" type="text" size="20" maxlength="20" />
		  </li>
		  <li>
			<label for="pass">Password:</label>
			<input id="passwd" name="passwd" type="password" size="20" maxlength="20" />
		  </li>
		  <li>
			<label for="pass2">Confirm Password:</label>
			<input id="passwd2" name="passwd2" type="password" size="20" maxlength="20" />
		  </li>
<li>
			<input id="submit" name="submit" type="submit" value="Register" />
		  </li>
		  <li><?php echo $_SESSION["errorMsg"]; ?></li>
		</ul>

and here is the pass through page it goes to:

<?php
//start the session
	session_start();
	
	$login = $_POST["userID"];
	//echo $login;
	$pass1 = md5($_POST["passwd"]);
	$passConfirm = md5($_POST["passwd2"]);
	$_SESSION["login"] = $_POST["userID"];
	$passMatch = false;
	
	if ((empty($_POST["login"])) || (empty($_POST["pass"])) || (empty($_POST["pass2"]))) /*||  (empty($_POST["lastName"])) || (empty($_POST["firstName"])) || (empty($_POST["email"])) || (empty($_POST["news"])))*/{
		$_SESSION["errorMsg"] = "Please make sure to fill out all the fields.";
		header("Location:index.php");
		exit;
	}
	
	include("includes/openDbConn.php");
	
	if ($pass1 == $passConfirm){
		$sql="SELECT Login FROM employee WHERE Login='".$login."'";
		//echo $sql;
		
		$result=mysql_query($sql);
		
		//nested if to check if the username is taken (if result)
		if(empty($result)){
			$num_results = 0;
		}else{
			$num_results = mysql_num_rows($result);
		}//end if result
		
		if ($num_resukts = 0){
			$sql="INSERT INTO employees(Login, Passwd) VALUES ('".$login."', '".$pass1."')";
			//echo $sql;
			
			$result=mysql_query($sql);
			
			$_SESSION["errorMsg"] ="You have been added successfully.";
			
			$passMatch = true;
			
		}else{
			$_SESSION["errorMessage"] = "User Name already exists";
		$passMatch = false;
		}//end if taken
	}else{
		$_SESSION["errorMessage"] = "Your Passwords do not match.";
		$passMatch =false;
	}
	
	if ($passMatch){
		CleanUp();
		header("Location:userHome.php");
		exit;
	}else{
		CleanUp();
		header("Location:createNewAccount.php");
		exit;
}

	
	
//this function clears out all variables used on this page.																																																																									
function CleanUp(){
	$login="";
	$pass1="";
	$passConfirm="";
	$passMatch="";
	$sql="";
	$result="";
	$num_results="";
}

?>

any help would be really great thanks. I will be adding more elements later but this is just to test and make sure it works.

This post has been edited by Hirthas: 05 October 2009 - 10:40 AM

[ShaneK]

That error signifies you're trying to access a key that doesn't exist on some array.

So you need to change things like:

$login = $_POST["userID"];

to

$login = (isset($_POST["userID"]) ? $_POST["userID"] : null); //Basically, "if $_POST["userID"] is set, then $login is set to it, otherwise it's null."

Or turn off the display of errors, the choice is yours.

Yours,
Shane~

This post has been edited by ShaneK: 05 October 2009 - 10:53 AM

[PsychoCoder]

I dont see a closing form tag in the HTML you posted (Is it in the HTML and you forgot to post it?). If it's not in the HTML page then add it.

When setting your variables try using isset() before setting a value (as the post array may not be populated when the page first executes). Something like this

if(isset($_POST["userID"]))
{
	   $login = $_POST["userID"];
}

[Hirthas]

OK I used the isset idea and it gets rid of the errors. However my big problem is it still isn't reading anything into the variables so I can't insert anything into the DB. Also, there is an </form> tag just didn't copy paste enough.

[cmwise]

Your Code:

if ($num_resukts = 0){

mispelled results!

[Valek]

You'll also want it to be if ($num_results === 0){, as the way you presently have it will set $num_resukts to 0 and evaluate to true. The line above fixes the typo and checks the value as opposed to assigning it. The fact that I used a triple equals there instead of a double shows that it's also checking the type, so if $num_results for some reason comes up false (and doesn't halt the script because of it), it doesn't execute the insert query.