imagecopyresampled og imagejpeg?

Du kunne også tage et kig på min artikel som netop samme emne:
Reize: http://www.eksperten.dk/artikler/1130
Og simpel upload: http://www.eksperten.dk/artikler/1098

Din $_FILES["image"]["tmp_name"] er IKKE en Image resource.
Du skal, ud fra hvilken type dit billede er, lave en image resource med funktionerne imagecreatefromgif, imagecreatefromjpeg eller imagecreatefrompng.

@ dkfire - Det virkede delvist.. Nu får jeg et sort billede. Jeg har prøvet at lave en fputs til at skrive informationerne til billedet, men det ser ikke ud til at virke.

Har du et forslag her også?

Hvad er der galt i at bruge mine 2 artikler?
De er gennemtestet og virker. :o)

Du tager også og ligger det forkerte billede ind i din database:
INSERT INTO .... `thumb` = '" . $thumb . "', ....
kan jo sætte et tomt billede ind.
Tror du skal bruge lade være med at bruge variabel navne flere forskellige gange.
Prøv at skrive din $image_type[1] ud for at se om den giver det rigtige til dine if sætninger.

jakobdo - Jeg har brugt lidt fra dine artikler, men dog ikke meget, da jeg står i en situation hvor jeg helst ikke vil gemme nogle filer på webserveren, men i stedet vil skrive det hele til databasen med det samme :) - og desuden virker resten af mit upload script jo udemærket, det er udelukkende resize jeg har problemer med :)

dkfire - Jeg har haft udskrevet samtlige variabler, og lige netop $image_type[1] skulle jo meget gerne være den samme hele vejen igennem scriptet da det thumbnail jeg vil generere skal være samme filformat som det uploadede billede er fra starten af :)

Det er muligt at se det i funktion her:

http://jordegern.dk.linux10.unoeuro.com

Så er spørgsmålet om hvordan du viser dit lille billede, samt om den går ind i den rigtige if sætning om laver et billede.

Jeg viser det lille billede på fuldstændig samme måde som jeg viser det store, så det ved jeg virker. - Poster lige den friske kode til jer :)

