Could use a little help if you're feeling generous!

jamesojackson

I keep getting the following error:

Parse error: parse error, unexpected $ in /home/html/overrideoffer.php on line 57

Here's my code:

<? 
session_start();

if ($_SESSION['membersarea']!="Active") {
	echo "<script>document.location.href='login.php'</script>";

} else { 



include_once ("config/config.php");



$page_title = 'View the Current Users';
include ("themes/".$setts['default_theme']."/header.php");

// Page header.
echo '<h1 id="mainhead">Registered Users</h1>';


// Make the query.
$aucid=$_REQUEST['aucofferid'];
$override= "SELECT * FROM realestate_auction_offers WHERE auctionid='$aucid'";
$oresult= mysql_query($override);


if ($num > 0) { // If it ran OK, display the records.

echo "<p>There are currently $num registered users.</p>\n";

// Table header.
echo '<table align="center" cellspacing="0" cellpadding="5">
<tr><td align="left"><b>Name</b></td><td align="left"><b>Date Registered</b></td></tr>
';

// Fetch and print all the records.
while ($row = mysql_fetch_array($oresult, MYSQL_ASSOC)) {
	echo '<tr><td align="left">' . $row['buyerid'] . '</td><td align="left">' . $row['amount'] . '</td></tr>
	';
}

echo '</table>';

mysql_free_result ($result);	

} else { 
	echo '<p class="error">There are currently no registered users.</p>';
}

mysql_close(); // Close the database connection.


?>  //[COLOR="Red"]this is line 57 shown in error[/COLOR]

can't figure out why I'm getting this error... when I remove session_start i don't get the error. any help would be greatly appreciated.

Scriptmaker

} else {

Do you ever close this first 'else' with a }?

bradgrafelman

Just to clarify, Scriptmaker is referring to the else statement opened on line 7 of your code. Speaking of your code...

When posting PHP code, please use the board's [PHP][/PHP] bbcode tags - they make the code much, much easier to read (as well as aiding the diagnosis of syntax errors). You should edit your post now and add these bbcode tags to help others in reading/analyzing your code.

NogDog

You have 5 "{" but only 4 "}".

A habit I've developed is to always type the closing "}" as soon as I type the opening brace.

First:

if($something == true)
{

}

Then start filling in the code for that block, indented appropriately:

if($something == true)
{
   myfunction($foo, $bar);
   echo "Hello, World!";
}

Developing that habit will avoid a lot of unbalanced braces.

bradgrafelman

NogDog wrote:
Developing that habit will avoid a lot of unbalanced braces.

In addition, searching the board a bit (there's a large topic devoted to this error) and using an editor that pairs up opening braces/brackets/parenthesis visually also helps! :p

EDIT: By the way, just to throw this out there: Your script appears to be vulnerable to SQL injection attacks. You should sanitize (using [man]mysql_real_escape_string/man, for example) all user-supplied data before placing it in a query.