[RESOLVED] Sending Quantity array values

[TheQ]

Hello:

I am having a problem with getting the actual values from the od_qty[] array in the code below to pass to the processing script.

while ($row = mysql_fetch_array ($result, MYSQL_ASSOC)) {
		echo '<tr><td align="left">' . $row['category'] . '</td>
		<td align="left"><input type="checkbox" name="selection[]" value="' .$row['dn'] .'"></td>
		<td align="left" width="35"><input type="text" name="od_qty[]" id="od_' .$row['d1'] .'_' . $row['price'] . '" size="2" onchange="CalculateTotal(this.form)"></td>
		<td align="left">' . $row['dn'] . '</td>
		<td align="left">' .$row['dd'] . '</td>
		<td align ="right">' . $row['price'] . '</td></tr>';
	}

I have the following in the process_form.php script to confirm the values sent.

<p id="content">You ordered:</p>
	<p id="content">';
		foreach ($_POST['selection'] as $key => $val) {
		echo $qty. ' ' .$val.'<br />';
		}

	echo '</p><p>';

// This is here for testing purposes to see what is being passed.
	foreach($_POST as $key => $value) {
    echo $key . ' has a value of: ' . $value . "<br>\n"; 
	}
	echo '</p>';

} else { // Invalid submitted values.
    echo '<h1 id="mainhead">Error!</h1>

The foreach is only counting the selection array and sending the dish_name.

My questions are how do I get the actual quantity of the items, the dish_name, and the dish_price of the items ordered to:

1) Post to the database along with the dish_name (fields: od_qty, dish_name, dish_price); I do have an INSERT INTO statement as follows:

$query = "INSERT INTO order_details (od_id, dish_id, od_qty, dish_price, order_date) VALUES ('', '$d1', '$qty', 'price', NOW())";

2) Give the form upon submission an order_id (I tried sessions but for some reason could not get it to work);

3) Display the information in the browser to confirm prior to the user entering their information??

Any assistance would be greatly appreciated, or being pointed to a source to help me.

Thank you in advance.

[bradgrafelman]

In general, what you're wanting to do (if I understand you correctly) is loop through multiple form entities (all being arrays) and insert data from each corresponding element into a SQL table? Normally, this would be done like so:

for($i = 0; $i < count($_POST['selection']); $i++) {
    // use $_POST['selection'][$i], $_POST['od_qty'][$i] when performing the SQL query
}

Also, unless you have a UNIQUE index set on the dish_name, you'd probably want to send the dish_id as the value of the checkbox (assuming you have a PRIMARY index on the id column.. right?) - otherwise you could run into duplicate matches.

As for the od_id, if this is the "order_id" that you're speaking of, I'm no DB guru and can't say what the best method would be to ensure that you generate a unique order ID that can be used throughout all INSERT queries for the order. Someone else may have an idea as to the best way to accomplish that.

What you'd probably want to do, however, is SELECT the appropriate data for each selection (e.g. the dish name and price) and then perform the INSERT query.

[TheQ]

Thank you for your reply.

Yes, the dish_id is the PRIMARY KEY with an INDEX set on the dish_name for queries.

As for the od_id, I do have it set for auto-increment but want to assign a value to the order itself, i.e. not unique to the details but unique to the order, so would be a foreign key, until which time I have the customer information, then it will be stored in an orders table with the total cost of the meal but referenced to an order_details (od) table for the individual dishes ordered.

Thanks for the code, I will try it out and hopefully will solve one problem. 🙂

[NogDog]

I think I see where at least one of your issues may be. Checkboxes only get included in the post data if they are checked, and using the "[]" element name suffix means that PHP gives the first checkbox of that name an index of 0, then 1, and so forth. But all of the text inputs elements are received, so if only some of the checkboxes are checked, their indexes will not line up with their corresponding quantities. Probably the simplest fix would be to assign the indexes while building the form, something like:

<?Php
$ix = 0; // form field index counter
while($row = mysql_fetch_array($result, MYSQL_ASSOC))
{
   echo '<tr><td align="left">' . $row['category'] . '</td>
        <td align="left"><input type="checkbox" name="selection[$ix]" value="' . $row['dn'] .
        '"></td>
        <td align="left" width="35"><input type="text" name="od_qty[$ix]" id="od_' .
        $row['d1'] . '_' . $row['price'] .
        '" size="2" onchange="CalculateTotal(this.form)"></td>
        <td align="left">' . $row['dn'] . '</td>
        <td align="left">' . $row['dd'] . '</td>
        <td align ="right">' . $row['price'] . '</td></tr>';
   $ix++; // increment index
}

