1 formulier met 2 types updaten werkt niet

Door Stephan op 01-06-2009 15:09

637 views

Hallo Allemaal,

Ik heb een formulier waarbij de gegevens uit de database komen dit werkt perfect.
Alleen als ik hem moet gaan updaten dan gaat het fout. Ik heb 2 types op de pagina een normaal tekstveld en een option list alleen er kunnen er meerdere toegevoegd worden.


<form method="post">
<!-- input 1 -->
<input type="hidden" name="id[]" value="1" />
<input type="hidden" name="type[]" value="1" />
<textarea name="content[]" >.$tekstuitdatabase.</textarea>
<!-- input 2 -->
<input type="hidden" name="id[]" value="3" />
<input type="hidden" name="type[]" value="2" />
<select name="afbeelding[]" >
<option value="1">Logo zonder ondertekst</option>
<option value="2">Logo met ondertekst</option>
</select>
<!-- input 3 -->
<input type="hidden" name="id[]" value="6" />
<input type="hidden" name="type[]" value="1" />
<textarea name="content[]" >.$tekstuitdatabase.</textarea>
base.</textarea>
<input type="submit" value="opslaan" />
</form>

php:


for ($i = 0; $i < count($_POST['type']); $i++) {
  if($_POST['type'][$i] == 1){
    $db->query("UPDATE blocks SET content = '".$_POST['content'][$i]."' WHERE block_id = '".$_POST['id'][$i]."' LIMIT 1;");
  }
  if($_POST['type'][$i] == 2){
    $db->query("UPDATE blocks SET afbeelding = '".$_POST['afbeelding'][$i]."' WHERE block_id = '".$_POST['id'][$i]."' LIMIT 1;");
  }
}

Door de count wordt de array alleen verkeerd.
1e = id[1], content[1]
2e = id[2], afbeelding[1]
3e = id[3], content[2]

Hoe kan ik dit oplossen zonder extra input velden oid neer te zetten.

Onbekende gebruiker

01-06-2009 15:11

Sowieso kun je geen twee elementen hebben met dezelfde name.
Andere names gebruiken dus.
Zie ook dit.

Stephan

01-06-2009 15:21

normaal gesproken niet nee maar als je een uploadformulier maakt voor een aantal afbeeldingen waarbij je meerdere afbeeldingen tegelijkertijd kan uploaden gebruik je ook <input name="file[]" type="file" id="file[]" size="50" /> dus dat is het probleem niet..

Onbekende gebruiker

01-06-2009 15:27

Ownee, sorry. My bad.
Ik zie het zo ook niet.

Stephan

01-06-2009 15:39

Het werkt wel met extra hidden fields maar dat is nou juist wat ik niet wil want dan krijg je allemaal nutteloze hidden fields. En dat is alleen maar troep in de broncode.

Zo werkt het wel..


<form method="post">
<!-- input 1 -->
<input type="hidden" name="id[]" value="1" />
<input type="hidden" name="type[]" value="1" />
<input type="hidden" name="afbeelding[]" value="NULL" />
<textarea name="content[]" >.$tekstuitdatabase.</textarea>
<!-- input 2 -->
<input type="hidden" name="id[]" value="3" />
<input type="hidden" name="type[]" value="2" />
<input type="hidden" name="content[]" value="NULL" />
<select name="afbeelding[]" >
<option value="1">Logo zonder ondertekst</option>
<option value="2">Logo met ondertekst</option>
</select>
<!-- input 3 -->
<input type="hidden" name="id[]" value="6" />
<input type="hidden" name="type[]" value="1" />
<input type="hidden" name="afbeelding[]" value="NULL" />
<textarea name="content[]" >.$tekstuitdatabase.</textarea>
base.</textarea>
<input type="submit" value="opslaan" />
</form>

Ik hoop toch dat iemand mij kan zeggen hoe ik dit op kan lossen zonder de hidden fields te gebruiken want hoe minder code hoe beter :)

Emmanuel Delay

01-06-2009 15:46

Zou je niet de index expliciet in de html zetten?

...
<input type="hidden" name="type[4]" value="1" />
<textarea name="content[4]" >'.$tekstuitdatabase.'</textarea>
...

Dan heb je die id[] niet meer nodig. Gewoon uit de key halen.

Stephan

01-06-2009 16:19

ja zou kunnen.
Maar de value van het veld id zou ook 300 kunnen zijn en dan samen met een value van het veld id die 10 is, dit gaat dan toch fout met updaten of zie ik dit verkeerd..

Reageren

Inloggen om te reageren