DIY 裁减功能中的BUG

裁图中  出现问题 提示 imagepreview_errorcode_1 错误


module/misc/misc_imgcropper.php中

1.         $picwidth = $cutwidth > $_GET['picwidth'] ? $cutwidth : $_GET['picwidth'];
   
2.         $picheight = $cutheight > $_GET['picheight'] ? $cutheight : $_GET['picheight'];

复制代码

$picwidth，$picheight  可以大于 图片高宽


source/class/class_image.php中

1.         function Thumb_GD() {
   
2.                 if(!function_exists('imagecreatetruecolor') || !function_exists('imagecopyresampled') || !function_exists('imagejpeg') || !function_exists('imagecopymerge')) {
   
3.                         return -4;
   
4.                 }
   
5. 
   
6.                 $imagefunc = &$this->imagefunc;
   
7.                 $attach_photo = $this->loadsource();
   
8.                 if($attach_photo < 0) {
   
9.                         return $attach_photo;
   
10.                 }
    
11.                 $copy_photo = imagecreatetruecolor($this->imginfo['width'], $this->imginfo['height']);
    
12.                 $bg = imagecolorallocate($copy_photo, 255, 255, 255);
    
13.                 imagefill($copy_photo, 0, 0, $bg);
    
14.                 imagecopy($copy_photo, $attach_photo ,0, 0, 0, 0, $this->imginfo['width'], $this->imginfo['height']);
    
15.                 $attach_photo = $copy_photo;
    
16. 
    
17.                 switch($this->param['thumbtype']) {
    
18.                         case 'fixnone':
    
19.                         case 1:
    
20. //*******************************************
    
21. // $picwidth，$picheight  可以大于 图片高宽
    
22. // 及 $this->param['thumbwidth'] 》$this->imginfo['width']  
    
23. // 所以出现了BUG   下面这段代码不会运行
    
24. //*******************************************
    
25.                                 if($this->imginfo['width'] > $this->param['thumbwidth'] || $this->imginfo['height'] > $this->param['thumbheight']) {
    
26.                                         $thumb = array();
    
27.                                         list(,,$thumb['width'], $thumb['height']) = $this->sizevalue(0);
    
28.                                         $cx = $this->imginfo['width'];
    
29.                                         $cy = $this->imginfo['height'];
    
30.                                         $thumb_photo = imagecreatetruecolor($thumb['width'], $thumb['height']);
    
31.                                         imagecopyresampled($thumb_photo, $attach_photo ,0, 0, 0, 0, $thumb['width'], $thumb['height'], $cx, $cy);
    
32.                                 }
    
33.                                 break;
    
34.                         case 'fixwr':
    
35.                         case 2:
    
36.                                 if(!($this->imginfo['width'] <= $this->param['thumbwidth'] || $this->imginfo['height'] <= $this->param['thumbheight'])) {
    
37.                                         list($startx, $starty, $cutw, $cuth) = $this->sizevalue(1);
    
38.                                         $dst_photo = imagecreatetruecolor($cutw, $cuth);
    
39.                                         imagecopymerge($dst_photo, $attach_photo, 0, 0, $startx, $starty, $cutw, $cuth, 100);
    
40.                                         $thumb_photo = imagecreatetruecolor($this->param['thumbwidth'], $this->param['thumbheight']);
    
41.                                         imagecopyresampled($thumb_photo, $dst_photo ,0, 0, 0, 0, $this->param['thumbwidth'], $this->param['thumbheight'], $cutw, $cuth);
    
42.                                 } else {
    
43.                                         $thumb_photo = imagecreatetruecolor($this->param['thumbwidth'], $this->param['thumbheight']);
    
44.                                         $bgcolor = imagecolorallocate($thumb_photo, 255, 255, 255);
    
45.                                         imagefill($thumb_photo, 0, 0, $bgcolor);
    
46.                                         $startx = ($this->param['thumbwidth'] - $this->imginfo['width']) / 2;
    
47.                                         $starty = ($this->param['thumbheight'] - $this->imginfo['height']) / 2;
    
48.                                         imagecopymerge($thumb_photo, $attach_photo, $startx, $starty, 0, 0, $this->imginfo['width'], $this->imginfo['height'], 100);
    
49.                                 }
    
50.                                 break;
    
51. //*******************************************
    
52. //自己加了一个 拿来临时解决此类BUG。不过也不完善。
    
53.                         case 3:
    
54.                                 $this->param['thumbheight'] = $this->imginfo['height'] * ($this->param['thumbwidth'] / $this->imginfo['width']);
    
55.                                 $thumb = array();
    
56.                                 list(,, $thumb['width'], $thumb['height']) = $this->sizevalue(0);
    
57.                                 $cx = $this->imginfo['width'];
    
58.                                 $cy = $this->imginfo['height'];
    
59.                                 $thumb_photo = imagecreatetruecolor($thumb['width'], $thumb['height']);
    
60.                                 $bgcolor = imagecolorallocate($thumb_photo, 255, 255, 255);
    
61.                                 imagefill($thumb_photo, 0, 0, $bgcolor);
    
62.                                 imagecopyresampled($thumb_photo, $attach_photo, 0, 0, 0, 0, $thumb['width'], $thumb['height'], $cx, $cy);
    
63.                                 break;
    
64. //*******************************************
    
65.                 }
    
66.                 clearstatcache();
    
67.                 if($this->imginfo['mime'] == 'image/jpeg') {
    
68.                         @$imagefunc($thumb_photo, $this->target, $this->param['thumbquality']);
    
69.                 } else {
    
70.                         @$imagefunc($thumb_photo, $this->target);
    
71.                 }
    
72.                 return 1;
    
73.         }

