---

Hejsa derude

Jeg har fået lavet mig en filuploader som virker upåklageligt...

Men så opdagede jeg en lille fejl!

Når jeg fx uploader en fil med et navn lidt ala "dette er en fil.jpg"
altså med mellemrum i filnavnet så vil enten PHP eller MySQL ikke
kendes ved filen!

Så derfor, hvordan renamer jeg den til "dette_er_en_fil.jpg" INDEN
kopiering, da PHP ikke kan "finde" filen igen når den er er blevet
kopieret (altså det virker fint hvis filen hedder "detteerenfil.jpg"
så kan den både slette og se billedet osv.

Indtil videre ser mit script sådan her ud:

<?php

$files_name = $_FILES["img"]["name"];
$renfile = time();
$total_filename = $renfile."-".$files_name;
$files_size = $_FILES["img"]["size"];

copy($_FILES["img"]["tmp_name"],
$relativ_sti."/images/varepics/".$_FILES["img"]["name"])
   or die("Kunne ikke uploade filen!");

rename($relativ_sti."/images/varepics/".$_FILES["img"]["name"],
$relativ_sti."/images/varepics/".$renfile."-".$_FILES["img"]["name"]);


$sql = sql("INSERT INTO pics
VALUES(NULL,'$vareid[0]','$total_filename','$files_size','0')");

$last_id = mysql_insert_id();

?>

---

On Sun, 10 Nov 2002 16:39:16 +0100, -Martin-
<martin_aarhof(AT)hotmail.com> wrote:

>Hejsa derude

Hej mig selv

>Så derfor, hvordan renamer jeg den til "dette_er_en_fil.jpg" INDEN
>kopiering, da PHP ikke kan "finde" filen igen når den er er blevet
>kopieret (altså det virker fint hvis filen hedder "detteerenfil.jpg"
>så kan den både slette og se billedet osv.

Jeg fik selv løs problemet

>
>Indtil videre ser mit script sådan her ud:
>
><?php
>


$edit = str_replace(" ","_",$_FILES["img"]["name"]);

>$renfile = time();

>$total_filename = $renfile."-".$edit;

>$files_size = $_FILES["img"]["size"];
>
>copy($_FILES["img"]["tmp_name"],
>$relativ_sti."/images/varepics/".$_FILES["img"]["name"])
>   or die("Kunne ikke uploade filen!");
>
>rename($relativ_sti."/images/varepics/".$_FILES["img"]["name"],
>$relativ_sti."/images/varepics/".$renfile."-".$_FILES["img"]["name"]);
>
>
>$sql = sql("INSERT INTO pics
>VALUES(NULL,'$vareid[0]','$total_filename','$files_size','0')");
>
>$last_id = mysql_insert_id();
>
>?>

---

-Martin- <martin_aarhof(AT)hotmail.com> wrote in
<2dvssucefpqn4snd31hb5aac9mkkd2ni4j@4ax.com>:
> Når jeg fx uploader en fil med et navn lidt ala "dette er en fil.jpg"
> altså med mellemrum i filnavnet så vil enten PHP eller MySQL ikke
> kendes ved filen!

Jeg kan ikke lige komme på noget problem med mellemrum i filnavne. Der er
dog en række steder, hvor der skal tages hensyn til visse tegn i filnavne.
Ud over mellemrum er der fx. dollar-tegn ($), single quote ('), double
quote ("), slash (/), backslash (\) og linjeskift.

Tegnet "form-feed" (sideskift, ^L) i filnavne har i øvrigt kostet mig et par
hårtotter. :)

> Så derfor, hvordan renamer jeg den til "dette_er_en_fil.jpg" INDEN
> kopiering,

Da dit problem sikkert ikke er "mellemrum", men nok "specialtegn", er det
kun starten på en løsning, som jeg i øvrigt vil fraråde, hvis der er
alternativer.

> da PHP ikke kan "finde" filen igen når den er er blevet kopieret

Det problem har jeg vist aldrig haft. PHP kan fint håndtere filnavne med
mellemrum.

> $files_name = $_FILES["img"]["name"];
> $renfile = time();
> $total_filename = $renfile."-".$files_name;
> $files_size = $_FILES["img"]["size"];
>
> copy($_FILES["img"]["tmp_name"],
> $relativ_sti."/images/varepics/".$_FILES["img"]["name"])
> or die("Kunne ikke uploade filen!");
>
> rename($relativ_sti."/images/varepics/".$_FILES["img"]["name"],
> $relativ_sti."/images/varepics/".$renfile."-".$_FILES["img"]["name"]);

Puha, det var da en værre smørre. :)

Jeg ville nok gøre noget i stil med dette:

$filename = $relativ_sti
.'/images/varepics/'
.time()
.' - '
.$_FILES["img"]["name"];

$uploaded = move_uploaded_file(
$_FILES["img"]["tmp_name"],
$relativ_sti. '/images/varepics/'.$filename
);

Der er adskillige fordele ved at gøre det på den måde i stedet. Både mht.
performance, sikkerhed og overskuelig kode.

Jeg fandt i øvrigt ikke noget i din kode, som kunne give problemer med
mellemrum. Din SQL der imod har problemer med bla. single-quotes. Filnavnet
er noget der (i hvert fald delvist) kommer udefra, derfor skal det escapes.

Det gøres vist traditionelt med addslashes(), men den tager ikke hensyn
hvilken dbm du bruger. Men det er en helt anden snak. :)

--
Mvh.

Niels Andersen
(la nels. anersyn.)

---

[ SNIP en helt lang smøre ]

>Der er adskillige fordele ved at gøre det på den måde i stedet. Både mht.
>performance, sikkerhed og overskuelig kode.
>
>Jeg fandt i øvrigt ikke noget i din kode, som kunne give problemer med
>mellemrum. Din SQL der imod har problemer med bla. single-quotes. Filnavnet
>er noget der (i hvert fald delvist) kommer udefra, derfor skal det escapes.
>
>Det gøres vist traditionelt med addslashes(), men den tager ikke hensyn
>hvilken dbm du bruger. Men det er en helt anden snak. :)

Jeg takker mange gange for de ting som jeg egentlig ikk har taget
hensyn til fx $ osv.

Kan du give mig en løsning på problemet så de hele bliver escapet
eller udeladt?

Måske noget med
str_replace("[SPECIAL-TEGN]",""$fil); ?

Men så ska den jo hele filnavnet igennem et array med special tegn
osv?

Så er der vel også muligheden med ereg_replace(); meeen det fatter jeg
virkelig ikk et hak af selvom jeg har prøvet i én uendelighed