Upload scriptje

Kan je de relevante code in het topic plaatsen, dan hebben andere er nog wat aan als de downloadlink straks verdwenen is.

ergens in je script (ik ga niet zoeken) zal iets staan als

<?php move_uploaded_file($_FILES['xxxx']['tmp_name'], $sometarget); ?> 

Daar kun je controleren of de file al bestaat met:

<?php
if(!file_exsists($sometarget)){
   move_uploaded_file($_FILES['xxxx']['tmp_name'], $sometarget);
}
?>

[size=xsmall]Toevoeging op 29/06/2015 18:19:40:[/size]

je kunt in plaats van het niet verplaatsen ook kiezen voor een nummer achter de filename:

<?php
$counter = 1;
$target = $sometarget;

while(file_exsists($sometarget)) {
  $sometarget = $target . $counter;
  $counter++;
}

move_uploaded_file($_FILES['xxxx']['tmp_name'], $sometarget);
?>

de code van upload.php is:

<?php

// A list of permitted file extensions
$allowed = array('png', 'jpg', 'gif');

if(isset($_FILES['upl']) && $_FILES['upl']['error'] == 0){

	$extension = pathinfo($_FILES['upl']['name'], PATHINFO_EXTENSION);

	if(!in_array(strtolower($extension), $allowed)){
		echo '{"status":"error"}';
		exit;
	}

	if(move_uploaded_file($_FILES['upl']['tmp_name'], 'uploads/'.$_FILES['upl']['name'])){
		echo '{"status":"success"}';
		exit;
	}
}

echo '{"status":"error"}';
exit;

@Ivo: Als $sometarget een extensie bevat gaat het vast niet zoals je wilt.

Ik kan denk ik niet zomaar van alles gaan toevoegen... ik weet niet precies het het met die "if" zit... die moet namelijk wel nog blijven werken anders wordt op de pagina zelf de status niet goed doorgegeven... dat is met namen het probleem waar ik mee zit...

foto.jpg
foto.jpg1
foto.jpg2

gaat prima. Zeker in een voorbeeldscript. Om dat weer te veranderen in
foto1.jpg kost wat extra regels, maar TS is vrij om eea aan te passen natuurlijk

[size=xsmall]Toevoeging op 29/06/2015 18:36:16:[/size]

sowieso is 'uploads/'.$_FILES['upl']['name'] niet zo'n goed plan over het algemeen.

De uploader laat je nu de naam bepalen. ook als er / of ../ oid in de filename staat.

Ok, je hebt gelijk, het gaat wel.
Alleen ben ik geen voorstander van extenties met een getal er aan vast.
Ik ga meer voor foto1.jpg.

Het is inderdaad wel de bedoeling dat de extensie hetzelfde blijft gezien het gaat om veel fotos van veel mensen... deze moeten dan wel een "screaning" krijgen maar om dan ook nog eens weet ik hoeveel foto`s te hernoemen is geen optie...

en de veiligheid is ook wel belangrijk ja... maar hoe ga ik het dan aanpassen zodat de statussen wel nog goed doorgenomen worden?

Die status kijkt niet naar de bestandsnaam, en alleen naar de exetntie en de voorwaarden voor de error-array in je $_FILES.

Dus de bestandsnaam kan je gewoon aanpassen naar elke waarde. Je kan zelfs een prefix/suffix meegeven zodat je kan onderscheiden of ze al gescreend zijn:

foto1.jpg v.s foto1_screened.jpg

het gaat een heel domme vraag zijn... maar gezien die regel begint met: if(move_uploaded_file........... hoe moet ik dan bijvoorbeeld: $counter = 1; $target = $sometarget;........... gaan invoegen? mijn eerste poging zou gewoon een lege if regel zijn maar heb het idee dat dat ook weer niet gaat werken...


is het niet mogelijk om de "tmp_name" aan te houden... neem aan dat hij daar toch al random is? of zie ik dat nou helemaal verkeerd?