[SOLVED] Form Won't Run

Xtremer360 · October 15, 2008

Any ideas on why when form is submitted it won't work? Meaning it won't actually run the form it just doesn't do anything.

<?php

/* addshowname.php */

/* This form after submission takes the results of the form and inserts the values into the database as a new show name is created. */

require ('database.php');
require ('style.css');

// Where the file is going to be placed
$target_path = "../defiant/images/";

/* Add the original filename to our target path. 
Result is "images/filename.extension" */
$target_path = $target_path . basename( $_FILES['uploadedfile']['name']);

//This code runs if the form has been submitted
if (isset($_POST['submit'])) {

$extension = explode(".", $image);
$extension = $extension[count($extension)-1];

if(strtolower($extension) == "jpg"){
if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path)) {
echo "The file ".basename( $_FILES['uploadedfile']['name'])." has been uploaded";
} else{
echo "There was an error uploading the file, please try again!";
}
}else {
echo "The file you chose to upload wasn't a valid jpg file, please try again!";
}


// checks if the show name is in use
if (!get_magic_quotes_gpc()) {
$_POST['showname'] = addslashes($_POST['showname']);
}
$showname = $_POST['showname'];
$check = mysql_query("SELECT showname FROM shows WHERE showname = '$showname'")
or die(mysql_error());
$check2 = mysql_num_rows($check);

//if the name exists it gives an error
if ($check2 != 0) {
die('Sorry, the show name '.$_POST['showname'].' is already in use.');
}

// now we insert it into the database
$insert = "INSERT INTO shows (showname, type, showimage, showlabel) VALUES ('".$_POST['showname']."','".$_POST['type']."','$target_path','1')";

$add_show = mysql_query($insert) or die(mysql_error());

}
print '<center><caption><strong>Add A Show</strong></caption></center>';
print '<form enctype="multipart/form-data" action="addshowname.php" method="post">';
print '<input name="MAX_FILE_SIZE" type="hidden" value="100000">';
print '<table border="1" style="margin: auto; width: 60%;">';
print '<tr><td>Enter Show Name:</td> ';
print '<td><input name="showname" type="text"></td></tr>';
print '<tr><td>Show Type:</td> ';
print '<td><select name="type"><option></option><option>Weekly Show</option><option>Pay Per View</option></select></td></tr>';
print '<tr><td>Show Image:</td> ';
print '<td><input name="uploadedfile" type="file"></td></tr>';
print '<tr><th colspan=2><input type="submit" value="Add Show Name" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';


print '<center><caption><strong>List of Shows</strong></caption></center>';
print '<table width="60%" border="1" align="center">';
print '<tr><th align="center">ID</th><th align="center">Type</th><th align="center">Show Name</th><th align="center">Show Image</th><th align="center">Edit</th><th align="center">Delete</th></tr>';

