PHP Edit form help

I am having trouble getting an edit record form to display records form my database table, i cant seem to get it to post the record into to form properly so that a user can edit the record. Can anyone help?

My code is:

<?php

if (isset($_POST['submit'])) {

$error_stat = 0;
$jobtitle_message = '';
$jobcatergory_message = '';
$joblocation_message = '';
$employmenttype_message = '';
$salary_message = '';
$date_message = '';
$educationallevel_message = '';
$description_message = '';
$filesize_message = '';
$filetype_message = '';


$jobtitle = trim($_POST['jobtitle']);
$jobcatergory = trim($_POST['jobcatergory']);
$joblocation = trim($_POST['joblocation']);
$employmenttype = trim($_POST['employmenttype']);
$salary = trim($_POST['salary']);
$date = trim($_POST['date']);
$educationallevel = trim($_POST['educationallevel']);
$description = trim($_POST['description']);



//Error checking



// Job Title check)
if (!$jobtitle) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$jobtitle_message = '*Please enter a job title*';
}

else if (ctype_digit($jobtitle)) {
$error_stat = 1;
$jobtitle_message .= '*Invalid Job Title*';
}

else if ( preg_match( '/\W/', $jobtitle)){
$error_stat = 1;
$jobtitle_message = '*Invalid jobtitle, letters only, no spaces*';

}

$jobtitle = $_POST['jobtitle'];
$jobtitle = trim($jobtitle);

if (strlen($jobtitle) > 30){
$error_stat = 1;
$jobtitle_message = '*Job Title must be 20 characters or less*';
}






// Job Catergory Check)
if ($jobcatergory == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$jobcatergory_message = '*Please select a Job Catergory*';
}





// Job Location Check)
if ($joblocation == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$joblocation_message = '*Please select a Job location*';
}


// Employment Type Check)
if ($employmenttype == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$employmenttype_message = '*Please select Employment type*';
}





// Salary check)
if (!$salary) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$salary_message = '*Please enter job salary*';
}

else if (!ctype_digit($salary)) {
$error_stat = 1;
$salary_message .= '*Invalid salary*';
}




//Date check)
if (empty($date)) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a dob
$date_message = '*Please enter job closing date*';
}

//Check the format and explode into $parts
elseif (!ereg("^([0-9]{2})/([0-9]{2})/([0-9]{4})$",
$date, $parts)){
$error_stat = 1;

//Set the message to tell the user the date is invalid
$date_message = '*Invalid date, must be DD/MM/YYYY format*';
}

elseif (!checkdate($parts[2],$parts[1],$parts[3]))
{
$error_stat = 1;

//Set the message to tell the date is invalid for the month entered
$date_message = '*Invalid date, month must be between 1-12*';
}


// Job Description check)
if (!$description) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$description_message = '*Please enter a job description*';
}

$description = $_POST['description'];
$description = trim($description);

if (strlen($description) > 150){
$error_stat = 1;
$description_message = '*Job Title must be 150 characters or less*';
}




// Educational Level Check)
if ($educationallevel == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$educationallevel_message = '*Please select Educational level required*';
}


if( $_FILES['userfile']['size'] > 2000000 ){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$filesize_message = '*Filesize too large *';

}

$fileTypes = array("application/pdf", "application/msword");

if( !in_array("{$_FILES['userfile']['type']}", $fileTypes) ){
$error_stat = 1;
$filetype_message = '*Filetype not allowed *';

}




$uploadDir = 'applicationforms/';

