开启Memcache缓存后后台社区QQ群搜索不到帖子的问题排查

问题描述:

此问题发现于Discuz! X2.5版本。
开启了Memcache缓存,后台社区QQ群推送消息页面按tid搜索不到帖子或搜索出来的帖子不对,如图。
开启Memcache缓存后后台社区QQ群按tid搜索帖子出现其他帖子

问题分析:

找到source\admincp\cloud\cloud_qqgroup.php文件,有如下代码:

if($srchtid) {
	$threads = C::t('forum_thread')->fetch_all_by_tid_displayorder($srchtid, 0);
}

这里会调用fetch_all_by_tid_displayorder方法,找到source\class\table\table_forum_thread.php文件,有如下代码:

public function fetch_all_by_tid_displayorder($tids, $displayorder = null, $glue = '>=', $fids = array(), $closed = null) {
    $data = array();
    if(!empty($tids)) {
        $data = $this->fetch_all_by_tid($tids);
        $fids = $fids && !is_array($fids) ? array($fids) : $fids;
        foreach($data as $tid => $value) {
            if($displayorder !== null && !(helper_util::compute($value['displayorder'], $displayorder, $glue))) {
                unset($data[$tid]);
            } elseif(!empty($fids) && !in_array($value['fid'], $fids)) {
                unset($data[$tid]);
            } elseif($closed !== null && $value['closed'] != $closed) {
                unset($data[$tid]);
            }
        }
    }
    return $data;
}

注意这个函数的参数:$tids应该为数组。这里调用了fetch_all_by_tid方法,还是这个文件,有如下代码:

public function fetch_all_by_tid($tids, $start = 0, $limit = 0, $tableid = 0) {
    $data = array();
    if(($data = $this->fetch_cache($tids)) === false || count($tids) != count($data)) {
        if(is_array($data) && !empty($data)) {
            $tids = array_diff($tids, array_keys($data));
        }
        if($data === false) $data = array();
        if(!empty($tids)) {
            $parameter = array($this->get_table_name($tableid), $tids);
            $query = DB::query("SELECT * FROM %t WHERE tid IN(%n)".DB::limit($start, $limit), $parameter);
            while($value = DB::fetch($query)) {
                $data[$value['tid']] = $value;
                $this->store_cache($value['tid'], $value, $this->_cache_ttl);
            }
        }
    }
    return $data;
}

注意这个函数的参数:$tids应该为数组。这里又调用到fetch_cache和store_cache方法。找到source\class\discuz\discuz_table.php文件,里面有这两个方法的定义,

public function fetch_cache($ids, $pre_cache_key = null) {
    $data = false;
    if($this->_allowmem) {
        if($pre_cache_key === null)	$pre_cache_key = $this->_pre_cache_key;
        $data = memory('get', $ids, $pre_cache_key);
    }
    return $data;
}

public function store_cache($id, $data, $cache_ttl = null, $pre_cache_key = null) {
    $ret = false;
    if($this->_allowmem) {
        if($pre_cache_key === null)	$pre_cache_key = $this->_pre_cache_key;
        if($cache_ttl === null)	$cache_ttl = $this->_cache_ttl;
        $ret = memory('set', $id, $data, $cache_ttl, $pre_cache_key);
    }
    return $ret;
}

这两个方法最终调用缓存类的get和set方法,找到source\class\discuz\discuz_memory.php文件,有如下代码:

public function get($key, $prefix = '') {
    static $getmulti = null;
    $ret = false;
    if($this->enable) {
        if(!isset($getmulti)) $getmulti = method_exists($this->memory, 'getMulti');
        $this->userprefix = $prefix;
        if(is_array($key)) {
            if($getmulti) {
                $ret = $this->memory->getMulti($this->_key($key));
                //格式化数组的KEY,去掉表前缀
                if($ret !== false && !empty($ret)) {
                    $_ret = array();
                    foreach((array)$ret as $_key => $value) {
                        $_ret[$this->_trim_key($_key)] = $value;
                    }
                    $ret = $_ret;
                }
            } else {
                $ret = array();
                $_ret = false;
                //循环取值
                foreach($key as $id) {
                    if(($_ret = $this->memory->get($this->_key($id))) !== false && isset($_ret)) {
                        $ret[$id] = $_ret;
                    }
                }
            }
            //无值返回false
            if(empty($ret)) $ret = false;
        } else {
            $ret = $this->memory->get($this->_key($key));
            if(!isset($ret)) $ret = false;
        }
    }
    return $ret;
}

public function set($key, $value, $ttl = 0, $prefix = '') {

    $ret = false;
    if($value === false) $value = '';
    if($this->enable) {
        $this->userprefix = $prefix;
        $ret = $this->memory->set($this->_key($key), $value, $ttl);
    }
    return $ret;
}

注意get方法里,有对$key进行是否为数组的判断,如下代码:

if(is_array($key)) {

这里如果是数组则返回的结果为二维数组,key => value的格式;反之返回的结果是一维数组,只有value。
通过记录log发现source\admincp\cloud\cloud_qqgroup.php文件调用fetch_all_by_tid_displayorder方法的返回结果为一维数组,只有value,而正常来说fetch_all_by_tid_displayorder方法返回的结果应该是二维数组结构。

所以推断是由于source\admincp\cloud\cloud_qqgroup.php文件调用fetch_all_by_tid_displayorder方法时传递的$srchtid参数类型非数组导致的问题,进一步分析发现是由于Discuz! 所有要求参数为数组的地方没有进行是否是数组类型的判断导致。

解决方法:

不给出暂时的解决方法了,静待Discuz! 修复。

标签: Discuz, Memcache

添加新评论