add info.....then be able to edit that info

Dutch_MaFia-boy

I am working on a project where I add information into a database via a form. That part is working.

What I am having trouble understanding is if I want to edit the data in the table- I want to be able to go back to the data and edit & update it.......How would I do this. I would want it to take me back to the forms page that I used to add the data into the database (with all the info I added still instact within the forms) - edit it and update the database....

You can help me also by giving me helpful links......

Thanks.

Adrian

GIJew

Hi Dutch MaFia-boy,

You can provide an edit link and use this code. I am going to use functions to accomplish what you want. Should work.

/*****************
  edit.php
*****************/

function dspData () {
//Display data and add an edit link

$sql = MySQL_Query("SELECT * FROM table");
  while ($row = MySQL_fetch_array($sql)) {
    echo $row['row1'];
    echo $row['row2'];
    echo "<a href=\"edit.php?action=editForm&id={$row['id']}\">Edit</a>";
  }
}

function dspEditForm () {
  $id = $_GET['id'];
  $sql = mysql_query("SELECT * FROM table where id=$id");

  while ($row = mysql_fetch_array($sql)) {
    echo "<form action=\"edit.php?action=editSub&id={$row['id']}\""."method=\"post\">".
    echo 'FORM CONTENTS';
    echo '<form>';
  }
}

function editSub () {
  $id = $_GET['id'];
  $var1 = $_POST['var1'];
  $var2 = $_POST['var2'];

  MySQL_query("UPDATE table SET id='$id', var1='$var1', var2='$var2' WHERE id='$id'");

  //If you want to go to another page you could always use the header function (which I think is neat).
  header ('Location: edit.php?action=success');
}

function success () {
  echo 'You got this far, should hopefully have worked!';
}

switch ($_GET['action']) {
  default:
    dspData ()
  break;

  case 'editForm';
    dspEditForm ();
  break;

  case 'editSub';
    editSub ();
  break;

  case 'success';
    success ();
  break;
}

I hope that can help you in some way. Even if you don't use functions the code is there and it works (should at least).

Dutch_MaFia-boy

Thanks for the script GIJew. However, I am having trouble understanding it and how it fits into my current work. I am also a newbie, so functions are a difficult concept to grasp for me (when, where, and why to use them)....so I will give you what I have and I would appreciate it if you would help me out filling in the blanks with my configuration....

Thanks.

Adrian.

Add.php

<html>
<head><title>Add Inventory</title></head>
<body>

<h2><center><font = "arial" color = "blue">Enter a New inventory</h2></center></font>

<form name=formMaker method=post action="insertinventory.php">
<table>

<tr height=30 border=1 valign=top>
<td align=right><b>Computer ID :</b></td>
<td>
    <input type="text" name="idField">
</td>
</tr>

<tr height=30 valign=top>
<td align=right><b>Computer Name:</b></td>
<td>
    <input type="text" name="nameField">
</td>
</tr>        

<tr height=30 valign=top>
<td align=right><b>Make :</b></td>
<td>
    <input type="text" name="makeField"><font size = 2> example: Dell
</td>
</tr>


<tr height=30 valign=top>
<td align=right><b>Model :</b></td>
<td>
    <input type="text" name="modelField"><font size = 2> example: Dimension 5500
</td>
</tr>

<tr height=30 valign=top>
<td align=right><b>CPU :</b></td>
<td>
    <select name="cpuField">
        <option value="Pentium4" selected>Pentium4</option>
        <option value="Pentium3">Pentium3</option>
        <option value="Pentium2">Pentium2</option>
        <option value="Pentium1">Pentium1</option>
        <option value="Centrino">Centrino (laptop)</option>
        <option value="Xeon">Xeon (server)</option> 
        <option value="AMD">AMD</option> 
    </select>
</td>
</tr>


<tr height=30 valign=top>
<td align=right><b>RAM :</b></td>
<td>
    <select name="ramField">
        <OPTION VALUE="1" selected>
        <option value="512" >512</option>
        <option value="256">256</option>
        <option value="128">128</option>
        <option value="64">64</option>
    </select>
Other<input type="text" name="other" size="4"  

     maxlength="4"> 
</td>
</tr>


