Resizing image with same aspect ratio

[gandalf458]

I have a script that uploads and if necessary resizes an image. It works fine except that if the uploaded image is not the same aspect ratio as the source image it gets distorted. Can anyone help me work out what I need to add to change the aspect ratio please?

$image = 'img'.date('U').'.'.$extn;
$destfile = '../video-img/'.$image;
$tempfile = '../temp/'.$image;
$tempname = $_FILES['image']['tmp_name'];
$size = getimagesize($tempname);
$width  = 800;
$height = 450;
if ( ($size[0] < $width) || ($size[1] < $height) )
  echo '<p class="error">Image ', $filename, ' is too small but will be resized and uploaded.</p>', "\n";

// if image is the right size, just save it
if ( ($size[0] === $width) || ($size[1] === $height) ) {
  if ( !(is_uploaded_file($tempname) && move_uploaded_file($tempname, $destfile)) )
    echo '<p class="error">Image ', $filename, ' could not be uploaded.</p>', "\n";
}
// if image is not the right size, save it to a temporary are and then resize it
else {
  if ( !(is_uploaded_file($tempname) && move_uploaded_file($tempname, $tempfile)) )
    echo '<p class="error">Image ', $filename, ' could not be uploaded.</p>', "\n";
  $resized = imagecreatetruecolor($width, $height);

  if ( $extn == 'jpg' or $extn == 'jpeg' ) {
    $srcfile = imagecreatefromjpeg($tempfile);
    imagecopyresampled($resized, $srcfile, 0, 0, 0, 0, $width, $height, $size[0], $size[1]);
    imagejpeg($resized, $destfile, 80);
  }
  elseif ( $extn == 'png' ) {
    $srcfile = imagecreatefrompng($tempfile);
    imagecopyresampled($resized, $srcfile, 0, 0, 0, 0, $width, $height, $size[0], $size[1]);
    imagepng($resized, $destfile, 6);
  }
  imagedestroy($resized);
  imagedestroy($srcfile);
  unlink($tempfile);
}

I'm not a number, I'm a free man

 

[spamjim]

The predefined $width and $height is causing the problem. You need to calculate that instead of defining 800 and 450.

Assuming that you want the image to be 800 pixels wide, calculate what percentage of the original $size[0] would equal 800 pixels ($width). Then apply that same percentage to $size[1] to determine the proper $height.

Don't forget to round the $height to an integer.

 

[spamjim]

...and as these may be video thumbnails/previews, you may also consider cropping to the target size of 800/450. There may be fault in the rest of your project if the images are not consistently sized.

https://www.google.com/search?q=php+image+cropping

 

[gandalf458]

Thanks spamjim.

I'm not a number, I'm a free man