插件Watermark Reloaded上传非图片文件出错修改打赏

今天在检查博客附件的时候发现有以前上传的一个注册表导入脚本reg文件丢了,重新上传的时候居然报错了。仔细看了才发现是之前安装的Watermark_Reloaded插件导致的错误,没多想,打开插件源代码看看原因,因为小博不是专注PHP的(不过今天发现PHP的语法跟Java很是相似啊),所以没那么快找到原因,后来分析可能是因为上传了非图片文件,而Watermark Reloaded又针对所以上传的附件进行检查并加水印,所以会导致程序出错。

错误提示



知道原因解决起来就很简单了,通篇看下来,发现错误的根源是出在doWatermark这个函数里面,这里在对附件进行加水印的操作前根本没有判断是不是图片附件,也许国外人就只上传图片附件,不传其他的吧,这里也不再深究了。

修改办法也很简单,函数默认获取了文件的mime_type,函数如下

public function doWatermark($filepath) {
$mime_type = wp_check_filetype($filepath);
$mime_type = $mime_type['type'];
$options = $this->get_options();
$image = $this->getImageResource($filepath, $mime_type);
$this->imageAddText($image, $options);
return $this->saveImageFile($image, $mime_type, $filepath);
}

文件类型的具体返回值不大清楚,于是在$mime_type = $mime_type['type'];这一句获取文件类型后面打印了下,发现是image\jpeg2822之类的,这样就好办了,图片文件肯定都包含image字样,于是我就在$options = $this->get_options();前面加入if判断语句,小博是很久没碰php了,还只是停留在基础的了解阶段,因此对php函数不是很熟悉,查了w3c的手册,一开始用了substr截取mime前五个字符与字符串image进行比较,发现不能正确加水印,可能mime返回值不是字符串或者有空格什么的,总之和image不是相等的,无奈这里就直接不用这个了,换个思路,检查mime里面是否包含image,依然查手册,最后使用了strstr函数,修改函数如下

public function doWatermark($filepath) {
$mime_type = wp_check_filetype($filepath);
$mime_type = $mime_type['type'];
if(strstr($mime_type,"image")){
$options = $this->get_options();
$image = $this->getImageResource($filepath, $mime_type);
$this->imageAddText($image, $options);
return $this->saveImageFile($image, $mime_type, $filepath);
}
}

为了便于读者查看,我将原程序复制过来删除了作者的注释,这里其实只用添加一句,在“$options = $this->get_options();”这句前面加入“if(strstr($mime_type,"image")){”,在最后再多加一个“}”即可。

其实插件已经很火了,出现这个错误也不是致命的,虽然报错但传文件不受影响,不知道是因为其他网友都不传图片文件之外的文件,还是怎么,在百度、谷歌无果后只有自己修改了,小小经验,希望大家批评指教!

插件Watermark Reloaded上传非图片文件出错修改
文章《插件Watermark Reloaded上传非图片文件出错修改》二维码
  • 微信打赏
  • 支付宝打赏

已有13条评论

  1. 淘宝特价

    来过了,欢迎回访哦。。

    2012-03-20 22:20 回复
  2. 小说网

    博主更新不给力啊..事太多吧…有时间给我的文学小说网http://www.wenx.net加加人气…

    2012-02-27 22:15 回复
  3. heui

    我发现问题了都没想过去解决,博主钻研精神值得我学习啊

    2012-02-27 20:47 回复
  4. 软件酷的Tmd

    来访了,支持下博主

    2012-02-27 17:28 回复
  5. 蕾丝连衣裙

    基本上都是传图片!

    2012-02-27 16:14 回复
    • 穷小子

      看来大家需求都是一样啊,只是我偶尔会传些其他文件,大的附件直接网盘了,呵呵

      2012-02-27 21:10 回复
  6. 蕾丝连衣裙

    这个还真的非常的强大

    2012-02-27 11:43 回复
  7. 柠檬绿茶

    博主好~ 谢谢分享。

    2012-02-27 10:50 回复
  8. 武林之人

    哈哈,不错不错,总算找到了!

    2012-02-26 23:01 回复
  9. Tmd

    本地制作,传图床

    2012-02-26 22:31 回复

(必填)

(必填)

(可选)