<tr height=30 valign=top>
<td align=right><b>OS :</b></td>
<td>
    <select name="osField">
        <option value="Windows XP" selected>Windows XP</option>
        <option value="Windows 2000 Professional">Windows 2000 Professional</option>
        <option value="Windows 98 SE">Windows 98 SE</option>
        <option value="Windows 98">Windows 98</option>
        <option value="Windows 95">Windows 95</option>
        <option value="Windows 2000 Server">Windows 2000 Server</option>
        <option value="Windows 2003 Server">Windows 2003 Server</option>
        <option value="Mac">Mac</option>
        <option value="Linux">Linux</option>

    </select>
</td>
</tr>
<tr height=30 valign=top>
<td align=right><b>Formfactor :</b></td>
<td>
    <input type="radio" name="formfactor" value="desktop" checked>Desktop	
    <input type="radio" name="formfactor" value="laptop">Laptop
</td>
</tr>
</table>


<input type=submit name=submit value="Submit">
<input type=reset name=reset value="Reset">
</form>

View.php

<?php

require_once("guestcon.inc.php");

$connect = mysql_connect($hostname,$username,$password)or die(mysql_error());

mysql_select_db($db)
   OR die(mysql_error());

if ($sortby!="")
{
        $sorted = " order by $sortby ";
}
$bareQuery = "select * from inventory ";
$queryall = $bareQuery.$sorted;
$resultall = MYSQL_QUERY($queryall);
$numberall = mysql_Numrows($resultall);

if ($numberall==0) {

echo "No Records Found !";

}
else if ($numberall>0) {

$x=0;

?>

<h3>List of Inventory </h3>
<table align = "center" border=1 cellspacing=0 cellpadding=10>
    <tr>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=id">id</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=name">Name</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=cpu">CPU</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=ram">RAM</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=os">OS</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=formfactor">Form</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=make">Make</a></b></td>
    <td><b><a href="<? echo $PHP_SELF; ?>?sortby=model">Model</a></b></td>

</tr>

<?php

    while ($x<$numberall)
    {

        // Changing Background color for each alternate row
        if (($x%2)==0) { $bgcolor="#FFFFFF"; } 
        else { 
        $bgcolor="#C0C0C0"; }



        // Retreiving data and putting it in local variables for each row
        $id=mysql_result($resultall,$x,"id");
        $name=mysql_result($resultall,$x,"name");
        $cpu=mysql_result($resultall,$x,"cpu");
        $ram=mysql_result($resultall,$x,"ram");
        $os=mysql_result($resultall,$x,"os");
        $formfactor=mysql_result($resultall,$x,"formfactor");
        $make=mysql_result($resultall,$x,"make");
        $model=mysql_result($resultall,$x,"model");


?>

     <tr bgcolor="<? echo $bgcolor; ?>" height=30>
        <td> <? echo $id; ?> </td>
        <td> <? echo $name; ?> </td>
        <td> <? echo $cpu; ?> </td>
        <td> <? echo $ram; ?> </td>
        <td> <? echo $os; ?> </td>
        <td> <? echo $formfactor; ?> </td>
        <td> <? echo $make; ?> </td>
        <td> <? echo $model; ?> </td>

        <td><a href="edit.php?id=<? echo $CPU; ?>">Edit</a></td>
        <td><a href="deleteinventory.php?id=<? echo $CPU; ?>">Delete</a></td>
        <td><a href="detailsinventory.php?id=<? echo $CPU; ?>">View</a></td>
    </tr>


<?php

    $x++;
} // end while

?>

</table>

<?php

} // end if numberall > 0

?>

insertinventory.php

<?php

require_once("guestcon.inc.php");

$connect = mysql_connect($hostname,$username,$password)or die(mysql_error());

mysql_select_db($db)
   OR die(mysql_error());

$query="insert into inventory (cpu,ram,os,id,name,formfactor,make,model,notes) VALUES 
('$cpuField','$ramField','$osField','$idField','$nameField','$formfactor','$makeField',
'$modelField','$notesField')";

$result = mysql_query($query);

//if(mysql_query($connect)) die(sql_error());
//echo "success in database entry.";
?>

<H2>New Inventory Created !!</H2>

<table>