[TheQ]

Thank you for your reply.

I tried both suggestions and I still show no change in the results. I am still getting just 1 as a qty, even when 2 is input into the quantity textbox and it isn't writing the amounts to the database with the exception of the 1 (this is the order_details table):

od_id order_id dish_id od_qty order_date dish_price shows as:

73 0 0 1 2008-06-01 16:13:11 0.00

on the process_form.php page I show:

Your TOTAL ORDER is: 35.85
(image to process order)

You ordered:

1 Mattar Paneer
1 Shaslik Tikka Masala
1 Shenai Bahar

od_qty has a value of: Array
selection has a value of: Array
od_total has a value of: 35.85
submitted has a value of: TRUE

Initially I was thinking because there was a js behavior on the input for the qty, but I am able to pass the order_total without hitch (but then again that is a single value and not an array).

I have such a headache as I've not had any sleep trying to figure this out and Larry Ullman's books haven't helped (lol).

[HalfaBee]

you need to handle od_qty and selection as arrays.

$od_qty = $_POST['od_qty'];
$selection = $_POST['selection'];

foreach( $od_qty as $key=>$value ) {
    echo $selection[$key] . " qty =  " . $od_qty[$key] . "<br\n";
}

[TheQ]

Thanks HalfaBee for your reply. I had done the selection but not the od_qty. Will try this.

[TheQ]

Well, I've made some of the changes and had to reword a few and am able to get the correct quantity to print but only the last selection/qty combination prints. If they made 2 selections (or different dishes) from the menu, only the last one will show. (i.e.: You ordered: 2 Tandoori Trout -- which is the only item to return but they ordered 1 Shenai Bahar as well).

The data still isn't writing to the database though. Not even a quantity. Here's the code changes:

$query = "INSERT INTO order_details (od_id, dish_id, od_qty, dish_price, order_date) VALUES ('', '$d1', '$qty', 'price', NOW())";

