Har en fejl i min upload kode

prøv med
echo substr($_FILES['upfile']['name'],-3);
og se hvad du får!?

mpj86> der er ingen grund til at lave en linie til filer med store bogstaver i navnet. Det checkes i forvejen.

$file_type = strtolower(substr($upfile,-3));
if (($file_type == "jpg") || ($file_type == "gif"))
{

if (!eregi(".jpg$", $upfil)) {

For at tilføje flere ->

if (!eregi(".jpg$|.gif$|.png$", $upfil)) {

$upfil = $_FILES['upfile']['name'];
if (!eregi(".jpg$|.gif$|.png$", $upfil)) {
  echo 'ugyldig filtype<br>';
  exit;
}

ahv> hvad hvis nogen finder på at kalde filen for: .jpg.exe

Den mener jeg den hapser.

hvordan? har du testet det? jeg ser det ikke nogen steder.

Det gør den også, hvilket $ jo sørger for.

jeg bliver nød til at lave det i to da jeg skal have hvad filtypen er i $fil_type. Altså hvis man har uploadet en fil ved navn billede.jpg så skal $fil_type være = .jpg

Det hapser den pga. $ til sidst!

ohmish, du kan jo selv teste efter ->

<?php
$upfil = "hej.jpg.exe";
if (!eregi(".jpg$|.gif$|.png$", $upfil)) {
  echo 'ugyldig filtype<br>';
  exit;
}
?>

mpj86, hvorfor det?

Hvis i har en nemmere måde så er hele koden her:

<?php
include('config.php');

if (isset($_FILES['upfile'])) {

$id = $_GET['id'];
$getvar = mysql_fetch_array(mysql_query("SELECT max(id) AS nyt_id FROM scooter_billeder")) or die (mysql_error());
$billede_id = ($getvar['nyt_id'])+1;

$dist = $_SERVER['DOCUMENT_ROOT']."/gallerisystem/billeder/";

for($i = 0; $i < count($_FILES['upfile']['name']); $i++) {

    if (isset($_FILES['upfile']['name'][$i]) && $_FILES['upfile']['name'][$i] !="") {
   
        if(is_uploaded_file($_FILES['upfile']['tmp_name'][$i])) {
           
        $files = $_FILES['upfile']['name'][$i];
        $temparr = explode(".", $files);
        $temparr[count($temparr)-2] = $id."_".$billede_id;
        $files = implode(".", $temparr);
        $upfil = $_FILES['upfile']['name'];
        if (preg_match ("/(jpg)$/i", $upfil)) { $fil_type = ".jpg"; }
        elseif (preg_match ("/(JPG)$/i", $upfil)) { $fil_type = ".JPG"; }
        else { echo 'ugyldig filtype<br>'; exit; }
       
        move_uploaded_file($_FILES['upfile']['tmp_name'][$i], $dist . $files . $fil_type);
        $billede_sti = 'http://www.mopedpower.dk/gallerisystem/billeder/'.$id.'_'.$billede_id.''.$fil_type;
        mysql_query("INSERT INTO scooter_billeder (scooter_id, URL) VALUES ('$id','$billede_sti')") or die (mysql_error());

        } else {

        echo 'Upload til serveren skete ikke!';

        }
        echo 'Uploadet!';
        }
    }
}

echo '
<form enctype="multipart/form-data" action="'.$_SERVER['REQUEST_URI'].'" method="post">
<input name="upfile[]" type="file" size="50" class="input"><br>
<input name="upfile[]" type="file" size="50" class="input"><br>
<input type="submit" value="Upload" size="50" class="submit">
</form>
';
?>

<?php
include('config.php');

if (isset($_FILES['upfile'])) {

$id = $_GET['id'];
$getvar = mysql_fetch_array(mysql_query("SELECT max(id) AS nyt_id FROM scooter_billeder")) or die (mysql_error());
$billede_id = ($getvar['nyt_id'])+1;

$dist = $_SERVER['DOCUMENT_ROOT']."/gallerisystem/billeder/";

for($i = 0; $i < count($_FILES['upfile']['name']); $i++) {

    if (isset($_FILES['upfile']['name'][$i]) && $_FILES['upfile']['name'][$i] !="") {
   
        if(is_uploaded_file($_FILES['upfile']['tmp_name'][$i])) {
           
        $files = $_FILES['upfile']['name'][$i];
        $temparr = explode(".", $files);
        $temparr[count($temparr)-2] = $id."_".$billede_id;
        $files = implode(".", $temparr);
        if (!eregi(".jpg$|.gif$|.png$", $files)) {
            echo 'ugyldig filtype<br>';
            exit;
        } 
        move_uploaded_file($_FILES['upfile']['tmp_name'][$i], $dist . $files);
        $billede_sti = 'http://www.mopedpower.dk/gallerisystem/billeder/$files';
        mysql_query("INSERT INTO scooter_billeder (scooter_id, URL) VALUES ('$id','$billede_sti')") or die (mysql_error());

        } else {

        echo 'Upload til serveren skete ikke!';

        }
        echo 'Uploadet!';
        }
    }
}

echo '
<form enctype="multipart/form-data" action="'.$_SERVER['REQUEST_URI'].'" method="post">
<input name="upfile[]" type="file" size="50" class="input"><br>
<input name="upfile[]" type="file" size="50" class="input"><br>
<input type="submit" value="Upload" size="50" class="submit">
</form>
';
?>

DAMN! det dur :)

Kan du ikke også tilføje i kode så den skriver dette hvis man uploader en fil på over 400 kb "Billederne om max fylde 400 kb pr. stk og et af de billeder du prøver på at uploade fylder 759 kb!" :)

Læs selv om det på www.webcafe.dk , man har godt af selv at læse lidt :o)