<tr height=30>
<td align=right><b>Cpu :</b></td>
<td><font color=blue><b><? echo $cpuField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Ram :</b></td>
<td><font color=blue><b><? echo $ramField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Os :</b></td>
<td><font color=blue><b><? echo $osField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Id :</b></td>
<td><font color=blue><b><? echo $idField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Name :</b></td>
<td><font color=blue><b><? echo $nameField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Formfactor :</b></td>
<td><font color=blue><b><? echo $formfactor; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Make :</b></td>
<td><font color=blue><b><? echo $makeField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Model :</b></td>
<td><font color=blue><b><? echo $modelField; ?></font></b></td>
</tr>

<tr height=30>
<td align=right><b>Notes :</b></td>
<td><font color=blue><b><? echo $notesField; ?></font></b></td>
</tr>

</table><br><br>

<a href="listoption.html">Main Menu</a><br>
<a href="view.php">View Inventory</a>

GIJew

I'm gonna comment some code and give it back to you in a day or so.

GIJew

Hey Adrian,

This is going to be long but I commented the hell out of everything. It is a Friday and I have tried to make it a point to never do crap on this day.

My laziness is your gain (hopefully).

I kept it as basic as I thought necessary throughout the entire thing. You will have to modify it some more to fit your needs since I did change bits and pieces. You're also no doubt going to have to clean up the formatting. The Message Board screwed it all up.

This all worked on my machine so it should work on yours as well. Give it a shot and enjoy!

Good luck learning.

<?php
/*************************************************
*               add_inventory.php			
**************************************************									
* 
* This is going to be your basic form.
*
* I cleaned the HTML up a bit.
* All of your fields are the same except I added 
* a notes field at the bottom since you had a 
* notes field in the INSERT page (your post on 
* the board).
**************************************************/
?>
<html> 
<head><title>Add Inventory</title></head> 
<body> 

<h2><center><font = "arial" color = "blue">Enter New inventory</h2></center></font> 

<form name="formMaker" method="post" action="insertinventory.php"> 

<table>
  <tr height="30" border="1" valign="top"> 
    <td align="right"><b>Computer ID :</b></td> 
		<td><input type="text" name="idField"></td> 
  </tr>

<tr height="30" valign="top"> 
  <td align="right"><b>Computer Name:</b></td> 
	<td><input type="text" name="nameField"></td> 
  </tr> 

<tr height="30" valign="top">  
  <td align="right"><b>Make :</b></td> 
	<td><input type="text" name="makeField"><font size = 2> example: Dell</td> 
  </tr>

<tr height="30" valign="top"> 
  <td align="right"><b>Model :</b></td> 
	<td><input type="text" name="modelField"><font size = 2> example: Dimension 5500</td> 
  </tr> 

  <tr height="30" valign="top">  

	  <td align="right"><b>CPU :</b></td> 
		<td>
      <select name="cpuField"> 
        <option selected>Pentium4</option> 
        <option>Pentium3</option> 
        <option>Pentium2</option> 
        <option>Pentium1</option> 
        <option>Centrino (laptop)</option> 
        <option>Xeon (server)</option>  

        <option>AMD</option>  

      </select> 
    </td> 
  </tr>

  <tr height="30" valign="top">  

	  <td align="right"><b>RAM :</b></td> 
		<td> 
      <select name="ramFieldMn"> 
        <option selected></option>
        <option>512</option> 
        <option>256</option> 
        <option>128</option> 
        <option>64</option> 
      </select> 
    Other<input type="text" name="ramFieldOther" size="4" maxlength="4">  

    </td> 
  </tr>

  <tr height="30" valign="top"> 
	  <td align="right"><b>OS :</b></td> 
		<td> 
      <select name="osField"> 
        <option selected>Windows XP</option> 
        <option>Windows 2000 Professional</option> 
        <option>Windows 98 SE</option> 
        <option>Windows 98</option> 
        <option>Windows 95</option> 
        <option>Windows 2000 Server</option> 
        <option>Windows 2003 Server</option> 
        <option>Mac</option> 
        <option>Linux</option> 
      </select> 
    </td> 
  </tr> 

<tr height="30" valign="top"> 
  <td align="right"><b>Formfactor :</b></td> 
	<td>
	  <input type="radio" name="formfactor" value="desktop" checked>Desktop     
  <input type="radio" name="formfactor" value="laptop">Laptop 
