[PHP] Uploaden bestand werkt niet

[Verwijderd]

Beste tweakers,

Ik gebruik onderstaand stukje php om onder ander een PDF te uploaden naar de map 'bijlagen' Op het moment dat ik dit een file probeer toe te voegen dan doet gaat deze wel uploaden (duurt eventjes) alleen wordt deze niet in de map /bijlagen geplaatst. DIR is writeable uiteraard en de PATH is ook goed. Doe ik toch iets verkeerd? of wat zou het kunnen zijn....

PHP:

if($actie == "toevoegen") {
if (is_uploaded_file($_FILES['bijlage']['tmp_name'])) {
        move_uploaded_file($_FILES['bijlage']['tmp_name'], "/var/www/vhost/httpdocs/bijlagen/".$_FILES['bijlage']['name']);
    }
$SQL_statement = "INSERT INTO vacatures SET titel='".$titel."', omschrijving='".$omschrijving."', bijlage='".$_FILES['bijlage']['name']."'  ";
$resultset = mysql_query($SQL_statement);   
$message = "De vacature is succesvol toegevoegd";

Alvast bedankt voor de genomen moeite om dit te lezen en/of eventueel te reageren.

[kalizec]

Kan het zijn dat de geuploade bestanden groter zijn dan in Apache en/of PHP toegestaan is?

[gertvdijk]

1. Zorg dat je MySQL injection niet meer mogelijk is! Als je niet weet wat dit is moet je je heel rap gaan inlezen.
2. Gebruik code=php tags. Dit maakt het voor iedereen duidelijk.

[Verwijderd]

kalizec schreef op zaterdag 17 januari 2009 @ 22:54:
Kan het zijn dat de geuploade bestanden groter zijn dan in Apache en/of PHP toegestaan is?

Ik heb juist een klein bestandje geprobeerd. LIMIT staat op 2048 dus dat moet wel lukken voor een tekt pdfje

[NMe]

Volgens mij moet de destination folder bij move_uploaded_file relatief zijn aan je webroot. Dus zonder /var/www etc.

edit:

Om het even te verduidelijken:

quote: http://nl3.php.net/move_uploaded_file

Note: move_uploaded_file() is both safe mode and open_basedir aware. However, restrictions are placed only on the destination path as to allow the moving of uploaded files in which filename may conflict with such restrictions. move_uploaded_file() ensures the safety of this operation by allowing only those files uploaded through PHP to be moved.

gertvdijk schreef op zaterdag 17 januari 2009 @ 22:55:
1. Zorg dat je MySQL injection niet meer mogelijk is! Als je niet weet wat dit is moet je je heel rap gaan inlezen.
2. Gebruik code=php tags. Dit maakt het voor iedereen duidelijk.

Als je dergelijke offtopic opmerkingen wil maken, kan dat dan iets vriendelijker?

[ Voor 38% gewijzigd door NMe op 17-01-2009 23:07 ]

[gertvdijk]

Verwijderd schreef op zaterdag 17 januari 2009 @ 22:57:
Ik heb juist een klein bestandje geprobeerd. LIMIT staat op 2048 dus dat moet wel lukken voor een tekt pdfje

Welke limit? En welke eenheid is die 2048? bytes, kB, MB?

[iH8]

Verwijderd schreef op zaterdag 17 januari 2009 @ 22:57:
[...]


Ik heb juist een klein bestandje geprobeerd. LIMIT staat op 2048 dus dat moet wel lukken voor een tekt pdfje

debuggen! begin eens met een:

PHP:

if($actie == "toevoegen") {
    print_r($_FILES);
}

[Verwijderd]

gertvdijk schreef op zaterdag 17 januari 2009 @ 22:55:
1. Zorg dat je MySQL injection niet meer mogelijk is! Als je niet weet wat dit is moet je je heel rap gaan inlezen.
2. Gebruik code=php tags. Dit maakt het voor iedereen duidelijk.

1. ik weet wat het is, tis maar voor heel simpel gebruik en verwacht geen kwade bedoelingen.
2. Ga we even regelen

[Verwijderd]

gertvdijk schreef op zaterdag 17 januari 2009 @ 22:59:
[...]

Welke limit? En welke eenheid is die 2048? bytes, kB, MB?

2 mb limiet meegegeven aan het uploaden van de bestandsgrote

[scorpie]

Heeft je form wel de goede enctype ?

[kalizec]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:01:
[...]


2 mb limiet meegegeven aan het uploaden van de bestandsgrote

In PHP of in Apache? Je kunt in beide de limiet instellen en het is de laagste die telt.

PS. Kijk ook even naar de reactie van NMe.

[ Voor 7% gewijzigd door kalizec op 17-01-2009 23:05 ]

[BramT]

iH8 schreef op zaterdag 17 januari 2009 @ 22:59:
[...]


debuggen! begin eens met een:

PHP:

if($actie == "toevoegen") {
    print_r($_FILES);
}

What he said... Die gaat je waarschijnlijk een hoop vertellen.

[Verwijderd]

NMe schreef op zaterdag 17 januari 2009 @ 22:58:
Volgens mij moet de destination folder bij move_uploaded_file relatief zijn aan je webroot. Dus zonder /var/www etc.

[...]

Als je dergelijke offtopic opmerkingen wil maken, kan dat dan iets vriendelijker?

Ik zal dat even nu gaan proberen!

[NMe]

Ah, trouwens: Waar hoort mijn topic?

WEB>>PRG

[Verwijderd]

scorpie schreef op zaterdag 17 januari 2009 @ 23:03:
Heeft je form wel de goede enctype ?

multipart/form-data, dat moet goed zijn volgens mij!

[Verwijderd]

iH8 schreef op zaterdag 17 januari 2009 @ 22:59:
[...]


debuggen! begin eens met een:

PHP:

if($actie == "toevoegen") {
    print_r($_FILES);
}

Dat heb ik nu net gedaan: Resultaat is

Array ( [bijlage] => Array ( [name] => vacature1.pdf [type] => application/pdf [tmp_name] => /tmp/phpHfuM4U [error] => 0 [size] => 399158 ) )

Maar wat gaat hier nu mis dan, zie het even niet meer.....

[gertvdijk]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:25:
Dat heb ik nu net gedaan: Resultaat is

Array ( [bijlage] => Array ( [name] => vacature1.pdf [type] => application/pdf [tmp_name] => /tmp/phpHfuM4U [error] => 0 [size] => 399158 ) )

Maar wat gaat hier nu mis dan, zie het even niet meer.....

Ziet er goed uit. Nu uitzoeken waarom het bestand dat tijdelijk in /tmp/phpHfuM4U staat niet naar /var/w... kan worden verplaatst. Je zei zelf dat hij schrijfbaar is, maar heb je rekening gehouden met het feit dat de webserver onder een andere user draait dan jezelf?

[scorpie]

De file komt dus blijkbaar goed aan alleen je verplaatst hem niet goed vanuit je tmp directory naar een andere directory denk ik.

BTW die move command heeft toch ook een return value die je uit kunt lezen?

[ Voor 23% gewijzigd door scorpie op 17-01-2009 23:29 ]

[iH8]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:25:
[...]


Dat heb ik nu net gedaan: Resultaat is

Array ( [bijlage] => Array ( [name] => vacature1.pdf [type] => application/pdf [tmp_name] => /tmp/phpHfuM4U [error] => 0 [size] => 399158 ) )

Maar wat gaat hier nu mis dan, zie het even niet meer.....

dan gaat er iets mis bij het verplaatsten. zet dit is bovenaan in je code:

PHP:

error_reporting( E_ALL | E_STRICT );
ini_set( 'display_errors', 1 );
ini_set( 'display_startup_errors', 1 );

[gertvdijk]

scorpie schreef op zaterdag 17 januari 2009 @ 23:28:
BTW die move command heeft toch ook een return value die je uit kunt lezen?

PHP functie move_uploaded_file:

bool move_uploaded_file ( string $filename , string $destination )
[...]
If filename is a valid upload file, but cannot be moved for some reason, no action will occur, and move_uploaded_file() will return FALSE. Additionally, a warning will be issued.

scorpie schreef op zaterdag 17 januari 2009 @ 23:35:
uuh.. ja? Dat zei ik toch ook?

Uit een boolean return value haal je niet zoveel, behalve wat je al weet: het gaat mis. Blijkbaar gooit ie alle debug info in E_WARNING.

[ Voor 28% gewijzigd door gertvdijk op 17-01-2009 23:37 ]

[scorpie]

uuh.. ja? Dat zei ik toch ook?

[iH8]

scorpie schreef op zaterdag 17 januari 2009 @ 23:28:

BTW die move command heeft toch ook een return value die je uit kunt lezen?

precies, die returned gewoon true als 't gelukt is.

PHP:

if(move_uploaded_file(...)){
    echo 'yeah';
} else {
    echo 'nope';
}

edit: late!

[ Voor 33% gewijzigd door iH8 op 17-01-2009 23:38 . Reden: late ]

[Verwijderd]

iH8 schreef op zaterdag 17 januari 2009 @ 23:32:
[...]


dan gaat er iets mis bij het verplaatsten. zet dit is bovenaan in je code:

PHP:

error_reporting( E_ALL | E_STRICT );
ini_set( 'display_errors', 1 );
ini_set( 'display_startup_errors', 1 );

HEE Fantastisch! Dit werkt, Ik weet niet wat het precies doet maar het werkt wel! Super! Dank je!

Nu heb ik weer weggehaald en het werkt nog steeds ??

[CH4OS]

iH8 schreef op zaterdag 17 januari 2009 @ 22:59:
debuggen! begin eens met een:

PHP:

if($actie == "toevoegen") {
    print_r($_FILES);
}

Doe dan gewoon

PHP:

if($_FILES)
{
    echo '<pre>';
    print_r($_FILES);
    echo '</pre>';
}
else
{
    echo 'No files! :(';
}

In jouw code maak je $_FILES afhankelijk van een variabele die moet bestaan en een waarde "toevoegen" moet hebben. Terwijl je juist wil weten of er überhaupt files gepost worden.

[ Voor 20% gewijzigd door CH4OS op 17-01-2009 23:40 ]

[gertvdijk]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:38:
HEE Fantastisch! Dit werkt, Ik weet niet wat het precies doet maar het werkt wel! Super! Dank je!

Nu heb ik weer weggehaald en het werkt nog steeds ??

Dat had geen enkel effect op de werking van de rest van je script. Zoals je zelf al aangeeft werkt het ook als je het weer weghaalt. Blijkbaar gewoon zelf iets niet goed gedaan met testen?

[iH8]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:38:
[...]


HEE Fantastisch! Dit werkt, Ik weet niet wat het precies doet maar het werkt wel! Super! Dank je!

Nu heb ik weer weggehaald en het werkt nog steeds ??

lol, dat zorgt er alleen voor dat je errors ook daadwerkelijk te zien krijgt. als je code nu wel werkt ligt het iig daar niet aan.

[scorpie]

Verwijderd schreef op zaterdag 17 januari 2009 @ 23:38:
[...]


HEE Fantastisch! Dit werkt, Ik weet niet wat het precies doet maar het werkt wel! Super! Dank je!

Nu heb ik weer weggehaald en het werkt nog steeds ??

Het zet wat config waarden om in de php config file.
Maar in het vervolg kan ik je aanraden dat je eerst opzoekt wat iets doet alvorens het uit te voeren, je weet nooit wat het met je systeem doet. Nu zijn we hier op GoT wel goedgezind met elkaar, maar je kan maar beter iets meer paranoïder zijn...

Anyway @gertvdijk: Er staat ook bij dat een warning geraised wordt waar je wat mee kan. Nergens had TS gezegd dat hij zeker wist dat die command niet (goed) uitgevoerd werd, dus kon je met een check op de returnvalue van die move_uploaded_file gewoon wel wat verder komen. Maargoed, het was ook een retorische vraag dus..

[CH4OS]

scorpie schreef op zaterdag 17 januari 2009 @ 23:43:
Het zet wat config waarden om in de php config file.

AFAIK doet ini_set alleen tijdens het draaien van dat specifieke script en zet het daarna weer terug:

quote: http://www.php.net/ini_set

Sets the value of the given configuration option. The configuration option will keep this new value during the script's execution, and will be restored at the script's ending.

Het is misschien een mierenneukertje en ik ga er van uit dat je dat ook bedoeld, alleen op de manier waarop je het schreef dacht ik in eerste instantie wat anders, vandaar.

[ Voor 24% gewijzigd door CH4OS op 18-01-2009 00:51 ]