noob help with T_VARIABLE error

natronp

I'm getting a T_VARIABLE error on line 19 of this code. I assume it means I'm incorrectly defining a variable? I know just enuff to be dangerous. thanks for any help:

<?php 

//this line includes the database connection variables
include_once("config.php"); 

$user=$_POST['user'];
$pass=$_POST['pass'];

if ($user && $pass){
$result = mysql_query("SELECT * FROM users ORDER BY user"); 

$num = mysql_num_rows( $result );
if ($num == 1){
	// And now we need to output an XML document 
// We use the names of columns as <row> properties. 
		echo '<?xml version="1.0" encoding="UTF-8"?>'; 
		echo '<datapacket>'; 
		while($row=mysql_fetch_array($result)){ 0
    	$line = '<user message="" user="'.$row[user].'" pass="'.$row[pass].'" first="'.$row[first].'" email="'.$row[email].'" last="'.$row[last].'" />'; 
    	echo $line; 
}; 
echo '</datapacket>'; 

} else {
echo '<?xml version="1.0" encoding="UTF-8"?>'; 
echo '<invalid message="Sorry, that Login ID and/or Password is not valid"/>' 
}
}



?>

this is line 19:

	$line = '<user message="" user="'.$row[user].'" pass="'.$row[pass].'" first="'.$row[first].'" email="'.$row[email].'" last="'.$row[last].'" />';

edwardsbc

Probably due to the "zero" after the open bracket on the previous line. Also, for correctness refer to your array indexes as strings. PHP first interprets your style as constants and then falls back to "assuming" they should be strings. This is probably generating E_NOTICE message in your error log (if you have E_NOTICE on).

Use this: $row['user']

natronp

thanks... yeah, that would probably screw things huh? (zero).

they are column names from my database. for all practical purposes is that interchangeable with the term "array index"???

I'll change them to strings. thanks!

bpat1434

No it's not......

When you use the mysql_fetch_array() you fetch one row of the result set into an array. The keys (or indexes) are derived from the column titles, and the values from the current row being retrieved.

A column title is just a header, nothing more. While sometimes they are interchangable (like now), most of the time, it's safer to say that you have column headers, and array indices.