insert values to database after looping through a form - php

Question

Briefly, my script includes form ask how many images to enter ,,the entered value will be in a while loop that will display a form with inputs equally to image value entered . okey , I made the submit button outside the loop ,I want now to add them to database ,,I used again a while loop with the query as I did in the form ,,but it only inserts the last item ..What could the problem be ? This is part of the script :
if (isset($_POST["chooseimagen"])) {
$i=0;
$imgvl = $_POST["imgvl"];
$cid = $_POST["cid"];
while($i<$imgvl){
echo '<form name="finalform" method="post" action="">
id of content :<input type="text" name="idcontent" value="'.$cid.'"/><br><br>
Enter id of image :<input type="text" name="imgid" value=""/><br><br>
Enter name of image : <input type="text" name="imgname" value=""/><br><br>
-----------------------------------------------------------------------<br>
';
$i++;
}
echo "<br><br><input type='hidden' name='imgvl' value='".$imgvl."'/><input type='submit' name='addto' value='insert images'/></form";
}
?>
<?php
if(isset($_POST["addto"])){
$imgvl = $_POST["imgvl"];
$i=0;
while ($i < $imgvl){
$idcontent = $_POST["idcontent"];
$imgid = $_POST["imgid"];
$imgname = $_POST["imgname"];
$queryin = "insert into content_images values ('$imgid','$idcontent','$imgname','ok')";
mysqli_query($con,$queryin);
$i++;
}
}
?>

Answer 1

Still new to development, but I think I know..
A few things, you are creating a new form with every loop, but only closing it once. Also you are missing a >, so technically you are not closing the forms at all.
while($i<$imgvl){
echo '<form name="finalform" method="post" action="">
...
$i++;
}
echo "<br><br><input type='hidden' name='imgvl' value='".$imgvl."'/>
<inputtype='submit' name='addto' value='insert images'/></form";
But I think the reason you are getting this problem is each form you create has the same names for the inputs. That's why you are only getting the last item, because every item you are inserting is just the last input set.
Because you have no action on the form, every form is essentially combined. This means there is no way to distinguish which input name you are reffering to and therefore it takes the last value that was assigned.
Hope this helps