Beste SD-ers,

Ik ben momenteel bezig met een mini upload script. Daarbij wil ik het volgende:

Als er een plaatje is meegepost, dan moet deze geüpload worden. Is er geen plaatje geselecteerd, dan moet er een standaard plaatje te voor schijn komen. Nu heb ik het als volgt gedaan:

PHP Code:

                                    $naam = $_FILES['foto']['name'];
                                    $temp = $_FILES['foto']['tmp_name'];
                                    $db = 'images/uploads/'.$naam;
                                    $link = 'http://www.naam.nl/images/uploads/'.$naam;

if(isset($naam)) {
    // Als er een plaatje is geupload, dan is de link $link.
    if(move_uploaded_file($temp, $db)) {
        // Het plaatje is geupload! Met succes!
        $plaatje = $link;
        echo 'Het plaatje is geupload.';
    } else {
        // Het uploaden is mislukt
        echo 'Je hebt geen plaatje ingevoegd of het uploaden is mislukt.';
        $plaatje = 'http://www.naam.nl/images/rooney.png';
    }
} else {
    // Er is geen plaatje geüpload, dus het plaatje wordt de standaard link
    $plaatje = 'http://www.naam.nl/images/rooney.png';
} 


De $naam is gewoon ingevuld, dus de eerste check doorloopt hij. Daarna wil hij hem echter niet uploaden. De $temp en $db link kloppen ook gewoon, dus ik snap niet waarom het niet werkt. De mappen waren ook goed gezet en hebben een bestandsrechten op 777 gezet..

Iemand met een geniaal idee?

Stefan

Is het domein op de plaats www.naam.nl ingevuld? Want de beschrijving snap ik niet echt.

Uiteraard =)

EDIT:

Mijn vraag is dus; waar kan het fout gegaan zijn? Want ik krijg geen foutmeldingen terug. Het enige dat ik weet is dat de plaatjes niet naar de map geüpload worden en dat ik daarom de melding terug krijg van een standaard plaatje.

$db = 'images/uploads/'.$naam;

Dit moet volgens mij een absolute path zijn en dit is het nu niet.
Dus de move file gaat altijd fout.

Wat zou in dit geval het absoluut path in mijn geval zijn?! Ik heb dit altijd gebruikt en dan werkte het. Nu heb ik echter ook .htaccess gebruikt, misschien dat het daarom fout gaat oid.

Heb je een voorbeeld?

$_SERVER['DOCUMENT_ROOT'];

$db = $_SERVER['DOCUMENT_ROOT'] . '/images/uploads/'.$naam;

Probeer die eens.

Ah natuurlijk! Hartstikke bedankt, het heeft gewerkt. Het script doet nu wat het moet doen.

En, ook belangrijk om te checken of je bestand wel een image is. Want als iemand een PHP upload, en dan die uitvoert, dan is je site straks ge /dev/null'ed.

Ook is het misschien handig als je kijkt naar de dubbele bestandsnamen ;)

Bedankt voor de feedback. Ik moet het inderdaad nog even wat verder optimaliseren (daar ben ik nu mee bezig).

Ook check je op isset($naam) terwijl de variabele $naam altijd gedefineerd is

ja gebruik ook !empty (not empty = niet leeg ;))

Ook jullie bedankt voor de bijdrage :) In het nieuwe script zal ik dit meenemen. Als het af is, zal ik het hier nogmaals posten, zodat je daar ook op kan schieten! Beveiligen van dit soort dingen is altijd lastig helaas..