[RESOLVED] HELP!!! Keep getting problem with undefined index....

RossBryan

I really dont get it now i keep having this problem and there always seems to be a different solutiion. Im defining variables and yet its telling me that they arent defined; so confusing :s

Notice: Undefined index: user in C:\xampp\htdocs\Care2Share\design\status.php on line 4
Notice: Undefined index: id in C:\xampp\htdocs\Care2Share\design\status.php on line 5

<?php 

$username = $_SESSION['user'];
$userid = $_SESSION['id'] ;

if ($username)
{
        echo "You are logged in as $username"; 
}

else
{
        echo "<form action='loginC2S.php' method='POST' >

<table style='margin-left: auto; margin-right: auto;'>
<tr>
        <td><input type='text' id='usernamebox' name='user' tabindex='1' value='Username' class='textbox' onfocus='usernamebox_focus();' onblur='usernamebox_blur();'></td>
        <td><a href='register.php'>Register</a></td>
</tr> 
<tr>
    <td><input type='text' id='passwordbox' name='pass' tabindex='2' value='Password' class='textbox' onfocus='passwordbox_focus();' onblur='passwordbox_blur();' /></td>
    <td><input type='submit' name='login' value='login' tabindex='3' class='button'> </td>
</tr>
</table>
</form>";
}

?>

I know coding can be frustrating at times but im seeing myself not having any hair left the way this is going lol

Derokorian

Unless this is included in a larger script you need to call session_start() before you try to access session variables. Also before any output is sent to the browser:

<?php 

session_start(); //Starts the previously initialized session, or creates a new one.

$username = $_SESSION['user']; 
$userid = $_SESSION['id'] ;

And then you should also be checking to verify that the session variables you want are initialized before trying to use them.

<?php 

session_start(); //Starts the previously initialized session, or creates a new one.

if( isset($_SESSION['user']) ) {
   $username = $_SESSION['user']; 
}
if( isset($_SESSION['id']) ) {
   $userid = $_SESSION['id'] ;
}

RossBryan

Derokorian;10986706 wrote:
Unless this is included in a larger script you need to call session_start() before you try to access session variables. Also before any output is sent to the browser:
<?php 

session_start(); //Starts the previously initialized session, or creates a new one.

$username = $_SESSION['user']; 
$userid = $_SESSION['id'] ;
And then you should also be checking to verify that the session variables you want are initialized before trying to use them.
<?php 

session_start(); //Starts the previously initialized session, or creates a new one.

if( isset($_SESSION['user']) ) {
   $username = $_SESSION['user']; 
}
if( isset($_SESSION['id']) ) {
   $userid = $_SESSION['id'] ;
}

Thanks that worked great but ended up causing another problem with another section of coding in an associated file:

Notice: Use of undefined constant numrows - assumed 'numrows' in C:\xampp\htdocs\Care2Share\loginC2S.php on line 62

$numrow =mysql_num_rows($query);


if (numrows == 1)
{
	$rows = mysql_fetch_assoc($query);
	$dbuser = $rows['username'];
	$dbid = $rows['id'];

	//logs user in		
	$_SESSION['user'] =$dbuser;
	$_SESSION['id'] = $dbid;	
	echo "<center>You have been logged in.</center>";
}

Derokorian

Look at these two lines closely:

$numrow =mysql_num_rows($query); 
    if (numrows == 1)

Notice the missing $ telling PHP numrows is a variable name, and also that the variable names don't match? Try:

if ($numrow == 1)

RossBryan

Haha im so sorry, and thank you so much, its obvious that ive been really lazy. Not a good start to using PHP, so ashamed

Thanks again all working perfectly now, and next time i wont be so fast to post an SOS. 🙂

Derokorian

NP, remember to mark the thread resolved if every thing is working =D