[php] form upload ok, foutmelding bij geen upload

[Verwijderd]

Hoi Allemaal,

Ik heb een html formpje waarbij ook eventueel een upload gedaan kan worden. Als er geupload wordt werkt het, besluit ik om geen upload te doen dan krijg ik een vervelende foutmelding die ikzelf aangemaakt heb.

heb dit stukje code uit script gepakt, het volgende gebeurt er, mijn script controleert of er een bestand wordt geupload vervolgens wordt de grootte gecontroleerd en daarna het type bestand. Als ik een plaatje upload gaat dit goed!

De user kan echter kiezen om wel of niet een jpeg up te loaden! als ik dus geen plaatje upload krijg ik wel altijd de foutmelding "bestandstype is niet toegestaan"...

Wat ik dus wil is dat als er in het form geen plaatje wordt gekozen om te uploaden er geen foutmeldingen komen. Wat zie ik over het hoofd en hoe kan ik het wel voor elkaar krijgen? Volgens mij klopt die isset functie niet...

bedankt voor jullie hulp!

PHP:

do {

    // controleer of $picture iets bevat, zoniet dan naar false
    if(isset($picture)) {
        
    // check max file size niet te groot
    if(getenv("CONTENT_LENGTH") > MAX_FILE_SIZE) {
        print_error("bestand is te groot: $picture_name");
            break;
    }

    // check voor juiste bestandstype
    if ($picture_type != "image/pjpeg") {
       print_error("bestandstype is niet toegestaan: $picture_type");
       break;
    }

[dusty]

Zet eens de lengte van je file als het bestandstype niet is toegestaan

Nu je dat weet, kan je zelf een oplossing aanbrengen hoe je het zou kunnen herkennen?

[Verwijderd]

hmmm... Klinkt heel mysterieus en vast heel simpel voor diegene die hem snapt maar ik weet het niet.

Wat bedoel je precies met lengte? De grootte van het bestand?
ik heb de grootte van het bestand nl met een define vastgesteld

code:

define("MAX_FILE_SIZE", 200*1024);

vage is wel dat hij nu als foutmelding geeft dat het octet/stream bestandstype niet geupload mag worden als ik geen bestand selecteer mbv mijn form...

[djluc]

De onderste if moet binnen de eerste if staan.

[drm]

Debian_Vs_Win2K:
vage is wel dat hij nu als foutmelding geeft dat het octet/stream bestandstype niet geupload mag worden als ik geen bestand selecteer mbv mijn form...

application/octet-stream is het bestandstype dat meegegeven wordt als het bestandstype door het OS niet bepaald kan worden. Dat betekent in heel veel gevallen dat je een bestand geselecteerd hebt dat niet bereikbaar is, leeg is, beschadigd is of (op windows) geen extensie heeft.

Overigens wordt al een tijdje aangeraden de $_FILES array te gebruiken. Zie voor meer informatie daarover de FAQ.

[T-MOB]

Lezen is natuurlijk een vak apart....
Volgens mij wil Debian_VS_Win2K dat zijn script het ge-uploade bestand alleen verwerkt als er daadwerkelijk een bestand wordt ge-upload. Waar het misgaat is idd in de isset functie, zoals hij zelf al aangeeft.

Blijkbaar wordt $picture eerder in zijn script 'geset' zodat $picture altijd bestaat en de isset dus geen onderscheid maakt.

Dit kun je waarschijnlijk verhelpen door de regel (regel 4) te veranderen in

PHP:

if ($picture != '') {

dit zijn twee single-quotes, geen dubbele.....

[ Voor 4% gewijzigd door T-MOB op 30-09-2003 09:17 . Reden: regel# - wel zo makkelijk ]

[drm]

T-MOB:
Lezen is natuurlijk een vak apart....
Volgens mij wil Debian_VS_Win2K dat zijn script het ge-uploade bestand alleen verwerkt als er daadwerkelijk een bestand wordt ge-upload. Waar het misgaat is idd in de isset functie, zoals hij zelf al aangeeft.

In dat geval kun je beter uit dit lijstje de laatste pakken.

PHP:

switch ( $_FILES [ 'file' ] [ 'error' ] ) {
   case UPLOAD_ERR_OK:
      //handle upload
   break;
   case UPLOAD_ERR_NO_FILE:
      //no problem, no uploaded file is also ok
   break;
   default:
      //print error message
   break;