variable from array not being passed through right

[mvanmeter]

I am trying to process the contents of a form, which contains multiple sets of the same basic fields - fields can either be previous ones that are being updated, or dynamically generated new fields. The problem I am running into is where there are multiple existing sets, the contents of any checkbox items are carried from the first set to all the others (i.e. - first set could have "taxable" checked at start - I uncheck (set to "N") it, and all other sets, regardless of their content are also set to "N" - the part of the form where processing occurs is below:

$product = $_POST["product"];
$product_name = $_POST["product_name"];
$owner = $owner_ID;
$price = $_POST["price"];
$comment = $_POST["product_note"];
$tax = $_POST["tax"];
$hourly = $_POST["hourly"];
$i=0;

foreach ($product as $prod) {

if (isset($tax[$i]))
{
$istax = 'Y';	
}
else {$istax = 'N';}
if (isset($hourly[$i]))
{
$ishourly = 'Y';	
}
else {$ishourly = 'N';}

if ($prod == 'new')
{ 
if (strlen($product_name[$i]) > 1) {
	$sql2 = "INSERT INTO product (product_ID,owner_ID,product_desc,product_price,taxable,product_detail,hourly) VALUES ('','$owner','$product_name[$i]','$price[$i]','$istax','$comment[$i]','$ishourly')";
						 }
}

else 
{
	$sql2 = "UPDATE product SET owner_ID='$owner',product_desc='$product_name[$i]',product_price='$price[$i]',taxable='$istax',product_detail='$comment[$i]',hourly='$ishourly' WHERE product_ID = '$prod'";
}
	$sql2result = $db->query($sql2);

$i++;


}


}

[Bjom]

for a better diagnosis add this line:

var_dump($tax);

right before the foreach. What do you get?

[mvanmeter]

Bjom,

It all depends on what was on the form page. For test purposes, I started with 3 entries. The first and second had the Tax field checked, the third did not. To test your method, I added the field before the for each loop, then ran it the first time by changing the first two tax boxes to unchecked, and checking the third. The result was:
array(1) { [0]=> string(2) "on" }
and when i reloaded the form data, the first check box was still checked the other two were not.

When i reset the data, and tried again, this time leaving the first box checked, unchecking the second and checking the third, I got this:
array(2) { [0]=> string(2) "on" [1]=> string(2) "on" }
and when i reloaded the form data, both of the first two checkboxes were still checked. Does that help?

Mike

[johanafm]

It makes perfect sense. Your html looks something like

Box 1<input type="checkbox" name="tax[]" />
Box 2<input type="checkbox" name="tax[]" />
Box 3<input type="checkbox" name="tax[]" />

But checkboxes are only sent when they are checked as opposed to text inputs which are sent without input (or with the empty string to be precise).
The posted tax array is filled out as the form is processed. If the first box is checked, add it to the end of tax array. If the second box is checkd, add it to the end of tax array (which now means either element 0 or element 1). And the last checkbox goes into element 0, 1 or 2 depending on which of the others were checked.

You could

Box 1<input type="checkbox" name="tax[1]" />
Box 2<input type="checkbox" name="tax[2]" />
Box 3<input type="checkbox" name="tax[3]" />

Or

Box 1<input type="checkbox" name="tax[]" value="1" />
Box 2<input type="checkbox" name="tax[]" value="2" />
Box 3<input type="checkbox" name="tax[]" value="3" />

[mvanmeter]

The only problem with that approach is that the input fields can be added dynamically by the user, so while

Box 1<input type="checkbox" name="tax[1]" />
Box 2<input type="checkbox" name="tax[2]" />
Box 3<input type="checkbox" name="tax[3]" />

may work for some, other users may be

Box 1<input type="checkbox" name="tax[1]" />
Box 2<input type="checkbox" name="tax[2]" />
.......
Box 80<input type="checkbox" name="tax[80]" />

That would seem to be a very hard thing to code? Or am I going after this wrong? My php is decent, but not up to this one I guess...

