imagecopyresize woes

fandelem

i've looked over the manual, but i just don't understand one (important) detail..

how do i make my $imagefile (for example, image1.jpg) a "valid Image resource" ??

here are my error msgs:

Warning: Supplied argument is not a valid Image resource in /home/kdavis/www/images/functions.inc on line 969

Warning: Supplied argument is not a valid Image resource in /home/kdavis/www/images/functions.inc on line 976

and here is my function:

function resizeimage($new_dir, $imagefile) {
// precondition: $imagefile must contain full path such as "/home/kdavis/www/images/image1.jpg"

     chdir($new_dir);
     $new_w = "75";
     $new_h = "75";
     $src_img = getimagesize($imagefile);
     $src_imagewidth = $src_img[0];
     $src_imageheight = $src_img[1];
     $dst_img = imagecreate($new_w,$new_h);
     imagecopyresized($dst_img,$imagefile,0,0,0,0,$new_w,$new_h,imagesx($src_imagewidth),imagesy($src_imageheight));
     chdir("thumbnails/");
     if ( file_exists("thumb_" . $imagefile) ) {
          unlink("thumb_" . $imagefile);
     }
     imagejpeg($dst_img,"thumb_$imagefile");

     ImageDestroy($src_img);
     ImageDestroy($dst_img);

}

thanks 🙂

[deleted]

Imagecopyresized() does not take a filename as argument.

Instead, you must first open the image using functions like "imagecreatefromjpg",
then use the resulting imagepointer in imagecopyresize.

fandelem

right, okay i kind of figured that.

however, i still am stumped to why this isn't working.. i have gd 1.6 or higher.. here the newest attempt:

function resizeimage($new_dir, $imagefile) {

     chdir($new_dir);
     $new_w = "75"; // resize width
     $new_h = "75"; // resize height
     $src_img = getimagesize($imagefile);
     $src_imagewidth = $src_img[0];
     $src_imageheight = $src_img[1];

     $src_img = @ImageCreateFromJPEG ($imagefile); /* Attempt to open */
        if (!$src_img) { /* See if it failed */
            $src_img = @ImageCreateFromGIF ($imgname); /* Attempt to open */
            if (!$src_img) { /* See if it failed */
                $src_img = @ImageCreateFromPNG ($imgname); /* Attempt to open */
                 if (!$src_img) { /* See if it failed */
                    $imageinfo = GetImagesize($imagefile);
                    if ( $imageInfo[2] == 1 ) {
                        //it's a gif
                        $fp = fopen($new_dir, "r");
                        $src_img = fread($fp,$imagefile, filesize($imagefile));
                        $src_img = imagecreate($src_img);
                        echo "all file types failed.";
                     }
                 }
             }
         }


     $dst_img = imagecreate($new_w,$new_h); // create thubmnail size image
     imagecopyresized($dst_img,$src_img,0,0,0,0,$new_w,$new_h,imagesx($src_imagewidth),imagesy($src_imageheight));
     chdir("thumbnails/");
     if ( file_exists("thumb_" . $imagefile) ) {
          unlink("thumb_" . $imagefile);
     }
     imagejpeg($dst_img,"thumb_$imagefile");

     ImageDestroy($src_img);
     ImageDestroy($dst_img);

}

what am i doing wrong? the same errors are appearing (not a valid Image resource)

thanks,

kyle

[deleted]

Well for one thing, you should stop using that "@" sign in front of every command.
It surpresses the very error message you are looking for.
Many people have lost several hours of sleep because of that "@" sign. :-)

And second, your IF statement is completely zany (pardon my french).

I guess what you are trying to do is open the image with the various imagecreatefromXXX functions to see if which one is successful.
But, your order is strange to say the least. 🙂

Look carefully at the IF structure, the image is only loaded if it's a NOT a JPG, NOT a GIF and NOT a PNG image. Uhm.. you just ran out of image types 🙂

Try looking at the filename to get the extention.
That will tell you that it's a GIF/JPG/PNG and you can call the correct function immediately instead of trying them all.

Anon

Change:
imagecopyresized($dst_img,$src_img,0,0,0,0,$new_w,$new_h,imagesx($src_imagewidth),imagesy($src_imageheight));

For:
imagecopyresized($dst_img,$src_img,0,0,0,0,$new_w,$new_h,$src_imagewidth,$src_imageheight);