Upload tiedoston nimenmuokkaus ja kielletyt merkit (PHP)

[anttinie]

Tarkoituksena olisi muuttaa tiedoston uploadauksessa tiedostonimi sopivaksi internetiin ääkköset -> aakkosiksi jne. Olen tehnyt tuota varten funktion "muokkaa_tiedostonimi" - se toimii hienosti esim. "Muuta nimi" osuudesta kutsuttuna, mutta upload-vaiheessa se toimii vain PC&Windows ympäristössä (linux ei testattuna). Osaako joku sanoo mitä on vialla? tai kuinka tuon saa toimiin myös Mac-ympäristöstä?

Tiedoston tallennus:

Koodi: Valitse kaikki

if (isset($_POST['send'])) { 
  		if($HTTP_POST_FILES['sentfile']['size'] > MAX_SIZE) die ("Liian iso tiedosto!");
		$filename = muokkaa_tiedostonimi($HTTP_POST_FILES['sentfile']['name']);  
		if(file_exists($root.$filename)) die("Tiedosto ".$filename." on jo serverillä"); 
		if (move_uploaded_file($HTTP_POST_FILES['sentfile']['tmp_name'], $root.$filename)) echo "Tiedosto lisättiin onnistuneesti serverille nimellä ".$filename; 
  		display_file_info ($filename); 
  	}  

Tiedostonimen muokkaus sopivaksi funktio:

Koodi: Valitse kaikki

function muokkaa_tiedostonimi ($newfile) {
	$newfile = stripslashes($newfile);
	$newfile = strtolower($newfile);
	
	$kielletyt = array('ä', 'ö', 'å', ' ', '#', '^' ,'!', '%', '&', '(', ')', '*', '¨', '\\','@', '\'');
	$korvaavat = array('a', 'o', 'a', '_', '-', '-','', '', '', '', '', '', '', '','','');
	$newfile = str_replace($kielletyt, $korvaavat, $newfile);
	
	return $newfile;
}

[nikoh]

Varsinaiseen kysymykseen en osaa vastata, mutta sellaisen huomion heittäisin että kiellettyjen merkkien listaamisen sijasta on kokolailla fiksumpaa listata vain sallitut merkit, tuota kiellettyjen listaa kun et koskaan saa idioottivarman kattavaksi (nyt siitä puuttuvat mm. ï, ë, ü, jne..). Sallittuja merkkejä sen sijaan on paaljon vähemmän (a-z, 0-9?)

[Kickarse]

Siis tarkoitatko, että tuohon

Koodi: Valitse kaikki

if (isset($_POST['send'])) { 

kohtaan ei koskaan mennä? Jos näin niin sattuuko tuo formin lähetys tapahtumaan kuvalla (<input type="image">)? Jos näin niin silloin sun pitää tehdä tuo tarkistus näin:

Koodi: Valitse kaikki

if (isset($_POST['send_x'])) { 

Osa selaimista nimittäin asettaa tuo send muuttujan kuvalle ja osa ei. Tai sitten laita formilla joku piilokenttä ja tarkista koodissasi onko se asetettu. Katso täältä lisäinfoa:

http://fi2.php.net/manual/en/language.v ... ternal.php ja kohta "IMAGE SUBMIT variable names". Jos ei tuosta ole kyse niin tarkennatko hiukan tuota ongelmaasi?

[anttinie]

Varsinaiseen kysymykseen en osaa vastata, mutta sellaisen huomion heittäisin että kiellettyjen merkkien listaamisen sijasta on kokolailla fiksumpaa listata vain sallitut merkit, tuota kiellettyjen listaa kun et koskaan saa idioottivarman kattavaksi (nyt siitä puuttuvat mm. ï, ë, ü, jne..). Sallittuja merkkejä sen sijaan on paaljon vähemmän (a-z, 0-9?)

Joo, tuo oli ideana aluksi, mutta en osannut tehdä sitä niin että se olisi korvannut ä -> a, ö -> o jne. Mutta noiden tarkistusten jälkeenhän olis mahdollista kieltää ne muut merkit ja korvata yhdellä tietyllä merkillä. esim, tyhjällä. Kiitoksia, pitää yrittää tehdä.

Siis tarkoitatko, että tuohon

Koodi: Valitse kaikki

if (isset($_POST['send'])) { 

kohtaan ei koskaan mennä? Jos näin niin sattuuko tuo formin lähetys tapahtumaan kuvalla (<input type="image">)?

Kyllä se tuohon if-lauseeseen menee ja muuttaa tiedostonimen kirjaimet pieneksi [strtolower()] funktiossa, mutta kirjainkorjauksia se ei tee Macin käyttäjille. Windowsilla ko. juttu on todettu toimivaksi. Oon testannut eri selaimillakin: IE, Opera, Firefox, Safari. (en tiedä siis onko alustan 'vika' vai joku muu, mutta toi nyt suurin ero koneilla). Ja kiitoksia vastauksesta muutenkin, suunnitelmissa oli laittaa siihen kuva

xhtml-lomakekoodi on siis:

Koodi: Valitse kaikki

<form enctype="multipart/form-data" action="file.php" method="post" >
<p>Lisää tiedosto: <input name="sentfile" type="file" />
<input type="submit" value="Lähetä" name="send" /></p>
</form>

Edit: Jahas, näemmä Firefoxilla lähti toimimaan! mutta Safari & Opera ainakin jäätyy. Eli ongelma lienee liian kiero ratkaistavaksi.

[anttinie]

Vein hommaa pidemmälle tuolla preg_replace():lla, jonka myötä ongelma katosi!

Jokin käsin syöttämistäni 'kielletyistä' merkeistä ei ollut 'yhteensopiva' safarin/operan kanssa. Nyt muokkaa_tiedostonimi() funktio näyttää tältä, mikäli joku tarvitsee:

Koodi: Valitse kaikki

function muokkaa_tiedostonimi ($newfile) {
	$newfile = stripslashes($newfile);
	$newfile = strtolower($newfile);
	
	$kielletyt = array('ä', 'ö', 'å', ' ', '#');
	$korvaavat = array('a', 'o', 'a', '_', '-');
	$newfile = str_replace($kielletyt, $korvaavat, $newfile);
	
	$newfile = preg_replace('#[^a-z0-9._-]#', '', $newfile);
	
	return $newfile;
}

Kiitän vastauksista!