if(!isset($_GET['action']) && !isset($_POST['name'])) {
//Define the query
$query = "SELECT * FROM shows";

if ($r = mysql_query ($query)){ // Run the query.
    if (mysql_num_rows($r) > 0)
    {

// Retrieve and print every record
        while ($row = mysql_fetch_array ($r)){
  print '<tr><td align="center">'.$row['id'].'</td><td align="center">'.$row['type'].'</td><td align="center">'.$row['showname'].'</td><td align="center">'.$row['showimage'].'</td><td align="center"><a href="addshowname.php?action=edit&id='.$row['id'].'"<center>Edit</center></a></td><td align="center"><a href="addshowname.php?action=delete&id='.$row['id'].'">Delete</a></td></tr>';
}
}
    else
    {
        print "No Shows\n";
    }
} else {
die ('<p>Could not retrieve the data because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
} //End of query IF

print '</table>';
}

if($_GET['action'] == 'edit') {
$query = "SELECT * FROM shows WHERE id = '".$_GET['id']."'";
$res = mysql_fetch_array(mysql_query($query));
print('<form action="'.$_SERVER['PHP_SELF'].'" method="post" name="form1">');
print('<table border=1 cellpadding=5 cellspacing=0 width=350>');
print('<tr><td>Name of show:</td><td><input type="text" name="name" value="'.$res['showname'].'"/></td></tr>');
print('<tr><td>Show Type Type:</td><td><select name="type">');
$types = array('Weekly Show','Pay Per View');
foreach($types as $type) {
  if($type == $res['type']) {
   print('<option value="'.$type.'" selected="selected">'.$type.'</option>');
  }
  else {
   print('<option value="'.$type.'">'.$type.'</option>');
  }
}
print('</select></td></tr>');
print('<tr><th colspan=2><input type="hidden" name="id" value="'.$_GET['id'].'" /><input type="submit" value="Edit Show" /></th></tr></table></form></center>');
} 
if(isset($_POST['name'])) {
$query = "UPDATE shows SET showname = '".mysql_real_escape_string($_POST['name'])."', location = '".mysql_real_escape_string($_POST['loc'])."', date = '".mysql_real_escape_string($_POST['date'])."' WHERE id = '".$_POST['id']."'"; if(mysql_query($query)) {
  echo "Show updated.";
}
else {
  die('<p>The show could not update because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
}
}

if($_GET['action'] == 'delete') {
$query = "DELETE FROM shows WHERE id = '".$_GET['id']."'";
if(mysql_query($query)) {

echo "Deletion successful.";
}
else {
die ('<p>Could not delete post because ' . mysql_error() . '. The query was '."$query.".'</p>');
}
}

?>

MadTechie · October 15, 2008

change Line 64 (i think)

print '<tr><th colspan=2><input type="submit" value="Add Show Name" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';

to

print '<tr><th colspan=2><input type="submit" value="Add Show Name" name="submit" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';

NOTE the name="submit"

OR

change

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

to

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

Xtremer360 · October 16, 2008

Thank you. I'm an idiot. lol However I ran a test run of the script and uploaded a jpg file and that's the only file I allow an upload on. Even though I did upload a jpg file it still says the error The file you chose to upload wasn't a valid jpg file, please try again!

MadTechie · October 16, 2008

Ahh well $image hasn't been set.. add the missing line

<?php
$image = $_FILES['uploadedfile']['name']; //<---Missing line
$extension = explode(".", $image);
$extension = $extension[count($extension)-1];
if(strtolower($extension) == "jpg")
{
?>

Xtremer360 · October 16, 2008

Thank you so much it works great it creates the record in my DB as well as correctly uploads the file to the correct directory except I ran a second test with a rtf file document. It did deny the file being able to upload and came up with the correct message saying it wasn't a valid file however it still creates a record. How do I change my code to make it to where if the file upload wasn't a valid jpg file then it doesn't run the query to insert the info into my DB.

MadTechie · October 16, 2008

I added an if statement to help

Please see the 3 $error lines

$error = true;

$error = false;

if(!$error);

in your code, i formatted a little as it was a pain to read

<?php

/* addshowname.php */
/* This form after submission takes the results of the form and inserts the values into the database as a new show name is created. */

require ('database.php');
require ('style.css');

// Where the file is going to be placed
$target_path = "../defiant/images/";

/* Add the original filename to our target path. 
Result is "images/filename.extension" */
$target_path = $target_path . basename( $_FILES['uploadedfile']['name']);

//This code runs if the form has been submitted
if (isset($_POST['submit']))
{
$error = true;
$image = $_FILES['uploadedfile']['name'];
$extension = explode(".", $image);
$extension = $extension[count($extension)-1];
if(strtolower($extension) == "jpg")
{
	if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path))
	{
		echo "The file ".basename( $_FILES['uploadedfile']['name'])." has been uploaded";
		$error = false;
	}else{
			echo "There was an error uploading the file, please try again!";
	}
}else{
	echo "The file you chose to upload wasn't a valid jpg file, please try again!";
}

	if(!$error)
	{
	// checks if the show name is in use
	if (!get_magic_quotes_gpc())
	{
		$_POST['showname'] = addslashes($_POST['showname']);
	}
	$showname = $_POST['showname'];
	$check = mysql_query("SELECT showname FROM shows WHERE showname = '$showname'")	or die(mysql_error());
	$check2 = mysql_num_rows($check);

	//if the name exists it gives an error
	if ($check2 != 0)
	{
		die('Sorry, the show name '.$_POST['showname'].' is already in use.');
	}

	// now we insert it into the database
	$insert = "INSERT INTO shows (showname, type, showimage, showlabel) VALUES ('".$_POST['showname']."','".$_POST['type']."','$target_path','1')";

	$add_show = mysql_query($insert) or die(mysql_error());
	}

}
print '<center><caption><strong>Add A Show</strong></caption></center>';
print '<form enctype="multipart/form-data"  method="post">';
print '<input name="MAX_FILE_SIZE" type="hidden" value="100000">';
print '<table border="1" style="margin: auto; width: 60%;">';
print '<tr><td>Enter Show Name:</td> ';
print '<td><input name="showname" type="text"></td></tr>';
print '<tr><td>Show Type:</td> ';
print '<td><select name="type"><option></option><option>Weekly Show</option><option>Pay Per View</option></select></td></tr>';
print '<tr><td>Show Image:</td> ';
print '<td><input name="uploadedfile" type="file"></td></tr>';
print '<tr><th colspan=2><input type="submit" name="submit" value="Add Show Name" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';


print '<center><caption><strong>List of Shows</strong></caption></center>';
print '<table width="60%" border="1" align="center">';
print '<tr><th align="center">ID</th><th align="center">Type</th><th align="center">Show Name</th><th align="center">Show Image</th><th align="center">Edit</th><th align="center">Delete</th></tr>';

if(!isset($_GET['action']) && !isset($_POST['name']))
{
//Define the query
$query = "SELECT * FROM shows";
if ($r = mysql_query ($query)) // Run the query.
{
    	if (mysql_num_rows($r) > 0)
    	{
			 // Retrieve and print every record
        	while ($row = mysql_fetch_array ($r))
        	{
        		print '<tr><td align="center">'.$row['id'].'</td><td align="center">'.$row['type'].'</td><td align="center">'.$row['showname'].'</td><td align="center">'.$row['showimage'].'</td><td align="center"><a href="addshowname.php?action=edit&id='.$row['id'].'"<center>Edit</center></a></td><td align="center"><a href="addshowname.php?action=delete&id='.$row['id'].'">Delete</a></td></tr>';
        	}
        }else{
        	print "No Shows\n";
        }
}else{
	die ('<p>Could not retrieve the data because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
} //End of query IF
print '</table>';
}

if($_GET['action'] == 'edit')
{
$query = "SELECT * FROM shows WHERE id = '".$_GET['id']."'";
$res = mysql_fetch_array(mysql_query($query));
print('<form action="'.$_SERVER['PHP_SELF'].'" method="post" name="form1">');
print('<table border=1 cellpadding=5 cellspacing=0 width=350>');
print('<tr><td>Name of show:</td><td><input type="text" name="name" value="'.$res['showname'].'"/></td></tr>');
print('<tr><td>Show Type Type:</td><td><select name="type">');
$types = array('Weekly Show','Pay Per View');
foreach($types as $type)
{
	if($type == $res['type'])
	{
		print('<option value="'.$type.'" selected="selected">'.$type.'</option>');
	}else{
		print('<option value="'.$type.'">'.$type.'</option>');
	}
}
print('</select></td></tr>');
print('<tr><th colspan=2><input type="hidden" name="id" value="'.$_GET['id'].'" /><input type="submit" value="Edit Show" /></th></tr></table></form></center>');
} 
if(isset($_POST['name']))
{
$query = "UPDATE shows SET showname = '".mysql_real_escape_string($_POST['name'])."', location = '".mysql_real_escape_string($_POST['loc'])."', date = '".mysql_real_escape_string($_POST['date'])."' WHERE id = '".$_POST['id']."'";
if(mysql_query($query))
{
	echo "Show updated.";
}else{
	die('<p>The show could not update because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
}
}

if($_GET['action'] == 'delete')
{
$query = "DELETE FROM shows WHERE id = '".$_GET['id']."'";
if(mysql_query($query))
{
	echo "Deletion successful.";
}else{
	die ('<p>Could not delete post because ' . mysql_error() . '. The query was '."$query.".'</p>');
}
}

?>

Xtremer360 · October 16, 2008

Incredible however one last final thing.

I have these messages come up when I load the page. Here's my error reporting code and it's showing these:

ini_set("display_errors",1);

error_reporting(E_ALL|E_STRICT);

Notice: Undefined index: uploadedfile in /home/content/y/a/n/yankeefaninkc/html/backstage/addshowname.php on line 17

Notice: Undefined index: action in /home/content/y/a/n/yankeefaninkc/html/backstage/addshowname.php on line 102

Notice: Undefined index: action in /home/content/y/a/n/yankeefaninkc/html/backstage/addshowname.php on line 134

MadTechie · October 17, 2008

Okay.. heres a clean up

please read the code and check it makes sense as i have added quite a few error checks

<?php

/* addshowname.php */
/* This form after submission takes the results of the form and inserts the values into the database as a new show name is created. */

require ('database.php');
require ('style.css');

//This code runs if the form has been submitted
if (isset($_POST['submit']))
{
$error = true;
if(!empty($_FILES['uploadedfile']['name']))
{
	// Where the file is going to be placed
	$target_path = "../defiant/images/";

	/* Add the original filename to our target path. 
	Result is "images/filename.extension" */
	$target_path = $target_path . basename( $_FILES['uploadedfile']['name']);

	$image = $_FILES['uploadedfile']['name'];
	$extension = explode(".", $image);
	$extension = $extension[count($extension)-1];
	if(strtolower($extension) == "jpg")
	{
		if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path))
		{
			echo "The file ".basename( $_FILES['uploadedfile']['name'])." has been uploaded";
			$error = false;
		}else{
 			echo "There was an error uploading the file, please try again!";
		}
	}else{
		echo "The file you chose to upload wasn't a valid jpg file, please try again!";
	}
	echo "No File uploaded, please try again!";
}

	if(!$error && !empty($_POST['showname']) && !empty($_POST['type']))
	{
	// checks if the show name is in use
	if (!get_magic_quotes_gpc())
	{
		$_POST['showname'] = addslashes($_POST['showname']);
	}
	$showname = $_POST['showname'];
	$check = mysql_query("SELECT showname FROM shows WHERE showname = '$showname'")	or die(mysql_error());
	$check2 = mysql_num_rows($check);

	//if the name exists it gives an error
	if ($check2 != 0)
	{
		die('Sorry, the show name '.$_POST['showname'].' is already in use.');
	}

	// now we insert it into the database
	$insert = "INSERT INTO shows (showname, type, showimage, showlabel) VALUES ('".$_POST['showname']."','".$_POST['type']."','$target_path','1')";

	$add_show = mysql_query($insert) or die(mysql_error());
	}
}
print '<center><caption><strong>Add A Show</strong></caption></center>';
print '<form enctype="multipart/form-data"  method="post">';
print '<input name="MAX_FILE_SIZE" type="hidden" value="100000">';
print '<table border="1" style="margin: auto; width: 60%;">';
print '<tr><td>Enter Show Name:</td> ';
print '<td><input name="showname" type="text"></td></tr>';
print '<tr><td>Show Type:</td> ';
print '<td><select name="type"><option></option><option>Weekly Show</option><option>Pay Per View</option></select></td></tr>';
print '<tr><td>Show Image:</td> ';
print '<td><input name="uploadedfile" type="file"></td></tr>';
print '<tr><th colspan=2><input type="submit" name="submit" value="Add Show Name" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';


print '<center><caption><strong>List of Shows</strong></caption></center>';
print '<table width="60%" border="1" align="center">';
print '<tr><th align="center">ID</th><th align="center">Type</th><th align="center">Show Name</th><th align="center">Show Image</th><th align="center">Edit</th><th align="center">Delete</th></tr>';

if(!isset($_GET['action']) && !isset($_POST['name']))
{
//Define the query
$query = "SELECT * FROM shows";
if ($r = mysql_query ($query)) // Run the query.
{
    	if (mysql_num_rows($r) > 0)
    	{
			 // Retrieve and print every record
        	while ($row = mysql_fetch_array ($r))
        	{
        		print '<tr><td align="center">'.$row['id'].'</td><td align="center">'.$row['type'].'</td><td align="center">'.$row['showname'].'</td><td align="center">'.$row['showimage'].'</td><td align="center"><a href="addshowname.php?action=edit&id='.$row['id'].'"<center>Edit</center></a></td><td align="center"><a href="addshowname.php?action=delete&id='.$row['id'].'">Delete</a></td></tr>';
        	}
        }else{
        	print "No Shows\n";
        }
}else{
	die ('<p>Could not retrieve the data because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
} //End of query IF
print '</table>';
}

if(!empty($_GET['action']) && !empty($_GET['id']))
{
switch($_GET['action'])
{
	case 'edit':
		$query = "SELECT * FROM shows WHERE id = '".$_GET['id']."'";
		$res = mysql_fetch_array(mysql_query($query));
		print('<form action="'.$_SERVER['PHP_SELF'].'" method="post" name="form1">');
		print('<table border=1 cellpadding=5 cellspacing=0 width=350>');
		print('<tr><td>Name of show:</td><td><input type="text" name="name" value="'.$res['showname'].'"/></td></tr>');
		print('<tr><td>Show Type Type:</td><td><select name="type">');
		$types = array('Weekly Show','Pay Per View');
		foreach($types as $type)
		{
			if($type == $res['type'])
			{
				print('<option value="'.$type.'" selected="selected">'.$type.'</option>');
			}else{
				print('<option value="'.$type.'">'.$type.'</option>');
			}
		}
		print('</select></td></tr>');
		print('<tr><th colspan=2><input type="hidden" name="id" value="'.$_GET['id'].'" /><input type="submit" value="Edit Show" /></th></tr></table></form></center>');
	break;
	case 'delete':
		$query = "DELETE FROM shows WHERE id = '".$_GET['id']."'";
		if(mysql_query($query))
		{
			echo "Deletion successful.";
		}else{
			die ('<p>Could not delete post because ' . mysql_error() . '. The query was '."$query.".'</p>');
		}
	break;
}
}

if(!empty($_POST['name']) && !empty($_POST['loc']) && !empty($_POST['date']) && !empty($_POST['id']))
{
$query = "UPDATE shows SET showname = '".mysql_real_escape_string($_POST['name'])."', location = '".mysql_real_escape_string($_POST['loc'])."', date = '".mysql_real_escape_string($_POST['date'])."' WHERE id = '".$_POST['id']."'";
if(mysql_query($query))
{
	echo "Show updated.";
}else{
	die('<p>The show could not update because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
}
}

?>

Xtremer360 · October 17, 2008

Looks good but even if it is a jpg file and is uploaded it says: The file aggression.jpg has been uploadedNo File uploaded, please try again!

MadTechie · October 17, 2008

ooops, forgot the else

change this

		echo "The file you chose to upload wasn't a valid jpg file, please try again!";
	}
	echo "No File uploaded, please try again!";
}

to

		echo "The file you chose to upload wasn't a valid jpg file, please try again!";
	}
}else{ //<---add this
	echo "No File uploaded, please try again!";	
}

