Warning: Invalid argument supplied for foreach()

deolsabh

$items = is_array($a_items);
if (isset($_POST['items']))
{
foreach($items as $itemval)
{
echo "a atimes = " .$a_items['isbn'];
$quoteid = 1;//$itemval['quoteid'];
$isbn = $itemval['isbn'];
$price = $itemval['price'];
$quantity = $itemval['quantity'];
// if ($itemval['sv_flag'] = "on" ){
$ret_val = insert_quote_items($quoteid,$isbn,$price,$quantity);
// }
}

Drakla

is_array returns true or false, so you're putting a value of true or false, not an array into the foreach loop.

halojoy

either you use all code
or take away those // comments
I think you have not same number LEFT and RIGHT brackets
At least that is my theory

deolsabh

} are close i just did not posted here

what else can be changed i am new to php

halojoy

It does not look bad.
You just have to make change like Drakla told.
$items has got to get correct value
Is it from $POST [ 'items' ] you want to do foreach?
In that case $items = $POST [ 'items' ] before foreach loop

deolsabh

No error nothing and does not insert values

halojoy

question:
Where does those $items come from?
Because there has got to be a list of items coming from somewhere...
Once you know this, then you can loop each item in this list.
Keep searching!

deolsabh

$items = get_items($catids);
?>
<form method=post action="insert_quote_item.php">
<table border="1" width="800" >

<tr>
<td><b>Items Type</b> </td>
<td><b>Items Name</b> </td>
<td><b>Price</b> </td>
<td><b>Description</b> </td>
<td><b>Quantity</b> </td>
</tr>

<?
$a_items[] = "quoteid";
$a_items[] = "isbn";
$a_items[] = "price";
$a_items[] = "quantity";

foreach ($items as $item)
{
$isbn = $item["isbn"];
?>
<tr>

<td><input disabled type=text name=author value="<? echo $item["author"] ?>"></td>
<td><input disabled type=text name=title value="<? echo $item["title"]?>"></td>
<td><input type=text name=price value="<? echo $item["price"]?>"></td>
<td><input disabled type=text name=description value="<? echo $item["description"]?>"></td>
<td><input type=text name=quantity value=""></td>

<td><input type=checkbox name=sv_flag ></td>

</tr>
<?
}

$a_items[quoteid] = $quoteid;
$a_items[isbn] = $item["isbn"];
$a_items[price] = $item["title"];
$a_items[quantity] = $quantity ;
?>
</table>
<input name=items type=submit value="Add items to Quote">
</form>
<?
}

halojoy

thanks.
now is only a matter of time before somebody will find a solution to this

kburger

This loop in your form is going to cause problems:

foreach ($items as $item)
{
$isbn = $item["isbn"];
?>
<tr>
<td><input disabled type=text name=author value="<? echo $item["author"] ?>"></td>
<td><input disabled type=text name=title value="<? echo $item["title"]?>"></td>
<td><input type=text name=price value="<? echo $item["price"]?>"></td>
<td><input disabled type=text name=description value="<? echo $item["description"]?>"></td>
<td><input type=text name=quantity value=""></td>
<td><input type=checkbox name=sv_flag ></td>
</tr>
<?
}

You're going to end up with multiple fields in your form, each with the same name. If you're going to do it like this you'll need to make them arrays. Something like this, for example:

<td><input type=text name=price[] value="<? echo $item["price"]?>"></td>

Then in your form handler price will be an array. I haven't really looked at all the rest of your code, but youi might need to rethink your logic.