Hoi iedereen,

Ik ben voor een klant een upload systeempje aan het maken, dat hij 15 foto's tegelijk kan uploaden.

Nu heb ik hiervoor een for loop gebruikt zodat je geen 15x hetzelfde script krijgt.
Alleen komt er nu steeds rijen met foutmeldingen te staan, dus 15x foutmelding image1, 15x foutmelding 2 enz.

PHP Code:

for($p=1;$p<16;$p++)
                        {
                                if(isset($_FILES['image'.$p.'']['name']) and !empty($_FILES['image'.$p.'']['name']))
                                
                                    {
                                        $x = strrchr($_FILES['image'.$p.'']['name'], ".");
   
                                      $image.$p = mktime()+$p . $x;
                                //veilingheid
                                $mimetype_image = strtolower($_FILES['image'.$p.'']['type']);
                                 if ((in_array($mimetype_image,$toegestane_mimetypes)) && (is_uploaded_file($_FILES['image'.$p.'']['tmp_name'])) && ($_FILES["image".$p.""]["size"] < $foto_size)) 
                                 {
                                
                                    $target_path = $target_path . basename( $image.$p);
                                    // File uploaden
                                    move_uploaded_file($_FILES['image'.$p.'']['tmp_name'], $target_path);
                                    }else{
                                        echo'Uploading image '.$p.' is mislukt.<br />';
                                        $image.$p = "";
                                    }
                                    }
                        } 


Er zit iets fout in de opbouw, want de if wordt steeds 15x uitgevoerd, en daarnaast de for 15x. De foutmelding die weergegeven moet wordt: Uploading image 'nummer ( t/m 15)' is mislukt. In plaats daarvan komt er fotobestand.jpg te staan. (zie bijlage)

Heb me al suf gestaard, maar zie de fout niet.

Hieronder mijn form code:

PHP Code:

 <?PHP
                        for($g=1;$g<16;$g++)
                        {
                        echo'Image '.$g.' (galerij):<br />
                        <input type="file" name="image'.$g.'" /><br />';
                        }
                        ?>

P.S Beveiliging zit er nog niet op, maar dat is niet mijn grootste prioriteit nu.

Je hebt de IF statement binnen de loop staan dus die zal dan ook 15x worden uitgevoerd.

Dat klopt inderdaad. Maar nu vraag ik mij af als ik 15 input file wil submitten hoe ik dit met het if-statement moet doen. Als ik de if buiten de for zet, dan gaat het script niet 15x de bestanden nalopen en uploaden naar de ftp neem ik aan.

Maak van de veldnaam <input type="file" name="upload[]" />

en werk vervolgens met; foreach($_FILES['upload'] as $upload) { }

binnen deze foreach kun je dan kijken of er een waarde is en de upload eventueel uitvoeren.

Thanx Joshua, ga hier vanmiddag even mee klooien!

Je hoort van me of het gelukt is.

Waarom niet gewoon SWFUploader gebruiken? (Hier kan je meerdere files tegelijkertijd selecteren i.p.v. 15x één enkele file te selecteren...)

Dat script niet op een live server gebruiken erg onveilig.

Checks op minetypes is erg link aangezien de http headers aangepast kunnen worden zodat men php bestanden kunnen uploaden

En je target_path wordt ook telkens langer op deze manier. Beter om even iets zoals
$basepath = '/a/b/c/';
for (...) {
$target_path = $basepath . basename (...);
....
}

te gebruiken.

Zoals ik had aangegeven heb ik hier nog geen beveiliging in verwerkt, dit was ook niet de issue van het probleem.

Het probleem heb ik op kunnen lossen met:

PHP Code:

while(list($key,$value) = each($_FILES['image']['name']))
{
         if(!empty($value)) 
              { 
              }
}