Fejl i upload af thumbs

if ($type == ".jpeg") {
                        imagejpeg($destinationImage, $thumbnail_dir.basename($billede, ".jpeg")."_thumb.jpeg");
                    } else {
                        imagejpeg($destinationImage, $thumbnail_dir.basename($billede, ".jpg")."_thumb.jpg");
                    }

Er jeg ret sikker på.

Men hvordan retter jeg den, kan ikke lige gennemskue det :-)

imagejpeg($destinationImage, $thumbnail_dir.basename($billede . "_thumb", ".jpg"), 100);
Prøv den.

Som kom filen til at hedde logo.jpglogo.jpg_thumb :-)

imagejpeg($destinationImage, "thumb_" . $billede, 100);

Prøv med en.

Er ikke elite php-programmør og har næsten ingen erfaring med billede upload/redigering i php, så prøver mig bare frem :D

Helt ok :-) Øv, nu laver den den slet ikke..

damnit,

Prøv så:
imagejpeg($destinationImage, basename($thumbnail_dir, ".jpeg") . "_thumb.jpeg", 100);
imagejpeg($destinationImage, basename($thumbnail_dir, ".jpg") . "_thumb.jpg", 100);

I kigger desværre det forkerte sted.

$thumbnail_dir  = $usermap_thumbs . $billede;// Stien til det thumbnail

bliver til /mappe/billede.jpg

prøv istedet bare at skrive

$thumbnail_dir  = $usermap_thumbs

Nej dkfire.

$thumbnail_dir = /mappe/billede.jpg

basename($thumbnail_dir, ".jpg") = billede

Når nu $thumbnail_dir = /mappe/billede.jpg

Så bliver :
imagejpeg($destinationImage, $thumbnail_dir.basename($billede, ".jpeg")."_thumb.jpeg");
da til:
imagejpeg($destinationImage, /mappe/billede.jpgbillede_thumb.jpeg)
'
For du lægger jo $thumbnail_dir + billede + _thumb.jpg, det må give:
/mappe/billede.jpgbillede_thumb.jpeg

dkfire du har ret, men i skal nu have tak begge 2. Smid et svar

Hov nu opstod der et andet problem, hvis jeg uploader to filer med samme navn omdøber den godt nok den store fil, men den genererer ingen thumb? Her er funktionen til tjek på navnet:

function tjekfilnavn($filnavn) {
    $i = 0;
    $ext = strrchr($filnavn, '.');
    $navn = substr($filnavn, 0, -strlen($ext));
    while (is_file($filnavn)) {
        $i++;
        $filnavn = $navn . "_" . sprintf("%03u", $i) . $ext;
    }
    return $filnavn;
}

Hvis du kan, prøv at smide en echo ind i din while-løkke, bare for at se om den ændre filnavnet.

Kan se den ændre navnet godt nok, det store billede bliver også gemt med det nye navn, men thumben bliver slet ikke gemt..

imagejpeg($destinationImage, basename($thumbnail_dir, ".jpeg") . "_thumb.jpeg", 100);
imagejpeg($destinationImage, basename($thumbnail_dir, ".jpg") . "_thumb.jpg", 100);

Har du prøvet den?
Fordi ifølge php.net skulle dette blive til
billedenavn_thumb.jpg

Det giver samme resultat, uploader man et billede med navnet f.eks. img.jpg virker det helt fint. Uploader man så igen et billede med samme navn kommer det store billede til at hedde img_001.jpg men thumben bliver ikke kopieret..

Hvornår bruger du tjekfilnavn ??

imagejpeg($destinationImage, basename($thumbnail_dir, ".jpg") . "_thumb.jpg", 100);

giver billede_thumb.jpg
har jeg selv lige testet.
Hvis du så vil kunne uploade med samme navn igen, må du bare smide et random nummer foran.

F.eks.
$random = date('ymhms');
imagejpeg($destinationImage, $random . "_" . basename($lort, ".jpg") . "_thumb.jpg";, 100);

Og evt. det samme med det store billede.

Min ovenstående eksempel giver:
0705100521_billede_thumb.jpg

Selvom du sætter et random nummer foran, bliver du stadig nød til at tjekke at filnavnet ikker er brugt i forvejen.

Nej, for så kan det bruges så ofte man har lyst..

Jeg tjekker øverst oppe

$dest = tjekfilnavn($dest);

