复制后图像损坏

Question

我将尝试将远程服务器上的图片复制到我的WordPress网站中作为缩略图使用。但是在复制过程中，有些图片会变得损坏。

以下是我的代码：

$url = 'http://media.cultserv.ru/i/1000x1000/'.$event->subevents[0]->image;
$timeout_seconds = 100;
$temp_file = download_url( $url, $timeout_seconds );

if(!is_wp_error( $temp_file )) {
  $file = array(
    'name' => basename($url),
    'type' => wp_check_filetype(basename($url), null),
    'tmp_name' => $temp_file,
    'error' => 0,
    'size' => filesize($temp_file),
  );
  $overrides = array(
    'test_form' => false,
    'test_size' => true,
    'test_upload' => true,
  );
  $results = wp_handle_sideload( $file, $overrides );
  if(empty($results['error'])) {
    $filename = $results['file'];
    $local_url = $results['url'];
    $type = $results['type'];
    $attachment = array(
      'post_mime_type' => $results['type'],
      'post_title' => preg_replace('/.[^.]+$/', '', basename( $results['file'] ) ),
      'post_content' => '',
      'post_status' => 'inherit',
      'post_type' => 'attachment',
      'post_parent' => $pID,
    );
    $attachment_id = wp_insert_attachment( $attachment, $filename );
    if($attachment_id) {
      set_post_thumbnail( $pID, $attachment_id );
    }
  }
}

这是一张截图，左边是原始图片，右边是我服务器上的复制品：

- Dmitry Kapusta

---

尝试在调用set_post_thumbnail之前使用$attachData = wp_generate_attachment_metadata($attachment_id, $filename);和wp_update_attachment_metadata($attach_id, $attachData);，看看结果是否会改善。同时确保在脚本的某个地方使用require_once( ABSPATH . 'wp-admin/includes/image.php' );。 - Will B.

问题在于，通过存储在$local_url中的url访问的图像已经损坏。这是在附件创建之前发生的。 - Dmitry Kapusta

Answer 1

我认为你的download_url( $url, $timeout_seconds )函数无法正常工作（您无法捕获网络/其他错误，这就是为什么您有损坏的图像），而且我认为下载URL不需要超时参数...。

为了解决这个问题，最好将此函数重写为以下内容：

function download_url($url)
{
    $saveto = 'temp.jpg'; // generate temp file
    $ch = curl_init($url);
    curl_setopt($ch, CURLOPT_HEADER, 0);
    curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
    curl_setopt($ch, CURLOPT_BINARYTRANSFER, 1);
    $raw = curl_exec($ch);
    if (curl_errno($ch)) {
        curl_close($ch);
        return false;
        // you probably have a network problem here.
        // you need to handle it, for example retry or skip and reqeue the image url
    }
    curl_close($ch);
    if (file_exists($saveto)) {
        unlink($saveto);
    }
    $fp = fopen($saveto, 'x');
    fwrite($fp, $raw);
    fclose($fp);
    return $saveto;
}

- mrbm