Xtremer360 · October 17, 2008

Thank you. Everything is PERFECT!!!

MadTechie · October 17, 2008

Cool just one thing remains.. clicking solved to end this thread

Xtremer360 · October 17, 2008

I take that back on the delete part. I want it to delete the entry and go back to the regular page instead of staying at http://kansasoutlawwrestling.com/backstage/addshowname.php?action=delete&id=7

I just want it to delete and go back to kansasoutlawwrestling.com/backstage/addshowname.php

MadTechie · October 17, 2008

Okay you can do this a few ways to redirect but i use header so.. heres an update

<?php
require ('database.php');
if(!empty($_GET['action']) && $_GET['action']=='delete')
{
$query = "DELETE FROM shows WHERE id = '".$_GET['id']."'";
if(mysql_query($query))
{
	echo "Deletion successful.";
}else{
	die ('<p>Could not delete post because ' . mysql_error() . '. The query was '."$query.".'</p>');
}
header("http://www.".$_SERVER["HTTP_HOST"].$_SERVER['PHP_SELF']); // you may need to tweak this line
}
/* addshowname.php */
/* This form after submission takes the results of the form and inserts the values into the database as a new show name is created. */

require ('style.css');

//This code runs if the form has been submitted
if (isset($_POST['submit']))
{
$error = true;
if(!empty($_FILES['uploadedfile']['name']))
{
	// Where the file is going to be placed
	$target_path = "../defiant/images/";

	/* Add the original filename to our target path. 
	Result is "images/filename.extension" */
	$target_path = $target_path . basename( $_FILES['uploadedfile']['name']);

	$image = $_FILES['uploadedfile']['name'];
	$extension = explode(".", $image);
	$extension = $extension[count($extension)-1];
	if(strtolower($extension) == "jpg")
	{
		if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path))
		{
			echo "The file ".basename( $_FILES['uploadedfile']['name'])." has been uploaded";
			$error = false;
		}else{
 			echo "There was an error uploading the file, please try again!";
		}
	}else{
		echo "The file you chose to upload wasn't a valid jpg file, please try again!";
	}
}else{
	echo "No File uploaded, please try again!";	
}

	if(!$error && !empty($_POST['showname']) && !empty($_POST['type']))
	{
	// checks if the show name is in use
	if (!get_magic_quotes_gpc())
	{
		$_POST['showname'] = addslashes($_POST['showname']);
	}
	$showname = $_POST['showname'];
	$check = mysql_query("SELECT showname FROM shows WHERE showname = '$showname'")	or die(mysql_error());
	$check2 = mysql_num_rows($check);

	//if the name exists it gives an error
	if ($check2 != 0)
	{
		die('Sorry, the show name '.$_POST['showname'].' is already in use.');
	}

	// now we insert it into the database
	$insert = "INSERT INTO shows (showname, type, showimage, showlabel) VALUES ('".$_POST['showname']."','".$_POST['type']."','$target_path','1')";

	$add_show = mysql_query($insert) or die(mysql_error());
	}
}
print '<center><caption><strong>Add A Show</strong></caption></center>';
print '<form enctype="multipart/form-data"  method="post">';
print '<input name="MAX_FILE_SIZE" type="hidden" value="100000">';
print '<table border="1" style="margin: auto; width: 60%;">';
print '<tr><td>Enter Show Name:</td> ';
print '<td><input name="showname" type="text"></td></tr>';
print '<tr><td>Show Type:</td> ';
print '<td><select name="type"><option></option><option>Weekly Show</option><option>Pay Per View</option></select></td></tr>';
print '<tr><td>Show Image:</td> ';
print '<td><input name="uploadedfile" type="file"></td></tr>';
print '<tr><th colspan=2><input type="submit" name="submit" value="Add Show Name" /><input name="sumbitted" type="hidden" value="TRUE"></th></tr></table></form><br><br><br>';