function upload_image() {
       
        if (isset($_POST["title"]) && $_FILES['image']['size'] > 0) {
            $file_name = $_FILES['image']['name'];
            $tmp_name  = $_FILES['image']['tmp_name'];
            $file_size = $_FILES['image']['size'];
            $file_type = $_FILES['image']['type'];
            $thumb_name = "thumb_" . $file_name;
           
            $fp      = fopen($tmp_name, 'r');
            $content = fread($fp, filesize($tmp_name));
            $content = addslashes($content);
            fclose($fp);   
               
            $image_type = explode("/", $file_type);   
           
            $image_dimensions = getimagesize($tmp_name);
            $width = $image_dimensions[0];
            $height = $image_dimensions[1];
           
               
            if ($file_size > 2097152) {
                echo "Billedet er større end 2 Mb, og kan derfor ikke uploades.";
            } elseif ($image_type[1] !== "jpeg" && $image_type[1] !== "png" && $image_type[1] !== "gif") {
                echo "Billedet er af typen " . $image_type[1] . ", og kan derfor ikke uploades. Formatet skal være JPEG, JPG, GIF eller PNG";
            } else {
           
               
                //Image resize
                //Target size in pixel                                               
                $target = 200; //px                                                       

                    if ($width > $height) {                                            
                    $percentage = ($target / $width);                                    
                    } else {                                                            
                    $percentage = ($target / $height);                                    
                    }                                                                    
                //gets the new value and applies the percentage, then rounds the value    
                    $thumb_width = round($width * $percentage);                        
                    $thumb_height = round($height * $percentage);                        
               
                //Create thumbnail
                switch($image_type[1])
                  {
                    case 'jpeg':
                    case 'jpe':
                    case 'jpg':
                      $srcim = imagecreatefromjpeg( $filename );
                      break;
                    case 'gif':
                      $srcim = imagecreatefromgif( $filename );
                      break;
                    case 'png':
                      $srcim = imagecreatefrompng( $filename );
                      break;
                    default:
                      return false;
                  }                                   
                $thumb = imagecreatetruecolor($thumb_width, $thumb_height);
                $fput_data = @imagecopyresampled ($thumb, $srcim, 0, 0, 0, 0, $thumb_width, $thumb_height, $width, $height);
                //imagecopyresampled(resource dst_image, resource src_image, int dst_x, int dst_y, int src_x, int src_y, int dst_w, int dst_h, int src_w, int src_h)
               
                switch($image_type[1])
                {
                  case 'jpeg':
                  case 'jpe':
                  case 'jpg':
                    imagejpeg( $thumb, $thumb_name, 100 );
                    break;
                  case 'gif':
                    imagegif( $thumb, $thumb_name );
                    break;
                  case 'png':
                    $png_q = floor( abs( 100 / 10 - 9.9 ) );
                    imagepng( $thumb, $thumb_name, $png_q );
                      break;
                  default:
                    return false;
                }
               
                $fp      = fopen($thumb_name, 'a+');
                $fput     = fwrite($fp, $fput_data);
                $thumb     = fread($fp, filesize($thumb_name));
                $thumb     = addslashes($thumb);
                fclose($fp);               
               
                //echo "Content: " . $content;
                //echo "<br />Thumb: " . $thumb . "<br />";
                   
                $image_sql = "INSERT INTO `" . _MYSQL_GALLERY . "` SET `thumb_width` = '" . $thumb_width . "', `thumb_height` = '" . $thumb_height . "', `thumb` = '" . $thumb . "', `width` = '" . $width . "', `height` = '" . $height . "', `image` = '" . $content . "', `title` = '" . mysql_real_escape_string($_POST["title"]) . "', `description` = '" . mysql_real_escape_string($_POST["description"]) . "', `user_id` = '1', `type` = '" . $file_type . "'";
                $image_query = mysql_query($image_sql) or die(mysql_error());
               
                    echo "Billede successfuldt uploadet.<br /><br /><a href=\"?show=all\">Galleriforside</a><br /><a href=\"?user=1\">Dit galleri</a><br />";
               
            }
        }
}
?>

Ingen der har noget?

Hvad giver den af fejl ??
Hvad sker der hvis du fjerner @ ved @imagecopyresampled() ?

Indeholder thumb i databasen noget data ??

Jeg kunne godt tænke mig at se hvordan du har lavet show_thumb.php.

Men der er en stor fejl du laver i den kode:
$fp      = fopen($thumb_name, 'a+');
                $fput    = fwrite($fp, $fput_data);
                $thumb    = fread($fp, filesize($thumb_name));
                $thumb    = addslashes($thumb);
                fclose($fp); 

Først åbner du en fil du allerede har lavet med imagejpeg, imagegif eller imagepng.
Dernæst forsøøger du at ligge ukomprimeret billeddata ned i filen.
Derefter læser du den samme data og tilføjer addslashes.
Det giver slet ingen mening at bruge ukompromeret billeddata, det kan du jo ikke bruge til at vise billeder med.
Du skal istedet bruge imagejpeg, imagegif eller imagepng til at lave den billeddata som du vil ligge i databasen.

Hvis jeg fjerner @ giver den denne fejl:

Warning: imagecopyresampled(): supplied argument is not a valid Image resource in /var/www/jordegern.dk/public_html/inc/functions.php on line 119, som jo egentlig nok er roden til alt ondt i mit tilfælde..

linie 119 ser således ud:

imagecopyresampled ($thumb, $srcim, 0, 0, 0, 0, $thumb_width, $thumb_height, $width, $height);


show_thumb.php ser således ud:

<?php
    require_once("inc/config.php");
    $sql = "SELECT `thumb`,`type` FROM `" . _MYSQL_GALLERY . "` WHERE `id` = '" . $_GET["id"] . "'";
    $do = mysql_query($sql);       
        if(mysql_num_rows($do) == 1) {
        $img = mysql_fetch_assoc($do);   
        $type = $img['type'];
        $data = $img['thumb'];       
        header("Content-type: $type");
        echo $data;
    } else {
        echo "Filen findes ikke!";
    }
?>

Jeg har nu fjernet den store fejl du påpegede - Smed det egentlig ind fordi jeg vidste det virkede længere oppe i mit script, men der er det selvfølgelig heller ikke rå billeddata.

Efter jeg har lavet de tilretninger du sagde jeg skulle lave, laver den ikke længere et thumbnail overhovedet..

Lige en hurtig rettelse:

Din variabel hedder ikke $filename men $file_name
Ret:
switch($image_type[1])
                  {
                    case 'jpeg':
                    case 'jpe':
                    case 'jpg':
                      $srcim = imagecreatefromjpeg( $filename );
                      break;
                    case 'gif':
                      $srcim = imagecreatefromgif( $filename );
                      break;
                    case 'png':
                      $srcim = imagecreatefrompng( $filename );
                      break;
                    default:
                      return false;
                  }
Til:
switch($image_type[1])
                  {
                    case 'jpeg':
                    case 'jpe':
                    case 'jpg':
                      $srcim = imagecreatefromjpeg( $file_name );
                      break;
                    case 'gif':
                      $srcim = imagecreatefromgif( $file_name );
                      break;
                    case 'png':
                      $srcim = imagecreatefrompng( $file_name );
                      break;
                    default:
                      return false;
                  }

Udover det, har du så husket at ligge resultatet af imagejpeg og de andre i en variabel, i stedet for en fil ??

Rettet ;)

Ja, den ligger i en variabel der hedder $final_thumb :)

Efter jeg har rettet dette til giver den lidt flere fejl:

Warning: imagecreatefromjpeg() [function.imagecreatefromjpeg]: Unable to access 2008_01_24_cat_not_drunk.jpg in /var/www/jordegern.dk/public_html/inc/functions.php on line 107

Warning: imagecreatefromjpeg(2008_01_24_cat_not_drunk.jpg) [function.imagecreatefromjpeg]: failed to open stream: No such file or directory in /var/www/jordegern.dk/public_html/inc/functions.php on line 107

Warning: imagecopyresampled(): supplied argument is not a valid Image resource in /var/www/jordegern.dk/public_html/inc/functions.php on line 119


Linie 107 og omliggende:

switch($image_type[1])
                  {
                    case 'jpeg':
                    case 'jpe':
                    case 'jpg':
                      $srcim = imagecreatefromjpeg( $file_name ); // LINE 107
                      break;
                    case 'gif':
                      $srcim = imagecreatefromgif( $file_name );
                      break;
                    case 'png':
                      $srcim = imagecreatefrompng( $file_name );
                      break;
                    default:
                      return false;
                  }

Ja se det skyldes så at den ikke kan finde filen med navnet fra $file_name.
Men tror du ikke også at du skal bruge $tmp_name i stedet for $file_name.

Så giver den ikke fejl længere.. Til gengæld virker det stadig ikke med thumbnailsne.. (Lidt pinligt jeg ikke selv opdagede min variabelfejl :/ )

Så fik jeg det til at virke. Jeg opdagede at den alligevel smed en fil på webserveren, så jeg lavede en fopen på den, læste indholdet, og smed det ind i databasen på den måde, og sletter det så igen med unlink bagefter :)

Smid et svar og du skal få points :)

og tusind tak for hjælpen forresten :)