</td> 
  </tr>

<tr height="30" valign="top">
  <td align="right"><b>Notes: </b></td>
	<td>
	  <textarea name="notesField" width="10"></textarea>
	</td>
</tr>
</table>

<input type=submit name=submit value="Submit"> 
<input type=reset name=reset value="Reset"> 

</form>

</body>
</html>



<?php 
// Connect to your MySQL database.
$connect = MySQL_Connect($host, $user, $password)or die(MySQL_Error()); 

//Select the database.    

MySQL_Select_DB($database) OR die(MySQL_Error()); 

// I am going to create a variable from each field in the previous form so
// I can reuse them as much as I want on this page.

// I use $_POST['field'] because of the register_globals issue.
// If you're curious about that, do a search in the forums or web, you'll find it!
$id         = $_POST['idField'];
$cpu        = $_POST['cpuField'];
$os         = $_POST['osField'];
$name       = $_POST['nameField'];
$formfactor = $_POST['formfactor'];
$make       = $_POST['makeField'];
$model      = $_POST['modelField'];
$notes      = $_POST['notesField'];

// RAM changes
// I decided to go this route to give a bit more functionality to this field.
// I re-named the HTML select for RAM to "ramFieldMn" and other to "ramFieldOther".

// Here, I assign the variable $ram to either "ramFieldOther" or "ramFieldMn"
// according to user input.  

// The strlen() function is checking to see whether the
// value of "ramFieldMn" is empty, if it is...assign the "ramFieldOther" variable 
// to $ram.  Else assign to "ramFieldMn".
if (strlen($_POST['ramFieldMn'] <= 0)) {
  $ram		  = $_POST['ramFieldOther'];
} else {
  $ram 	    = $_POST['ramFieldMn'];
}
// If you wanted to do some error checking this is where you would put it.

// If you are curious on how to do error checking you could put something like...
if (strlen($id <= 0)) {
  echo 'You didn\'t put anything for the ID field.<br><a href="javascript:history.go(-1);">Go back</a>';
	exit ();
} else {
// Store the variables into the database.
$add = MySQL_Query("INSERT INTO inventory(id         , 
		 	 										 								cpu 			 , 
																					ram 			 , 
																					os 				 , 
																					name       , 
																					formfactor , 
																					make 			 , 
																					model 		 , 
																					notes)" . " 
										VALUES ('$id'         , 
													  '$cpu'				, 
														'$ram'				, 
														'$os'					, 
														'$name'       , 
														'$formfactor' ,
														'$make'				, 
														'$model'			,	
														'$notes')"); 
}
?>

<!--Display the results that were successfully stored into the DB-->
<html>
<head><title>Results</title></head>
<body>

<H2>New Inventory Created !!</H2>

<table> 
  <tr height="30"> 
    <td align="right"><b>Cpu :</b></td> 
		<td><font color="blue"><b><? echo $cpu; ?></font></b></td> 
  </tr> 

  <tr height="30">
	  <td align="right"><b>Ram :</b></td> 
		<td><font color="blue"><b><? echo $ram; ?></font></b></td> 
  </tr> 

  <tr height="30"> 
	  <td align="right"><b>Os :</b></td> 
		<td><font color="blue"><b><? echo $os; ?></font></b></td> 
  </tr> 

  <tr height="30"> 
	  <td align="right"><b>Id :</b></td> 
		<td><font color="blue"><b><? echo $id; ?></font></b></td> 
  </tr> 

<tr height="30">
  <td align="right"><b>Name :</b></td> 
	<td><font color="blue"><b><? echo $name; ?></font></b></td> 
  </tr> 

  <tr height="30">
	  <td align="right"><b>Formfactor :</b></td> 
		<td><font color="blue"><b><? echo $formfactor; ?></font></b></td> 
  </tr> 

  <tr height="30">
	  <td align="right"><b>Make :</b></td> 
		<td><font color="blue"><b><? echo $make; ?></font></b></td> 
  </tr> 

  <tr height="30"> 
	  <td align="right"><b>Model :</b></td> 
		<td><font color="blue"><b><? echo $model; ?></font></b></td> 
  </tr> 

  <tr height="30"> 
	  <td align="right"><b>Notes :</b></td> 
		<td><font color="blue"><b><? echo $notes; ?></font></b></td> 
  </tr> 