Øhhh nu jeg slet ikke med.

Hvis man kun må uploade 2 gange, hvordan vil du holde styr på det ?

Hvis man gerne må uploade flere end 2 gange, hvordan undgår du så at overskrive en existerende fil selv med random nummer ?
Du bliver squ da nød til at tjekke om filen existere eller ej.

wolstrup: så bliver du nød til at gøre det samme med thumbnail.

dkfire >
Mit "random" nummer, er datoen, og tiden for når man uploader, så medmindre et billede med samme navn, og i samme sekund bliver uploadet, så skulle der ikke være noget problem.

www.mmkay.dk/test.php

Der kan du se hvad der kommer ud af mit script.

Der har jeg angivet stien som /snejlefarphg/bruum/billede.jpg

zurekk => Den kopierer slet ikke thumben når jeg laver dine ændringer

dkfire => Hvordan laver jeg tjek på thumben? Troede egentlig at thumben blev genereret ud fra navnet på det store billede som jo i så fald ville være ændret hvis der fandtes et billede med samme navn i forvejen..

Du huskede at ændre $lort til $thumbnail_dir ikke?

<?php

$billede        = $_FILES['upfile']['name'];
    $type            = strrchr($billede, ".");// For at finde filtypen til tjek af jpeg kontra jpg
    $billede        = strtolower($billede);// ændre alt til små bogstaver
    $dest            = $usermap . $billede;// Stien til det store billede
    $dest            = tjekfilnavn($dest);
    $thumbnail_dir  = $usermap_thumbs . $billede;// Stien til det thumbnail
    $overskrift        = $_POST['kommentar'];



    $oprettelsesDato = date('Ymd');
    $source = $_FILES['upfile']['tmp_name'];
    if (is_uploaded_file($source))
        {
            if (tjektype($_FILES['upfile']['type']))
                {
                    if (move_uploaded_file($source, $dest))
                        {
                            list($width, $height) = getimagesize($dest);
                            $new_width = 100;
                            $krympefaktor = $new_width/$width;
                            $new_height = round($height * $krympefaktor);
                            $destinationImage = imagecreatetruecolor($new_width, $new_height);
                           
                            if(tjektype($_FILES['upfile']['type'] == "image/pjpeg"))
                                {
                               
                                $sourceImage = imagecreatefromjpeg($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                       
                            if ($type == ".jpeg") {
                                imagejpeg($destinationImage, date('ymdhms') . "_" basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg");
                                $type = '.jpg';
                                }
                            else
                                {
                                imagejpeg($destinationImage, date('ymdhms') . "_" basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg");
                                $type = '.jpeg';
                                }
                           
                                }
                            else if(tjektype($_FILES['upfile']['type'] == "image/png"))
                                {
                                $sourceImage = imagecreatefrompng($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                                imagepng($destinationImage, $thumbnail_dir.basename($billede, ".png")."_thumb.png");
                                $type = '.png';
                                }
                   
                $name = date('ymdhms') . "_" basename( $thumbnail_dir, $type ) . "_thumb" . $type;
                mysql_query("INSERT INTO images (user, name, text, date)
                            VALUES ('$userid', '$name', '$overskrift', '$oprettelsesDato')") or die (mysql_error());
                mysql_close();
                $besked = "Tekst og billede gemt";
               
                        }
                    else
                        {
                       
                $besked = "Der blev ikke gemt noget";
               
                }
            }
        else
            {
                $besked = "Du prøvede at gemme et billedeformat som ikke er tilladt. Du kan gemme billeder i JPEG og PNG formaterne.";
            }
           
        }
    else
        {
        mysql_query("INSERT INTO RS_Nyhed SET overskrift='$overskrift', sti='images/intetBillede.png',
                    tekst='$tekst', dato='$oprettelsesDato'");
        mysql_close();
        header("location: adminNyhed.php");
        }
}


Så prøv den der..

Får Parse error: syntax error, unexpected T_STRING in (stien) on line 89

Hvad er der på linje 89 =)?

Du havde bare lige glemt nogle "." foran basename.. Nu virker det en smule, bortset fra at et billeder ved navn michael1.jpg, kommer til at hedde 070503120550_michael1.jpg_thumb.jpeg, og så ender det i roden istedet for i mappen thumbnails.

Prøv med

                            if ($type == ".jpg") {
                                imagejpeg($destinationImage, "dinmappe/" . date('ymdhms') . "_" basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg");
                                $type = '.jpg';
                                }
                            else
                                {
                                imagejpeg($destinationImage, "dinmappe/" . date('ymdhms') . "_" basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg");
                                $type = '.jpeg';
                                }

Perfekt... Kan jeg lave mere harmoni mellem de store og de små billeder, så der er et mønster i det. Nu har jeg f.eks. to store billeder:

michael1.jpg
michael1_001.jpg

og to små:

070503010517_michael1_thumb.jpg
070503010538_michael1_thumb.jpg

Lidt svært at skelne hvad der hører til hvad :)

Det kommer an på hvordan dit script ser ud når du uploader de originale, om du gør det på samme side og om du gør det på samme tid =)

Men du kan evt. smide det script som tilføjer original billederne, så skal jeg kigge på det.

Evt. smide hele filen.

Hele filen ser nu således ud:

if ($submit) {

if(is_dir($usermap)){
}else{
mkdir($usermap, 0777);
mkdir($usermap_thumbs, 0777);
}

function tjektype($type) {
    $tilladt = array('image/x-png', 'image/png', 'image/pjpeg', 'image/jpeg');
    if(!in_array($type, $tilladt)) {
        return false;
    }else{
        return true;
    }
}

function tjekfilnavn($filnavn) {
    $i = 0;
    $ext = strrchr($filnavn, '.');
    $navn = substr($filnavn, 0, -strlen($ext));
    while (is_file($filnavn)) {
        $i++;
        $filnavn = $navn . "_" . sprintf("%03u", $i) . $ext;
    }
    return $filnavn;
}

  $billede        = $_FILES['upfile']['name'];
    $type            = strrchr($billede, ".");// For at finde filtypen til tjek af jpeg kontra jpg
    $billede        = strtolower($billede);// ændre alt til små bogstaver
    $dest            = $usermap . $billede;// Stien til det store billede
    $dest            = tjekfilnavn($dest);
    $thumbnail_dir  = $usermap_thumbs . $billede;// Stien til det thumbnail
    $overskrift        = $_POST['kommentar'];



    $oprettelsesDato = date('Ymd');
    $source = $_FILES['upfile']['tmp_name'];
    if (is_uploaded_file($source))
        {
            if (tjektype($_FILES['upfile']['type']))
                {
                    if (move_uploaded_file($source, $dest))
                        {
                            list($width, $height) = getimagesize($dest);
                            $new_width = 100;
                            $krympefaktor = $new_width/$width;
                            $new_height = round($height * $krympefaktor);
                            $destinationImage = imagecreatetruecolor($new_width, $new_height);
                           
                            if(tjektype($_FILES['upfile']['type'] == "image/pjpeg"))
                                {
                               
                                $sourceImage = imagecreatefromjpeg($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                       
                                if ($type == ".jpg") {
                                imagejpeg($destinationImage, $usermap_thumbs . date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg");
                                $type = '.jpg';
                                $thumb = date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg";
                                }
                            else
                                {
                                imagejpeg($destinationImage, $usermap_thumbs . date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg");
                                $type = '.jpeg';
                                $thumb = date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg";
                                }
                           
                                }
                            else if(tjektype($_FILES['upfile']['type'] == "image/png"))
                                {
                                $sourceImage = imagecreatefrompng($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                                imagepng($destinationImage, $thumbnail_dir.basename($billede, ".png")."_thumb.png");
                                $type = '.png';
                                }
                   
                mysql_query("INSERT INTO images (id, user, name, thumb, text, date)
                            VALUES ('', '$userid', '$billede', '$thumb', '$overskrift', '$oprettelsesDato')") or die (mysql_error());
                mysql_close();

            echo  "<script>alert('Your picture has been uploaded');</script>";
            echo "<script>location.href='ucp.php?i=diary&mode=gallery';</script>";
               
                        }
                    else
                        {
                       
                $besked = "Der blev ikke gemt noget";
               
                }
            }
        else
            {
                $besked = "Du prøvede at gemme et billedeformat som ikke er tilladt. Du kan gemme billeder i JPEG og PNG formaterne.";
            }
           
        }
    else
        {
        mysql_query("INSERT INTO RS_Nyhed SET overskrift='$overskrift', sti='images/intetBillede.png',
                    tekst='$tekst', dato='$oprettelsesDato'");
        mysql_close();
        header("location: adminNyhed.php");
        }
}

<?php

if ($submit) {

if(is_dir($usermap)){
}else{
mkdir($usermap, 0777);
mkdir($usermap_thumbs, 0777);
}

function tjektype($type) {
    $tilladt = array('image/x-png', 'image/png', 'image/pjpeg', 'image/jpeg');
    if(!in_array($type, $tilladt)) {
        return false;
    }else{
        return true;
    }
}

function tjekfilnavn($filnavn) {
    $i = 0;
    $ext = strrchr($filnavn, '.');
    $navn = substr($filnavn, 0, -strlen($ext));
    while (is_file($filnavn)) {
        $i++;
        $filnavn = $navn . "_" . sprintf("%03u", $i) . $ext;
    }
    return $filnavn;
}

  $billede        = $_FILES['upfile']['name'];
    $type            = strrchr($billede, ".");// For at finde filtypen til tjek af jpeg kontra jpg
    $billede        = strtolower($billede);// ændre alt til små bogstaver
    $dest            = $usermap . $billede;// Stien til det store billede
    $dest            = tjekfilnavn($dest);
    $thumbnail_dir  = $usermap_thumbs . $billede;// Stien til det thumbnail
    $overskrift        = $_POST['kommentar'];


    $random = date('ymdhms');
    $oprettelsesDato = date('Ymd');
    $source = $_FILES['upfile']['tmp_name'];
    if (is_uploaded_file($source))
        {
            $newsource = $random . "_" . $billede;
            rename($source, $newsource);
            if (tjektype($_FILES['upfile']['type']))
                {
                    if (move_uploaded_file($newsource, $dest))
                        {
                           
                            list($width, $height) = getimagesize($dest);
                            $new_width = 100;
                            $krympefaktor = $new_width/$width;
                            $new_height = round($height * $krympefaktor);
                            $destinationImage = imagecreatetruecolor($new_width, $new_height);
                         
                            if(tjektype($_FILES['upfile']['type'] == "image/pjpeg"))
                                {
                             
                                $sourceImage = imagecreatefromjpeg($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                     
                                if ($type == ".jpg") {
                                imagejpeg($destinationImage, $usermap_thumbs . $random . "_" . basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg");
                                $type = '.jpg';
                                $thumb = date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpg' ) . "_thumb.jpg";
                                }
                            else
                                {
                                imagejpeg($destinationImage, $usermap_thumbs . $random . "_" . basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg");
                                $type = '.jpeg';
                                $thumb = date('ymdhms') . "_" . basename( $thumbnail_dir, '.jpeg' ) . "_thumb.jpeg";
                                }
                         
                                }
                            else if(tjektype($_FILES['upfile']['type'] == "image/png"))
                                {
                                $sourceImage = imagecreatefrompng($dest);
                                imagecopyresampled($destinationImage, $sourceImage, 0, 0, 0, 0, $new_width, $new_height, imagesx($sourceImage), imagesy($sourceImage));
                                imagejpeg($destinationImage, $usermap_thumbs . $random . "_" . basename( $thumbnail_dir, '.png' ) . "_thumb.png");
                                $type = '.png';
                                }
                 
                mysql_query("INSERT INTO images (user, name, thumb, text, date)
                            VALUES ('$userid', '$billede', '$thumb', '$overskrift', '$oprettelsesDato')") or die (mysql_error());
                mysql_close();

            echo  "<script>alert('Your picture has been uploaded');</script>";
            echo "<script>location.href='ucp.php?i=diary&mode=gallery';</script>";
             
                        }
                    else
                        {
                     
                $besked = "Der blev ikke gemt noget";
             
                }
            }
        else
            {
                $besked = "Du prøvede at gemme et billedeformat som ikke er tilladt. Du kan gemme billeder i JPEG og PNG formaterne.";
            }
         
        }
    else
        {
        mysql_query("INSERT INTO RS_Nyhed SET overskrift='$overskrift', sti='images/intetBillede.png',
                    tekst='$tekst', dato='$oprettelsesDato'");
        mysql_close();
        header("location: adminNyhed.php");
        }
}
?>



Prøv den, man vil ikke love noget, da jeg synes din kode er en smule rodet.

Smid et svar

Ved ikke om det er mig du mener, men hvis det er, så, her :)