$result = @mysql_query ($query); // Run the query
	if ($result) { // If it ran OK.

// Print the results.
echo '<p id="process">Your TOTAL ORDER is:  ' . $_REQUEST['od_total'] . ' &nbsp;&nbsp;</p>
<p><a href="customer.php"><img src="images/proc_ord_but.gif" border="0"></a></p>
<p id="content">You ordered:</p>
<p id="content">';
	foreach ($_POST['od_qty'] as $key => $val) {
	echo $qty. ' ' .$val.' ';
	}
	foreach ($_POST['selection'] as $key => $val) {
	echo ' ' .$val. ' ';
	}

I still don't understand why the values aren't being sent to the database, particularly when they are being passed to the page.

All and any help is greatly appreciated and thank you for what you've suggested so far. It's coming a little bit at a time.

[HalfaBee]

As you have numerous lines to insert, you need to do a query for each line.

This would be in the same loop where you display the data.

[TheQ]

That makes sense, thanks! (Wished I thought of it...😉)

[TheQ]

HalfaBee wrote:

As you have numerous lines to insert, you need to do a query for each line.

This would be in the same loop where you display the data.

Ok, I did this but I'm certain I've not got it right:

<p id="content">';

	foreach ($_POST['od_qty'] as $key => $val) {
	echo $qty. ' ' .$val.' ';
	}
	foreach ($_POST['selection'] as $key => $val) {
	echo ' ' .$val. ' ';
	}
	$query = "INSERT INTO order_details (od_id, dish_id, od_qty, dish_price, order_date) VALUES ('', '$d1', '$qty', 'price', NOW())";
	$result = @mysql_query ($query); // Run the query
	if ($result) { // If it ran OK.

Although it is listing each, it lists it like this:

You ordered:

1 1 Shenai Bahar Tandoori Trout

I tried other combo's to try to get it but none worked.

Also, now it is looping through each menu item and just entering the od_id but no other values 50 times 🙁.

Have to work in the morning, so enough for tonight. Will check again in the morning for any suggestions. Thanks again.

[TheQ]

Hello,

I have made some progress on this but am a bit stumped still.

I've tried all the suggestions but nothing is working the way it is supposed to. I have gotten it to where it does pass the correct quantities but still can't get the dish_name next to the quantity.

Here is the code as I have it now:

echo '<p id="process">Your TOTAL ORDER is:  ' . $_REQUEST['od_total'] . ' &nbsp;&nbsp;</p>
	<p><a href="customer.php"><img src="images/proc_ord_but.gif" border="0"></a></p>
	<p id="content">You ordered:</p>
	<table id="content">';
		echo '<tr>';
		foreach ($_POST['od_qty'] as $val)

if ($val != NULL) {
//		foreach ($_POST['selection'] as $sel) THIS CAUSES THE ITEM NAME TO REPEAT EACH VALUE
		echo '<td> ' .$val.' ' .$sel. '</td></tr>';
        $query = "INSERT INTO order_details (od_id, order_id, dish_id, od_qty, order_date, dish_price) VALUES ('', '$ix', '$d1', '$val', NOW(), '$price')";
		$result = @mysql_query ($query); // Run the query
		}
	}

	echo '</table>';
	echo '<p>';

This is the result is gives:

Your TOTAL ORDER is: 13.85

You ordered:
1
2

[Weedpacket]

Well, you're not getting the selection displayed of course because you're no longer asking
for it 🙂

You need to know the key for each value in $POST['od_qty'] so that you can find the
appropriate value in $POST['selection'] - which, thanks to previous work, would have the same key. Or vice-versa. Whichever makes the
more sense to the programmer.

foreach ($_POST['od_qty'] as $key=>$val)
	$sel = $_POST['selection'][$key];

One possible way to simplify the code at this point is to use the fact that
od_qtys and selections are supposed to be paired with each other and name the
form fields to reflect this.

<input type="checkbox" name="od[<?php echo $ix?>][selection]" />
<input type="text" name="od[<?php echo $ix?>][qty]" />

(After of course, all the other display requirements are taken into account.)
Then you would have $POST['od'], which could have an array of individual
orders, $POST['od'][0], $POST['od'][1] and so on. Each of those would be an
array with elements $POST['od'][0]['qty'] and $_POST['od'][0]['selection'].

Then the loop for processing the orders would be

foreach($_POST['od'] as $order)
{
	$qty = $order['qty']
	$sel = $order['selection'];
	// ...
}

[TheQ]

Thank you, Weedpacket, for your reply.

I made the changes to code as suggested (the 2nd way to do it) and now I don't receive any results 🙁. It shows only this:

Your TOTAL ORDER is: 34.90

You ordered:

(Nothing lists)

Here is the new code in my form:

$ix = 0; // form field index counter
	while($row = mysql_fetch_array($result, MYSQL_ASSOC))
	{
   	echo '<tr><td align="left">' . $row['category'] . '</td>
        <td align="left"><input type="checkbox" name="od[<?php echo $ix?>][selection]"></td>
        <td align="left" width="35"><input type="text" name="od[<?php echo $ix?>][qty]" id="od_' . $row['d1'] . '_' . $row['price'] . '" size="2" onchange="CalculateTotal(this.form)"></td>
        <td align="left">' . $row['dn'] . '</td>
        <td align="left">' . $row['dd'] . '</td>
        <td align ="right">' . $row['price'] . '</td></tr>';
	$ix++; // increment index
	}

And the code in the processing script:

	echo '<p id="process">Your TOTAL ORDER is:  ' . $_REQUEST['od_total'] . ' &nbsp;&nbsp;</p>
	<p><a href="customer.php"><img src="images/proc_ord_but.gif" border="0"></a></p>
	<p id="content">You ordered:</p>
	<table id="content">';
//	for ($ix= 1; $ix < count($_POST['order_id']); $ix++) 
		echo '<tr>';
		foreach($_POST['od'] as $order)
{
    $qty = $order['qty'];
    $sel = $order['selection'];

if ($qty != NULL) {
	echo '<td> ' . $qty . ' ' . $sel . ' </td></tr>';
    $query = "INSERT INTO order_details (od_id, order_id, dish_id, od_qty, order_date, dish_price) VALUES ('', '$oid', '$d1', '$qty', NOW(), '$price')";
	$result = @mysql_query ($query); // Run the query
	}

}
		echo '</table>';
		echo '<p>';

[Weedpacket]

        <td align="left" width="35"><input type="text" name="od[<?php echo $ix?>][qty]"

Since I didn't have any PHP tags open or an echo statement, I opened the PHP tag and started the echo statement inside the <input> element. Since you already had those, you didn't need to put those in as well. Yours would of course be more like

        <td align="left" width="35"><input type="text" name="od['.$ix.'][qty]"

TheQ wrote:

the 2nd way to do it

It's called programming. One rule of thumb is to get your data into the format that is most convenient for working with. A lot of the time it's not clear what that format is until you try working with it.

[TheQ]

Thanks everyone for your assistance in this. I've finally got it worked out. May need a little help on sending a couple of emails later lol.

You guys (& gals) are the best 😃.