</table>
  <br>
	<br> 
  <center>
	  <a href="listoption.html">Main Menu</a><br> 
		<a href="view.php">View Inventory</a><br>
		<a href="add_inventory.php">Add More</a>
</body>
</html>


<?php 
/*************************************************
*               view.php			
*************************************************/
// Connect to the MySQL Database
$connect = MySQL_Connect($host, $user, $password)or die(MySQL_Error());    

// Grab the database.  

MySQL_Select_DB($database) OR die(MySQL_Error()); 

$sql = MySQL_Query ("SELECT * from inventory");

// Get the number of rows from MySQL
$numofrows = MySQL_Num_Rows($sql);

echo '<html><head>'.
		 '<title>View Inventory</title>'.
		 '</head><body>';

echo '<h3>List of Inventory </h3>';

// Table header information
echo '<table border="1" cellpadding="0" cellspacing="0">';
echo '<tr>'.
		 '<td><b>id</b></td>'.
		 '<td><b>Name</b></td>'.
		 '<td><b>CPU</b></td>'.
     '<td><b>RAM</b></td>'.
     '<td><b>OS</b></td>'.
     '<td><b>Form</b></td>'.
     '<td><b>Make</b></td>'.
     '<td><b>Model</b></td>'.
		 '</tr>';
	// This for loop will change the row color.  I changed yours for no reason 
	// other than I like this version better.
  for($i = 0; $i < $numofrows; $i++) {
    $row = MySQL_Fetch_Array($sql); 
    if($i % 2) { 
      echo '<tr bgcolor="#C0C0C0">';
    } else { 
      echo '<TR bgcolor="#FFFFFF">';
    }
		// Display MySQL Data
    echo '<td>'.$row['id'].'</td>'.
				 '<td>'.$row['name'].'</td>'.
				 '<td>'.$row['cpu'].'</td>'.
				 '<td>'.$row['ram'].'</td>'.
				 '<td>'.$row['os'].'</td>'.
				 '<td>'.$row['formfactor'].'</td>'.
				 '<td>'.$row['make'].'</td>'.
				 '<td>'.$row['model'].'</td>'.
				 // Provide an edit/delete/view link for each record.
				 "<td><a href=\"edit.php?id={$row['id']}\">Edit</a></td>".

			 // I am going to show you a neat JS trick too that I like.
			 // It pops up a confirm box before deleting to prevent 
			 // accidental deletion.
			 "<td><a href=\"delete.php?id={$row['id']}\" onclick=\"javascript: return confirm('ARE YOU SURE YOU WANT TO DELETE THIS ITEM?');\">Delete</a></td>".
			 "<td><a href=\"detailsinventory.php?id={$row['id']}\">View</a></td>".
		 '</tr>';
  }
echo '</table>';

echo '</body></html>';
?>

GIJew

Continued...

<?php
/*****************************************
* edit.php
*
* This can be simplified using functions.
* You didn't want to go that route so...
*
* You could also $PHP_SELF this page and
* insert the query that edit's the data
* in here but I am still assuming basic
* knowledge (no offense).
*****************************************/
// We need to get the $id from view.php and store it in a variable.
$id = $_GET['id'];

// Connect to the MySQL Database
$connect = mysql_connect($host, $user, $password)or die(MySQL_Error());    

// Grab the database.  

MySQL_Select_db($database) OR die(MySQL_Error()); 

// This MySQL Query basically says grab the row that matches the id number.
$sql = MySQL_Query ("SELECT * from inventory WHERE id LIKE $id");

