Не работает создание миниатюры JPEG картинки

[Bars]

Тупо ничего не выводится, даже error_reporting(E_ALL); ничего не показывает. В чём проблема?

<?php
$url = $_GET['url'];
if (strpos($url, 'scontent.cdninstagram.com') == false || strpos($url, 't51.2885-15') == false) echo 'Access Denied!';

$src = imagecreatefromjpeg($url);
$oldW = imagesx($src);
$oldH = imagesy($src);
// Даже с этим:
// $newW = $oldW, $newH = $oldH;
// $max_width = 640, $max_height = 640;
if ($oldW < $oldH) {
  $newW = $oldW * ($max_width / $oldH);
  $newH = $max_height;
} else {
  $newW = $max_width;
  $newH = $oldH * ($max_height / $oldW);
}

$new = imagecreatetruecolor($newW, $newH);
imagecopyresampled($new, $src, 0, 0, 0, 0, $newW, $newH, $oldW, $oldH);
header('Content-Type: image/jpeg');

imagejpeg($new);
imagedestroy($new);
imagedestroy($src);
?>

 

[c0dex]

ну есть еще display_errors

 

[Bars]

ну есть еще display_errors

Вот он мне вывел ошибки, я исправил, теперь даже с ним ничего не показывает. Добавил строки:
$newW = $oldW, $newH = $oldH;
$max_width = 640, $max_height = 640;

 

[WMix]

я убрал бы на время отладки header

imagedestroy($new);
imagedestroy($src);

эту педантичность
ну и возможно браузер подглючивает, размера нет заранее

 

[AnrDaemon]

Я бы вообще на месте ТС взял IMagick и не морочился.
После того, как я нарыл по нему нормальную доументацию, я кучу кода на помойку отправил.

 

[weregod]

@AnrDaemon, IMagik не везде установлен, мало ли для чего там ТС ваяет

 

[AnrDaemon]

Если такие задачи возникают постоянно - его тупо надо взять и тупо установить.
Вся вон та портянка элементарно помещается в три строчки.

        $img = new Imagick(GALLERY_BASE_DIR."/{$fname}");
        $img->thumbnailImage(GALLERY_PREVIEW_X, GALLERY_PREVIEW_Y, true);
        $img->writeImage(GALLERY_BASE_DIR."/.preview/{$fname}");

 

[WMix]

@AnrDaemon предлагаю свободу выбора

WideImage::load(GALLERY_BASE_DIR."/{$fname}")
  ->resize(GALLERY_PREVIEW_X, GALLERY_PREVIEW_Y)
  ->saveToFile(GALLERY_BASE_DIR."/.preview/{$fname}");

 

[Bars]

Всё решено. Рабочий оконечный вариант:

<?php
$url = $_GET['url'];
if (strpos($url, 'scontent.cdninstagram.com') == false
 || strpos($url, 't51.2885-15') == false)
   die('Сторонняя картинка! Обрезать не буду.');

$src = imagecreatefromjpeg($url);
$oldW = imagesx($src);
$oldH = imagesy($src);
if ($oldW == $oldH) {
  die('Размеры изображения совпадают. Обрезать нечего!');
} else if ($oldW < $oldH) {
  $pad = round(($oldH - $oldW) / 2);
  $img = ImageCreateTrueColor($oldW, $oldW);
  imagecopy($img, $src, 0, 0, 0, $pad, $oldW, $oldH);
} else {
  $pad = round(($oldW - $oldH) / 2);
  $img = ImageCreateTrueColor($oldH, $oldH);
  imagecopy($img, $src, 0, 0, $pad, 0, $oldW, $oldH);
}

header('Content-Type: image/jpeg');
imagejpeg($img);
imagedestroy($img);
imagedestroy($src);
?>

Строки 3-6 можно смело удалять, это для себя.

 

[grigori]

а если в адрес сторонней картинки добавят &scontent.cdninstagram.com ?
а что если я дам адрес очень большой картинки, и сервер уйдет в своп?

 

[weregod]

и сервер уйдет в своп?

вроде процесс по памяти тупо треснуть должен и всё, не?

 

[grigori]

да я не знаю, выделит gd память перед тем, как ze определит превышение, или нет
если на сервере пара свободных гигабайт и 10 процессов с дефолтным лимитом - ничего не случится, но разрешение изображения надо проверять
http://www.unforgettable.dk/42.zip