Scriptet endte med at se sådan ud:

    function upload_image() {
       
        if (isset($_POST["title"]) && $_FILES['image']['size'] > 0) {
            $file_name = $_FILES['image']['name'];
            $tmp_name  = $_FILES['image']['tmp_name'];
            $file_size = $_FILES['image']['size'];
            $file_type = $_FILES['image']['type'];
            $thumb_name = "thumb_" . $file_name;
           
            $fp      = fopen($tmp_name, 'r');
            $content = fread($fp, filesize($tmp_name));
            $content = addslashes($content);
            fclose($fp);   
               
            $image_type = explode("/", $file_type);   
           
            $image_dimensions = getimagesize($tmp_name);
            $width = $image_dimensions[0];
            $height = $image_dimensions[1];
           
               
            if ($file_size > 2097152) {
                echo "Billedet er større end 2 Mb, og kan derfor ikke uploades.";
            } elseif ($image_type[1] !== "jpeg" && $image_type[1] !== "png" && $image_type[1] !== "gif") {
                echo "Billedet er af typen " . $image_type[1] . ", og kan derfor ikke uploades. Formatet skal være JPEG, JPG, GIF eller PNG";
            } else {
           
               
                //Image resize
                //Target size in pixel                                               
                $target = 200; //px                                                       

                    if ($width > $height) {                                            
                    $percentage = ($target / $width);                                    
                    } else {                                                            
                    $percentage = ($target / $height);                                    
                    }                                                                    
                //gets the new value and applies the percentage, then rounds the value    
                    $thumb_width = round($width * $percentage);                        
                    $thumb_height = round($height * $percentage);                        
               
                //Create thumbnail
                switch($image_type[1])
                  {
                    case 'jpeg':
                    case 'jpe':
                    case 'jpg':
                      $srcim = imagecreatefromjpeg( $tmp_name );
                      break;
                    case 'gif':
                      $srcim = imagecreatefromgif( $tmp_name );
                      break;
                    case 'png':
                      $srcim = imagecreatefrompng( $tmp_name );
                      break;
                    default:
                      return false;
                  }                                   
                $thumb = imagecreatetruecolor($thumb_width, $thumb_height);
                imagecopyresampled ($thumb, $srcim, 0, 0, 0, 0, $thumb_width, $thumb_height, $width, $height);
                //imagecopyresampled(resource dst_image, resource src_image, int dst_x, int dst_y, int src_x, int src_y, int dst_w, int dst_h, int src_w, int src_h)
               
                switch($image_type[1])
                {
                  case 'jpeg':
                  case 'jpe':
                  case 'jpg':
                    imagejpeg( $thumb, $thumb_name, 100 );
                    break;
                  case 'gif':
                    imagegif( $thumb, $thumb_name );
                    break;
                  case 'png':
                    $png_q = floor( abs( 100 / 10 - 9.9 ) );
                    imagepng( $thumb, $thumb_name, $png_q );
                      break;
                  default:
                    return false;
                }
               
                $fp2      = fopen($thumb_name, 'r');
                $thumb = fread($fp2, filesize($thumb_name));
                $thumb = addslashes($thumb);
                fclose($fp2);
               
                unlink($thumb_name);
               
                //echo "Content: " . $content;
                //echo "<br />Thumb: " . $thumb . "<br />";
                   
                $image_sql = "INSERT INTO `" . _MYSQL_GALLERY . "` SET `thumb_width` = '" . $thumb_width . "', `thumb_height` = '" . $thumb_height . "', `thumb` = '" . $thumb . "', `width` = '" . $width . "', `height` = '" . $height . "', `image` = '" . $content . "', `title` = '" . mysql_real_escape_string($_POST["title"]) . "', `description` = '" . mysql_real_escape_string($_POST["description"]) . "', `user_id` = '1', `type` = '" . $file_type . "'";
                $image_query = mysql_query($image_sql) or die(mysql_error());
               
                    echo "Billede successfuldt uploadet.<br /><br /><a href=\"?show=all\">Galleriforside</a><br /><a href=\"?user=1\">Dit galleri</a><br />";
               
            }
        }
}

Jamen jeg ligger et svar