// Display the data using MySQL_Fetch_Array and display it in a form.
while ($row = MySQL_Fetch_Array($sql)) {

// Display form.
// For each value=" " attribute I am going to display the data from the DB.
// I use the format $row['column_name']
echo '<html>'.
		 '<head><title>Edit Record</title></head>'.
		 '<body>';

echo "<form action=\"edit_record.php?id={$row['id']}\" method=\"post\">";

echo '<table>'.
  	 '<tr height="30" border="1" valign="top">'.
     '<td align="right"><b>Computer ID :</b></td>'. 
		 '<td><input type="text" name="idField" value="'.$row['id'].'"></td>'. 
  	 '</tr>'.

	 '<tr height="30" valign="top">'.
   '<td align="right"><b>Computer Name:</b></td>'. 
	 '<td><input type="text" name="nameField" value="'.$row['name'].'"></td>'. 
 '</tr>'.

	 '<tr height="30" valign="top">'.  
   '<td align="right"><b>Make :</b></td>'.
	 '<td><input type="text" name="makeField" value="'.$row['make'].'"><font size = 2> example: Dell</td>'. 
 '</tr>'.

	 '<tr height="30" valign="top">'.
   '<td align="right"><b>Model :</b></td>'.
	 '<td><input type="text" name="modelField" value="'.$row['model'].'"><font size = 2> example: Dimension 5500</td>'.
 '</tr>'.

 '<tr height="30" valign="top">'.  
   '<td align="right"><b>CPU :</b></td>'. 
	 '<td>'.
 '<select name="cpuField">'.
	 '<option selected>'.$row['cpu'].'</option>'.
	 '<option>-------------</option>'.
 '<option>Pentium4</option>'.
 '<option>Pentium3</option>'. 
 '<option>Pentium2</option>'. 
 '<option>Pentium1</option>'.
 '<option>Centrino (laptop)</option>'.
 '<option>Xeon (server)</option>'.
 '<option>AMD</option>'.
 '</select>'.
 '</td>'.
 '</tr>'.

 '<tr height="30" valign="top">'.
   '<td align="right"><b>RAM :</b></td>'.
	 '<td>'.
 '<select name="ramFieldMn">'. 
 '<option selected>'.$row['ram'].'</option>'.
	 '<option></option>'.
 '<option>512</option>'. 
 '<option>256</option>'. 
 '<option>128</option>'. 
 '<option>64</option>'. 
 '</select>'. 
 'Other<input type="text" name="ramFieldOther" size="4" maxlength="4">'.  
 '</td>'. 
 '</tr>'.

 '<tr height="30" valign="top">'. 
   '<td align="right"><b>OS :</b></td>'. 
	 '<td>'. 
 '<select name="osField">'. 
	 '<option selected>'.$row['os'].'</option>'.
	 '<option></option>'.
 '<option>Windows XP</option>'.
 '<option>Windows 2000 Professional</option>'.
 '<option>Windows 98 SE</option>'. 
 '<option>Windows 98</option>'. 
 '<option>Windows 95</option>'.
 '<option>Windows 2000 Server</option>'. 
 '<option>Windows 2003 Server</option>'. 
 '<option>Mac</option>'. 
 '<option>Linux</option>'. 
 '</select>'.
 '</td>'.
 '</tr>'.

	 '<tr height="30" valign="top">'. 
   '<td align="right"><b>Formfactor :</b></td>'. 
	 '<td>';

	 // I need to see which one is checked so I am going to test for that here
	 // and check the appropriate one.		 
	 if ($row['formfactor'] == "desktop") {
	   echo '<input type="radio" name="formfactor" value="desktop" checked>Desktop'.     
 	 			'<input type="radio" name="formfactor" value="laptop">Laptop';
	 } elseif ($row['formfactor'] == "laptop") {
	   echo '<input type="radio" name="formfactor" value="desktop">Desktop'.     
 	 			'<input type="radio" name="formfactor" value="laptop" checked>Laptop';
	 }

echo '</td>'. 
  	 '</tr>'.

	 '<tr height="30" valign="top">'.
   '<td align="right"><b>Notes: </b></td>'.
	 '<td>'.
	 '<textarea name="notesField" width="10">'.$row['notes'].'</textarea>'.
	 '</td>'.
	 '</tr>'.
	 '</table>'.

	 '<input type=submit name=submit value="Submit">'. 
	 '<input type=reset name=reset value="Reset">'; 


echo '</form>';
}
?>


<?PHP
/**********************************
* edit_record.php
**********************************/
// Get the id again from the previous form.
$id = $_GET['id'];

// Connect to your MySQL database.
$connect = MySQL_Connect($host, $user, $password)or die(MySQL_Error()); 

// Select the database.    