en lille fejl i db står der: http://www.mopedpower.dk/gallerisystem/billeder/$files

Webcafe har en hel artikel om uplaod -> http://www.webcafe.dk/artikler/php/upload/

Fejlen, prøv med ->
$billede_sti = "http://www.mopedpower.dk/gallerisystem/billeder/".$files;

Det dur...

Men før i hen ville den godt uploade flere filer på en gang men det vil den ikke mere hvorfor?

Hvad mener du, hvis der er fejl i det ene billede, så vil den ikke uploade det andet eller?

Hvis jeg vælger 2 filer som er .jpg til at uploade så bliver der kun uploadet det første billede men der bliver oprettet 2 (nye ids) i db som der skal men i begge i feltet URL er adressen den samme nemlig adressen til det første billede der bliver uploadet...

Prøv ->

<?php
include('config.php');

if (isset($_FILES['upfile'])) {
    $id = $_GET['id'];
    $getvar = mysql_fetch_array(mysql_query("SELECT max(id) AS nyt_id FROM scooter_billeder")) or die (mysql_error());
    $billede_id = ($getvar['nyt_id'])+1;
    $dist = $_SERVER['DOCUMENT_ROOT']."/gallerisystem/billeder/";
    foreach ($_FILES['upfile']['name'] as $name => $value) {
        if ($value) {
            if (is_uploaded_file($_FILES['upfile']['tmp_name'][$i])) {
                $temparr = explode(".", $value);
                $temparr[count($temparr)-2] = $id."_".$billede_id;
                $files = implode(".", $temparr);
                if (!eregi(".jpg$|.gif$|.png$", $files)) {
                    echo 'ugyldig filtype<br>';
                    exit;
                } 
                move_uploaded_file($_FILES['upfile']['tmp_name'][$i], $dist . $files);
                $billede_sti = "http://www.mopedpower.dk/gallerisystem/billeder/".$files;
                mysql_query("INSERT INTO scooter_billeder (scooter_id, URL) VALUES ('$id','$billede_sti')") or die (mysql_error());
            } else {
                echo 'Upload til serveren skete ikke!';
            }
            echo 'Uploadet!';
        }
    }
}

echo '
<form enctype="multipart/form-data" action="'.$_SERVER['REQUEST_URI'].'" method="post">
<input name="upfile[]" type="file" size="50" class="input"><br>
<input name="upfile[]" type="file" size="50" class="input"><br>
<input type="submit" value="Upload" size="50" class="submit">
</form>
';
?>

Hov ->

<?php
include('config.php');

if (isset($_FILES['upfile'])) {
    $id = $_GET['id'];
    $getvar = mysql_fetch_array(mysql_query("SELECT max(id) AS nyt_id FROM scooter_billeder")) or die (mysql_error());
    $billede_id = ($getvar['nyt_id'])+1;
    $dist = $_SERVER['DOCUMENT_ROOT']."/gallerisystem/billeder/";
    $i = 0;
    foreach ($_FILES['upfile']['name'] as $name => $value) {
        if ($value) {
            if (is_uploaded_file($_FILES['upfile']['tmp_name'][$i])) {
                $temparr = explode(".", $value);
                $temparr[count($temparr)-2] = $id."_".$billede_id;
                $files = implode(".", $temparr);
                if (!eregi(".jpg$|.gif$|.png$", $files)) {
                    echo 'ugyldig filtype<br>';
                    exit;
                } 
                move_uploaded_file($_FILES['upfile']['tmp_name'][$i], $dist . $files);
                $billede_sti = "http://www.mopedpower.dk/gallerisystem/billeder/".$files;
                mysql_query("INSERT INTO scooter_billeder (scooter_id, URL) VALUES ('$id','$billede_sti')") or die (mysql_error());
            } else {
                echo 'Upload til serveren skete ikke!';
            }
            echo 'Uploadet!';
        }
        $i++;
    }
}

echo '
<form enctype="multipart/form-data" action="'.$_SERVER['REQUEST_URI'].'" method="post">
<input name="upfile[]" type="file" size="50" class="input"><br>
<input name="upfile[]" type="file" size="50" class="input"><br>
<input type="submit" value="Upload" size="50" class="submit">
</form>
';
?>

den skriver: Upload til serveren skete ikke!Uploadet! men den uploader ikke nogle billeder...

tester lige den nye.....

Første gang man uploader, uploader den kun et billede anden gang man uploader, uploader den begge billeder så der er problemer med den første gang der bliver uploadet jeg tror fejlen ligger her:

    $getvar = mysql_fetch_array(mysql_query("SELECT max(id) AS nyt_id FROM scooter_billeder")) or die (mysql_error());
    $billede_id = ($getvar['nyt_id'])+1;

Anden gang man så har uploadet, uploader den begge billeder men i db hedder begge det samme i feltet URL...

Prøv ->

$temparr[count($temparr)-2] = $id."_".$billede_id."_".$i;

Nu virker det så kan jeg bare lave min kode lidt om så det bliver sådan $id_$scooter_id_$i

Hvad betyder $i?

$i er bare den variabel der tæller på, så vi for de rigtige informtioner, den gør så også at de ikke for det samme navn, ved at sætte den bagved navnene. :o)

ok takker :)