print '<center><caption><strong>List of Shows</strong></caption></center>';
print '<table width="60%" border="1" align="center">';
print '<tr><th align="center">ID</th><th align="center">Type</th><th align="center">Show Name</th><th align="center">Show Image</th><th align="center">Edit</th><th align="center">Delete</th></tr>';

if(!isset($_GET['action']) && !isset($_POST['name']))
{
//Define the query
$query = "SELECT * FROM shows";
if ($r = mysql_query ($query)) // Run the query.
{
    	if (mysql_num_rows($r) > 0)
    	{
			 // Retrieve and print every record
        	while ($row = mysql_fetch_array ($r))
        	{
        		print '<tr><td align="center">'.$row['id'].'</td><td align="center">'.$row['type'].'</td><td align="center">'.$row['showname'].'</td><td align="center">'.$row['showimage'].'</td><td align="center"><a href="addshowname.php?action=edit&id='.$row['id'].'"<center>Edit</center></a></td><td align="center"><a href="addshowname.php?action=delete&id='.$row['id'].'">Delete</a></td></tr>';
        	}
        }else{
        	print "No Shows\n";
        }
}else{
	die ('<p>Could not retrieve the data because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
} //End of query IF
print '</table>';
}

if(!empty($_GET['action']) && !empty($_GET['id']))
{
switch($_GET['action'])
{
	case 'edit':
		$query = "SELECT * FROM shows WHERE id = '".$_GET['id']."'";
		$res = mysql_fetch_array(mysql_query($query));
		print('<form action="'.$_SERVER['PHP_SELF'].'" method="post" name="form1">');
		print('<table border=1 cellpadding=5 cellspacing=0 width=350>');
		print('<tr><td>Name of show:</td><td><input type="text" name="name" value="'.$res['showname'].'"/></td></tr>');
		print('<tr><td>Show Type Type:</td><td><select name="type">');
		$types = array('Weekly Show','Pay Per View');
		foreach($types as $type)
		{
			if($type == $res['type'])
			{
				print('<option value="'.$type.'" selected="selected">'.$type.'</option>');
			}else{
				print('<option value="'.$type.'">'.$type.'</option>');
			}
		}
		print('</select></td></tr>');
		print('<tr><th colspan=2><input type="hidden" name="id" value="'.$_GET['id'].'" /><input type="submit" value="Edit Show" /></th></tr></table></form></center>');
	break;
}
}

if(!empty($_POST['name']) && !empty($_POST['loc']) && !empty($_POST['date']) && !empty($_POST['id']))
{
$query = "UPDATE shows SET showname = '".mysql_real_escape_string($_POST['name'])."', location = '".mysql_real_escape_string($_POST['loc'])."', date = '".mysql_real_escape_string($_POST['date'])."' WHERE id = '".$_POST['id']."'";
if(mysql_query($query))
{
	echo "Show updated.";
}else{
	die('<p>The show could not update because <b>' . mysql_error() . '</b>. The query was '."$query.".'</p>');
}
}

?>

MadTechie · October 17, 2008

Bug fix..

change the if and remove the echo

if(!empty($_GET['action']) && $_GET['action']=='delete')
{
   $query = "DELETE FROM shows WHERE id = '".$_GET['id']."'";
   if(!mysql_query($query))
   {
      die ('<p>Could not delete post because ' . mysql_error() . '. The query was '."$query.".'</p>');
   }
   header("http://www.".$_SERVER["HTTP_HOST"].$_SERVER['PHP_SELF']); // you may need to tweak this line
}

Sign In

[SOLVED] Form Won't Run

Recommended Posts

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Link to comment

Share on other sites

Join the conversation

Important Information