MySQL_Select_DB($database) OR die(MySQL_Error()); 

// Once again I do my RAM check.
if (strlen($_POST['ramFieldMn'] <= 0)) {
  $ram		  = $_POST['ramFieldOther'];
} else {
  $ram 	    = $_POST['ramFieldMn'];
}

// Once again store the variables.
$id         = $_POST['idField'];
$cpu        = $_POST['cpuField'];
$os         = $_POST['osField'];
$name       = $_POST['nameField'];
$formfactor = $_POST['formfactor'];
$make       = $_POST['makeField'];
$model      = $_POST['modelField'];
$notes      = $_POST['notesField'];

// MySQL Query to update the data.
// I'm not sure how you are going to use the id field so in the event your
// are going to change it, don't!  It has to be unique.
MySQL_Query("UPDATE inventory SET id = '$id', cpu = '$cpu', ram = '$ram', os = '$os', name = '$name', formfactor = '$formfactor', make	= '$make', model = '$model', notes = '$notes' WHERE id = '$id' ");

// The header() function will redirect you back to the view page once the 
// record has been edited.
header ("Location: View.php");


<?PHP
/*************************
* delete.php
*************************/

// Get the id again.
$id = $_GET['id'];

// Connect to your MySQL database.
$connect = mysql_Connect($host, $user, $password)or die(MySQL_Error()); 

// Select the database.    

MySQL_Select_DB($database) OR die(MySQL_Error()); 

// MySQL Query that deletes the record that matches the $id.
MySQL_Query("DELETE FROM inventory WHERE id = $id");

// Re-direct back to the view page once completed using the header () function.
header ('Location: View.php');
?>

Dutch_MaFia-boy

GIJew:

Thank-you so much for your time and efforts.....and thank-you for recongnizing that I am learning. I will most likely reply back to this and ask you why you did something a particular way.

Thanks again.

Dutch_MaFia-boy

I am getting a error:

Warning: mysql_fetch_array(): supplied argument is not a valid MySQL result resource in /usr/local/apache/htdocs/edit.php on line 27

Line 27 is the while loop......I'm pretty sure everything is connected to the database, b/c I can add data passed from the forms and view them - I get this error when I hit the Edit link on the view.php page....

Here is my guestcon.inc.php:

<?php
$host = "localhost";
$user = "safe_mysqld";
$pass = "";
$db = "techigloo";
//$DBConn= mysql_pconnect($host, $user, $pass, $db) 
//or trigger_error(mysql_error(),E_USER_ERROR);
$date = date("l F d, Y");
?>

Here is my edit.php:

<?php
$id = $_GET['id'];

// Connect to the MySQL Database
require_once("guestcon.inc.php");
$connect = mysql_connect($host,$user,$pass)or die(mysql_error());

// Grab the database.  

mysql_select_db($db) OR die(mysql_error()); 

// This MySQL Query basically says grab the row that matches the id number.
$sql = mysql_query ("SELECT * from inventory WHERE id LIKE $id");