[Bjom]

You will know in advance whether it is the 3 or the 80 fields case, right?

Then just put that into a standard for next loop

$html = ''; //some html code before those boxes
for ($i = 1, $ub = 1 + $theNumberOfCheckboxesGoesHere; $i < $ub; $i++) {
//add a checkbox
$html .= '<input type="checkbox" name="tax[' . $i . ']" id="cb' . $i . '"/><label for="cb' . $i . '">Box ' . $i . '</label>';
}
$html = ''; //some more html can go here

echo $html;

[mvanmeter]

i won't really use in advance - the user adds fields dynamically via javascript, and can delete new and existing fields at the same time (the deletes on the existing take place via ajax calls)

Here is the full code, and after is the js file that adds the fields

<?php
require_once ("DB.php");
if (!$GET["owner_ID"]) {$owner_ID = 1;}
else {$owner_ID = $GET["owner_ID"];}
require_once ("../common/global_inc.php");
$db = DB::connect($dsn);
if( DB::isError($db) ) {
die ($db->getMessage());
}

$js = "
<script type='text/javascript' src='delete_product.js'></script>
<script type='text/javascript' src='product.js.php'></script>

";
include ("../inc/header.inc.php");
?>

<script language='Javascript' type='text/javascript'>

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

				</script>

<div id="pagecontent">
<br/>
<div class="sectionheading"><table width='100%' cellspacing='0' cellpadding='0' border='0'><tr><td align='left' width='600'>
Manage Products&nbsp;&nbsp;&nbsp;</td><td align='right'>
</td></tr></table></div>
<form action='manage_product.php' method='post' name='manage_product'>
<?php

if (!$_POST[submit]) {
$query3 = "SELECT product.* FROM product WHERE product.owner_ID = '$owner_ID' ";
$result3 = $db->query($query3);
$j = 1;
echo "<div id=\"myDiv\">";
while ($row3 = $result3->fetchRow(DB_FETCHMODE_ASSOC))
{
$counter++;
if ($counter%2===0){
$class = "lighter";
$class2 = "bold";
}else{
$class = "darker";
$class2= "bold";
}

echo "<div id='my".$j."Div'><table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"$class\"><td><input type=\"hidden\" name=\"product[]\" value=\"$row3[product_ID]\"> Name:<input type=\"text\" name=\"product_name[]\" value=\"$row3[product_desc]\" style=\"width:170px;\">&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[]\" style=\"border:0\" ";
if ($row3["taxable"] == 'Y') {echo "checked";}
echo ">&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" ";
if ($row3["hourly"] == 'Y') {echo "checked";}
echo ">&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"  value='${row3[product_price]}'><button class=\"button\" style=\"float:right;\" onclick=\"showUser($row3[product_ID]);removeElement(my".$j."Div)\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\">${row3[product_detail]}</textarea></td></tr></table></div>";
$j++;}
$j = $j;
echo "

<input type='hidden' value=\"$j\" id=\"theValue\" />
</div><p><a href='javascript:;' onClick=\"addElement();\">Add New Product</a></p>
";
?>

<br/><input type='submit' name='submit' value='Update Products'>
</form></body>

<?php } else {

$product = $POST["product"];
$product_name = $POST["product_name"];
$owner = $owner_ID;
$price = $POST["price"];
$comment = $POST["product_note"];
$tax = $POST["tax"];
$hourly = $POST["hourly"];
$i=0;

var_dump($tax);

foreach ($product as $prod) {

if (isset($tax[$i]))
{
$istax = 'Y';
}
else {$istax = 'N';}
if (isset($hourly[$i]))
{
$ishourly = 'Y';
}
else {$ishourly = 'N';}

if ($prod == 'new')
{
if (strlen($product_name[$i]) > 1) {
$sql2 = "INSERT INTO product (product_ID,owner_ID,product_desc,product_price,taxable,product_detail,hourly) VALUES ('','$owner','$product_name[$i]','$price[$i]','$istax','$comment[$i]','$ishourly')";
}
}

else
{
$sql2 = "UPDATE product SET owner_ID='$owner',product_desc='$product_name[$i]',product_price='$price[$i]',taxable='$istax',product_detail='$comment[$i]',hourly='$ishourly' WHERE product_ID = '$prod'";
}
$sql2result = $db->query($sql2);

$i++;

}

}