if (isset($_POST['submit']) && $error_stat == 0) {


$fileName = $_FILES['userfile']['name'];
$tmpName = $_FILES['userfile']['tmp_name'];
$fileSize = $_FILES['userfile']['size'];
$fileType = $_FILES['userfile']['type'];

// the files will be saved in filePath
$filePath = $uploadDir . $fileName;

// move the files to the specified directory
// if the upload directory is not writable or
// something else went wrong $result will be false
$result = move_uploaded_file($tmpName, $filePath);


include("database.php");

if(!get_magic_quotes_gpc())
{
$fileName = addslashes($fileName);
$filePath = addslashes($filePath);
}


mysql_query("UPDATE job SET username='" . $_POST["username"] . "',jobtitle='" . $_POST["jobtitle"] . "',jobcatergory='" . $_POST["jobcatergory"] . "',joblocation='" . $_POST["joblocation"] . "',employmenttype='" . $_POST["employmenttype"] . "',salary='" . $_POST["salary"] . "',date='" . $_POST["date"] . "',educationallevel='" . $_POST["educationallevel"] . "',description='" . $_POST["description"] . "',name='$fileName', type='$fileType', size='$fileSize', path='$filePath' WHERE username='" . $_SESSION["username"] . "'");
?>

<br />
<a href="index.php">Back to main page</a>
<br />
<br />
<br />
You have successfully updated your account .
<?php
}
else
{
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='" . $_SESSION["username"] . "'"));
?>
<form method="post" class="addform" action="" enctype="multipart/form-data">
<fieldset>
<label for="cvtitle">Edit Job</label><fieldset>
<p align="right">&nbsp;</p>
</fieldset>
<label for="username">Username:</label>
<input readonly name="username" type="text" id="username" value="<?php echo $_SESSION["username"]; ?>" /><br />
</fieldset>

<hr class="hr_blue"/>

<fieldset>
<label for="jobtitle">Job Title:</label>
<input name="jobtitle" type="text" id="jobtitle" value="<?php echo $_POST['jobtitle']; ?>"/>
<span class="redboldtxt"><?php echo "$jobtitle_message";?></fieldset></span>


<fieldset>
<label for="jobcatergory">Job Catergory:</label><p></p>
<select name="jobcatergory">
<option value="Please Select">Please Select</option>
<?php
$jobcatergory_opts = array(
"Accountancy and Finance",
"Banking and Insurance",
"Construction",
"Customer Service",
"Engineering",
"Management",
"Hotel and Catering",
"Information Technology",
"Legal",
"Marketing",
"Medical",
"Retail",
"Sales",
"Secretarial",
"Transport and Distribution",
"Working from home",
);
foreach($jobcatergory_opts as $opt){
$selected = $_POST['jobcatergory'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$jobcatergory_message";?><?php echo $error['jobcatergory']; ?></span></fieldset>

<fieldset>
<label for="joblocation">Location:</label>
<p></p>
<select name="joblocation">
<option value="Please Select">Please Select</option>
<?php
$joblocation_opts = array(
"Co.Antrim",
"Co.Armagh",
"Co.Down",
"Co.Fermanagh",
"Co.Londonderry",
"Co.Tyrone",
);
foreach($joblocation_opts as $opt){
$selected = $_POST['joblocation'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$joblocation_message";?><?php echo $error['joblocation']; ?></span></fieldset>


<fieldset>
<label for="employmenttype">Job Type:</label><p></p>
<select name="employmenttype">
<option value="Please Select">Please Select</option>
<?php
$employmenttype_opts = array(
"permanent fulltime",
"permanent parttime",
"temporary fulltime",
"temporary parttime",
);
foreach($employmenttype_opts as $opt){
$selected = $_POST['employmenttype'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$employmenttype_message";?><?php echo $error['employmenttype']; ?></span></fieldset>


<fieldset>
<label for="salary">Salary:</label>
<input name="salary" type="text" id="salary" value="<?php echo $_POST['salary']; ?>"/>
<span class="redboldtxt"><?php echo "$salary_message";?></span></fieldset>

<fieldset>
<label for="date">Closing Date:</label>
<input name="date" type="text" id="date" value="<?php echo $_POST['date']; ?>"/>
<span class="redboldtxt"><?php echo "$date_message";?></span></fieldset>


<fieldset>
<label for="educationallevel">Qualification Level Required:</label><p></p>
<select name="educationallevel">
<option value="Please Select">Please Select</option>
<?php
$educationallevel_opts = array(
"GCSE",
"A-Level",
"Third Level Certification",
"Third Level Diploma",
"Third Level Degree",
"Post Graduate Qualification",
"Masters",
"PHD",
"Professional Qualification",
"Part Professional Qualification",
"Trade Qualification",
);
foreach($educationallevel_opts as $opt){
$selected = $_POST['educationallevel'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$educationallevel_message";?><?php echo $error['educationallevel']; ?></span></fieldset>

<hr class="hr_blue"/>

<fieldset>
Job Description -<br />
</fieldset>

<fieldset>
<textarea rows="2" name="description" cols="20"><?php echo $account["description"]; ?></textarea><p></p>
<span class="redboldtxt"><?php echo "$description_message";?></span></fieldset>

Upload Application Form</label>
<input type="hidden" name="MAX_FILE_SIZE" value="2000000"><input name="userfile" type="file" class="box" id="userfile">
<span class="redboldtxt"><?php echo "$filesize_message";?></span>
<span class="redboldtxt"><?php echo "$filetype_message";?></span>
</fieldset>

<p></p>
<fieldset>
<p class="submit"><input type="submit" name="submit" value="Add Job" />

</fieldset>
</form>
<?php
}
?>

Your business logic should be something like
if(POST)
Check for errors and update, if there is no error
Get all required data from DB
Display the form with these data (it may be old, or it may be new- if an update occurred just above )

ie. after this query
else
{
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='" . $_SESSION["username"] . "'"));

?>
you have to display the data in the form as
<input name="jobtitle" type="text" id="jobtitle" value="<?php echo $account['jobtitle']; ?>"/>, where jobtitle is a field in DB table.

Your business logic should be something like
if(POST)
Check for errors and update, if there is no error
Get all required data from DB
Display the form with these data (it may be old, or it may be new- if an update occurred just above )

ie. after this query
else
{
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='" . $_SESSION["username"] . "'"));

?>
you have to display the data in the form as
<input name="jobtitle" type="text" id="jobtitle" value="<?php echo $account['jobtitle']; ?>"/>, where jobtitle is a field in DB table.

abduraooft thanks for the reply, i have a viewjob page with the editjob link beside each job record, the link is:

"<a href="editjob.php?username=$username&&id=$id">"

this gets the username and id assigned to the job record. I am having trouble getting the record back into the edit form, it wont display the record that has been selected to be edited from the viewjob page. Is there a way i can get the data to display in the edit form. I think i need a "get" statement but im not sure how to do this.

I have got the data for the first record going to the form using the code:

<input name="jobtitle" type="text" id="jobtitle" value="<?php echo $account['jobtitle']; ?>"/>

but when i select another record it displays the same record, i doesnt same to be selecting the record based on the id.

Do i need to use:

$username = $_GET['username'];
$id = $_GET['id'];


To display the record that has been selected from the viewjob page?

I don't know the setup of your DB, but if the records are based on a username and id of the record, you have to include id also in to your query's where clause, otherwise you will get the same data.
You don't need to pass username through link, as it is available in session (and this is the secure way). So take it from session and get record id through $_GET[] ( from link).

PS:1) You have to ensure that the id received is in a valid format.
2) select * is not considered as good practice, specify the required fields only.
3) Use limit ( in this case limit 1) to speed up your queries.

I don't know the setup of your DB, but if the records are based on a username and id of the record, you have to include id also in to your query's where clause, otherwise you will get the same data.
You don't need to pass username through link, as it is available in session (and this is the secure way). So take it from session and get record id through $_GET[] ( from link).


Hey thanks for that. I am tryin the code below in the where clause, but im not sure if the syntax is correct, im getting an error: "Parse error: syntax error, unexpected $end"

<?php

if (isset($_POST['submit'])) {

$username = $_GET['username'];
$id = $_GET['id'];


$error_stat = 0;
$jobtitle_message = '';
$jobcatergory_message = '';
$joblocation_message = '';
$employmenttype_message = '';
$salary_message = '';
$date_message = '';
$educationallevel_message = '';
$description_message = '';
$filesize_message = '';
$filetype_message = '';


$jobtitle = trim($_POST['jobtitle']);
$jobcatergory = trim($_POST['jobcatergory']);
$joblocation = trim($_POST['joblocation']);
$employmenttype = trim($_POST['employmenttype']);
$salary = trim($_POST['salary']);
$date = trim($_POST['date']);
$educationallevel = trim($_POST['educationallevel']);
$description = trim($_POST['description']);



//Error checking



// Job Title check)
if (!$jobtitle) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$jobtitle_message = '*Please enter a job title*';
}

else if (ctype_digit($jobtitle)) {
$error_stat = 1;
$jobtitle_message .= '*Invalid Job Title*';
}

else if ( preg_match( '/\W/', $jobtitle)){
$error_stat = 1;
$jobtitle_message = '*Invalid jobtitle, letters only, no spaces*';

}

$jobtitle = $_POST['jobtitle'];
$jobtitle = trim($jobtitle);

if (strlen($jobtitle) > 30){
$error_stat = 1;
$jobtitle_message = '*Job Title must be 20 characters or less*';
}






// Job Catergory Check)
if ($jobcatergory == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$jobcatergory_message = '*Please select a Job Catergory*';
}





// Job Location Check)
if ($joblocation == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$joblocation_message = '*Please select a Job location*';
}


// Employment Type Check)
if ($employmenttype == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$employmenttype_message = '*Please select Employment type*';
}





// Salary check)
if (!$salary) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$salary_message = '*Please enter job salary*';
}

else if (!ctype_digit($salary)) {
$error_stat = 1;
$salary_message .= '*Invalid salary*';
}




//Date check)
if (empty($date)) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a dob
$date_message = '*Please enter job closing date*';
}

//Check the format and explode into $parts
elseif (!ereg("^([0-9]{2})/([0-9]{2})/([0-9]{4})$",
$date, $parts)){
$error_stat = 1;

//Set the message to tell the user the date is invalid
$date_message = '*Invalid date, must be DD/MM/YYYY format*';
}

elseif (!checkdate($parts[2],$parts[1],$parts[3]))
{
$error_stat = 1;

//Set the message to tell the date is invalid for the month entered
$date_message = '*Invalid date, month must be between 1-12*';
}


// Job Description check)
if (!$description) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$description_message = '*Please enter a job description*';
}

$description = $_POST['description'];
$description = trim($description);

if (strlen($description) > 150){
$error_stat = 1;
$description_message = '*Job Title must be 150 characters or less*';
}




// Educational Level Check)
if ($educationallevel == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$educationallevel_message = '*Please select Educational level required*';
}


if( $_FILES['userfile']['size'] > 2000000 ){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$filesize_message = '*Filesize too large *';

}

$fileTypes = array("application/pdf", "application/msword");

if( !in_array("{$_FILES['userfile']['type']}", $fileTypes) ){
$error_stat = 1;
$filetype_message = '*Filetype not allowed *';

}




$uploadDir = 'applicationforms/';

if (isset($_POST['submit']) && $error_stat == 0) {


$fileName = $_FILES['userfile']['name'];
$tmpName = $_FILES['userfile']['tmp_name'];
$fileSize = $_FILES['userfile']['size'];
$fileType = $_FILES['userfile']['type'];

// the files will be saved in filePath
$filePath = $uploadDir . $fileName;

// move the files to the specified directory
// if the upload directory is not writable or
// something else went wrong $result will be false
$result = move_uploaded_file($tmpName, $filePath);


include("database.php");

if(!get_magic_quotes_gpc())
{
$fileName = addslashes($fileName);
$filePath = addslashes($filePath);
}
}


mysql_query("UPDATE job SET username='" . $_POST["username"] . "',jobtitle='" . $_POST["jobtitle"] . "',jobcatergory='" . $_POST["jobcatergory"] . "',joblocation='" . $_POST["joblocation"] . "',employmenttype='" . $_POST["employmenttype"] . "',salary='" . $_POST["salary"] . "',date='" . $_POST["date"] . "',educationallevel='" . $_POST["educationallevel"] . "',description='" . $_POST["description"] . "',name='$fileName', type='$fileType', size='$fileSize', path='$filePath' WHERE username='" . $_SESSION["username"] . "'");
?>

<br />
<a href="index.php">Back to main page</a>
<br />
<br />
<br />
You have successfully updated the job account .
<?php
}
else
{
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='$username' AND id='$id'"));
if ($account['username'] == $username && $account['id'] == $id)
{
?>

What is the exact error? Without line numbers its hard to tell you where the error is at. This usually means you have to many brackets.

What is the exact error? Without line numbers its hard to tell you where the error is at. This usually means you have to many brackets.

Its says the error is on the last line of the code which i dont understand:

<?php
session_start();
if(!isset($_SESSION['username'])){
header("Location: index2.php");
}
include("database.php");
include("loginemployer.php");
?>


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Jobs4U</title>


<!--CSS Stylesheet -->
<link rel="stylesheet" type="text/css" href="./css/main.css" />
<link rel="stylesheet" type="text/css" href="./css/registerform.css" />
</head>

<body style="margin-left:1px;margin-right:1px;margin-bottom:0px;margin-top:0px;">

<!--Main Container -->
<div id="container">

<!--Main Header -->
<div id="header" style="width: 988px; height: 154px">
&nbsp;
</div>


<!-- Navigation Menu -->
<div id="nav_menu">
<div class="link_container" style="width: 852px; height: 32px">
<a href="Index.php" class="main_nav">home</a><span class="link_separator">::</span>
<a href="page.htm" class="main_nav">products</a><span class="link_separator">::</span>
<a href="page.htm" class="main_nav">services</a><span class="link_separator">::</span>
<a href="page.htm" class="main_nav">about us</a><span class="link_separator">::</span>
<a href="page.htm" class="main_nav">contact</a>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;
<input type="text" name="search0" maxlength="30" size="30" >&nbsp;&nbsp;
<a href="page.htm" class="main_nav">Quick Search&gt;&gt;&gt;</a></div>
</div>

<!-- Container (left) -->
<div id="body">
<div id="body-left-panel" style="width: 654px; height: 981px">
<div style="padding:10px;">

<!-- Left container 1(Top) -->
<div class="box_upper_partviewdetails">
<img src="img/box_up-left_edge.png" alt="" class="align-left"/>
<img src="img/box_up-right_edge.png" alt="" class="align-right"/>
<span class="box_header">:: Edit Job</span></div>

<!--(Middle) -->
<div class="box_middle_partaddjob">
<div class="pad_15tah">

<!--Register Form -->


<?php

if (isset($_POST['submit'])) {

$username = $_GET['username'];
$id = $_GET['id'];


$error_stat = 0;
$jobtitle_message = '';
$jobcatergory_message = '';
$joblocation_message = '';
$employmenttype_message = '';
$salary_message = '';
$date_message = '';
$educationallevel_message = '';
$description_message = '';
$filesize_message = '';
$filetype_message = '';


$jobtitle = trim($_POST['jobtitle']);
$jobcatergory = trim($_POST['jobcatergory']);
$joblocation = trim($_POST['joblocation']);
$employmenttype = trim($_POST['employmenttype']);
$salary = trim($_POST['salary']);
$date = trim($_POST['date']);
$educationallevel = trim($_POST['educationallevel']);
$description = trim($_POST['description']);



//Error checking



// Job Title check)
if (!$jobtitle) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$jobtitle_message = '*Please enter a job title*';
}

else if (ctype_digit($jobtitle)) {
$error_stat = 1;
$jobtitle_message .= '*Invalid Job Title*';
}

else if ( preg_match( '/\W/', $jobtitle)){
$error_stat = 1;
$jobtitle_message = '*Invalid jobtitle, letters only, no spaces*';

}

$jobtitle = $_POST['jobtitle'];
$jobtitle = trim($jobtitle);

if (strlen($jobtitle) > 30){
$error_stat = 1;
$jobtitle_message = '*Job Title must be 20 characters or less*';
}






// Job Catergory Check)
if ($jobcatergory == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$jobcatergory_message = '*Please select a Job Catergory*';
}





// Job Location Check)
if ($joblocation == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$joblocation_message = '*Please select a Job location*';
}


// Employment Type Check)
if ($employmenttype == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$employmenttype_message = '*Please select Employment type*';
}





// Salary check)
if (!$salary) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$salary_message = '*Please enter job salary*';
}

else if (!ctype_digit($salary)) {
$error_stat = 1;
$salary_message .= '*Invalid salary*';
}




//Date check)
if (empty($date)) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a dob
$date_message = '*Please enter job closing date*';
}

//Check the format and explode into $parts
elseif (!ereg("^([0-9]{2})/([0-9]{2})/([0-9]{4})$",
$date, $parts)){
$error_stat = 1;

//Set the message to tell the user the date is invalid
$date_message = '*Invalid date, must be DD/MM/YYYY format*';
}

elseif (!checkdate($parts[2],$parts[1],$parts[3]))
{
$error_stat = 1;

//Set the message to tell the date is invalid for the month entered
$date_message = '*Invalid date, month must be between 1-12*';
}


// Job Description check)
if (!$description) {
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;

//Set the message to tell the user to enter a username
$description_message = '*Please enter a job description*';
}

$description = $_POST['description'];
$description = trim($description);

if (strlen($description) > 150){
$error_stat = 1;
$description_message = '*Job Title must be 150 characters or less*';
}




// Educational Level Check)
if ($educationallevel == 'Please Select'){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$educationallevel_message = '*Please select Educational level required*';
}


if( $_FILES['userfile']['size'] > 2000000 ){
//Set the error_stat to 1, which means that an error has occurred
$error_stat = 1;
$filesize_message = '*Filesize too large *';

}

$fileTypes = array("application/pdf", "application/msword");

if( !in_array("{$_FILES['userfile']['type']}", $fileTypes) ){
$error_stat = 1;
$filetype_message = '*Filetype not allowed *';

}




$uploadDir = 'applicationforms/';

if (isset($_POST['submit']) && $error_stat == 0) {


$fileName = $_FILES['userfile']['name'];
$tmpName = $_FILES['userfile']['tmp_name'];
$fileSize = $_FILES['userfile']['size'];
$fileType = $_FILES['userfile']['type'];

// the files will be saved in filePath
$filePath = $uploadDir . $fileName;

// move the files to the specified directory
// if the upload directory is not writable or
// something else went wrong $result will be false
$result = move_uploaded_file($tmpName, $filePath);


include("database.php");

if(!get_magic_quotes_gpc())
{
$fileName = addslashes($fileName);
$filePath = addslashes($filePath);
}
}


mysql_query("UPDATE job SET username='" . $_POST["username"] . "',jobtitle='" . $_POST["jobtitle"] . "',jobcatergory='" . $_POST["jobcatergory"] . "',joblocation='" . $_POST["joblocation"] . "',employmenttype='" . $_POST["employmenttype"] . "',salary='" . $_POST["salary"] . "',date='" . $_POST["date"] . "',educationallevel='" . $_POST["educationallevel"] . "',description='" . $_POST["description"] . "',name='$fileName', type='$fileType', size='$fileSize', path='$filePath' WHERE username='" . $_SESSION["username"] . "'");
?>

<br />
<a href="index.php">Back to main page</a>
<br />
<br />
<br />
You have successfully updated the job account .
<?php
}
else
{
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='$username' AND id='$id'"));
if ($account['username'] == $username && $account['id'] == $id)
{
?>
<form method="post" class="addform" action="" enctype="multipart/form-data">
<fieldset>
<label for="cvtitle">Edit Job</label><fieldset>
<p align="right">&nbsp;</p>
</fieldset>
<label for="username">Username:</label>
<input readonly name="username" type="text" id="username" value="<?php echo $_SESSION["username"]; ?>" /><br />
</fieldset>



<hr class="hr_blue"/>

<fieldset>
<label for="jobtitle">Job Title:</label>
<input name="jobtitle" type="text" id="jobtitle" value="<?php echo $account['jobtitle']; ?>"/>
<span class="redboldtxt"><?php echo "$jobtitle_message";?></fieldset></span>


<fieldset>
<label for="jobcatergory">Job Catergory:</label><p></p>
<select name="jobcatergory">
<option value="Please Select">Please Select</option>
<?php
$jobcatergory_opts = array(
"Accountancy and Finance",
"Banking and Insurance",
"Construction",
"Customer Service",
"Engineering",
"Management",
"Hotel and Catering",
"Information Technology",
"Legal",
"Marketing",
"Medical",
"Retail",
"Sales",
"Secretarial",
"Transport and Distribution",
"Working from home",
);
foreach($jobcatergory_opts as $opt){
$selected = $_POST['jobcatergory'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$jobcatergory_message";?><?php echo $error['jobcatergory']; ?></span></fieldset>

<fieldset>
<label for="joblocation">Location:</label>
<p></p>
<select name="joblocation">
<option value="Please Select">Please Select</option>
<?php
$joblocation_opts = array(
"Co.Antrim",
"Co.Armagh",
"Co.Down",
"Co.Fermanagh",
"Co.Londonderry",
"Co.Tyrone",
);
foreach($joblocation_opts as $opt){
$selected = $_POST['joblocation'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$joblocation_message";?><?php echo $error['joblocation']; ?></span></fieldset>


<fieldset>
<label for="employmenttype">Job Type:</label><p></p>
<select name="employmenttype">
<option value="Please Select">Please Select</option>
<?php
$employmenttype_opts = array(
"permanent fulltime",
"permanent parttime",
"temporary fulltime",
"temporary parttime",
);
foreach($employmenttype_opts as $opt){
$selected = $_POST['employmenttype'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$employmenttype_message";?><?php echo $error['employmenttype']; ?></span></fieldset>


<fieldset>
<label for="salary">Salary:</label>
<input name="salary" type="text" id="salary" value="<?php echo $_POST['salary']; ?>"/>
<span class="redboldtxt"><?php echo "$salary_message";?></span></fieldset>

<fieldset>
<label for="date">Closing Date:</label>
<input name="date" type="text" id="date" value="<?php echo $_POST['date']; ?>"/>
<span class="redboldtxt"><?php echo "$date_message";?></span></fieldset>


<fieldset>
<label for="educationallevel">Qualification Level Required:</label><p></p>
<select name="educationallevel">
<option value="Please Select">Please Select</option>
<?php
$educationallevel_opts = array(
"GCSE",
"A-Level",
"Third Level Certification",
"Third Level Diploma",
"Third Level Degree",
"Post Graduate Qualification",
"Masters",
"PHD",
"Professional Qualification",
"Part Professional Qualification",
"Trade Qualification",
);
foreach($educationallevel_opts as $opt){
$selected = $_POST['educationallevel'] == $opt ? " selected=true":"";
print "<option value=\"{$opt}\"{$selected}>{$opt}</option>";
}
?>
</select>
<span class="redboldtxt"><?php echo "$educationallevel_message";?><?php echo $error['educationallevel']; ?></span></fieldset>

<hr class="hr_blue"/>

<fieldset>
Job Description -<br />
</fieldset>

<fieldset>
<textarea rows="2" name="description" cols="20"><?php echo $account["description"]; ?></textarea><p></p>
<span class="redboldtxt"><?php echo "$description_message";?></span></fieldset>

Upload Application Form</label>
<input type="hidden" name="MAX_FILE_SIZE" value="2000000"><input name="userfile" type="file" class="box" id="userfile">
<span class="redboldtxt"><?php echo "$filesize_message";?></span>
<span class="redboldtxt"><?php echo "$filetype_message";?></span>
</fieldset>

<p></p>
<fieldset>
<p class="submit"><input type="submit" name="submit" value="Edit Job" />

</fieldset>
</form>
<?php
}
?>
</div>
</div>



<!-- (bottom) -->
<div class="box_lower_part">
<img src="img/box_low-left_edge.png" alt="" class="align-left"/>
<img src="img/box_low-right_edge.png" alt="" class="align-right"/>
</div>

<!--Left Container 2 (Top) -->
</div>
</div>
<p>

<!--Right Panel 1 -->
</p>
<div id="body-right-panel" style="width: 333px; height: 981px">
<div style="padding:10px;">


<!--Top -->
<div class="box_upper_part">
<img src="img/box_up-left_edge_dark.png" alt="" class="align-left"/>
<img src="img/box_up-right_edge_dark.png" alt="" class="align-right"/>
<span class="box_header">Employer Login</span></div>


<!--Middle -->
<div class="box_middle_part">
<div style="padding:5px;">
<ul class="img_red">


<!--Login -->
<?php displayLoginemployer(); ?>
<p><br /></p>
</div>
</div>


<!-- (bottom) -->
<div class="box_lower_part">
<img src="img/box_low-left_edge_dark.png" alt="" class="align-left"/>
<img src="img/box_low-right_edge_dark.png" alt="" class="align-right"/>
</div>


<!--Right Panel 2 (top) -->
<div class="box_upper_part" style="width: 313px; height: 29px">
<img src="img/box_up-left_edge_dark_nodot.png" alt="" class="align-left"/>
<img src="img/box_up-right_edge_dark.png" alt="" class="align-right"/>
<span class="box_header">:: Resource Centre</span>
</div>


<!--Middle -->
<div class="box_middle_part">
<div style="padding:10px;">
<div align="center">
<table border="0" width="65%" id="table12" cellspacing="0" cellpadding="0" bgcolor="#E7E6E4">
<tr>
<td>
<img border="0" src="images/cv.gif" width="221" height="23"></td>
</tr>
<tr>
<td><p>&nbsp;</p>
<p><span class="navyboldtxt">CV Advice</span>. </p>
<p>&nbsp;- <a href="cvadvice.php">How to create a CV</a> </p>
<p>&nbsp;- <a href="cvreferences.php">CV References</a> </p>
<p>&nbsp;</p>
</td>
</tr>
</table>
<table border="0" width="65%" id="table13" cellspacing="0" cellpadding="0" bgcolor="#E7E6E4">
<tr>
<td>
<img border="0" src="images/interview.gif" width="221" height="23">
</td>
</tr>
<tr>
<td>&nbsp;<p><span class="navyboldtxt">
Interview Advice</span> - </p>
<p>&nbsp;&nbsp;- <a href="interviewadvice.php">Interview tips</a> </p>
<p>&nbsp;</p><p>&nbsp;</td>
</tr>
</table>
</div>
</div>
</div>


<!-- (bottom) -->
<div class="box_lower_part" style="width: 313px; height: 22px">
<img src="img/box_low-left_edge_dark.png" alt="" class="align-left"/>
<img src="img/box_low-right_edge_dark.png" alt="" class="align-right"/>
</div>
</div>
</div>
</div>


<!-- Footer -->
<div id="footer">
<div id="footer_designby" style="width: 987px; height: 30px">
<div class="designedby">Designed by
<a href=""></a></div>
</div>
</div>
</div>

</body>
</html>

Again what is the EXACT error? Copy and paste EXACTLY what it tells you.

Again what is the EXACT error? Copy and paste EXACTLY what it tells you.

sorry, the error is:

"Parse error: syntax error, unexpected $end in C:\wamp\www\Jobs4U\editjob.php on line 567"

count your openning curly braces and closing i'm guessing the numbers arent even

openning == {
closing == }

count your openning curly braces and closing i'm guessing the numbers arent even

openning == {
closing == }

I cant seem to find the error, i dont know what is wrong with the code.

Is the code you posted editjob.php?

Is the code you posted editjob.php?

Thanks for the reply aerospace, yeah the file is editjob.php

Hmm I noticed you have this in one of the codes you posted
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='$username' AND id='$id'"));
if ($account['username'] == $username && $account['id'] == $id)
{
Where is the closing } (bracket)?

Hmm I noticed you have this in one of the codes you posted
$account = mysql_fetch_array(mysql_query("SELECT * FROM job WHERE username='$username' AND id='$id'"));
if ($account['username'] == $username && $account['id'] == $id)
{
Where is the closing } (bracket)?

Do i need to include a closing bracket in then?

That if statement isn't doing anything by itself. Its on post #7 if you care to look. What were you trying to do there? I really think you should go back to the basics and read up on php and mysql rather than trying to piece things together from what we tell you.

That if statement isn't doing anything by itself. Its on post #7 if you care to look. What were you trying to do there? I really think you should go back to the basics and read up on php and mysql rather than trying to piece things together from what we tell you.

I got it workin thanks for help.