// Display the data using MySQL_Fetch_Array and display it in a form.
while ($row = mysql_fetch_array($sql)) {
// Display form.
// For each value=" " attribute I am going to display the data from the DB.
// I use the format $row['column_name']
echo '<html>'.
         '<head><title>Edit Record</title></head>'.
         '<body>';

echo "<form action=\"edit_record.php?id={$row['id']}\" method=\"post\">";

echo '<table>'.
       '<tr height="30" border="1" valign="top">'.
     '<td align="right"><b>Computer ID :</b></td>'. 
         '<td><input type="text" name="idField" value="'.$row['id'].'"></td>'. 
       '</tr>'.

     '<tr height="30" valign="top">'.
   '<td align="right"><b>Computer Name:</b></td>'. 
     '<td><input type="text" name="nameField" value="'.$row['name'].'"></td>'. 
   '</tr>'.

     '<tr height="30" valign="top">'.  
   '<td align="right"><b>Make :</b></td>'.
     '<td><input type="text" name="makeField" value="'.$row['make'].'"><font size = 2> example: Dell</td>'. 
   '</tr>'.

     '<tr height="30" valign="top">'.
   '<td align="right"><b>Model :</b></td>'.
     '<td><input type="text" name="modelField" value="'.$row['model'].'"><font size = 2> example: Dimension 5500</td>'.
   '</tr>'.

   '<tr height="30" valign="top">'.  
   '<td align="right"><b>CPU :</b></td>'. 
     '<td>'.
 '<select name="cpuField">'.
     '<option selected>'.$row['cpu'].'</option>'.
     '<option>-------------</option>'.
 '<option>Pentium4</option>'.
 '<option>Pentium3</option>'. 
 '<option>Pentium2</option>'. 
 '<option>Pentium1</option>'.
 '<option>Centrino (laptop)</option>'.
 '<option>Xeon (server)</option>'.
 '<option>AMD</option>'.
 '</select>'.
 '</td>'.
   '</tr>'.

   '<tr height="30" valign="top">'.
   '<td align="right"><b>RAM :</b></td>'.
     '<td>'.
 '<select name="ramFieldMn">'. 
 '<option selected>'.$row['ram'].'</option>'.
     '<option></option>'.
 '<option>512</option>'. 
 '<option>256</option>'. 
 '<option>128</option>'. 
 '<option>64</option>'. 
 '</select>'. 
 'Other<input type="text" name="ramFieldOther" size="4" maxlength="4">'.  
 '</td>'. 
   '</tr>'.

   '<tr height="30" valign="top">'. 
   '<td align="right"><b>OS :</b></td>'. 
     '<td>'. 
 '<select name="osField">'. 
     '<option selected>'.$row['os'].'</option>'.
     '<option></option>'.
 '<option>Windows XP</option>'.
 '<option>Windows 2000 Professional</option>'.
 '<option>Windows 98 SE</option>'. 
 '<option>Windows 98</option>'. 
 '<option>Windows 95</option>'.
 '<option>Windows 2000 Server</option>'. 
 '<option>Windows 2003 Server</option>'. 
 '<option>Mac</option>'. 
 '<option>Linux</option>'. 
 '</select>'.
 '</td>'.
   '</tr>'.

     '<tr height="30" valign="top">'. 
   '<td align="right"><b>Formfactor :</b></td>'. 
     '<td>';

     // I need to see which one is checked so I am going to test for that here
     // and check the appropriate one.        
     if ($row['formfactor'] == "desktop") {
       echo '<input type="radio" name="formfactor" value="desktop" checked>Desktop'.     
                  '<input type="radio" name="formfactor" value="laptop">Laptop';
     } elseif ($row['formfactor'] == "laptop") {
       echo '<input type="radio" name="formfactor" value="desktop">Desktop'.     
                  '<input type="radio" name="formfactor" value="laptop" checked>Laptop';
     }

echo '</td>'. 
       '</tr>'.

     '<tr height="30" valign="top">'.
   '<td align="right"><b>Notes: </b></td>'.
     '<td>'.
     '<textarea name="notesField" width="10">'.$row['notes'].'</textarea>'.
     '</td>'.
     '</tr>'.
     '</table>'.

     '<input type=submit name=submit value="Submit">'. 
     '<input type=reset name=reset value="Reset">'; 


echo '</form>';
}
?>

GIJew

Hey,

I spotted an error in the code before line 27 which might be producing that error.

// This MySQL Query basically says grab the row that matches the id number. 
$sql = mysql_query ("SELECT * from inventory WHERE id LIKE $id");

// It should read...
$sql = mysql_query ("SELECT * from inventory WHERE id=$id");

That looks like it might be the problem your are having. The while loop looks fine at a glance though.

I don't mind the questions but I rarely come to this board so if you want, email me at joakley@vivame.com

Dutch_MaFia-boy

Thanks GIJew for your last suggestion....but I am still getting the same error pointing to line 27.

This is sometimes frustrating to a newbie like me - the code looks fine, but it is still not working! Uggh!

Any other suggestions would be much appreciated!.

Thanks.

Adrian

aaron_karp

You're getting this problem almost certainly because your $id variable is blank.

First, you should probably be putting your variables in single quotes, even if they're integers:

$sql = mysql_query ("SELECT * from inventory WHERE id='$id'");

Before you run the query, try echoing out the value of $id to make sure that it's set.

If the variable is set, add this code:

echo mysql_error();

It will display on the screen the specifics of what's causing the error.