复制代码

希望官方早日解决   

请问如何重现 裁图中  出现问题 提示 imagepreview_errorcode_1 错误  。

请告之详细重现步骤。

m.king 发表于 2012-7-19 17:42
请问如何重现 裁图中  出现问题 提示 imagepreview_errorcode_1 错误  。

请告之详细重现步骤。

比如  我的diy模块限制的图片大小是 300x300  实际图片是 400X600
我把这个图拉大  成为800X1200的时候 好像就会这样！
还有一种情况是  diy模块限制的图片大小是 300x300  实际图片 200X200
也会出现问题！  

m.king 发表于 2012-7-19 17:42
请问如何重现 裁图中  出现问题 提示 imagepreview_errorcode_1 错误  。

请告之详细重现步骤。

其实看代码很明显就知道     这个逻辑会冲突

在diy 裁图 是先 生成与缩放大小一致的 图片 然后再根据位置进行裁剪

而在原有的Thumb_GD中  如果 缩放的大小 超过原图 则不会执行相关代码了  这样就会出现问题了

if($this->imginfo['width'] > $this->param['thumbwidth'] || $this->imginfo['height'] > $this->param['thumbheight']) {


thumbwidth 是 等于 传参过来的 width
但是thumbheight 是等于 原图的等高
这个判断中就根本不会执行了

yovei 发表于 2012-7-19 21:40
其实看代码很明显就知道     这个逻辑会冲突

在diy 裁图 是先 生成与缩放大小一致的 图片 然后再根据位 ...

该问题我们再看看。

yovei 发表于 2012-7-19 21:40
其实看代码很明显就知道     这个逻辑会冲突

在diy 裁图 是先 生成与缩放大小一致的 图片 然后再根据位 ...

该问题我们再看看。

m.king 发表于 2012-7-20 15:47
该问题我们再看看。

DIY里面 图片限制267x437的宽高
图片使用这个  

m.king 发表于 2012-7-20 15:47
该问题我们再看看。

或者可以QQ CALL 我下 我演示给你看！  

yovei 发表于 2012-7-20 16:34
或者可以QQ CALL 我下 我演示给你看！  281762391

等会再看看吧。我现在忙其他。