echo "</div>"; ?>

<?php
include ("../inc/footer.inc.php");
?>

To add a new fieldset:
<?php
echo "

function addElement() {
var ni = document.getElementById('myDiv');
var numi = document.getElementById('theValue');
var num = (document.getElementById('theValue').value -1)+ 2;
numi.value = num;
var newdiv = document.createElement('div');
var divIdName = 'my'+num+'Div';
newdiv.setAttribute('id',divIdName);
if(num % 2 == 0){
var divIdClass = 'darker';
}else{
var divIdClass = 'lighter';

}

newdiv.innerHTML =

'<table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"'+divIdClass+'\"><td><input type=\"hidden\" name=\"product[]\" value=\"new\" > Name:<input type=\"text\" name=\"product_name[]\" style=\"width:170px;\" />&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[]\" style=\"border:0\" >&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" >&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"><button class=\"button\" style=\"float:right;\" onclick=\"removeElement('+divIdName+')\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\"></textarea></td></tr></table>';
ni.appendChild(newdiv);
}

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

";

?>

[louisl]

Adding to what johanafm said, set up you html so your ProductID is in the tax field ie.

<input type="checkbox" name="tax['.$productID.']" />

Then in your processing loop check if the id is in that array

foreach ($product as $productID) { 

if(in_array($productID, $tax){
 	$taxchecked = true;	
}else{
	$taxchecked = false;	
}

}

[mvanmeter]

I like where you are going with that - but one quick question. For the newly added fieldsets, the product ID has a value of "new" - when it's inserted into the database, it's replaced with an auto-increment number. How would i implement your solution for those?

[louisl]

mvanmeter;10943148 wrote:

I like where you are going with that - but one quick question. For the newly added fieldsets, the product ID has a value of "new" - when it's inserted into the database, it's replaced with an auto-increment number. How would i implement your solution for those?

You could create a dummy ProductID with a counter to create new_1, new_2 etc.

Also just a tip when using html in javascript or php use single quotes to enclose strings, it makes it much easier to read and work with and most editors will highlight errors easier.

eg.
echo "<div id=/"some_id/" class=/"some_class/">;
becomes...
echo('<div id="some_id" class="some_class">');

[mvanmeter]

I made the changes and it's still not working.

The live file is here (add or delete to your heart's content - wiping the database when i fix this :-) )

http://tekweb.ampltek.net/billing/manage_product.php

Updated code:
Main File

<?php 
	require_once ("DB.php");
	if (!$_GET["owner_ID"]) {$owner_ID = 1;}
	else {$owner_ID = $_GET["owner_ID"];}
	require_once ("../common/global_inc.php");
	$db = DB::connect($dsn);
		if( DB::isError($db) ) {
			die ($db->getMessage());
		}


$js = "
<script type='text/javascript' src='delete_product.js'></script>
<script type='text/javascript' src='product.js.php'></script>

";
include ("../inc/test.inc.php");
?>

<script language='Javascript' type='text/javascript'>

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

				</script>
<div id="pagecontent">
<br/>
<div class="sectionheading"><table width='100%' cellspacing='0' cellpadding='0' border='0'><tr><td align='left' width='600'>
Manage Products&nbsp;&nbsp;&nbsp;</td><td align='right'>
</td></tr></table></div>
<form action='manage_product.php' method='post' name='manage_product'>
<?php 


if (!$_POST[submit]) {
	$query3 = "SELECT product.* FROM product WHERE product.owner_ID = '$owner_ID' ";
	$result3 = $db->query($query3);	
	$j = 1;
	echo "<div id=\"myDiv\">";
	while ($row3 = $result3->fetchRow(DB_FETCHMODE_ASSOC))
	{
$counter++; 
if ($counter%2===0){ 
$class = "lighter";
$class2 = "bold"; 
}else{ 
$class = "darker";
$class2= "bold"; 
} 

echo "<div id='my".$j."Div'><table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"$class\"><td><input type=\"hidden\" name=\"product[]\" value=\"$row3[product_ID]\"> Name:<input type=\"text\" name=\"product_name[]\" value=\"$row3[product_desc]\" style=\"width:170px;\">&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[".$row3["product_ID"]."]\" style=\"border:0\" ";
if ($row3["taxable"] == 'Y') {echo "checked";}
echo ">&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" ";
if ($row3["hourly"] == 'Y') {echo "checked";}
echo ">&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"  value='${row3[product_price]}'><button class=\"button\" style=\"float:right;\" onclick=\"showUser($row3[product_ID]);removeElement(my".$j."Div)\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\">${row3[product_detail]}</textarea></td></tr></table></div>";
$j++;}
$j = $j;
echo "
<input type='hidden' value=\"$j\" id=\"theValue\" />
</div><p><a href='javascript:;' onClick=\"addElement();\">Add New Product</a></p>	
	";
?>

<br/><input type='submit' name='submit' value='Update Products'>
</form></body>


<?php } else {

$product = $_POST["product"];
$product_name = $_POST["product_name"];
$owner = $owner_ID;
$price = $_POST["price"];
$comment = $_POST["product_note"];
$tax = $_POST["tax"];
$hourly = $_POST["hourly"];
$i=0;

var_dump($tax);

foreach ($product as $prod) {

if (in_array($prod,$tax))
{
$istax = 'Y';	
}
else {$istax = 'N';}
if (isset($hourly[$i]))
{
$ishourly = 'Y';	
}
else {$ishourly = 'N';}

$new = substr('$prod', 0, 3); 

if ($new == 'new')
{ 
if (strlen($product_name[$i]) > 1) {
	$sql2 = "INSERT INTO product (product_ID,owner_ID,product_desc,product_price,taxable,product_detail,hourly) VALUES ('','$owner','$product_name[$i]','$price[$i]','$istax','$comment[$i]','$ishourly')";
						 }
}

else 
{
	$sql2 = "UPDATE product SET owner_ID='$owner',product_desc='$product_name[$i]',product_price='$price[$i]',taxable='$istax',product_detail='$comment[$i]',hourly='$ishourly' WHERE product_ID = '$prod'";
}
	$sql2result = $db->query($sql2);

$i++;

}

}

echo "</div>"; ?>

<?php
include ("../inc/footer.inc.php");
?>



Include File
<?php 
	require_once ("DB.php");
	require_once ("../common/global_inc.php");
	$db = DB::connect($dsn);
		if( DB::isError($db) ) {
			die ($db->getMessage());
		}

echo "

  function addElement() {
  var ni = document.getElementById('myDiv');
  var numi = document.getElementById('theValue');
  var num = (document.getElementById('theValue').value -1)+ 2;
  numi.value = num;
  var newdiv = document.createElement('div');
  var divIdName = 'my'+num+'Div';
  newdiv.setAttribute('id',divIdName);
  if(num % 2 == 0){ 
       var divIdClass = 'darker'; 
     }else{ 
       var divIdClass = 'lighter';  

     }       


  newdiv.innerHTML = 

  '<table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"'+divIdClass+'\"><td><input type=\"hidden\" name=\"product[]\" value=\"new'+divIdName+'\" > Name:<input type=\"text\" name=\"product_name[]\" style=\"width:170px;\" />&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[new'+divIdName+']\" style=\"border:0\" >&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" >&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"><button class=\"button\" style=\"float:right;\" onclick=\"removeElement('+divIdName+')\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\"></textarea></td></tr></table>';
  ni.appendChild(newdiv);
}

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

";

?>

[Weedpacket]

When posting, please use the formatting tags described in the FAQ to make your code easier to read; thank you. (See the difference it makes where bradgrafelman added them to your first post.)

[mvanmeter]

Sorry Weedpacket - let's try this again :-)

I made the changes and it's still not working.

The live file is here (add or delete to your heart's content - wiping the database when i fix this :-) )

http://tekweb.ampltek.net/billing/manage_product.php

Updated code:
Main File

<?php 
require_once ("DB.php");
if (!$_GET["owner_ID"]) {$owner_ID = 1;}
else {$owner_ID = $_GET["owner_ID"];}
require_once ("../common/global_inc.php");
$db = DB::connect($dsn);
if( DB::isError($db) ) {
die ($db->getMessage());
}


$js = "
<script type='text/javascript' src='delete_product.js'></script>
<script type='text/javascript' src='product.js.php'></script>

";
include ("../inc/test.inc.php");
?>

<script language='Javascript' type='text/javascript'>

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

</script>
<div id="pagecontent">
<br/>
<div class="sectionheading"><table width='100%' cellspacing='0' cellpadding='0' border='0'><tr><td align='left' width='600'>
Manage Products&nbsp;&nbsp;&nbsp;</td><td align='right'>
</td></tr></table></div>
<form action='manage_product.php' method='post' name='manage_product'>
<?php 


if (!$_POST[submit]) {
$query3 = "SELECT product.* FROM product WHERE product.owner_ID = '$owner_ID' ";
$result3 = $db->query($query3);	
$j = 1;
echo "<div id=\"myDiv\">";
while ($row3 = $result3->fetchRow(DB_FETCHMODE_ASSOC))
{
$counter++; 
if ($counter%2===0){ 
$class = "lighter";
$class2 = "bold"; 
}else{ 
$class = "darker";
$class2= "bold"; 
} 

echo "<div id='my".$j."Div'><table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"$class\"><td><input type=\"hidden\" name=\"product[]\" value=\"$row3[product_ID]\"> Name:<input type=\"text\" name=\"product_name[]\" value=\"$row3[product_desc]\" style=\"width:170px;\">&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[".$row3["product_ID"]."]\" style=\"border:0\" ";
if ($row3["taxable"] == 'Y') {echo "checked";}
echo ">&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" ";
if ($row3["hourly"] == 'Y') {echo "checked";}
echo ">&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\" value='${row3[product_price]}'><button class=\"button\" style=\"float:right;\" onclick=\"showUser($row3[product_ID]);removeElement(my".$j."Div)\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\">${row3[product_detail]}</textarea></td></tr></table></div>";
$j++;}
$j = $j;
echo "
<input type='hidden' value=\"$j\" id=\"theValue\" />
</div><p><a href='javascript:;' onClick=\"addElement();\">Add New Product</a></p>	
";
?>

<br/><input type='submit' name='submit' value='Update Products'>
</form></body>


<?php } else {

$product = $_POST["product"];
$product_name = $_POST["product_name"];
$owner = $owner_ID;
$price = $_POST["price"];
$comment = $_POST["product_note"];
$tax = $_POST["tax"];
$hourly = $_POST["hourly"];
$i=0;

var_dump($tax);

foreach ($product as $prod) {

if (in_array($prod,$tax))
{
$istax = 'Y';	
}
else {$istax = 'N';}
if (isset($hourly[$i]))
{
$ishourly = 'Y';	
}
else {$ishourly = 'N';}

$new = substr('$prod', 0, 3); 

if ($new == 'new')
{ 
if (strlen($product_name[$i]) > 1) {
$sql2 = "INSERT INTO product (product_ID,owner_ID,product_desc,product_price,taxable,product_detail,hourly) VALUES ('','$owner','$product_name[$i]','$price[$i]','$istax','$comment[$i]','$ishourly')";
}
}

else 
{
$sql2 = "UPDATE product SET owner_ID='$owner',product_desc='$product_name[$i]',product_price='$price[$i]',taxable='$istax',product_detail='$comment[$i]',hourly='$ishourly' WHERE product_ID = '$prod'";
}
$sql2result = $db->query($sql2);

$i++;

}

}

echo "</div>"; ?>

<?php
include ("../inc/footer.inc.php");
?>
</php]


Include File
[code=php]
<?php 
require_once ("DB.php");
require_once ("../common/global_inc.php");
$db = DB::connect($dsn);
if( DB::isError($db) ) {
die ($db->getMessage());
}

echo "

function addElement() {
var ni = document.getElementById('myDiv');
var numi = document.getElementById('theValue');
var num = (document.getElementById('theValue').value -1)+ 2;
numi.value = num;
var newdiv = document.createElement('div');
var divIdName = 'my'+num+'Div';
newdiv.setAttribute('id',divIdName);
if(num % 2 == 0){ 
var divIdClass = 'darker'; 
}else{ 
var divIdClass = 'lighter'; 
} 


newdiv.innerHTML = 

'<table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"'+divIdClass+'\"><td><input type=\"hidden\" name=\"product[]\" value=\"new'+divIdName+'\" > Name:<input type=\"text\" name=\"product_name[]\" style=\"width:170px;\" />&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[new'+divIdName+']\" style=\"border:0\" >&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" >&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"><button class=\"button\" style=\"float:right;\" onclick=\"removeElement('+divIdName+')\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\"></textarea></td></tr></table>';
ni.appendChild(newdiv);
}

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

";

?>

[mvanmeter]

Sorry Weedpacket - let's try this again :-)

I made the changes and it's still not working.

The live file is here (add or delete to your heart's content - wiping the database when i fix this :-) )

http://tekweb.ampltek.net/billing/manage_product.php

Updated code:
Main File

<?php 
require_once ("DB.php");
if (!$_GET["owner_ID"]) {$owner_ID = 1;}
else {$owner_ID = $_GET["owner_ID"];}
require_once ("../common/global_inc.php");
$db = DB::connect($dsn);
if( DB::isError($db) ) {
die ($db->getMessage());
}


$js = "
<script type='text/javascript' src='delete_product.js'></script>
<script type='text/javascript' src='product.js.php'></script>

";
include ("../inc/test.inc.php");
?>

<script language='Javascript' type='text/javascript'>

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

</script>
<div id="pagecontent">
<br/>
<div class="sectionheading"><table width='100%' cellspacing='0' cellpadding='0' border='0'><tr><td align='left' width='600'>
Manage Products&nbsp;&nbsp;&nbsp;</td><td align='right'>
</td></tr></table></div>
<form action='manage_product.php' method='post' name='manage_product'>
<?php 


if (!$_POST[submit]) {
$query3 = "SELECT product.* FROM product WHERE product.owner_ID = '$owner_ID' ";
$result3 = $db->query($query3);	
$j = 1;
echo "<div id=\"myDiv\">";
while ($row3 = $result3->fetchRow(DB_FETCHMODE_ASSOC))
{
$counter++; 
if ($counter%2===0){ 
$class = "lighter";
$class2 = "bold"; 
}else{ 
$class = "darker";
$class2= "bold"; 
} 

echo "<div id='my".$j."Div'><table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"$class\"><td><input type=\"hidden\" name=\"product[]\" value=\"$row3[product_ID]\"> Name:<input type=\"text\" name=\"product_name[]\" value=\"$row3[product_desc]\" style=\"width:170px;\">&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[".$row3["product_ID"]."]\" style=\"border:0\" ";
if ($row3["taxable"] == 'Y') {echo "checked";}
echo ">&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" ";
if ($row3["hourly"] == 'Y') {echo "checked";}
echo ">&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\" value='${row3[product_price]}'><button class=\"button\" style=\"float:right;\" onclick=\"showUser($row3[product_ID]);removeElement(my".$j."Div)\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\">${row3[product_detail]}</textarea></td></tr></table></div>";
$j++;}
$j = $j;
echo "
<input type='hidden' value=\"$j\" id=\"theValue\" />
</div><p><a href='javascript:;' onClick=\"addElement();\">Add New Product</a></p>	
";
?>

<br/><input type='submit' name='submit' value='Update Products'>
</form></body>


<?php } else {

$product = $_POST["product"];
$product_name = $_POST["product_name"];
$owner = $owner_ID;
$price = $_POST["price"];
$comment = $_POST["product_note"];
$tax = $_POST["tax"];
$hourly = $_POST["hourly"];
$i=0;

var_dump($tax);

foreach ($product as $prod) {

if (in_array($prod,$tax))
{
$istax = 'Y';	
}
else {$istax = 'N';}
if (isset($hourly[$i]))
{
$ishourly = 'Y';	
}
else {$ishourly = 'N';}

$new = substr('$prod', 0, 3); 

if ($new == 'new')
{ 
if (strlen($product_name[$i]) > 1) {
$sql2 = "INSERT INTO product (product_ID,owner_ID,product_desc,product_price,taxable,product_detail,hourly) VALUES ('','$owner','$product_name[$i]','$price[$i]','$istax','$comment[$i]','$ishourly')";
}
}

else 
{
$sql2 = "UPDATE product SET owner_ID='$owner',product_desc='$product_name[$i]',product_price='$price[$i]',taxable='$istax',product_detail='$comment[$i]',hourly='$ishourly' WHERE product_ID = '$prod'";
}
$sql2result = $db->query($sql2);

$i++;

}

}

echo "</div>"; ?>

<?php
include ("../inc/footer.inc.php");
?>

Include File

<?php 
require_once ("DB.php");
require_once ("../common/global_inc.php");
$db = DB::connect($dsn);
if( DB::isError($db) ) {
die ($db->getMessage());
}

echo "

function addElement() {
var ni = document.getElementById('myDiv');
var numi = document.getElementById('theValue');
var num = (document.getElementById('theValue').value -1)+ 2;
numi.value = num;
var newdiv = document.createElement('div');
var divIdName = 'my'+num+'Div';
newdiv.setAttribute('id',divIdName);
if(num % 2 == 0){ 
var divIdClass = 'darker'; 
}else{ 
var divIdClass = 'lighter'; 
} 


newdiv.innerHTML = 

'<table cellpadding=\"0\" width=\"100%\" border=\"0\" border=\"0\"><tr class=\"'+divIdClass+'\"><td><input type=\"hidden\" name=\"product[]\" value=\"new'+divIdName+'\" > Name:<input type=\"text\" name=\"product_name[]\" style=\"width:170px;\" />&nbsp;Taxable?<input type=\"checkbox\" name=\"tax[new'+divIdName+']\" style=\"border:0\" >&nbsp;Hourly?<input type=\"checkbox\" name=\"hourly[]\" style=\"border:0\" >&nbsp;&nbsp;Price $<input type=\"text\" size=\"5\" name=\"price[]\"><button class=\"button\" style=\"float:right;\" onclick=\"removeElement('+divIdName+')\"> X </button><br/>Notes: <textarea name=\"product_note[]\" cols=\"50\" rows=\"1\"></textarea></td></tr></table>';
ni.appendChild(newdiv);
}

function removeElement(divNum) {
var d = document.getElementById('myDiv');
d.removeChild(divNum);
}

";

?>

[louisl]

I've just created this example for you. You should be able to see how it works and impliment the same thing in your code.

<?php

define('EOL',"\n");

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

processForm();

}else{

showForm();

}

function isChecked($val){

if($val === 1){

	return ' checked="checked" ';

}
}

function processForm(){

$arrProductID ='';
$arrTax = '';
$arrHourly = '';
$tax = '';
$hourly = '';


if(isset($_POST["ProductID"])){
	$arrProductID = $_POST["ProductID"];
}
if(isset($_POST["tax"])){
	$arrTax = $_POST["tax"];
}
if(isset($_POST["hourly"])){
	$arrHourly = $_POST["hourly"];
}


if ( is_array($arrProductID) ){

	foreach ($arrProductID as $productID) {

		if ( is_array($arrTax) && in_array($productID, $arrTax) ){
			$tax = 'Y';
		}else{
			$tax = 'N';
		}

		if ( is_array($arrHourly) && in_array($productID, $arrHourly) ){
			$hourly = 'Y';
		}else{
			$hourly = 'N';
		}

		if(strpos($productID, 'new_') === false){
			//update an existing record
			echo('update: productID: '.$productID.' tax: '.$tax.' hourly: '.$hourly.'<br />');
		}else{
			//insert an new record
			echo('insert: productID: '.$productID.' tax: '.$tax.' hourly: '.$hourly.'<br />');
		}

	}

}

}

function showForm(){

echo('<form action="'.$_SERVER['PHP_SELF'].'" method="post">'.EOL);

echo(	'<table cellpadding="0" width="100%" border="0" border="0">'.EOL);

$productID = 33;
echo(		'<tr>'.EOL);
echo(			'<td>'.EOL);
echo(				'<input type="hidden" name="ProductID['.$productID.']" value="'.$productID.'" />'.EOL);
echo(				'Name:<input type="text" name="product_name['.$productID.']" value="product 1" />'.EOL);
echo(				'Taxable?<input type="checkbox" name="tax['.$productID.']" value="'.$productID.'"'.isChecked(0).' />'.EOL);
echo(				'Hourly?<input type="checkbox" name="hourly['.$productID.']" value="'.$productID.'"'.isChecked(1).' />'.EOL);
echo(				'Price $<input type="text" size="5" name="price['.$productID.']" value="100" />'.EOL);
echo(			'</td>'.EOL);
echo(		'</tr>'.EOL);


$productID = 200;
echo(		'<tr>'.EOL);
echo(			'<td>'.EOL);
echo(				'<input type="hidden" name="ProductID['.$productID.']" value="'.$productID.'" />'.EOL);
echo(				'Name:<input type="text" name="product_name['.$productID.']" value="product 2" />'.EOL);
echo(				'Taxable?<input type="checkbox" name="tax['.$productID.']" value="'.$productID.'"'.isChecked(1).' />'.EOL);
echo(				'Hourly?<input type="checkbox" name="hourly['.$productID.']" value="'.$productID.'"'.isChecked(0).' />'.EOL);
echo(				'Price $<input type="text" size="5" name="price['.$productID.']" value="100" />'.EOL);
echo(			'</td>'.EOL);
echo(		'</tr>'.EOL);

// imagine this is one you created from javascript, you just need to create a unique ID with a prefix eg new_1, new_2
$productID = 'new_1';
echo(		'<tr>'.EOL);
echo(			'<td>'.EOL);
echo(				'<input type="hidden" name="ProductID['.$productID.']" value="'.$productID.'" />'.EOL);
echo(				'Name:<input type="text" name="product_name['.$productID.']" value="product 3" />'.EOL);
echo(				'Taxable?<input type="checkbox" name="tax['.$productID.']" value="'.$productID.'"'.isChecked(0).' />'.EOL);
echo(				'Hourly?<input type="checkbox" name="hourly['.$productID.']" value="'.$productID.'"'.isChecked(0).' />'.EOL);
echo(				'Price $<input type="text" size="5" name="price['.$productID.']" value="100" />'.EOL);
echo(			'</td>'.EOL);
echo(		'</tr>'.EOL);

echo(	'</table>'.EOL);

echo(	'<input name="submit" type="submit" value="Submit">'.EOL);

echo('</form>'.EOL);

}

?>