☆小BUG（002）——必须重视DZ关于搜索引擎优化的一个BUG

主要分析DZ底层关于获取后台设置的模块关键字和描述，对于搜索引擎优化非常重要。






如图所示，在helper_seo类的获取seo设置方法get_seosetting里面。或者看下面代码颜色标注段。
蓝色的是获取title，只要不为空就可以获取。

if($titletext) {
$seotitle = helper_seo::strreplace_strip_split($searchs, $replaces, $titletext);
}
if($descriptiontext && (CURSCRIPT == 'forum' || IS_ROBOT || $_G['adminid'] == 1)) {
$seodescription = helper_seo::strreplace_strip_split($searchs, $replaces, $descriptiontext);
}
if($keywordstext && (CURSCRIPT == 'forum' || IS_ROBOT || $_G['adminid'] == 1)) {
$seokeywords = helper_seo::strreplace_strip_split($searchs, $replaces, $keywordstext);
}

红色的是获取keyword和description，为什么在获取title时，可以直接获取，而在获取keyword和description时就需要其他的判断条件呢？

CURSCRIPT == 'forum' || IS_ROBOT || $_G['adminid'] == 1

多出的这一句判断条件到底有什么作用？如果当前脚本模块是forum或者是爬虫或者是管理员，就可以获取keyword和description，爬虫还好理解。但是，如果当前脚本模块是forum之外的同时不是爬虫或者管理员就不能正常获取后台设置的keyword和description了。

我的问题：能不能把这一句去掉呢，就是所有页面都可以获取keyword和description，有没有什么坏处呢？

1. public static function get_seosetting($page, $data = array(), $defset = array()) {
3. global $_G;
4. $searchs = array('{bbname}');
5. $replaces = array($_G['setting']['bbname']);
7. $seotitle = $seodescription = $seokeywords = '';
8. $titletext = $defset['seotitle'] ? $defset['seotitle'] : $_G['setting']['seotitle'][$page];
9. $descriptiontext = $defset['seodescription'] ? $defset['seodescription'] : $_G['setting']['seodescription'][$page];
10. $keywordstext = $defset['seokeywords'] ? $defset['seokeywords'] : $_G['setting']['seokeywords'][$page];
11. preg_match_all("/{([a-z0-9_-]+?)}/", $titletext.$descriptiontext.$keywordstext, $pageparams);
12. if($pageparams) {
13. foreach($pageparams[1] as $var) {
14. $searchs[] = '{'.$var.'}';
15. if($var == 'page') {
16. $data['page'] = $data['page'] > 1 ? lang('core', 'page', array('page' => $data['page'])) : '';
17. }
18. $replaces[] = $data[$var] ? strip_tags($data[$var]) : '';
19. }
20. if($titletext) {
21. $seotitle = helper_seo::strreplace_strip_split($searchs, $replaces, $titletext);
22. }
23. if($descriptiontext && (CURSCRIPT == 'forum' || IS_ROBOT || $_G['adminid'] == 1)) {
24. $seodescription = helper_seo::strreplace_strip_split($searchs, $replaces, $descriptiontext);
25. }
26. if($keywordstext && (CURSCRIPT == 'forum' || IS_ROBOT || $_G['adminid'] == 1)) {
27. $seokeywords = helper_seo::strreplace_strip_split($searchs, $replaces, $keywordstext);
28. }
29. }
30. return array($seotitle, $seodescription, $seokeywords);
31. }
33. [code]<blockquote>

复制代码